base.action.rule is repeated ignoring the date it was last run

This is related to:
http://help.openerp.com/question/2303/how-can-a-baseactionrule-be-run-just-one-time/

If you look at the code here:
http://bazaar.launchpad.net/~openerp/openobject-addons/trunk/view/head:/base_action_rule/base_action_rule.py#L272

                if last_run and (last_run <= action_dt < now) or (action_dt < now):

you see a misuse of the python and-or trick. What happens in that specific case is that when (last_run <= action_dt < now) is False, then Python checks (action_dt < now) which can easily be True.

I think this can be fixed using the inline if-else instead of the and-or trick.

The bug seems to have been introduced just before releasing 7.0:
http://bazaar.launchpad.net/~openerp/openobject-addons/trunk/revision/8098.3.29

Thanks!