Issue 4905 in pharo: Clean SystemAnnouncement private/public

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

Issue 4905 in pharo: Clean SystemAnnouncement private/public

pharo
Status: Accepted
Owner: [hidden email]
CC: [hidden email],  [hidden email]
Labels: Milestone-1.4

New issue 4905 by [hidden email]: Clean SystemAnnouncement  
private/public
http://code.google.com/p/pharo/issues/detail?id=4905

We should remove half of the annoucements in SystemAnnouncement (so that we  
do not deal with private one the way it is). What we should have is two  
announcers

Announcer announcePublicly
        should send two announcements

the private one only one.

I should have enter this issue when it was clear in my mind. geessshhh.



Igor I will have a look and sketch something and we sit friday afternoon  
and do it for real


_______________________________________________
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 4905 in pharo: Clean SystemAnnouncement private/public

pharo

Comment #1 on issue 4905 by [hidden email]: Clean SystemAnnouncement  
private/public
http://code.google.com/p/pharo/issues/detail?id=4905

first
        - changed all the tests to not refer to Internal annoucements anymore,
        - checked and fixed all the methods in the two hierarchies
        - started to introduce another announcer in SystemAnnouncer
                        SystemAnnouncer current
                        SystemAnnouncer private.

Attachments:
        SystemAnnouncement.1.cs  40.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 4905 in pharo: Clean SystemAnnouncement private/public

pharo

Comment #2 on issue 4905 by [hidden email]: Clean SystemAnnouncement  
private/public
http://code.google.com/p/pharo/issues/detail?id=4905

New version cleaning a bit more.
Now I will try to apply it in a RPackage image to see...

Attachments:
        SystemAnnouncement.2.cs  41.8 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 4905 in pharo: Clean SystemAnnouncement private/public

pharo

Comment #3 on issue 4905 by [hidden email]: Clean SystemAnnouncement  
private/public
http://code.google.com/p/pharo/issues/detail?id=4905

The second version was tested on RPackage. Now waited from Benjamin to know  
if the fixes work with Nautilus.
We should remove Internal hierarchy


_______________________________________________
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 4905 in pharo: Clean SystemAnnouncement private/public

pharo
Updates:
        Status: Closed

Comment #4 on issue 4905 by [hidden email]: Clean SystemAnnouncement  
private/public
http://code.google.com/p/pharo/issues/detail?id=4905

in 14197


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