Fork me on GitHub
#clara
<
2016-01-22
>
ryanbrush20:01:51

@pguillebert: Ugh, I think you came across a bug here. Issue logged: https://github.com/rbrush/clara-rules/issues/165

ryanbrush20:01:41

@pguillebert It looks like it's specific to the case of having a test within a negation, which somehow we haven't covered. Thanks for reporting it here.

pguillebert20:01:37

you’re welcone

pguillebert20:01:00

usually when I report something I am the bug simple_smile

ryanbrush20:01:16

@pguillebert Haha, I know that feeling very well. I got a simple fix for this bug committed on the issue linked above. The fix is a bit ugly and depends on some other refactoring, but I expect to get this into master next week and released soon afterwards. If you do need this sooner then you can build against the issue-165 branch I created.

pguillebert21:01:29

well thanks but the simple workaround of negating the condition works well for me

ryanbrush21:01:31

Good to know. I mostly just didn't want that mistake hanging around out here any longer than it already has. 😉

pguillebert23:01:56

Mmmh what is the preferred way to configure rules ?

pguillebert23:01:29

like, should I add a configuration record to my facts or is there a better facility ?