Extend the listener to receive events when a rule evaluates to false

[pklovesbacon]

I would like to be notified via the listener interface when a rule evaluates to false. Thanks

[yisraelU]

I'd also like to request this feature

[fmbenhassine]

Hi,

Good Idea! Thank you for this suggestion.

I've added a method void afterEvaluate(Rule rule, boolean evaluationResult) in the RuleListener API to get notified with the evaluation result of each rule.

This method will be invoked in both cases, so you can just grab the result you need to get notified on. You can already give a try with version 2.5.0-SNAPSHOT.

Looking forward for your feedback.

Kr
Mahmoud

[yisraelU]

where do i find 2.5.0 ?

[fmbenhassine]

v2.5.0 is not released yet.

You can try the new feature with version 2.5.0-SNAPSHOT that can be found in sonatype's repository. See here

[yisraelU]

would it be bad practice to use the listener to provide actions for the false side to the rule?

[fmbenhassine]

Not a bad practice but it's not how it is designed to be used.

Easy rules is a production system (no else statement), so I guess you can create a rule for "else" and you are done.

Did you manage to use the snapshot version and test the new listener method?

[yisraelU]

Yes I have . So far it's working great no troubles

…

On May 12, 2017 5:14 AM, "Mahmoud Ben Hassine" ***@***.***> wrote: Not a bad practice but it's not how it is designed to be used. Easy rules is a production system (no else statement), so I guess you can create a rule for "else" and you are done. Did you manage to use the snapshot version and test the new listener method? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#72 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AEILSmlZ8pFFhcUlcr8518UqQFURfbptks5r5CMOgaJpZM4NTBgW> .

[fmbenhassine]

@pklovesbacon Have you got a chance to test the feature you requested?

[pklovesbacon]

Hi Benas, Thanks for responding to this request. I really appreciate you adding this feature.
I'm currently tied up on another project but should be able to get back you on this in a few days time.

[pklovesbacon]

I got a chance to run it up in my test environment this morning.
Yes, it suits my need perfectly.
Thanks again , Paul.

[fmbenhassine]

Great! Thank you for your time.

I'll include the fix in the upcoming release, keep tuned.