Changeset 38307 for trunk/src/wp-admin/includes/media.php

Timestamp:

08/22/2016 06:24:48 PM (9 years ago)

Author:

DrewAPicture

Message:

Hooks: Standardize naming of dynamic hooks to use interpolation vs concatenation.

Benefits gained in discoverability and self-documentation throughout core trump the negligible performance hit in using interpolation in hook names.

Props ramiy.
See #37748.

File:

• trunk/src/wp-admin/includes/media.php (modified) (2 diffs)

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

@@ -619 +619 @@
      * @param string $upload_iframe_src The upload iframe source URL by type.
      */
-    $upload_iframe_src = apply_filters( $type . '_upload_iframe_src', $upload_iframe_src );
+    $upload_iframe_src = apply_filters( "{$type}_upload_iframe_src", $upload_iframe_src );
 
     return add_query_arg('TB_iframe', true, $upload_iframe_src);
@@ -787 +787 @@
              * @param string $title Media title.
              */
-            $html = apply_filters( $type . '_send_to_editor_url', $html, esc_url_raw( $src ), $title );
+            $html = apply_filters( "{$type}_send_to_editor_url", $html, esc_url_raw( $src ), $title );
         } else {
             $align = '';