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

Timestamp:

12/11/2023 12:09:42 PM (10 months ago)

Author:

SergeyBiryukov

Message:

Docs: Consistently document the $body_id global as a string.

Includes declaring the global at the beginning of wp_iframe() and iframe_header().

Follow-up to [32642], [32643].

Props mukesh27, upadalavipul.
Fixes #60032.

File:

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

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

@@ -528 +528 @@
  *              by adding it to the function signature.
  *
- * @global int $body_id
+ * @global string $body_id
  *
  * @param callable $content_func Function that outputs the content.
@@ -534 +534 @@
  */
 function wp_iframe( $content_func, ...$args ) {
+    global $body_id;
+
     _wp_admin_html_begin();
     ?>
@@ -604 +606 @@
     $body_id_attr = '';
 
-    if ( isset( $GLOBALS['body_id'] ) ) {
-        $body_id_attr = ' id="' . $GLOBALS['body_id'] . '"';
+    if ( isset( $body_id ) ) {
+        $body_id_attr = ' id="' . $body_id . '"';
     }