Changeset 6133

Timestamp:

09/19/2007 12:51:21 AM (18 years ago)

Author:

ryan

Message:

Script loader cleanup from mdawaffe. fixes #5003

File:

• trunk/wp-includes/script-loader.php (modified) (5 diffs)

trunk/wp-includes/script-loader.php

@@ -3 +3 @@
     var $scripts = array();
     var $queue = array();
+    var $to_print = array();
     var $printed = array();
     var $args = array();
@@ -155 +156 @@
      */
     function print_scripts( $handles = false ) {
+        global $wp_db_version;
+
         // Print the queue if nothing is passed.  If a string is passed, print that script.  If an array is passed, print those scripts.
         $handles = false === $handles ? $this->queue : (array) $handles;
-        $handles = $this->all_deps( $handles );
-        $this->_print_scripts( $handles );
-        return $this->printed;
-    }
-
-    /**
-     * Internally used helper function for printing script tags
-     *
-     * @param array handles Hierarchical array of scripts to be printed
-     * @see WP_Scripts::all_deps()
-     */
-    function _print_scripts( $handles ) {
-        global $wp_db_version;
-
-        foreach( array_keys($handles) as $handle ) {
-            if ( !$handles[$handle] )
-                return;
-            elseif ( is_array($handles[$handle]) )
-                $this->_print_scripts( $handles[$handle] );
+        $this->all_deps( $handles );
+
+        $to_print = apply_filters( 'print_scripts_array', array_keys($this->to_print) );
+
+        foreach( $to_print as $handle ) {
             if ( !in_array($handle, $this->printed) && isset($this->scripts[$handle]) ) {
                 if ( $this->scripts[$handle]->src ) { // Else it defines a group.
@@ -196 +185 @@
             }
         }
+
+        $this->to_print = array();
+        return $this->printed;
     }
 
@@ -220 +212 @@
      * Determines dependencies of scripts
      *
-     * Recursively builds hierarchical array of script dependencies.  Does NOT catch infinite loops.
+     * Recursively builds array of scripts to print taking dependencies into account.  Does NOT catch infinite loops.
      *
      * @param mixed handles Accepts (string) script name or (array of strings) script names
      * @param bool recursion Used internally when function calls itself
-     * @return array Hierarchical array of dependencies
      */
     function all_deps( $handles, $recursion = false ) {
-        if ( ! $handles = (array) $handles )
-            return array();
-        $return = array();
+        if ( !$handles = (array) $handles )
+            return false;
+
         foreach ( $handles as $handle ) {
             $handle = explode('?', $handle);
@@ -235 +226 @@
                 $this->args[$handle[0]] = $handle[1];
             $handle = $handle[0];
-            if ( is_null($return[$handle]) ) // Prime the return array with $handles
-                $return[$handle] = true;
-            if ( $this->scripts[$handle]->deps ) {
-                if ( false !== $return[$handle] && array_diff($this->scripts[$handle]->deps, array_keys($this->scripts)) )
-                    $return[$handle] = false; // Script required deps which don't exist
+
+            if ( isset($this->to_print[$handle]) ) // Already grobbed it and its deps
+                continue;
+
+            $keep_going = true;
+            if ( !isset($this->scripts[$handle]) )
+                $keep_going = false; // Script doesn't exist
+            elseif ( $this->scripts[$handle]->deps && array_diff($this->scripts[$handle]->deps, array_keys($this->scripts)) )
+                $keep_going = false; // Script requires deps which don't exist (not a necessary check.  efficiency?)
+            elseif ( $this->scripts[$handle]->deps && !$this->all_deps( $this->scripts[$handle]->deps, true ) )
+                $keep_going = false; // Script requires deps which don't exist
+
+            if ( !$keep_going ) { // Either script or its deps don't exist.
+                if ( $recursion )
+                    return false; // Abort this branch.
                 else
-                    $return[$handle] = $this->all_deps( $this->scripts[$handle]->deps, true ); // Build the hierarchy
-            }
-            if ( $recursion && false === $return[$handle] )
-                return false; // Cut the branch
+                    continue; // We're at the top level.  Move on to the next one.
+            }                   
+
+            $this->to_print[$handle] = true;
         }
-        return $return;
+
+        return true;
     }