Load User action by email

you need to create the patch form the rules root directory http://drupal.org/patch/create

And you need to provide the changes in one patch, please.

+++ rules/modules/user.rules.inc	Mon Jan 19 23:26:36 1970
@@ -162,7 +162,7 @@
+      'eval input' => array('userid', 'username','mail'),

missing white space before 'mail'

Powered by Dreditor.

any testers? :)

@code:
"User mail", "mail", "email" - please be consistent with whatever label you use. Best do it the same way as core.

oh
>+ $userarray = array('name' => $settings['username'], 'uid' => $settings['userid'], 'mail' => $settings['mail']);

This will generate a notice for old $settings without 'mail'.

I just used the patch in #13 and it worked fine for what I needed it for. Thanks!

subscribing

+++ rules/modules/user.rules.inc	2011-04-16 08:56:16.000000000 -0400
@@ -215,7 +215,10 @@ function rules_action_user_removerole($u
+  if ($settings['mail'])
+    $userarray = array('name' => $settings['username'], 'uid' => $settings['userid']);
+  else
+    $userarray = array('name' => $settings['username'], 'uid' => $settings['userid'], 'mail' => $settings['mail']);

please use braces as required by the Drupal coding standards. Furthermore you have to use isset(), because the 'mail' array key may not be set and could generate a notice. And I think you swapped the if/else branches by mistake.

And please create one patch file that contains all changes.

And please create patches in the new -p1 format, see http://drupal.org/patch/create ;-)

Powered by Dreditor.

#28: load_user_action_by_email-932484.patch queued for re-testing.

but maybe others want to pick this up.

This feature is available in Rules in D7 and D8, and won't be implemented in the now-obsolete D6 version of Rules.