Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi,
I was reading about the issue http://drupal.org/node/1338768 and I was wandering if someone can integrate into Rules module. It could be a very useful addition.
Thanks,
Perry.
Comment | File | Size | Author |
---|---|---|---|
#2 | rules-user-has-permission-1418978.patch | 2.98 KB | mavame |
#1 | rules-user-has-permission-1418978.patch | 2.19 KB | mavame |
Comments
Comment #1
mavame CreditAttribution: mavame commentedHi, I wrote a patch.
Comment #2
mavame CreditAttribution: mavame commentedThe previous patch was missing the eval callback.
Comment #3
Mac_Weber CreditAttribution: Mac_Weber commentedSending to testbot
Comment #6
TR CreditAttribution: TR commentedPatch still applies, tests pass. However, there should be a new test for this new condition ... also there are two minor coding standards issues (which weren't issues when the patch was originally written ...)
Those things need to be fixed if this is going to be committed, but I'm going to leave the status at "Needs review" for now to see if that will prompt some people to try this patch and review it ...
Comment #7
TR CreditAttribution: TR commentedMoving to 8.x-3.x. There doesn't seem to be much interest in adding this, but if it is added it should go into the current version of Rules first. It can then be backported to 7.x-2.x if there is community interest.