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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.