Issue 5105 in pharo: it allows me to install a method when the class doesn't exists.

Status: Accepted
Owner: ----
Labels: Milestone-1.4 Type-Bug

New issue 5105 by [hidden email]: it allows me to install a method  
when the class doesn't exists.
http://code.google.com/p/pharo/issues/detail?id=5105

Pharo image: Pharo1.4a#14256
Virtual machine used: CoInterpreter VMMaker.oscog-eem.138 uuid:  
f9a63bdf-7bbc-4ae5-9634-ecd6fd814c9d Nov 17 2011,  
StackToRegisterMappingCogit VMMaker.oscog-eem.139 uuid:  
c2849383-9768-4948-b9b2-a5c22d482b07 Nov 17 2011, r2515  
http://www.squeakvm.org/svn/squeak/branches/Cog
Platform Name: unix
Class browser used (if applicable): Browser

the MNU raise because it allows me to install the method testCapacity
when the class HashedCollectionIntegrityTest doesn't exist in the image.

Steps to reproduce:
#. World>>Monticello Browser
#. open repository http://source.squeak.org/trunk
#. select Tests
#. select Tests-ul.75.mcz
#. click on history
#. select Tests-laza.74.mcz
#. right click and view changes -> Tests-ul.75
#. select the method testCapacity
#. right click and install incoming version
#.

Actual Result:
#. MNU: receiver of "compile:classified:withStamp:notifying:" is nil
Expected Result:
#. you are not able to that method in the image.


Attachments:
        PharoDebug.log  27.7 KB


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

Comment #1 on issue 5105 by [hidden email]: it allows me to install a  
method when the class doesn't exists.
http://code.google.com/p/pharo/issues/detail?id=5105

Expected Result:
#. you are not able to install that method in the image.



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

Comment #2 on issue 5105 by [hidden email]: it allows me to install a  
method when the class doesn't exists.
http://code.google.com/p/pharo/issues/detail?id=5105

Could NOT reproduce this bug on 1.4-14260 on Mac


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

Comment #3 on issue 5105 by [hidden email]: it allows me to install a  
method when the class doesn't exists.
http://code.google.com/p/pharo/issues/detail?id=5105

could reproduce in


Pharo image: Pharo1.4a#14260
Virtual machine used: CoInterpreter VMMaker.oscog-eem.134 uuid:  
a0d46c48-3d9a-43f9-a769-51f31ce120e9 Oct 19 2011,  
StackToRegisterMappingCogit VMMaker.oscog-eem.134 uuid:  
a0d46c48-3d9a-43f9-a769-51f31ce120e9 Oct 19 2011, r2502  
http://www.squeakvm.org/svn/squeak/branches/Cog
Platform Name: unix
Class browser used (if applicable): Browser

just in case, im attaching another trace.

Attachments:
        PharoDebug.2.log  27.7 KB


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

Comment #4 on issue 5105 by [hidden email]: it allows me to install  
a method when the class doesn't exists.
http://code.google.com/p/pharo/issues/detail?id=5105

I could reproduce the problem following the steps exactly as mentioned.

Even, if you just do:

- MC browser
- open a repository
- browse a version
- select a method of a class you don't have in your image
- "load method"

It barks the same


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

Updates:
        Status: FixReviewNeeded

Comment #5 on issue 5105 by [hidden email]: it allows me to install  
a method when the class doesn't exists.
http://code.google.com/p/pharo/issues/detail?id=5105

He, this bug seems to come from old times :).

I attach a suggested fix

Attachments:
        Issue 5105: it allows me to install a method when the class doesn't  
exists..1.cs  865 bytes


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

Updates:
        Status: FixToInclude

Comment #6 on issue 5105 by [hidden email]: it allows me to install  
a method when the class doesn't exists.
http://code.google.com/p/pharo/issues/detail?id=5105

looks good


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

Updates:
        Status: Integrated

Comment #7 on issue 5105 by [hidden email]: it allows me to install  
a method when the class doesn't exists.
http://code.google.com/p/pharo/issues/detail?id=5105

in 14381


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