On Fri, Sep 2, 2011 at 9:14 AM, Thomas Mortagne <[email protected]> wrote: > On Fri, Sep 2, 2011 at 9:11 AM, Sergiu Dumitriu <[email protected]> wrote: >> On 09/02/2011 02:52 AM, Thomas Mortagne wrote: >>> Hi devs, >>> >>> I started to move (very) old notification system to legacy but I'm >>> stuck because there is one thing still not supported by new >>> observation system: event sent before executing an action. >>> >>> So here it is: I propose to add it with the name ActionExecutingEvent >>> in the bridge. >> >> -1 for the bridge, unless we rename it from model-bridge to >> oldcore-bridge, since it actually bridges the connection between >> components and the oldcore. Initially it was supposed to be a mockup of >> the future model component only, but it looks like it's far from it. > > Yes it's far from it since a long time now.
Also its name is bridge, not model-bridge so there is nothing to rename. Even the packages of this modules never refer to "model". There is user, skins, and right related classes/methods since a very long time now so I think the moved as been already done... > >> >>> At the same time I would also like to come with ActionExecutedEvent in >>> the bridge to replace the ActionExecuteEvent which is currently in >>> observation API (and that should never have been there). >> >> +1. >> >>> WDYT ? >>> >>> here is my +1 >> >> +1. >> >> I think they should also be cancellable, blocking requests if they fail >> some conditions. >> >> -- >> Sergiu Dumitriu >> http://purl.org/net/sergiu/ >> _______________________________________________ >> devs mailing list >> [email protected] >> http://lists.xwiki.org/mailman/listinfo/devs >> > > > > -- > Thomas Mortagne > -- Thomas Mortagne _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
