Changeset 9580 for trunk

Timestamp:

Mar 9, 2011, 11:17:14 AM (13 years ago)

Author:

patdenice

Message:

feature:2114
footer_script can use multiple required scripts (like combine_script).
automaticaly load know scripts.

File:

• trunk/include/template.class.php (modified) (5 diffs)

trunk/include/template.class.php

@@ -659 +659 @@
     if ( !empty($content) )
     { // second call
-      $this->scriptLoader->add_inline( $content, @$params['require'] );
+      $this->scriptLoader->add_inline(
+        $content,
+        empty($params['require']) ? array() : explode(',', $params['require'])
+      );
     }
   }
@@ -935 +938 @@
     if(!empty($require))
     {
-      if(!isset($this->registered_scripts[$require]))
-        fatal_error("inline script not found require $require");
-      $s = $this->registered_scripts[$require];
-      if($s->load_mode==2)
-        $s->load_mode=1; // until now the implementation does not allow executing inline script depending on another async script
+      foreach ($require as $id)
+      {
+        if(!isset($this->registered_scripts[$id]))
+          $this->load_known_required_script($id, 1) or fatal_error("inline script not found require $id");
+        $s = $this->registered_scripts[$id];
+        if($s->load_mode==2)
+          $s->load_mode=1; // until now the implementation does not allow executing inline script depending on another async script
+      }
     }
     $this->inline_scripts[] = $code;
@@ -974 +980 @@
       foreach (self::$ui_core_dependencies as $script_id => $required_ids)
         $this->add($script_id, $load_mode, $required_ids, null, $version);
+    }
+
+    // Try to load undefined required script
+    foreach ($script->precedents as $script_id)
+    {
+      if (! isset( $this->registered_scripts[$script_id] ) )
+        $this->load_known_required_script($script_id, $load_mode);
     }
   }
@@ -1114 +1127 @@
 
         if ( empty($script->path) )
-          $script->path = dirname(self::$known_paths['jquery.ui'])."/$id.min.js";
+          $script->path = dirname(self::$known_paths['jquery.effects'])."/$id.min.js";
       }
 
@@ -1123 +1136 @@
       }
     }
+  }
+
+  private function load_known_required_script($id, $load_mode)
+  {
+    if ( isset(self::$known_paths[$id]) or strncmp($id, 'jquery.ui.', 10)==0 or strncmp($id, 'jquery.effetcs.', 15)==0 )
+    {
+      $this->add($id, $load_mode, array(), null);
+      return true;
+    }
+    return false;
   }