Changeset 47857

Timestamp:

05/26/2020 06:01:18 PM (6 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 );
         }