Having a rules event in place for when T&C's are updated would be useful for those wishing to perform additional actions beyond just the basic functionality this module provides. For example, all users affected perhaps should be emailed.

Currently, the only sensible parameter to be passed to the rules event seems to be language, as a two digit text. Open to other suggestions.

Patch to follow.

CommentFileSizeAuthor
#4 rules_integration_tc_update-2743623-4.patch1.21 KBabarpetia
#2 add_rules_event_tc_update-2743623-1.patch1.2 KBjnicola

Comments

jnicola created an issue. See original summary.

jnicola’s picture

jnicola commented
StatusFileSize
new add_rules_event_tc_update-2743623-1.patch1.2 KB

Adding patch. Preliminary testing has this working. It is placed just one line above where the module itself sets the Drupal message for T&C's updated, so it seemed like the most appropriate place. If there's a false positive here, there's a false positive for that Drupal message as well!

jnicola’s picture

jnicola commented
Status: Needs work » Needs review
abarpetia’s picture

abarpetia commented
StatusFileSize
new rules_integration_tc_update-2743623-4.patch1.21 KB

Language variable is used by Rules module and overriding that variable to 'text' format will ended up in "Error: The data type of the configured argument does not match the parameter's language requirement." error message. Attached patch updating the name of variable and adding missing ',' on line 868.

robert castelo’s picture

robert castelo commented
Status: Needs review » Fixed

Alternative patch has been comitted to dev branch to add Rules integration

https://www.drupal.org/node/1832904

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.