Changeset 18561 for trunk/wp-includes/functions.wp-styles.php

Timestamp:

08/17/2011 09:02:43 PM (12 years ago)

Author:

nacin

Message:

Call _doing_it_wrong() individually in wp_scripts and wp_styles functions. Partially reverts [18556], [18557], removes wp_styles_init(), wp_scripts_init(). fixes #11526.

File:

• trunk/wp-includes/functions.wp-styles.php (modified) (8 diffs)

trunk/wp-includes/functions.wp-styles.php

@@ -24 +24 @@
 
     global $wp_styles;
-    if ( !is_a($wp_styles, 'WP_Styles') ) {
+    if ( ! is_a( $wp_styles, 'WP_Styles' ) ) {
+        if ( ! did_action( 'init' ) )
+            _doing_it_wrong( __FUNCTION__, sprintf( __( 'Scripts and styles should not be registered or enqueued until the %1$s, %2$s, or %3$s hooks.' ),
+                '<code>wp_enqueue_scripts</code>', '<code>admin_enqueue_scripts</code>', '<code>init</code>' ), '3.3' );
+
         if ( !$handles )
             return array(); // No need to instantiate if nothing is there.
@@ -47 +51 @@
 function wp_add_inline_style( $handle, $data ) {
     global $wp_styles;
-    if ( !is_a($wp_styles, 'WP_Styles') )
-        return false;
+    if ( ! is_a( $wp_styles, 'WP_Styles' ) ) {
+        if ( ! did_action( 'init' ) )
+            _doing_it_wrong( __FUNCTION__, sprintf( __( 'Scripts and styles should not be registered or enqueued until the %1$s, %2$s, or %3$s hooks.' ),
+                '<code>wp_enqueue_scripts</code>', '<code>admin_enqueue_scripts</code>', '<code>init</code>' ), '3.3' );
+        $wp_styles = new WP_Styles();
+    }
 
     return $wp_styles->add_inline_style( $handle, $data );
@@ -71 +79 @@
 function wp_register_style( $handle, $src, $deps = array(), $ver = false, $media = 'all' ) {
     global $wp_styles;
-
-    wp_styles_init();
+    if ( ! is_a( $wp_styles, 'WP_Styles' ) ) {
+        if ( ! did_action( 'init' ) )
+            _doing_it_wrong( __FUNCTION__, sprintf( __( 'Scripts and styles should not be registered or enqueued until the %1$s, %2$s, or %3$s hooks.' ),
+                '<code>wp_enqueue_scripts</code>', '<code>admin_enqueue_scripts</code>', '<code>init</code>' ), '3.3' );
+        $wp_styles = new WP_Styles();
+    }
 
     $wp_styles->add( $handle, $src, $deps, $ver, $media );
@@ -88 +100 @@
 function wp_deregister_style( $handle ) {
     global $wp_styles;
-
-    wp_styles_init();
+    if ( ! is_a( $wp_styles, 'WP_Styles' ) ) {
+        if ( ! did_action( 'init' ) )
+            _doing_it_wrong( __FUNCTION__, sprintf( __( 'Scripts and styles should not be registered or enqueued until the %1$s, %2$s, or %3$s hooks.' ),
+                '<code>wp_enqueue_scripts</code>', '<code>admin_enqueue_scripts</code>', '<code>init</code>' ), '3.3' );
+        $wp_styles = new WP_Styles();
+    }
 
     $wp_styles->remove( $handle );
@@ -115 +131 @@
 function wp_enqueue_style( $handle, $src = false, $deps = array(), $ver = false, $media = 'all' ) {
     global $wp_styles;
-
-    wp_styles_init();
+    if ( ! is_a( $wp_styles, 'WP_Styles' ) ) {
+        if ( ! did_action( 'init' ) )
+            _doing_it_wrong( __FUNCTION__, sprintf( __( 'Scripts and styles should not be registered or enqueued until the %1$s, %2$s, or %3$s hooks.' ),
+                '<code>wp_enqueue_scripts</code>', '<code>admin_enqueue_scripts</code>', '<code>init</code>' ), '3.3' );
+        $wp_styles = new WP_Styles();
+    }
 
     if ( $src ) {
@@ -133 +153 @@
 function wp_dequeue_style( $handle ) {
     global $wp_styles;
-
-    wp_styles_init();
+    if ( ! is_a( $wp_styles, 'WP_Styles' ) ) {
+        if ( ! did_action( 'init' ) )
+            _doing_it_wrong( __FUNCTION__, sprintf( __( 'Scripts and styles should not be registered or enqueued until the %1$s, %2$s, or %3$s hooks.' ),
+                '<code>wp_enqueue_scripts</code>', '<code>admin_enqueue_scripts</code>', '<code>init</code>' ), '3.3' );
+        $wp_styles = new WP_Styles();
+    }
 
     $wp_styles->dequeue( $handle );
@@ -153 +177 @@
 function wp_style_is( $handle, $list = 'queue' ) {
     global $wp_styles;
-
-    wp_styles_init();
+    if ( ! is_a( $wp_styles, 'WP_Styles' ) ) {
+        if ( ! did_action( 'init' ) )
+            _doing_it_wrong( __FUNCTION__, sprintf( __( 'Scripts and styles should not be registered or enqueued until the %1$s, %2$s, or %3$s hooks.' ),
+                '<code>wp_enqueue_scripts</code>', '<code>admin_enqueue_scripts</code>', '<code>init</code>' ), '3.3' );
+        $wp_styles = new WP_Styles();
+    }
 
     $query = $wp_styles->query( $handle, $list );
@@ -163 +191 @@
     return $query;
 }
-
-/**
- * Initializes $wp_styles global (if it hasn't already been initialized by a faulty plugin or theme).
- *
- * @since 3.3
- */
-function wp_styles_init() {
-    global $wp_styles;
-    static $done = false;
-
-    if ( !$done && !is_a($wp_styles, 'WP_Styles') ) {
-        if ( !did_action('after_setup_theme') )
-            _doing_it_wrong( __FUNCTION__, __( '$wp_styles should not be accessed before the "init" hook.' ), '3.3' );
-
-        $wp_styles = new WP_Styles();
-        $done = true;
-    }
-}
-