WordPress.org

Make WordPress Core

Changeset 48552


Ignore:
Timestamp:
07/21/2020 06:42:24 PM (6 months ago)
Author:
SergeyBiryukov
Message:

Site Health: Simplify the logic for displaying whether auto-updates are enabled for plugins and themes.

Remove some extra variables.

Follow-up to [48546], [48549].

See #50663.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/class-wp-debug-data.php

    r48549 r48552  
    915915
    916916        foreach ( $plugins as $plugin_path => $plugin ) {
    917             $auto_updates_enabled_string  = __( 'Auto-updates enabled' );
    918             $auto_updates_disabled_string = __( 'Auto-updates disabled' );
    919 
    920917            $plugin_part = ( is_plugin_active( $plugin_path ) ) ? 'wp-plugins-active' : 'wp-plugins-inactive';
    921918
     
    952949            if ( $auto_updates_enabled ) {
    953950                if ( in_array( $plugin_path, $auto_updates, true ) ) {
    954                     $enabled = true;
     951                    $auto_updates_string = __( 'Auto-updates enabled' );
     952                    $enabled             = true;
    955953
    956954                    /**
     
    959957                     * @since 5.5.0
    960958                     *
    961                      * @param string $auto_updates_enabled_string The string output for the auto-updates column.
    962                      * @param array  $plugin                      An array of plugin data.
    963                      * @param bool   $enabled                     True if auto-updates are enabled for this item,
    964                      *                                            false otherwise.
     959                     * @param string $auto_updates_string The string output for the auto-updates column.
     960                     * @param array  $plugin              An array of plugin data.
     961                     * @param bool   $enabled             Whether auto-updates are enabled for this item.
    965962                     */
    966                     $auto_updates_enabled_string = apply_filters( 'plugin_auto_update_debug_string', $auto_updates_enabled_string, $plugin, $enabled );
    967 
    968                     $plugin_version_string       .= ' | ' . $auto_updates_enabled_string;
    969                     $plugin_version_string_debug .= ', ' . $auto_updates_enabled_string;
     963                    $auto_updates_string = apply_filters( 'plugin_auto_update_debug_string', $auto_updates_string, $plugin, $enabled );
    970964                } else {
    971                     $enabled = false;
     965                    $auto_updates_string = __( 'Auto-updates disabled' );
     966                    $enabled             = false;
    972967
    973968                    /** This filter is documented in wp-admin/includes/class-wp-debug-data.php */
    974                     $auto_updates_disabled_string = apply_filters( 'plugin_auto_update_debug_string', $auto_updates_disabled_string, $plugin, $enabled );
    975 
    976                     $plugin_version_string       .= ' | ' . $auto_updates_disabled_string;
    977                     $plugin_version_string_debug .= ', ' . $auto_updates_disabled_string;
     969                    $auto_updates_string = apply_filters( 'plugin_auto_update_debug_string', $auto_updates_string, $plugin, $enabled );
    978970                }
     971
     972                $plugin_version_string       .= ' | ' . $auto_updates_string;
     973                $plugin_version_string_debug .= ', ' . $auto_updates_string;
    979974            }
    980975
     
    10731068            ),
    10741069        );
     1070
    10751071        if ( $auto_updates_enabled ) {
    10761072            if ( in_array( $active_theme->stylesheet, $auto_updates, true ) ) {
    1077                 $auto_updates_enabled_string = __( 'Enabled' );
    1078                 $enabled                     = true;
     1073                $auto_updates_string = __( 'Enabled' );
     1074                $enabled             = true;
    10791075
    10801076                /**
     
    10831079                 * @since 5.5.0
    10841080                 *
    1085                  * @param string $auto_updates_enabled_string The string output for the auto-updates column.
    1086                  * @param object $theme                       An object of theme data.
    1087                  * @param bool   $enabled                     True if auto-updates are enabled for this item,
    1088                  *                                            false otherwise.
     1081                 * @param string $auto_updates_string The string output for the auto-updates column.
     1082                 * @param object $theme               An object of theme data.
     1083                 * @param bool   $enabled             Whether auto-updates are enabled for this item.
    10891084                 */
    1090                 $auto_updates_enabled_string = apply_filters( 'theme_auto_update_debug_string', $auto_updates_enabled_string, $active_theme, $enabled );
    1091 
     1085                $auto_updates_string = apply_filters( 'theme_auto_update_debug_string', $auto_updates_string, $active_theme, $enabled );
    10921086            } else {
    1093                 $auto_updates_disabled_string = __( 'Disabled' );
    1094                 $enabled                      = false;
     1087                $auto_updates_string = __( 'Disabled' );
     1088                $enabled             = false;
    10951089
    10961090                /** This filter is documented in wp-admin/includes/class-wp-debug-data.php */
    1097                 $auto_updates_disabled_string = apply_filters( 'theme_auto_update_debug_string', $auto_updates_disabled_string, $active_theme, $enabled );
    1098 
    1099             }
    1100             $theme_auto_update_string = $enabled ? $auto_updates_enabled_string : $auto_updates_disabled_string;
     1091                $auto_updates_string = apply_filters( 'theme_auto_update_debug_string', $auto_updates_string, $active_theme, $enabled );
     1092            }
    11011093
    11021094            $info['wp-active-theme']['fields']['auto_update'] = array(
    11031095                'label' => __( 'Auto-update' ),
    1104                 'value' => $theme_auto_update_string,
    1105                 'debug' => $theme_auto_update_string,
    1106             );
    1107         }
     1096                'value' => $auto_updates_string,
     1097                'debug' => $auto_updates_string,
     1098            );
     1099        }
     1100
    11081101        $parent_theme = $active_theme->parent();
    11091102
     
    11791172                continue;
    11801173            }
    1181 
    1182             $auto_updates_enabled_string  = __( 'Auto-updates enabled' );
    1183             $auto_updates_disabled_string = __( 'Auto-updates disabled' );
    11841174
    11851175            $theme_version = $theme->version;
     
    12181208            if ( $auto_updates_enabled ) {
    12191209                if ( in_array( $theme_slug, $auto_updates, true ) ) {
    1220                     $enabled = true;
     1210                    $auto_updates_string = __( 'Auto-updates enabled' );
     1211                    $enabled             = true;
    12211212
    12221213                    /** This filter is documented in wp-admin/includes/class-wp-debug-data.php */
    1223                     $auto_updates_enabled_string = apply_filters( 'theme_auto_update_debug_string', $auto_updates_enabled_string, $theme, $enabled );
    1224 
    1225                     $theme_version_string       .= ' | ' . $auto_updates_enabled_string;
    1226                     $theme_version_string_debug .= ',' . $auto_updates_enabled_string;
     1214                    $auto_updates_string = apply_filters( 'theme_auto_update_debug_string', $auto_updates_string, $theme, $enabled );
    12271215                } else {
    1228                     $enabled = false;
     1216                    $auto_updates_string = __( 'Auto-updates disabled' );
     1217                    $enabled             = false;
    12291218
    12301219                    /** This filter is documented in wp-admin/includes/class-wp-debug-data.php */
    1231                     $auto_updates_disabled_string = apply_filters( 'theme_auto_update_debug_string', $auto_updates_disabled_string, $theme, $enabled );
    1232 
    1233                     $theme_version_string       .= ' | ' . $auto_updates_disabled_string;
    1234                     $theme_version_string_debug .= ', ' . $auto_updates_disabled_string;
     1220                    $auto_updates_string = apply_filters( 'theme_auto_update_debug_string', $auto_updates_string, $theme, $enabled );
    12351221                }
     1222
     1223                $theme_version_string       .= ' | ' . $auto_updates_string;
     1224                $theme_version_string_debug .= ',' . $auto_updates_string;
    12361225            }
    12371226
Note: See TracChangeset for help on using the changeset viewer.