Changeset 49677

Timestamp:

11/22/2020 01:01:46 PM (4 years ago)

Author:

azaozz

Message:

Upgrade/Install: Replace the conditionals that check the AUTOMATIC_UPDATER_DISABLED constant and the automatic_updater_disabled filter in update-core.php with a call to WP_Automatic_Updater::is_disabled(). This prevents a PHP warning, fixes the logic, and considers wp_is_file_mod_allowed( 'automatic_updater' ) when determining the UI state.

Props jamesros161, pbiron, audrasjb, azaozz.
Fixes #51827.

File:

• trunk/src/wp-admin/update-core.php (modified) (3 diffs)

trunk/src/wp-admin/update-core.php

@@ -292 +292 @@
     }
 
+    require_once ABSPATH . 'wp-admin/includes/class-wp-upgrader.php';
+    $updater = new WP_Automatic_Updater();
+
     // Defaults:
     $upgrade_dev   = get_site_option( 'auto_update_core_dev', 'enabled' ) === 'enabled';
@@ -324 +327 @@
     }
 
-    if ( defined( 'AUTOMATIC_UPDATER_DISABLED' )
-        || has_filter( 'automatic_updater_disabled' )
-    ) {
-        if ( true === AUTOMATIC_UPDATER_DISABLED
-            /** This filter is documented in wp-admin/includes/class-wp-automatic-updater.php */
-            || true === apply_filters( 'automatic_updater_disabled', false )
-        ) {
-            $upgrade_dev   = false;
-            $upgrade_minor = false;
-            $upgrade_major = false;
-        }
-        // The UI is overridden by the AUTOMATIC_UPDATER_DISABLED constant.
+    if ( $updater->is_disabled() ) {
+        $upgrade_dev   = false;
+        $upgrade_minor = false;
+        $upgrade_major = false;
+
+        // The UI is overridden by the AUTOMATIC_UPDATER_DISABLED constant
+        // or the automatic_updater_disabled filter,
+        // or by wp_is_file_mod_allowed( 'automatic_updater' ).
+        // See WP_Automatic_Updater::is_disabled().
         $can_set_update_option = false;
     }
@@ -371 +371 @@
     <p class="auto-update-status">
         <?php
-        require_once ABSPATH . 'wp-admin/includes/class-wp-upgrader.php';
-        $updater = new WP_Automatic_Updater();
+
         if ( $updater->is_vcs_checkout( ABSPATH ) ) {
             _e( 'This site appears to be under version control. Automatic updates are disabled.' );