Issue 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

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

Issue 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

pharo
Status: Accepted
Owner: [hidden email]

New issue 4708 by [hidden email]: [Gofer] MessageNotUnderstood:  
ByteString>>goferReferences
http://code.google.com/p/pharo/issues/detail?id=4708

Pharo image: Pharo1.4a#14109
Virtual machine used: Squeak4.1 of 17 April 2010 [latest update: #9957]
Platform Name: unix
Class browser used (if applicable): Browser

Steps to reproduce:
#. evaluate
Gofer it
     repository: 'http://ss3.gemstone.com/ss/SystemReporter';
     package: 'SystemReporter';
     load

Actual Result:
#. MessageNotUnderstood: ByteString>>goferReferences

Expected Result:
#. load SystemReporter



Attachments:
        PharoDebug.log  21.7 KB


_______________________________________________
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 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

pharo
Updates:
        Owner: ---

Comment #1 on issue 4708 by [hidden email]: [Gofer]  
MessageNotUnderstood: ByteString>>goferReferences
http://code.google.com/p/pharo/issues/detail?id=4708

(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 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

pharo

Comment #2 on issue 4708 by [hidden email]: [Gofer]  
MessageNotUnderstood: ByteString>>goferReferences
http://code.google.com/p/pharo/issues/detail?id=4708

It seems like #repository: takes aRepository, not a string. Try:
   Gofer new
     url: 'http://ss3.gemstone.com/ss/SystemReporter';
     package: 'SystemReporter';
     load


_______________________________________________
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 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

pharo

Comment #3 on issue 4708 by [hidden email]: [Gofer]  
MessageNotUnderstood: ByteString>>goferReferences
http://code.google.com/p/pharo/issues/detail?id=4708

thanks, but i think that the api could warn you and not give you a MNU


_______________________________________________
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 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

pharo
Updates:
        Cc: renggli

Comment #4 on issue 4708 by [hidden email]: [Gofer]  
MessageNotUnderstood: ByteString>>goferReferences
http://code.google.com/p/pharo/issues/detail?id=4708

(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 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

pharo
Updates:
        Status: ReviewNeeded

Comment #5 on issue 4708 by [hidden email]: [Gofer]  
MessageNotUnderstood: ByteString>>goferReferences
http://code.google.com/p/pharo/issues/detail?id=4708

im adding
SLICE-Issue-4708-MessageNotUnderstood-ByteStringgoferReferences-BernardoContreras.1
to validate the given repository.



_______________________________________________
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 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

pharo

Comment #6 on issue 4708 by renggli: [Gofer] MessageNotUnderstood:  
ByteString>>goferReferences
http://code.google.com/p/pharo/issues/detail?id=4708

I don't strongly object of adding this change, but then it is not in the  
style of Smalltalk (or any other dynamic language).

The method #repository: states in the selector name, the argument name and  
the method name that an argument understanding the MCRepository protocol is  
expected. Adding a type check breaks the possibility to use a different  
object that is not in the MCRepository hierarch but that supports the same  
protocol (i.e. a MCRepositoryGroup). Also note that Code Critics will  
complain about 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 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

pharo
Updates:
        Status: WontFix

Comment #7 on issue 4708 by [hidden email]: [Gofer]  
MessageNotUnderstood: ByteString>>goferReferences
http://code.google.com/p/pharo/issues/detail?id=4708

thanks, lukas. i should check code critics :)


_______________________________________________
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 4708 in pharo: [Gofer] MessageNotUnderstood: ByteString>>goferReferences

pharo

Comment #8 on issue 4708 by [hidden email]: [Gofer]  
MessageNotUnderstood: ByteString>>goferReferences
http://code.google.com/p/pharo/issues/detail?id=4708

could someone remove that slice from pharoinbox, please?


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