Issue 5927 in pharo: When removing a method from a list

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 5927 in pharo: When removing a method from a list

pharo
Status: Accepted
Owner: [hidden email]
CC: [hidden email]
Labels: Type-Bug Milestone-2.0

New issue 5927 by [hidden email]: When removing a method from a list
http://code.google.com/p/pharo/issues/detail?id=5927

we get a problem because the code looks to display the code of the method.

eventRegistration
       
        listModel whenSelectedItemChanged: [:item |
                        toolbarModel method: item.
                        textModel behavior: (item ifNil: [ nil ] ifNotNil: [ item methodClass ]).
                        textModel text: (textConverter method: item; getText)].
                                                                                             
^^^^^^^^^^^
               
        listModel whenListChanged: [ self setTitle ].
               
        titleHolder
                whenChangedDo: [ self setTitle ]

getText
      somehow invokes

sourceCode

        self isActive
                ifTrue: [ ^ self compiledMethod sourceCode ].
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

        self isHistorical
                ifTrue: [ ^ self sourceCodeAtPointer ifNil:[ self compiledMethod ifNil:[  
sourceCode ] ifNotNil:[ :cm| cm sourceCode ] ] ].
        ^ sourceCode


but since the compiledMethod has been removed it does not work. The ring  
object should be send asHistorical
.




_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 5927 in pharo: When removing a method from a list

pharo
Updates:
        Status: Closed

Comment #1 on issue 5927 by [hidden email]: When removing a method  
from a list
http://code.google.com/p/pharo/issues/detail?id=5927

I think this has been fixed


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker