Changeset 59954 for trunk/src/wp-includes/media.php

Timestamp:

03/09/2025 09:14:28 AM (12 months ago)

Author:

audrasjb

Message:

Media: Improve HTML5 compliance of wp_video_shortcode() boolean attributes.

This changeset updates wp_video_shortcode() to improve boolean attributes handling in accordance with HTML5 standards. Technically, it replaces attr="1" with attr for the loop, autoplay and muted attributes. The preload attribute is also updated to accept only allowed values: none, metadata, and auto. If a value outside of this list is provided, it will be ignored, preventing invalid attribute outputs.

Props jongycastillo, sabernhardt, joedolson, audrasjb, shub07, debarghyabanerjee.
Fixes #60178.

File:

• trunk/src/wp-includes/media.php (modified) (1 diff)

trunk/src/wp-includes/media.php

@@ -3483 +3483 @@
     $attr_strings = array();
 
-    foreach ( $html_atts as $k => $v ) {
-        $attr_strings[] = $k . '="' . esc_attr( $v ) . '"';
+    foreach ( $html_atts as $attribute_name => $attribute_value ) {
+        if ( in_array( $attribute_name, array( 'loop', 'autoplay', 'muted' ), true ) && true === $attribute_value ) {
+            // Add boolean attributes without their value for true.
+            $attr_strings[] = esc_attr( $attribute_name );
+        } elseif ( 'preload' === $attribute_name && ! empty( $attribute_value ) ) {
+            // Handle the preload attribute with specific allowed values.
+            $allowed_preload_values = array( 'none', 'metadata', 'auto' );
+            if ( in_array( $attribute_value, $allowed_preload_values, true ) ) {
+                $attr_strings[] = sprintf( '%s="%s"', esc_attr( $attribute_name ), esc_attr( $attribute_value ) );
+            }
+        } elseif ( ! empty( $attribute_value ) ) {
+            // For non-boolean attributes, add them with their value.
+            $attr_strings[] = sprintf( '%s="%s"', esc_attr( $attribute_name ), esc_attr( $attribute_value ) );
+        }
     }