On 09/05/2011 08:54 AM, Thomas Mortagne wrote: > 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".
Indeed, my bad. So, +1 for this. > 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
