Skip to content

fix cron-based triggers

JonGold requested to merge JonGold/civirules:event-cron-fixes into master

Cron-based event date triggers are broken in three different ways. One of those ways breaks all CiviRules based on this trigger. I could break these into separate PRs, but it'd be difficult to test without all three fixed, and they're each very small changes.

  • The SQL that does date arithmetic is incorrect, so any trigger with a date offset will return 0 records.
  • The parameterized SQL statement references %3 but there is no $params[3]. This should be %2 and this returns an error any time the Scheduled Job is run with such a rule active.
  • A typo on the form meant that the date offset wasn't being loaded when editing the trigger.

Merge request reports