Changeset 34118 for branches/4.3/src/wp-admin/network/site-new.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-new.php (modified) (1 diff)

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

@@ -80 +80 @@
             wp_die( __( 'There was an error creating the user.' ) );
         else
-            wp_new_user_notification( $user_id, 'both' );
+            wp_new_user_notification( $user_id, null, 'both' );
     }