Issue 5357 in pharo: bulk removal of methods

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

Issue 5357 in pharo: bulk removal of methods

pharo
Status: Accepted
Owner: marianopeck
CC: [hidden email]
Labels: Type-Nautilus

New issue 5357 by marianopeck: bulk removal of methods
http://code.google.com/p/pharo/issues/detail?id=5357

when selecting several methods and  try to remove them, do not ask each  
method individually… "it has xx senders.. what do you want to do?" … give  
options according to the fact that there are several methods, but it only  
once. For example: "remove all and see all senders" "or just remove all".  
If you ask that for each method…then it is not a bulk remove but again,  
having to remove method by method…

Ben answered:

I should, but it's some work :)



_______________________________________________
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 5357 in pharo: bulk removal of methods

pharo
Updates:
        Status: Integrated
        Blockedon: 5368

Comment #1 on issue 5357 by [hidden email]: bulk removal of methods
http://code.google.com/p/pharo/issues/detail?id=5357

Done :)

But since some methods have been implemented on AbstractTool, I am waiting  
for integration :)


_______________________________________________
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 5357 in pharo: bulk removal of methods

pharo

Comment #2 on issue 5357 by marianopeck: bulk removal of methods
http://code.google.com/p/pharo/issues/detail?id=5357

Hi Ben. I was doing this: take the class ShortRunArrayTest and I selected  
the methods: testDo, testLengthAndValuesDo, testLengthAtRun and testSize.  
And then I tried to remove them. The systems just freezes. After  
interrupting, I see it is in
self buildSelfContainedCallsFrom: aCollection in: aClass

because of confirmRemovalOfSelectors: aCollection on: aClass

I am using last Nautilus and last Pharo 1.4. If I find something else I  
will let you know.


_______________________________________________
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 5357 in pharo: bulk removal of methods

pharo

Comment #3 on issue 5357 by [hidden email]: bulk removal of methods
http://code.google.com/p/pharo/issues/detail?id=5357

I think i will just remove that since it's not really useful


_______________________________________________
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 5357 in pharo: bulk removal of methods

pharo

Comment #4 on issue 5357 by [hidden email]: bulk removal of methods
http://code.google.com/p/pharo/issues/detail?id=5357

Fixed


_______________________________________________
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 5357 in pharo: bulk removal of methods

pharo

Comment #5 on issue 5357 by marianopeck: bulk removal of methods
http://code.google.com/p/pharo/issues/detail?id=5357

Excellent. I have just tested and works :)


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