Issue 4700 in pharo: hasTraitComposition

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

Issue 4700 in pharo: hasTraitComposition

pharo
Status: FixProposed
Owner: [hidden email]

New issue 4700 by [hidden email]: hasTraitComposition
http://code.google.com/p/pharo/issues/detail?id=4700

this method test is traitComposition is nil or not.

But actually, traitComposition seems to be initialized as a TraitComposition

So instead of checking if it's nil or no, we should check if it's empty or  
not.


It's broken since 2006 :D

Attachments:
        FixHasTraitComposition.1.cs  493 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 4700 in pharo: hasTraitComposition

pharo

Comment #1 on issue 4700 by [hidden email]: hasTraitComposition
http://code.google.com/p/pharo/issues/detail?id=4700

New version, please forget the previous one ;)

Attachments:
        FixHasTraitComposition.2.cs  600 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 4700 in pharo: hasTraitComposition

pharo
Updates:
        Labels: Milestone-1.4

Comment #2 on issue 4700 by [hidden email]: hasTraitComposition
http://code.google.com/p/pharo/issues/detail?id=4700

(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 4700 in pharo: hasTraitComposition

pharo

Comment #3 on issue 4700 by [hidden email]: hasTraitComposition
http://code.google.com/p/pharo/issues/detail?id=4700

There is #isEmptyOrNil... so using

^traitComposition isEmptyOrNil not

instead of:

^traitComposition notNil and: [ traitComposition notEmpty ]


_______________________________________________
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 4700 in pharo: hasTraitComposition

pharo
Updates:
        Status: Closed

Comment #4 on issue 4700 by [hidden email]: hasTraitComposition
http://code.google.com/p/pharo/issues/detail?id=4700

in 14130


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