Changeset 39932 for trunk/src/wp-includes/option.php

Timestamp:

01/20/2017 05:32:13 PM (8 years ago)

Author:

flixos90

Message:

Multisite: Do not check for is_super_admin() when trying to set user settings.

The checks were introduced in [22256] to prevent user settings to be set for super admins that were not a member of the current site. However the latter should apply to any kind of user, so the is_super_admin() check is redundant. Furthermore, removing these checks is necessary for the ongoing effort to get rid of is_super_admin() checks in general.

Props chandrapatel for initial patch.
Fixes #39199. See #37616.

File:

• trunk/src/wp-includes/option.php (modified) (2 diffs)

trunk/src/wp-includes/option.php

@@ -809 +809 @@
     }
 
-    if ( is_super_admin() && ! is_user_member_of_blog() ) {
+    if ( ! is_user_member_of_blog() ) {
         return;
     }
@@ -973 +973 @@
     }
 
-    if ( is_super_admin() && ! is_user_member_of_blog() ) {
+    if ( ! is_user_member_of_blog() ) {
         return;
     }