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