Convert register_pending_approval_admin email to config system and provide the ability to configure it in UI.

  body: "[user:name] has applied for an account. [user:edit-url]"

This is not precisely the same as it was in Drupal 7. It should be:

  body: "[user:name] has applied for an account.\n\n[user:edit-url]"

Just added the "\n\n"s..

I'm so confused about this. Where does this ever gets called? Can it be set in the UI? Where?

It's not actually set in the UI anywhere but after a bit of a trace I think I've found where it gets called.

register_pending_approval_admin is the mail key set when admin approval is required for user registration and the mail is going to the admin. This is called from the user module.
The drupal_mail function calls any functions which are of the type $module . '_mail' (user_mail function).

Within user_mail the subject and body are created by calling _user_mail_text with the $key (register_pending_approval_admin) followed by .subject or .body respectively.

_user_mail_text then does a config lookup with what is passed to it (register_pending_approval_admin.subject etc).

Perhaps then this should be a user configurable setting - to allow the email sent to admins when there is a new user awaiting approval to be customised?

I'm still relatively new to the whole CMI scene so my user_update_N function may not be correct. As far as I can tell though, that config does need to be set as it does get called.

It looks like this functionality is also being addressed by patch 809806-Register pending approval admin mails not configurable.patch in issue Register pending approval admin mails not configurable?.

No idea which is the better approach.

Please ignore, I was getting my versions confused.

The last section of the patch in #7 above failed to apply for me (the user.install changes)

It may need rerolling for updates to HEAD

+++ b/core/modules/user/user.installundefined
@@ -1042,3 +1042,20 @@ function _user_install_picture_field(array $settings = array()) {
+function user_update_8017() {
+  $config_key = 'user.mail';
+  $config = config($config_key);
+  $config_data = array(
+    'register_pending_approval_admin.body' => "[user:name] has applied for an account.\n\n[user:edit-url]",
+    'register_pending_approval_admin.subject' => 'Account details for [user:name] at [site:name] (pending admin approval)',
+  );
+  $config->set($config_key, $config_data);
+  $config->save();
+}

This function needs to migrate the variable from Drupal 7. There is a helper function to do this. See http://api.drupal.org/api/drupal/core!modules!user!user.install/function... for how to do this.

Actually this is a special case since the user.mail.yml in already updated by user_update_8006() so you can just add the two new variables to the conversion function.

in case @edrupal or anyone else wants to try a reroll, here is the doc page for rerolls http://drupal.org/patch/reroll

I've had a go at re-rolling the patch in #7. Also made the following changes:

1. Slight correction to the help text for the new field on the UI.
2. Moved the new variables from user_update_8017() to user_update_8006 as suggested by @alexpott (Have I understood what's needed correctly?)

Ed

Setting the issue status to 'needs review' will trigger the review bot to test the patch.

Looks good to me. Does not have upgrade test coverage, but I don't think that we need explicit coverage for every single trivial variable.

Per #17 removing tag

Note this is blocking #494518: Allow for custom user registration approval email address

#15: register_pending_approval_admin-1804926-15.patch queued for re-testing.

Since it had been 5 days, and d8 is moving fast, I used the re-test link on the most recent patch to have to testbot test it again. That way we can reroll it if needed.

Committed/pushed to 8.x, thanks!