Changeset 61323

Timestamp:

11/30/2025 12:54:26 AM (3 months ago)

Author:

westonruter

Message:

Script Loader: Emit notices when enqueueing a script, style, or script module with missing dependencies.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/10545

Follow-up to [60999].

Props deepakprajapati, westonruter.
See #63486.
Fixes #64229.

Location:

trunk

Files:

• src/wp-includes/class-wp-dependencies.php (modified) (3 diffs)
• src/wp-includes/class-wp-script-modules.php (modified) (2 diffs)
• src/wp-includes/class-wp-scripts.php (modified) (1 diff)
• src/wp-includes/class-wp-styles.php (modified) (1 diff)
• tests/phpunit/tests/dependencies/scripts.php (modified) (1 diff)
• tests/phpunit/tests/dependencies/styles.php (modified) (1 diff)
• tests/phpunit/tests/script-modules/wpScriptModules.php (modified) (2 diffs)

trunk/src/wp-includes/class-wp-dependencies.php

@@ -104 +104 @@
      */
     private $queued_before_register = array();
+
+
+
+
+
+
+
+
+
+
+
+
 
     /**
@@ -200 +212 @@
             }
 
-            $keep_going = true;
+            $keep_going           = true;
+            $missing_dependencies = array();
+            if ( isset( $this->registered[ $handle ] ) && count( $this->registered[ $handle ]->deps ) > 0 ) {
+                $missing_dependencies = array_diff( $this->registered[ $handle ]->deps, array_keys( $this->registered ) );
+            }
             if ( ! isset( $this->registered[ $handle ] ) ) {
                 $keep_going = false; // Item doesn't exist.
-            } elseif ( $this->registered[ $handle ]->deps && array_diff( $this->registered[ $handle ]->deps, array_keys( $this->registered ) ) ) {
+            } elseif ( count( $missing_dependencies ) > 0 ) {
+                if ( ! in_array( $handle, $this->dependencies_with_missing_dependencies, true ) ) {
+                    _doing_it_wrong(
+                        get_class( $this ) . '::add',
+                        $this->get_dependency_warning_message( $handle, $missing_dependencies ),
+                        '7.0.0'
+                    );
+                    $this->dependencies_with_missing_dependencies[] = $handle;
+                }
                 $keep_going = false; // Item requires dependencies that don't exist.
             } elseif ( $this->registered[ $handle ]->deps && ! $this->all_deps( $this->registered[ $handle ]->deps, true, $new_group ) ) {
@@ -536 +560 @@
         return 'W/"' . md5( $etag ) . '"';
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }

trunk/src/wp-includes/class-wp-script-modules.php

@@ -70 +70 @@
         'high',
     );
+
+
+
+
+
+
+
+
+
+
+
 
     /**
@@ -723 +734 @@
 
         // If the item requires dependencies that do not exist, fail.
-        if ( count( array_diff( $dependency_ids, array_keys( $this->registered ) ) ) > 0 ) {
+        $missing_dependencies = array_diff( $dependency_ids, array_keys( $this->registered ) );
+        if ( count( $missing_dependencies ) > 0 ) {
+            if ( ! in_array( $id, $this->modules_with_missing_dependencies, true ) ) {
+                _doing_it_wrong(
+                    get_class( $this ) . '::register',
+                    sprintf(
+                        /* translators: 1: Script module ID, 2: Comma-separated list of missing dependency IDs. */
+                        __( 'The script module with the ID "%1$s" was enqueued with dependencies that are not registered: %2$s.' ),
+                        $id,
+                        implode( ', ', $missing_dependencies )
+                    ),
+                    '7.0.0'
+                );
+                $this->modules_with_missing_dependencies[] = $id;
+            }
+
             return false;
         }

trunk/src/wp-includes/class-wp-scripts.php

@@ -1165 +1165 @@
         $this->ext_handles    = '';
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }

trunk/src/wp-includes/class-wp-styles.php

@@ -494 +494 @@
         $this->print_html     = '';
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }

trunk/tests/phpunit/tests/dependencies/scripts.php

@@ -4094 +4094 @@
         $this->assertStringNotContainsStringIgnoringCase( 'sourceURL=', $translations_script_data );
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }

trunk/tests/phpunit/tests/dependencies/styles.php

@@ -815 +815 @@
         $this->assertEqualHTML( $expected, $printed );
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }

trunk/tests/phpunit/tests/script-modules/wpScriptModules.php

@@ -2060 +2060 @@
         );
 
-        $deregister( array( 'b', 'c ' ) );
+        $deregister( array( 'b', 'c' ) );
 
         // Test that registered dependency in footer doesn't place dependant in footer.
@@ -2313 +2313 @@
         );
     }
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
 }