Changeset 47857

Timestamp:

05/26/2020 06:01:18 PM (3 years ago)

Author:

whyisjake

Message:

Coding Standards: Change auto-update filters to be more consistent.

• send_theme_auto_update_email 👉 auto_theme_update_send_email
• wp_plugins_auto_update_enabled 👉 plugins_auto_update_enabled
• wp_themes_auto_update_enabled 👉 themes_auto_update_enabled

Want to make sure that @ronalfy gets props for his work in #50052 too.

See #50052.
Props: ronalfy, pbiron, azaozz, audrasjb, whyisjake.

Location:

trunk/src/wp-admin/includes

Files:

• class-wp-automatic-updater.php (modified) (1 diff)
• update.php (modified) (2 diffs)

trunk/src/wp-admin/includes/class-wp-automatic-updater.php

@@ -903 +903 @@
          * @param bool $enabled True if notifications are enabled, false otherwise.
          */
-        $notifications_enabled = apply_filters( 'send_theme_auto_update_email', true );
+        $notifications_enabled = apply_filters( 'auto_theme_update_send_email', true );
 
         if ( ! empty( $update_results['theme'] ) && $notifications_enabled ) {

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

@@ -958 +958 @@
              * @param bool $enabled True if plugins auto-update is enabled, false otherwise.
              */
-            return apply_filters( 'wp_plugins_auto_update_enabled', true );
+            return apply_filters( 'plugins_auto_update_enabled', true );
         case 'theme':
             /**
@@ -967 +967 @@
              * @param bool True if themes auto-update is enabled, false otherwise.
              */
-            return apply_filters( 'wp_themes_auto_update_enabled', true );
+            return apply_filters( 'themes_auto_update_enabled', true );
     }