Changeset 52338 for trunk/tests/phpunit/tests/dependencies.php

Timestamp:

12/07/2021 05:44:46 PM (3 years ago)

Author:

audrasjb

Message:

Script Loader: Allow for wp_register_script() to be called after wp_enqueue_script().

When a plugin registers styles/scripts on wp_enqueue_scripts (as plugin authors are encouraged to do), and conditionally enqueues their script/style on the_content filter, things "just work". In block themes, the_content is run prior to the header being processed, which results in the above scenario failing.

This change makes a wp_enqueue_script( 'example' ); wp_register_script( 'example' ); work, where as currently the enqueue silently fails (no "doing it wrong" message) and the following register has no impact. Scripts can therefore be enqueued and dequeued (by "handle") before they are registered.

Fixes #54529.

File:

• trunk/tests/phpunit/tests/dependencies.php (modified) (1 diff)

trunk/tests/phpunit/tests/dependencies.php

@@ -137 +137 @@
 
     }
+
+    function test_enqueue_before_register() {
+        $dep = new WP_Dependencies;
+
+        $this->assertArrayNotHasKey( 'one', $dep->registered );
+
+        $dep->enqueue( 'one' );
+
+        $this->assertNotContains( 'one', $dep->queue );
+
+        $this->assertTrue( $dep->add( 'one', '' ) );
+
+        $this->assertContains( 'one', $dep->queue );
+    }
 }