This patch adds yet another Rules integration. New Rules condition is added - "User has accepted T&C".

CommentFileSizeAuthor
#2 legal-rules_integration-1873354-2.patch2.06 KBmaximpodorov
legal-rules_integration.patch2.3 KBmaximpodorov

Comments

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Active » Needs work

+1 to this

+++ b/legal.info
@@ -1,6 +1,7 @@
+files[] = legal.rules.inc

No need, only files that contains classes should be listed here

+++ b/legal.rules.inc
@@ -0,0 +1,62 @@
+  if ($uid == 1) {
+    return TRUE;
+  }

Please add a comment that UID=1 skips the check, does the module works the same way?

maximpodorov’s picture

maximpodorov commented
Status: Needs work » Needs review
StatusFileSize
new legal-rules_integration-1873354-2.patch2.06 KB

The updated patch.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Great!

jherencia’s picture

jherencia commented

Worked for me too, thank you :).

It would be perfect if the maintainer could commit the patch.

jherencia’s picture

jherencia commented
Issue summary: View changes

Updated issue summary.

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Title: Rules integration (adding condition) » Rules integration: Condition "T&C accepted"
Issue summary: View changes
Related issues: +#1832904: Rules integration: Event "T&C accepted"

  • Robert Castelo committed 9792490 on 7.x-1.x 
    Issue #1873354 by maximpodorov: Rules integration: Condition "T...
robert castelo’s picture

robert castelo commented
Status: Reviewed & tested by the community » Fixed

Thanks maximpodorov!

Added to dev, will be in the next release (soon).

Status: Fixed » Closed (fixed)

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