Changeset 12826 for trunk/src/bp-core/admin/bp-core-admin-tools.php

Timestamp:

12/18/2020 12:54:41 PM (4 years ago)

Author:

slaffik

Message:

Tools: add WP MultiSite checks for "Repair site tracking site icons" operation

On regular sites checking and running the "Repair site tracking site icons" operation on the Tools > BuddyPress page generates a fatal error due to the get_sites() function usage, which is MultiSite specific. So, hide the operation on non-multisite and properly return an error when ran directly.

Fixes #8414 (Trunk)

File:

• trunk/src/bp-core/admin/bp-core-admin-tools.php (modified) (2 diffs)

trunk/src/bp-core/admin/bp-core-admin-tools.php

@@ -145 +145 @@
         );
 
-        if ( bp_is_active( 'blogs', 'site-icon' ) ) {
+        if ( is_multisite() && bp_is_active( 'blogs', 'site-icon' ) ) {
             $repair_list[91] = array(
                 'bp-blog-site-icons',
@@ -332 +332 @@
     /* translators: %s: the result of the action performed by the repair tool */
     $statement = __( 'Repairing site icons/blog avatars synchronization… %s', 'buddypress' );
+
+    if ( ! is_multisite() ) {
+        return array( 0, sprintf( $statement, __( 'Failed!', 'buddypress' ) ) );
+    }
 
     // Run function if blogs component is active.