Re: allowing content rules to be launched by other rules
Yuri <[email protected]>
| Newsgroups | gmane.comp.web.zope.plone.devel |
|---|---|
| Message-ID | <[email protected]> |
Il 09/07/2013 08:48, thomas desvenain ha scritto: > > > > On Mon, Jul 8, 2013 at 8:18 PM, ajung <[email protected] > <mailto:[email protected]>> wrote: > > -1 - sounds like yet another level of complexity and featurism > that is not > often needed. > Explict configuration of multiple rules is more obvious and easy > to handle > than stuff happening magically under the hood. > > > Keep it simple, keep it clear > and obvious. > > > The current behavior is absolutely not obvious, not clear and not > simple. Anyone think that if he has got a "publish when added" rule > and a "notify when publish" rule, > then if he adds a content, it will be published and a notification > will be send. > It doesn't, and only people that read the code understand this is a > feature, not a bug. > The alternative for user, having two rules (a rule that publish AND > notify when added, and an other rule that notify when added), and > assign them on different parts of the website, do not keep it simple. > I agree, why just don't queue the events and execute them once? Shouldn't this things handled asyncronously? For loops, assign to every pool of events in the queue a session, then you can limit every session with a number of events, cutting loops before executing anything. I think tomcat does this, for example gsearch webapps. I think this is dangerous if not well designed (if you move content, the context change and when you apply an event, maybe something is broken at that stage). So the better is to keep as it is, and add a warning when multiple rules insist on the same event (as publish before), with a documented workaround. ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk