The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Thanks for the nice LoginToboggan module for D7.
I really like the rules integration. I spent a couple of minutes finding the rules event, till I found it on top of the list. I was looking for a LoginToboggan group first but there was none. This patch adds the missing LoginToboggan group (optgroup) and puts the rule inside. This is just a one-liner but an important one. ;)
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | add_rule_into_group.patch | 544 bytes | Anonymous (not verified) |
| #1 | LT_Rules.jpg | 97.19 KB | 1sp |
| add_rule_into_group.patch | 550 bytes | Anonymous (not verified) |
Comments
Comment #1
1sp commented@mikewink i can see the group declaration in the git master already. are you celebrating april 1st on me :P
Nevermind , this actually pulled in a good argument. Shouldn't the LoginToboggan events be in User group ?
Please see the attached image for detail.
Comment #2
1sp commented@hunmonk what do you think on this?
Comment #3
hunmonk commented@sudhirporwal: i don't use rules, so i defer to your judgement on this -- whatever seems like the most sensible place people would look to find it, that's where it should go. :)
Comment #4
1sp commented@hunmonk thanks :) I believe It must go in the User group as it is where normally any user would expect it.
@mikewink can you please re-submit the patch so that the event gets listed in the User group? Thanks for the effort so far !
Comment #5
Anonymous (not verified) commentedHere is the altered patch as you wished my lord. ;) It puts the rule into the user group.
Comment #6
1sp commentedPatch tested and confirmed to work. I have committed it to the source tree.