Issue 5464 in pharo: SystemNavigation>>browseHierarchy:selector: should use Browser>> #spawnHierarchyForClass:selector:

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

Issue 5464 in pharo: SystemNavigation>>browseHierarchy:selector: should use Browser>> #spawnHierarchyForClass:selector:

pharo
Status: Accepted
Owner: ----
Labels: Milestone-1.4 Type-Cleanup Difficulty-Easy

New issue 5464 by [hidden email]:  
SystemNavigation>>browseHierarchy:selector: should use Browser>>  
#spawnHierarchyForClass:selector:
http://code.google.com/p/pharo/issues/detail?id=5464

SystemNavigation>>browseHierarchy: aClass selector: aSelector
        "Open a browser"
        (aClass == nil)  ifTrue: [^ self].

        ^ Smalltalk tools browser new
                        setClass: aClass
                        selector: aSelector;
                        spawnHierarchy;
                        yourself.

should be

SystemNavigation>>browseHierarchy: aClass selector: aSelector
        "Open a browser"
        (aClass == nil)  ifTrue: [^ self].

        ^ Smalltalk tools browser new
                        spawnHierarchyForClass: aClass
                         selector: aSelector;
                        yourself.


_______________________________________________
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 5464 in pharo: SystemNavigation>>browseHierarchy:selector: should use Browser>> #spawnHierarchyForClass:selector:

pharo
Updates:
        Status: FixReviewNeeded

Comment #1 on issue 5464 by [hidden email]:  
SystemNavigation>>browseHierarchy:selector: should use Browser>>  
#spawnHierarchyForClass:selector:
http://code.google.com/p/pharo/issues/detail?id=5464

(No comment was entered for this change.)

Attachments:
        issue-5464.1.cs  391 bytes


_______________________________________________
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 5464 in pharo: SystemNavigation>>browseHierarchy:selector: should use Browser>> #spawnHierarchyForClass:selector:

pharo
Updates:
        Status: Integrated

Comment #2 on issue 5464 by [hidden email]:  
SystemNavigation>>browseHierarchy:selector: should use Browser>>  
#spawnHierarchyForClass:selector:
http://code.google.com/p/pharo/issues/detail?id=5464

in 14388


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