Having problems with Rulehelper

15 views
Skip to first unread message

boadie

unread,
Aug 22, 2010, 9:52:00 PM8/22/10
to rul...@googlegroups.com
I finally got some time this weekend to try the new version with the Rulehelper, I was excited when I saw the new
syntax but have been buried for the last two months.

I opened two issues, one minor one that the "-snapshot" in version string breaks rake.
The second is that the new Rulehelper example is not working.

I cannot seem to get rulehelper to work in my test progs. Spec and Test both seem to react
adversely to Include Ruleby::RuleHelper getting some bizarre traceback about method arguments,
I assume some namespace clash but I have not had the time to track it down.

- Ric


Matt Smith

unread,
Aug 22, 2010, 10:22:56 PM8/22/10
to rul...@googlegroups.com, rul...@googlegroups.com
Hi Ric,

It might be the binding method. Can you give the or_transform branch a try?

Thanks

Matt

Sent from my iPhone

> --
> You received this message because you are subscribed to the Google Groups "Ruleby" group.
> To post to this group, send email to rul...@googlegroups.com.
> To unsubscribe from this group, send email to ruleby+un...@googlegroups.com.
> For more options, visit this group at http://groups.google.com/group/ruleby?hl=en.
>

Boadie

unread,
Aug 23, 2010, 12:03:12 AM8/23/10
to rul...@googlegroups.com
Yep spot on, it is the binding method. The or_transform branch seems to have the same issue.
The fibonacci example 3 also fails with the or_transform with the same complaint about undefined method 'pattern' in assert_rule
as I logged in the issue.

- Ric

Matt Smith

unread,
Aug 23, 2010, 2:16:15 AM8/23/10
to rul...@googlegroups.com
Hi RIc,

Thanks for catching that.  This was do to the "rule" method returning rules as an array, which we need as "or" rules get converted into multiple rules.  This has been updated in the or_transform branch.

Also, I got rid of the binding and condition methods as they cause conflicts with some environments, use the "b" and "c" methods instead.

Thanks,

Matt
Reply all
Reply to author
Forward
0 new messages