If I have 5 minutes today, I'll add a test :P. On Tue, Apr 12, 2011 at 8:17 AM, Benjamin < benjamin.vanryseghem.pharo@gmail.com> wrote:
Adding 'asString' again could solve this problem, but I think it's hiding something because
item := selector, ' (' , defClass name , ')'.
So if item is a Text, it means selector is a Text, which sounds strange to me...
But yes, let's put asString ^^
Ben
On Apr 12, 2011, at 12:51 AM, Bernardo Ezequiel Contreras wrote:
Guillermo Polito and Benjamin Van Ryseghem,
Could you check your changes made to ProtocolBrowser>>initListFrom: selectorCollection highlighting: aClass ? It's related to http://code.google.com/p/pharo/issues/detail?id=3088
Basically, Guillermo added the line
stringVersion: item asString
and then Benjamin replaced it with
stringVersion: item
So, now the Protocol Browser is creating SourceMethodReferences with Texts as stringVersion. you can view those instances if you follow the steps and then execute
SourcedMethodReference allInstances select: [:each | each stringVersion class == Text ]
And it leads to http://code.google.com/p/pharo/issues/detail?id=3088
For what i saw, the fix will be to add Guillermo's line again but im not sure. its confusing.
What do you think?
Thanks