Issue 6398 in pharo: MCPackageManager class >> methodMoved: uses old SystemChangeNotifier

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

Issue 6398 in pharo: MCPackageManager class >> methodMoved: uses old SystemChangeNotifier

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

New issue 6398 by [hidden email]: MCPackageManager class >>  
methodMoved: uses old SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=6398

MCPackageManager class >> methodMoved: gets a system announcement  
MethodRecategorized as anEvent but it still expects RecategorizedEvent from  
SystemChangeNotifier.

I disovered this this while trying to load latest versions of Gofer to the  
Pharo Kernel. Fix changes call of oldCategory to itemOldProtocol.


_______________________________________________
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 6398 in pharo: MCPackageManager class >> methodMoved: uses old SystemChangeNotifier

pharo

Comment #1 on issue 6398 by [hidden email]: MCPackageManager  
class >> methodMoved: uses old SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=6398

SLICE-Issue-6398-MCPackageManager-class-gtgt-methodMoved-uses-old-SystemChangeNotifier-PavelKrivanek.1


_______________________________________________
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 6398 in pharo: MCPackageManager class >> methodMoved: uses old SystemChangeNotifier

pharo
Updates:
        Status: FixReviewNeeded

Comment #2 on issue 6398 by [hidden email]: MCPackageManager  
class >> methodMoved: uses old SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=6398

(No comment was entered for this change.)


_______________________________________________
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 6398 in pharo: MCPackageManager class >> methodMoved: uses old SystemChangeNotifier

pharo

Comment #3 on issue 6398 by [hidden email]: MCPackageManager  
class >> methodMoved: uses old SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=6398

hmm, I tried to change a category of some method in an image with this  
slice and I got DNU because in this case it gets RecategorizedEvent as  
anEvent. :-(


_______________________________________________
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 6398 in pharo: MCPackageManager class >> methodMoved: uses old SystemChangeNotifier

pharo

Comment #4 on issue 6398 by [hidden email]: MCPackageManager  
class >> methodMoved: uses old SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=6398

I added itemOldProtocol category to RecategorizedEvent for compatibility of  
this classes.

SLICE-Issue-6398-MCPackageManager-class-gtgt-methodMoved-uses-old-SystemChangeNotifier-PavelKrivanek.2


_______________________________________________
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 6398 in pharo: MCPackageManager class >> methodMoved: uses old SystemChangeNotifier

pharo
Updates:
        Status: FixToInclude

Comment #5 on issue 6398 by [hidden email]: MCPackageManager  
class >> methodMoved: uses old SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=6398

It looks ok, makes MethodRecategorized and ClassRecategorized polymorphic.


_______________________________________________
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 6398 in pharo: MCPackageManager class >> methodMoved: uses old SystemChangeNotifier

pharo
Updates:
        Status: Integrated

Comment #6 on issue 6398 by [hidden email]: MCPackageManager class  
>> methodMoved: uses old SystemChangeNotifier
http://code.google.com/p/pharo/issues/detail?id=6398

in 2.0 224


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