Issue 5594 in pharo: ClassOrganizer use printString for changeSet

Status: Accepted
Owner: [hidden email]
Labels: Type-Bug Milestone-1.4 Importance-High

New issue 5594 by [hidden email]: ClassOrganizer use printString for  
changeSet
http://code.google.com/p/pharo/issues/detail?id=5594

This is a random ClassOrganizer printString

('private' canBeRemoved)
('accessing' methods name protocolOrganizer protocolOrganizer:)

Thanks to that, they do not had to introduce a new method dedicated to the  
file out.


Let's fix that because it's too ugly to live.

Slice is incoming


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

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 5594 by [hidden email]: ClassOrganizer use  
printString for changeSet
http://code.google.com/p/pharo/issues/detail?id=5594

Slice is in the inbox


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

Comment #2 on issue 5594 by [hidden email]: ClassOrganizer use  
printString for changeSet
http://code.google.com/p/pharo/issues/detail?id=5594

Can't we do that in 1.5?

Changes in old parts of the system seldomly have no side effect.


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

Updates:
        Labels: -Milestone-1.4 Milestone-1.5

Comment #3 on issue 5594 by [hidden email]: ClassOrganizer use  
printString for changeSet
http://code.google.com/p/pharo/issues/detail?id=5594

It can be in 1.5, but since it's really ugly, I want to kill it before it  
lays eggs ^^


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

Issue 5594: ClassOrganizer use printString for changeSet
http://code.google.com/p/pharo/issues/detail?id=5594

This issue is now blocking issue 5593.
See http://code.google.com/p/pharo/issues/detail?id=5593

--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

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

Comment #6 on issue 5594 by [hidden email]: ClassOrganizer use  
printString for changeSet
http://code.google.com/p/pharo/issues/detail?id=5594

I agree marcus :)
Ben BTW I when I do ClassOrganizer printString I get 'ClassOrganizer'


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

Comment #7 on issue 5594 by [hidden email]: ClassOrganizer use  
printString for changeSet
http://code.google.com/p/pharo/issues/detail?id=5594

With or without this change loaded ?


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

Updates:
        Status: Integrated

Comment #9 on issue 5594 by [hidden email]: ClassOrganizer use  
printString for changeSet
http://code.google.com/p/pharo/issues/detail?id=5594

in 2.0 020


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