Changeset 34118 for branches/4.3/src/wp-admin/network/site-users.php

Timestamp:

09/14/2015 01:02:03 PM (9 years ago)

Author:

ocean90

Message:

Passwords: Deprecate second parameter of wp_new_user_notification().

The second parameter $plaintext_pass was removed in [33023] and restored as $notify in [33620] with a different behavior. If you have a plugin overriding wp_new_user_notification() which hasn't been updated you would get a notification with your username and the password "both".
To prevent this the second parameter is now deprecated and reintroduced as the third parameter.

Adds unit tests.

Merge of [34116] to the 4.3 branch.

Props kraftbj, adamsilverstein, welcher, ocean90.
See #33654.

File:

• branches/4.3/src/wp-admin/network/site-users.php (modified) (1 diff)

branches/4.3/src/wp-admin/network/site-users.php

@@ -78 +78 @@
                     $update = 'err_new_dup';
                 } else {
-                    wp_new_user_notification( $user_id, 'both' );
+                    wp_new_user_notification( $user_id, null, 'both' );
                     add_user_to_blog( $id, $user_id, $_POST['new_role'] );
                     $update = 'newuser';