On Wed, Nov 10, 2010 at 09:08, Stefan abageru <[email protected]> wrote: > Hi devs ! > > I thought about adding some changes in Events and Activity Stream. > > First of all, I want to remove all the listeners from Activity Stream > and make this class listen to AllEvent, making it capable of cathcing > all events that occur in the wiki (and this way, when adding a new > event, you wouldn't need any modifications to this class in order to > catch it). > > Secondly, I would like to modify onEvent function in the class so that > it uses the simpleClassName of the event occurred in order to log the > event (the eventType would be this simpleClassName...for example > DocumentSavedEvent, AnnotationAddedEvent and so on). > > And thirdly i thought about adding one field in AbstractFilterableEvent > called /identifier/ and make all events extend directly this class. And > that /identifier/ would be used in a different way by the events > themselves. But it would be used to retrieve infos about that events. > And so, when logging an event, we could use classname + identifier. > For example > AnnotationAddedEvent + "This is my annotation" . > CommentAddedEvent + "This is my comment" or the number of the comment.
-1 for that, you can't force all events to extend AbstractFilterableEvent it does not always m > > > WDYT ? > > Stefan > _______________________________________________ > devs mailing list > [email protected] > http://lists.xwiki.org/mailman/listinfo/devs > -- Thomas Mortagne _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
