Changeset 39600 for trunk/src/wp-admin/network/site-themes.php

Timestamp:

12/14/2016 04:17:38 AM (9 years ago)

Author:

johnbillion

Message:

General: Use interpolation instead of concatenation for all dynamic hook names.

This fixes the rendering of the hook names on developer.wordpress.org.

Props keesiemeijer
Fixes #39148

File:

• trunk/src/wp-admin/network/site-themes.php (modified) (3 diffs)

trunk/src/wp-admin/network/site-themes.php

@@ -125 +125 @@
                 $themes = (array) $_POST['checked'];
                 $n = count( $themes );
+                $screen = get_current_screen()->id;
+
                 /**
                  * Fires when a custom bulk action should be handled.
@@ -130 +132 @@
                  * The redirect link should be modified with success or failure feedback
                  * from the action to be used to display feedback to the user.
+                 *
+                 * The dynamic portion of the hook name, `$screen`, refers to the current screen ID.
                  *
                  * @since 4.7.0
@@ -136 +140 @@
                  * @param string $action       The action being taken.
                  * @param array  $items        The items to take the action on.
-                 * @param int    $site_id      The site id.
+                 * @param int    $site_id      The site ID.
                  */
-                $referer = apply_filters( 'handle_network_bulk_actions-' . get_current_screen()->id, $referer, $action, $themes, $id );
+                $referer = apply_filters( "handle_network_bulk_actions-{$screen}", $referer, $action, $themes, $id );
             } else {
                 $action = 'error';