Index: includes/theme.inc
===================================================================
RCS file: /cvs/drupal/drupal/includes/theme.inc,v
retrieving revision 1.426
diff -u -r1.426 theme.inc
--- includes/theme.inc	6 Jun 2008 01:50:20 -0000	1.426
+++ includes/theme.inc	7 Jun 2008 04:58:23 -0000
@@ -158,7 +158,7 @@
 
   // Add scripts used by this theme.
   foreach ($final_scripts as $script) {
-    drupal_add_js($script, 'theme');
+    drupal_add_js($script, array('weight' => JS_THEME_WEIGHT));
   }
 
   $theme_engine = NULL;
@@ -1284,7 +1284,7 @@
 
   // Add sticky headers, if applicable.
   if (count($header)) {
-    drupal_add_js('misc/tableheader.js');
+    drupal_add_js('misc/tableheader.js', array('weight' => JS_CORE_WEIGHT));
     // Add 'sticky-enabled' class to the table to identify it for JS.
     // This is needed to target tables constructed by this function.
     $attributes['class'] = empty($attributes['class']) ? 'sticky-enabled' : ($attributes['class'] . ' sticky-enabled');
@@ -1401,7 +1401,7 @@
  * Returns a header cell for tables that have a select all functionality.
  */
 function theme_table_select_header_cell() {
-  drupal_add_js('misc/tableselect.js');
+  drupal_add_js('misc/tableselect.js', array('weight' => JS_CORE_WEIGHT));
 
   return array('class' => 'select-all');
 }
Index: includes/batch.inc
===================================================================
RCS file: /cvs/drupal/drupal/includes/batch.inc,v
retrieving revision 1.16
diff -u -r1.16 batch.inc
--- includes/batch.inc	6 Jun 2008 16:53:48 -0000	1.16
+++ includes/batch.inc	7 Jun 2008 04:58:22 -0000
@@ -73,7 +73,7 @@
   // and the initialization and error messages.
   $current_set = _batch_current_set();
   drupal_set_title($current_set['title']);
-  drupal_add_js('misc/progress.js', 'core', 'header', FALSE, FALSE);
+  drupal_add_js('misc/progress.js', array('cache' => FALSE, 'weight' => JS_CORE_WEIGHT));
 
   $url = url($batch['url'], array('query' => array('id' => $batch['id'])));
   $js_setting = array(
@@ -84,7 +84,7 @@
     ),
   );
   drupal_add_js($js_setting, 'setting');
-  drupal_add_js('misc/batch.js', 'core', 'header', FALSE, FALSE);
+  drupal_add_js('misc/batch.js', array('cache' => FALSE, 'weight' => JS_CORE_WEIGHT));
 
   $output = '<div id="progress"></div>';
   return $output;
Index: includes/form.inc
===================================================================
RCS file: /cvs/drupal/drupal/includes/form.inc,v
retrieving revision 1.272
diff -u -r1.272 form.inc
--- includes/form.inc	6 May 2008 12:18:45 -0000	1.272
+++ includes/form.inc	7 Jun 2008 04:58:23 -0000
@@ -1488,7 +1488,7 @@
  */
 function theme_fieldset($element) {
   if ($element['#collapsible']) {
-    drupal_add_js('misc/collapse.js');
+    drupal_add_js('misc/collapse.js', array('weight' => JS_CORE_WEIGHT));
 
     if (!isset($element['#attributes']['class'])) {
       $element['#attributes']['class'] = '';
@@ -1786,8 +1786,8 @@
   // Adding the same javascript settings twice will cause a recursion error,
   // we avoid the problem by checking if the javascript has already been added.
   if (isset($element['#ahah']['path']) && isset($element['#ahah']['event']) && !isset($js_added[$element['#id']])) {
-    drupal_add_js('misc/jquery.form.js');
-    drupal_add_js('misc/ahah.js');
+    drupal_add_js('misc/jquery.form.js', array('weight' => JS_CORE_WEIGHT));
+    drupal_add_js('misc/ahah.js', array('weight' => JS_CORE_WEIGHT));
 
     $ahah_binding = array(
       'url'      => url($element['#ahah']['path']),
@@ -1812,7 +1812,7 @@
 
     // Add progress.js if we're doing a bar display.
     if ($ahah_binding['progress']['type'] == 'bar') {
-      drupal_add_js('misc/progress.js');
+      drupal_add_js('misc/progress.js', array('weight' => JS_CORE_WEIGHT));
     }
 
     drupal_add_js(array('ahah' => array($element['#id'] => $ahah_binding)), 'setting');
@@ -2000,7 +2000,7 @@
   $output = '';
 
   if ($element['#autocomplete_path']) {
-    drupal_add_js('misc/autocomplete.js');
+    drupal_add_js('misc/autocomplete.js', array('weight' => JS_CORE_WEIGHT));
     $class[] = 'form-autocomplete';
     $extra =  '<input class="autocomplete" type="hidden" id="' . $element['#id'] . '-autocomplete" value="' . check_url(url($element['#autocomplete_path'], array('absolute' => TRUE))) . '" disabled="disabled" />';
   }
@@ -2052,16 +2052,14 @@
 
   // Add teaser behavior (must come before resizable)
   if (!empty($element['#teaser'])) {
-    drupal_add_js('misc/teaser.js');
-    // Note: arrays are merged in drupal_get_js().
-    drupal_add_js(array('teaserCheckbox' => array($element['#id'] => $element['#teaser_checkbox'])), 'setting');
-    drupal_add_js(array('teaser' => array($element['#id'] => $element['#teaser'])), 'setting');
+    drupal_add_js('misc/teaser.js', array('weight' => JS_CORE_WEIGHT));
+    drupal_add_js(array('teaserCheckbox' => array($element['#id'] => $element['#teaser_checkbox']), 'teaser' => array($element['#id'] => $element['#teaser'])), 'setting');
     $class[] = 'teaser';
   }
 
   // Add resizable behavior
   if ($element['#resizable'] !== FALSE) {
-    drupal_add_js('misc/textarea.js');
+    drupal_add_js('misc/textarea.js', array('weight' => JS_CORE_WEIGHT));
     $class[] = 'resizable';
   }
 
Index: includes/common.inc
===================================================================
RCS file: /cvs/drupal/drupal/includes/common.inc,v
retrieving revision 1.770
diff -u -r1.770 common.inc
--- includes/common.inc	30 May 2008 17:41:51 -0000	1.770
+++ includes/common.inc	7 Jun 2008 04:58:22 -0000
@@ -25,6 +25,26 @@
 define('SAVED_DELETED', 3);
 
 /**
+ * Weight of JavaScript for files loaded from core (files in /misc/).
+ */
+define('JS_CORE_WEIGHT', -20);
+
+/**
+ * Weight of JavaScript files loaded by modules.
+ */
+define('JS_MODULE_WEIGHT', 0);
+
+/**
+ * Weight of JavaScript files loaded by themes.
+ */
+define('JS_THEME_WEIGHT', 20);
+
+/**
+ * Weight of JavaScript libraries needed by other scripts, eg. jQuery.
+ */
+define('JS_LIBRARY_WEIGHT', -100);
+
+/**
  * Set content for a specified region.
  *
  * @param $region
@@ -1430,9 +1450,9 @@
 function l($text, $path, $options = array()) {
   // Merge in defaults.
   $options += array(
-      'attributes' => array(),
-      'html' => FALSE,
-    );
+    'attributes' => array(),
+    'html' => FALSE,
+  );
 
   // Append active class.
   if ($path == $_GET['q'] || ($path == '<front>' && drupal_is_front_page())) {
@@ -1894,16 +1914,17 @@
  * reference to an existing file or as inline code. The following actions can be
  * performed using this function:
  *
- * - Add a file ('core', 'module' and 'theme'):
- *   Adds a reference to a JavaScript file to the page. JavaScript files
- *   are placed in a certain order, from 'core' first, to 'module' and finally
- *   'theme' so that files, that are added later, can override previously added
- *   files with ease.
+ * - Add a file ('file'):
+ *   Adds a reference to a JavaScript file to the page and supports files on
+ *   external servers. JavaScript files are placed in a certain order depending
+ *   on their given weight. Files from core, modules, and themes have predefined
+ *   constants for their weights of JS_CORE_WEIGHT (-20), JS_MODULE_WEIGHT (0),
+ *   and JS_THEME_WEIGHT (20).
  *
  * - Add inline JavaScript code ('inline'):
  *   Executes a piece of JavaScript code on the current page by placing the code
- *   directly in the page. This can, for example, be useful to tell the user that
- *   a new message arrived, by opening a pop up, alert box etc.
+ *   directly in the page. This can, for example, be useful to tell the user
+ *   that a new message arrived, by opening a pop up, alert box, etc.
  *
  * - Add settings ('setting'):
  *   Adds a setting to Drupal's global storage of JavaScript settings. Per-page
@@ -1911,92 +1932,176 @@
  *   will be accessible at Drupal.settings.
  *
  * @param $data
- *   (optional) If given, the value depends on the $type parameter:
- *   - 'core', 'module' or 'theme': Path to the file relative to base_path().
+ *   (optional) If given, the value depends on the $options parameter:
+ *   - 'file': Either the path to the file relative to base_path(), or the
+ *       absolute URL to the Javascript file on an external server.
  *   - 'inline': The JavaScript code that should be placed in the given scope.
  *   - 'setting': An array with configuration options as associative array. The
- *       array is directly placed in Drupal.settings. You might want to wrap your
- *       actual configuration settings in another variable to prevent the pollution
- *       of the Drupal.settings namespace.
- * @param $type
- *   (optional) The type of JavaScript that should be added to the page. Allowed
- *   values are 'core', 'module', 'theme', 'inline' and 'setting'. You
- *   can, however, specify any value. It is treated as a reference to a JavaScript
- *   file. Defaults to 'module'.
- * @param $scope
- *   (optional) The location in which you want to place the script. Possible
- *   values are 'header' and 'footer' by default. If your theme implements
- *   different locations, however, you can also use these.
- * @param $defer
- *   (optional) If set to TRUE, the defer attribute is set on the <script> tag.
- *   Defaults to FALSE. This parameter is not used with $type == 'setting'.
- * @param $cache
- *   (optional) If set to FALSE, the JavaScript file is loaded anew on every page
- *   call, that means, it is not cached. Defaults to TRUE. Used only when $type
- *   references a JavaScript file.
- * @param $preprocess
- *   (optional) Should this JS file be aggregated if this
- *   feature has been turned on under the performance section?
- * @return
- *   If the first parameter is NULL, the JavaScript array that has been built so
- *   far for $scope is returned. If the first three parameters are NULL,
- *   an array with all scopes is returned.
+ *       array is directly placed in Drupal.settings. You might want to wrap 
+ *       your actual configuration settings in another variable to prevent the 
+ *       pollution of the Drupal.settings namespace.
+ * @param $options
+ *   (optional) A string for the type ('file', 'inline', 'setting'), or an array
+ *   which can have any or all of the following keys (these are not valid with 
+ *   type => 'setting'):
+ *   - type
+ *       The type of JavaScript that should be added to the page. Allowed
+ *       values are 'file', 'inline' and 'setting'.
+ *   - scope
+ *       The location in which you want to place the script. Possible
+ *       values are 'header' and 'footer' by default. If your theme implements
+ *       different locations, however, you can also use these.
+ *   - weight
+ *       The weight determines the order of files in output. Lower weights mean
+ *       the scripts are loaded sooner. Default to JS_MODULE_WEIGHT if none 
+ *       specified. Constants are available as JS_CORE_WEIGHT (-20), 
+ *       JS_MODULE_WEIGHT (0), and JS_THEME_WEIGHT (20).
+ *   -  cache
+ *       If set to FALSE, the JavaScript file is loaded anew on every page
+ *       call, that means, it is not cached. Defaults to TRUE. Used only when 
+ *       $type references a JavaScript file.
+ *   - preprocess
+ *       Should this JS file be aggregated if this feature has been turned
+ *       on under the performance section.
+ *   - attributes
+ *       An array of attributes to be added to the script.  Run through 
+ *       drupal_attributes().
+ * @param $reset
+ *   Resets the currently loaded JavaScript.
+ * @return
+ *   An array of JavaScript files to include - the array has three keys:
+ *    - 'file'
+ *        An array of files keyed by path (the value is the $options that was 
+ *        passed in).
+ *    - 'inline'
+ *        An array of code to be put inline.  The key is the code and the value
+ *        is the $options.
+ *    - 'setting'
+ *        This one is different; it is keyed by scope (region to put the
+ *        JavaScript in). Inside each scope is an aggregation of the variables 
+ *        that were added to the settings.
  */
-function drupal_add_js($data = NULL, $type = 'module', $scope = 'header', $defer = FALSE, $cache = TRUE, $preprocess = TRUE) {
+function drupal_add_js($data = NULL, $options = array(), $reset = FALSE) {
   static $javascript = array();
 
-  if (isset($data)) {
-
-    // Add jquery.js and drupal.js, as well as the basePath setting, the
-    // first time a Javascript file is added.
-    if (empty($javascript)) {
-      $javascript['header'] = array(
-        'core' => array(
-          'misc/jquery.js' => array('cache' => TRUE, 'defer' => FALSE, 'preprocess' => TRUE),
-          'misc/drupal.js' => array('cache' => TRUE, 'defer' => FALSE, 'preprocess' => TRUE),
+  // Add jquery.js and drupal.js, as well as the basePath setting, the
+  // first time a Javascript file is added.
+  if (empty($javascript) || $reset) {
+    $javascript = array(
+      'file' => array(
+        'misc/jquery.js' => array(
+          'weight' => JS_LIBRARY_WEIGHT,
+          'scope' => 'header',
+          'cache' => TRUE,
+          'preprocess' => TRUE,
+          'attributes' => array('type' => 'text/javascript'),
+          'data' => 'misc/jquery.js',
         ),
-        'module' => array(),
-        'theme' => array(),
-        'setting' => array(
-          array('basePath' => base_path()),
+        'misc/drupal.js' => array(
+          'weight' => -99,
+          'scope' => 'header',
+          'cache' => TRUE,
+          'preprocess' => TRUE,
+          'attributes' => array('type' => 'text/javascript'),
+          'data' => 'misc/drupal.js',
         ),
-        'inline' => array(),
-      );
-    }
+      ),
+      'setting' => array(
+        'header' => array(
+          'basePath' => base_path(),
+        ),
+      ),
+      'inline' => array(),
+    );
+  }
+  
+  // Reset to default if necessary.
+  if ($reset && isset($data)) {
+    $javascript = $data;
+    return $javascript;
+  }
+  
 
-    if (isset($scope) && !isset($javascript[$scope])) {
-      $javascript[$scope] = array('core' => array(), 'module' => array(), 'theme' => array(), 'setting' => array(), 'inline' => array());
+  if (isset($data)) {
+    if (isset($options) && !is_array($options)) {
+      $options = array('type' => $options);
     }
 
-    if (isset($type) && isset($scope) && !isset($javascript[$scope][$type])) {
-      $javascript[$scope][$type] = array();
-    }
+    $options += array(
+      'type' => 'file',
+      'weight' => JS_MODULE_WEIGHT,
+      'scope' => 'header',
+      'cache' => TRUE,
+      'preprocess' => ((isset($options['cache']) && $options['cache']) || !isset($options['cache']) ? TRUE : FALSE),
+      'remove' => FALSE,
+      'data' => $data,
+      'attributes' => array('type' => 'text/javascript'),
+    );
 
+    $type = $options['type'];
+    unset($options['type']);
+    
+    // Remove given JavaScript file if it exists, otherwise do nothing.
+    if ($options['remove']) {
+    	if (isset($javascript[$type][$data])) {
+        unset($javascript[$type][$data]);
+    	}
+      return $javascript;
+    }
+    
     switch ($type) {
       case 'setting':
-        $javascript[$scope][$type][] = $data;
+        $javascript['setting'][$options['scope']] = $data + $javascript['setting'][$options['scope']];
         break;
       case 'inline':
-        $javascript[$scope][$type][] = array('code' => $data, 'defer' => $defer);
+        if (!isset($javascript['file'][$data])) {
+          $javascript['inline'][$data] = $options;
+          $sort = array();
+          foreach ($javascript['inline'] as $code => $settings) {
+            $sort[$code] = $settings['weight'];
+          }
+          $inline_data = $javascript['inline'];
+          array_multisort($sort, SORT_NUMERIC, $inline_data);
+          $javascript['inline'] = $inline_data;
+        }
+        break;
+      case 'file':
+        if (!isset($javascript['file'][$data]) && !$options['remove']) {
+          $javascript['file'][$data] = $options;
+          $sort = array();
+          foreach ($javascript['file'] as $uri => $settings) {
+            $sort[$uri] = $settings['weight'];
+          }
+          $files = $javascript['file'];
+          array_multisort($sort, SORT_ASC, $files);
+          $javascript['file'] = $files;
+        }
         break;
-      default:
-        // If cache is FALSE, don't preprocess the JS file.
-        $javascript[$scope][$type][$data] = array('cache' => $cache, 'defer' => $defer, 'preprocess' => (!$cache ? FALSE : $preprocess));
     }
   }
 
-  if (isset($scope)) {
+  return $javascript;
+}
 
-    if (isset($javascript[$scope])) {
-      return $javascript[$scope];
-    }
-    else {
-      return array();
-    }
-  }
-  else {
-    return $javascript;
-  }
+/**
+ * Removes a specified JavaScript include from the include list. This is only
+ * allowed for JavaScript files, as it matches based on file URL.
+ *
+ * @param $data
+ *   Same as for drupal_add_js.
+ * @param $type
+ *   (optional) The type of JavaScript that should be added to the page. Allowed
+ *   values are 'file', 'inline' and 'setting'.
+ * @return
+ *   The resulting call to drupal_add_js.
+ * @see drupal_add_js
+ */
+function drupal_remove_js($data, $type = 'file') {
+  $options = array(
+    'type' => $type,
+    'remove' => TRUE
+  );
+  return drupal_add_js($data, $options);
 }
 
 /**
@@ -2015,26 +2120,23 @@
  *   JavaScript array for the given scope.
  * @return
  *   All JavaScript code segments and includes for the scope as HTML tags.
+ * @see drupal_add_js
  */
 function drupal_get_js($scope = 'header', $javascript = NULL) {
-  if ((!defined('MAINTENANCE_MODE') || MAINTENANCE_MODE != 'update') && function_exists('locale_update_js_files')) {
-    locale_update_js_files();
-  }
 
   if (!isset($javascript)) {
-    $javascript = drupal_add_js(NULL, NULL, $scope);
+    $javascript = drupal_add_js();
   }
 
   if (empty($javascript)) {
     return '';
   }
 
-  $output = '';
-  $preprocessed = '';
-  $no_preprocess = array('core' => '', 'module' => '', 'theme' => '');
-  $files = array();
+  // Check if JS preprocessing is enabled.
   $preprocess_js = (variable_get('preprocess_js', FALSE) && (!defined('MAINTENANCE_MODE') || MAINTENANCE_MODE != 'update'));
+  // Get the file directory path.
   $directory = file_directory_path();
+  // Check whether the file directory path is writable.
   $is_writable = is_dir($directory) && is_writable($directory) && (variable_get('file_downloads', FILE_DOWNLOADS_PUBLIC) == FILE_DOWNLOADS_PUBLIC);
 
   // A dummy query-string is added to filenames, to gain control over
@@ -2045,43 +2147,126 @@
   // page request.
   $query_string = '?' . substr(variable_get('css_js_query_string', '0'), 0, 1);
 
+  // The first step is to format it nicely for altering.
+  $alterable_js = array();
+  if (!empty($javascript['file'])) {
+    $files = array();
+    foreach ($javascript['file'] as $path => $settings) {
+      // Array of files to include in the preprocessor
+      if ($settings['scope'] == $scope) {
+        if (!$settings['preprocess'] || !$is_writable || !$preprocess_js) {
+          $alterable_js['file'][$path] = $settings;
+        }
+        else {
+          $files[$path] = $settings;
+        }
+      }
+    }
+    // Process the files to be pre-processed.
+    if ($is_writable && $preprocess_js && count($files) > 0) {
+      $filename = md5(serialize($files) . $query_string) . '.js';
+      $unprocessed = array();
+      $preprocess_file = drupal_build_js_cache($files, $filename, $unprocessed);
+      $alterable_js['file'][$preprocess_file] = array(
+        'weight' => 0,
+        'scope' => 'header',
+        'cache' => TRUE,
+        'preprocess' => TRUE,
+        'data' => $preprocess_file,
+        'attributes' => array(),
+      );
+      $alterable_js += $unprocessed;
+    }
+  }
+  
+  // Add any inline scripts defined for this scope.
+  if (!empty($javascript['inline'])) {
+    foreach ($javascript['inline'] as $code => $settings) {
+      if ($settings['scope'] == $scope) {
+        $alterable_js['inline'][$path] = $settings;
+      }
+    }
+  }
+  
+  // Call hook_js_alter() in all relevant modules.
+  $alterable_js['setting'] = (isset($javascript['setting'][$scope])? $javascript['setting'][$scope] : array());
+  drupal_alter('js', $alterable_js, $scope);
+  $javascript = $alterable_js;
+
+  $renderable_js = array();
   foreach ($javascript as $type => $data) {
 
     if (!$data) continue;
 
     switch ($type) {
       case 'setting':
-        $output .= '<script type="text/javascript">jQuery.extend(Drupal.settings, ' . drupal_to_js(call_user_func_array('array_merge_recursive', $data)) . ");</script>\n";
+        $renderable_js[] = array(
+          'type' => 'setting',
+          'data' => $data,
+        );
         break;
       case 'inline':
         foreach ($data as $info) {
-          $output .= '<script type="text/javascript"' . ($info['defer'] ? ' defer="defer"' : '') . '>' . $info['code'] . "</script>\n";
+          $renderable_js[] = array(
+            'type' => 'inline',
+            'data' => $info['data'],
+            'attributes' => $info['attributes'],
+          );
         }
         break;
       default:
-        // If JS preprocessing is off, we still need to output the scripts.
-        // Additionally, go through any remaining scripts if JS preprocessing is on and output the non-cached ones.
         foreach ($data as $path => $info) {
-          if (!$info['preprocess'] || !$is_writable || !$preprocess_js) {
-            $no_preprocess[$type] .= '<script type="text/javascript"' . ($info['defer'] ? ' defer="defer"' : '') . ' src="' . base_path() . $path . ($info['cache'] ? $query_string : '?' . time()) . "\"></script>\n";
+          // Create the source of the Javscript, checking if base_path()
+          // is required, as well as placing the dummy query string at the
+          // end of the path. For external files, check if they should be cached
+          // and do that, using that new path as the source. Done after 
+          // hook_js_alter() so it's easier for scripts to be removed.
+          if (valid_url($path, TRUE)) {
+			      $filename = md5($path) . '.js';
+			      $unprocessed = array();
+			      $preprocess_file = drupal_build_js_cache($files, $filename, $unprocessed);
+            $info['attributes']['src'] = $preprocess_file;
           }
           else {
-            $files[$path] = $info;
+            $info['attributes']['src'] = base_path() . $path . ($info['cache'] ? $query_string : '?' . time());
           }
+          $renderable_js[] = array(
+            'type' => 'file',
+            'attributes' => $info['attributes'],
+            'path' => $path,
+          );
         }
     }
   }
+  return theme('scripts', $renderable_js, $scope);
+}
 
-  // Aggregate any remaining JS files that haven't already been output.
-  if ($is_writable && $preprocess_js && count($files) > 0) {
-    $filename = md5(serialize($files) . $query_string) . '.js';
-    $preprocess_file = drupal_build_js_cache($files, $filename);
-    $preprocessed .= '<script type="text/javascript" src="' . base_path() . $preprocess_file . '"></script>' . "\n";
-  }
+/**
+ * Themes a list of scripts, which are then rendered to the page.
+ *
+ * @param $scripts
+ *   An array of scripts to be output.
+ * @param $scope
+ *   The scope of the JavaScript: where it's going to be located
+ *   on the page.
+ * @return
+ *   HTML containing the script tags.
+ */
+function theme_scripts($scripts, $scope) {
+  $output = '';
 
-  // Keep the order of JS files consistent as some are preprocessed and others are not.
-  // Make sure any inline or JS setting variables appear last after libraries have loaded.
-  $output = $preprocessed . implode('', $no_preprocess) . $output;
+  foreach ($scripts as $script) {
+    switch ($script['type']) {
+      case 'setting':
+        $output .= '<script type="text/javascript">jQuery.extend(Drupal.settings, ' . drupal_to_js($script['data']) . ");</script>\n";
+        break;
+      case 'inline':
+        $output .= '<script' . drupal_attributes($script['attributes']) . '>' . $script['data'] . "</script>\n";
+        break;
+      case 'file':
+        $output .= '<script' . drupal_attributes($script['attributes']) . "></script>\n";
+    }
+  }
 
   return $output;
 }
@@ -2196,7 +2381,7 @@
 function drupal_add_tabledrag($table_id, $action, $relationship, $group, $subgroup = NULL, $source = NULL, $hidden = TRUE, $limit = 0) {
   static $js_added = FALSE;
   if (!$js_added) {
-    drupal_add_js('misc/tabledrag.js', 'core');
+    drupal_add_js('misc/tabledrag.js', array('weight' => -20));
     $js_added = TRUE;
   }
 
@@ -2221,10 +2406,12 @@
  *   An array of JS files to aggregate and compress into one file.
  * @param $filename
  *   The name of the aggregate JS file.
+ * @param $unprocessed
+ *   An array that holds the paths of any files that couldn't be processed.
  * @return
  *   The name of the JS file.
  */
-function drupal_build_js_cache($files, $filename) {
+function drupal_build_js_cache($files, $filename, &$unprocessed) {
   $contents = '';
 
   // Create the js/ within the files folder.
@@ -2235,8 +2422,22 @@
     // Build aggregate JS file.
     foreach ($files as $path => $info) {
       if ($info['preprocess']) {
-        // Append a ';' after each JS file to prevent them from running together.
-        $contents .= file_get_contents($path) . ';';
+        // Allow Javascript to be loaded from external sources.
+        if (valid_url($path, TRUE)) {
+          // Request the external Javascript resource.
+          $request = drupal_http_request($path);
+          if ($request->status == 200) {
+            $contents .= $request->data . ';';
+          }
+          else {
+          	$unprocessed[$path] = $info;
+          }
+        }
+        else {
+          // Append a ';' after each JS file to prevent them from running 
+          // together.
+          $contents .= file_get_contents($path) . ';';
+        }
       }
     }
 
@@ -2256,9 +2457,14 @@
 }
 
 /**
- * Converts a PHP variable into its Javascript equivalent.
+ * Converts a PHP variable into its Javascript equivalent.  We use
+ * HTML-safe strings, i.e. with <, > and & escaped.
  *
- * We use HTML-safe strings, i.e. with <, > and & escaped.
+ * @param $var
+ *   The variable to encode.
+ * @return
+ *   A string, which is a JavaScript-encoded version of $var.
+ * @see drupal_json
  */
 function drupal_to_js($var) {
   // json_encode() does not escape <, > and &, so we do it with str_replace()
@@ -2273,6 +2479,7 @@
  *
  * @param $var
  *   (optional) If set, the variable will be converted to JSON and output.
+ * @see drupal_to_js
  */
 function drupal_json($var = NULL) {
   // We are returning JavaScript, so tell the browser.
@@ -3015,6 +3222,10 @@
     'form_element' => array(
       'arguments' => array('element' => NULL, 'value' => NULL),
     ),
+    // theme_scripts() exists in form.inc.
+    'scripts' => array(
+      'arguments' => array('scripts' => NULL, 'scope' => NULL),
+    ),
   );
 }
 
Index: modules/user/user.admin.inc
===================================================================
RCS file: /cvs/drupal/drupal/modules/user/user.admin.inc,v
retrieving revision 1.22
diff -u -r1.22 user.admin.inc
--- modules/user/user.admin.inc	7 May 2008 19:34:24 -0000	1.22
+++ modules/user/user.admin.inc	7 Jun 2008 04:58:25 -0000
@@ -83,7 +83,7 @@
     );
   }
 
-  drupal_add_js('misc/form.js', 'core');
+  drupal_add_js('misc/form.js', array('weight' => JS_CORE_WEIGHT));
 
   return $form;
 }
Index: modules/user/user.module
===================================================================
RCS file: /cvs/drupal/drupal/modules/user/user.module,v
retrieving revision 1.909
diff -u -r1.909 user.module
--- modules/user/user.module	26 May 2008 17:12:55 -0000	1.909
+++ modules/user/user.module	7 Jun 2008 04:58:26 -0000
@@ -2111,7 +2111,7 @@
   global $user;
   // Only need to do once per page.
   if (!$complete) {
-    drupal_add_js(drupal_get_path('module', 'user') . '/user.js', 'module');
+    drupal_add_js(drupal_get_path('module', 'user') . '/user.js');
 
     drupal_add_js(array(
       'password' => array(
Index: modules/system/system.test
===================================================================
RCS file: /cvs/drupal/drupal/modules/system/system.test,v
retrieving revision 1.5
diff -u -r1.5 system.test
--- modules/system/system.test	2 Jun 2008 17:32:19 -0000	1.5
+++ modules/system/system.test	7 Jun 2008 04:58:25 -0000
@@ -295,3 +295,195 @@
     }
   }
 }
+
+class CommonIncTest extends DrupalWebTestCase {
+  /**
+   * Implementation of getInfo().
+   */
+  function getInfo() {
+    return array(
+      'name' => t('Common.inc tests'),
+      'description' => t('Tests for common.inc'),
+      'group' => t('Core'),
+    );
+  }
+
+  function test_drupal_add_js() {
+  	
+  	// Clear out all cached JavaScript files.
+  	drupal_clear_js_cache();
+  	
+    // Reset the JavaScript.
+    $original_js = drupal_add_js(NULL, array(), TRUE);
+
+    // Default JavaScript array.
+    $default_js = array(
+      'file' => array(
+        'misc/jquery.js' => array(
+          'weight' => -100,
+          'scope' => 'header',
+          'cache' => TRUE,
+          'preprocess' => TRUE,
+          'attributes' => array('type' => 'text/javascript'),
+          'data' => 'misc/jquery.js',
+        ),
+        'misc/drupal.js' => array(
+          'weight' => -99,
+          'scope' => 'header',
+          'cache' => TRUE,
+          'preprocess' => TRUE,
+          'attributes' => array('type' => 'text/javascript'),
+          'data' => 'misc/drupal.js',
+        ),
+      ),
+      'setting' => array(
+        'header' => array(
+          'basePath' => base_path(),
+        ),
+      ),
+      'inline' => array(),
+    );
+
+    // Second time - make sure it retains the defaults.
+    $this->assertIdentical(drupal_add_js(), $default_js, t('Test drupal_add_js() without parameters.'));
+
+    // Try adding a duplicate file.
+    drupal_add_js('misc/jquery.js');
+    $this->assertIdentical(drupal_add_js(), $default_js, t('Adding a duplicate file was ignored.'));
+
+    // Try the reset parameter.
+    drupal_add_js('misc/'. $this->randomName(8) . '.js');
+    drupal_add_js(NULL, array(), TRUE);
+    $this->assertIdentical(drupal_add_js(), $default_js, t('Reset actually reset the JavaScript.'));
+
+    // Test the weight system with files.
+    $weights = array(1, 10, -10);
+    $files = array();
+    $sort = array();
+    foreach ($weights as $weight) {
+      $js_file = 'misc/' . $this->randomName(8) . '.js';
+      $files[$js_file] = array(
+        'weight' => $weight,
+        'scope' => 'header',
+        'cache' => TRUE,
+        'preprocess' => TRUE,
+        'remove' => FALSE,
+        'data' => $js_file,
+        'attributes' => array('type' => 'text/javascript'),
+      );
+      $sort[$js_file] = $weight;
+    }
+    
+    foreach ($files as $file) {
+      drupal_add_js($file['data'], array('weight' => $file['weight']));
+    }
+    array_multisort($sort, SORT_NUMERIC, $files);
+
+    // Make a copy of the default JS.
+    $new_js = $default_js;
+    $new_js['file'] += $files;
+    $this->assertIdentical(drupal_add_js(), $new_js, t('The weights system works with files.'));
+
+    // Test deleting with $options['remove'] = TRUE.
+    $file = array_shift($files);
+    $new_js = $default_js;
+    $new_js['file'] += $files;
+    drupal_add_js($file['data'], array('remove' => TRUE));
+    $this->assertIdentical(drupal_add_js(), $new_js, t('The JavaScript was deleted from the array with $options[\'remove\'] = TRUE.'));
+
+    // Test deleting with drupal_remove_js().
+    $file = array_shift($files);
+    $new_js = $default_js;
+    $new_js['file'] += $files;
+    drupal_remove_js($file['data']);
+    $this->assertIdentical(drupal_add_js(), $new_js, t('The JavaScript was deleted from the array with drupal_remove_js().'));
+
+    // Test resetting.  Although I can't trigger the if (empty($javascript)), I can trigger the $reset.
+    drupal_add_js(NULL, array(), TRUE);
+    $this->assertIdentical(drupal_add_js(), $default_js, t('The JavaScript was reset.'));
+    
+    // Test deleting without adding first.
+    $js_name = $this->randomName();
+    drupal_remove_js($js_name);
+    $final_js = drupal_add_js();
+    $this->assertFalse(isset($final_js['file'][$js_name]), t('A file that has not been added, but removed, doesn\'t show up.'));
+    
+    // Test settings.
+    $new_js = drupal_add_js(NULL, array(), TRUE);
+    
+    $settings = array();
+    $settings[$this->randomName()] = $this->randomName();
+    $settings[$this->randomName()] = array($this->randomName(), $this->randomName());
+    $settings[$this->randomName()] = array('test' => $this->randomName(), 'test2' => $this->randomName());
+    $new_js['setting']['header'] = $settings + $new_js['setting']['header'];
+    drupal_add_js($settings, 'setting');
+    $this->assertIdentical(drupal_add_js(), $new_js, t('Added settings with type => \'setting\'.'));
+
+    drupal_add_js($settings, 'setting');
+    $this->assertIdentical(drupal_add_js(), $new_js, t('Duplicate settings had no effect.'));
+
+    // Test adding settings with type => 'setting'.
+    $settings = array();
+    $settings[$this->randomName()] = $this->randomName();
+    $settings[$this->randomName()] = array($this->randomName(), $this->randomName());
+    $settings[$this->randomName()] = array('test' => $this->randomName(), 'test2' => $this->randomName());
+    $new_js['setting']['header'] = $settings + $new_js['setting']['header'];
+    drupal_add_js($settings, 'setting');
+    $this->assertIdentical(drupal_add_js(), $new_js, t('Added more settings with type => \'setting\'.'));
+
+    // Test inline.
+    $new_js = drupal_add_js(NULL, array(), TRUE);
+    drupal_add_js("$(function() { $('test').hide(); });", 'inline');
+    $new_js['inline']["$(function() { $('test').hide(); });"] = array(
+      'weight' => 0,
+      'scope' => 'header',
+      'cache' => TRUE,
+      'preprocess' => TRUE,
+      'remove' => FALSE,
+      'data' => "$(function() { $('test').hide(); });",
+      'attributes' => array('type' => 'text/javascript'),
+    );
+    $this->assertIdentical(drupal_add_js(), $new_js, t('Added inline JavaScript.'));
+    
+    // Test externally available JavaScript without caching.
+    drupal_clear_js_cache();
+    $new_js = drupal_add_js(NULL, array(), TRUE);
+    $filepath = url('misc/teaser.js', array('absolute' => TRUE));
+    $final_js = drupal_add_js($filepath, array('cache' => FALSE));
+//    var_dump(drupal_get_js('header', $final_js));
+    $this->assertTrue(isset($final_js['file'][$filepath]), t('Add an available external JavaScript file without caching.'));
+    
+    // Test externally unavailable JavaScript without caching.
+    drupal_clear_js_cache();
+    $new_js = drupal_add_js(NULL, array(), TRUE);
+    $filepath = url('no_real_file', array('absolute' => TRUE));
+    $final_js = drupal_add_js($filepath, array('cache' => FALSE));
+//    var_dump(drupal_get_js('header', $final_js));
+    $this->assertTrue(isset($final_js['file'][$filepath]), t('Add an unavailable external JavaScript file without caching.'));
+    
+    // Test externally available JavaScript with caching.
+    drupal_clear_js_cache();
+    $new_js = drupal_add_js(NULL, array(), TRUE);
+    $filepath = url('misc/teaser.js', array('absolute' => TRUE));
+    $final_js = drupal_add_js($filepath);
+//    var_dump(drupal_get_js('header', $final_js));
+    $this->assertTrue(isset($final_js['file'][md5($filepath)]) && !isset($final_js['file'][$filepath]), t('Add an available external JavaScript file with caching.'));
+    
+    // Test externally unavailable JavaScript with caching.
+    drupal_clear_js_cache();
+    $new_js = drupal_add_js(NULL, array(), TRUE);
+    $filepath = url('no_real_file', array('absolute' => TRUE));
+//    var_dump(drupal_get_js('header', $final_js));
+    $this->assertTrue(isset($final_js['file'][$filepath]), t('Add an unavailable external JavaScript file with caching.'));
+
+    // Test caching.
+    $new_js = drupal_add_js(NULL, array(), TRUE);
+    $file = $this->randomName();
+    $new_js = drupal_add_js($file, array('cache' => FALSE));
+    $this->assertFalse($new_js['files'][$file]['cache'], t('The setting to turn caching off went through.'));
+    $this->assertFalse($new_js['files'][$file]['preprocess'], t('The setting to turn caching off, in turn turning preprocess off, went through.'));
+
+    // Reset the JavaScript.
+    drupal_add_js($original_js, array(), TRUE);
+  }
+}
Index: modules/system/system.admin.inc
===================================================================
RCS file: /cvs/drupal/drupal/modules/system/system.admin.inc,v
retrieving revision 1.76
diff -u -r1.76 system.admin.inc
--- modules/system/system.admin.inc	10 May 2008 07:32:02 -0000	1.76
+++ modules/system/system.admin.inc	7 Jun 2008 04:58:24 -0000
@@ -1522,7 +1522,7 @@
  * @see system_date_time_settings_submit()
  */
 function system_date_time_settings() {
-  drupal_add_js(drupal_get_path('module', 'system') . '/system.js', 'module');
+  drupal_add_js(drupal_get_path('module', 'system') . '/system.js');
   drupal_add_js(array('dateTime' => array('lookup' => url('admin/settings/date-time/lookup'))), 'setting');
 
   // Date settings:
@@ -1730,7 +1730,7 @@
 
   if (!variable_get('clean_url', 0)) {
     if (strpos(request_uri(), '?q=') !== FALSE) {
-      drupal_add_js(drupal_get_path('module', 'system') . '/system.js', 'module');
+      drupal_add_js(drupal_get_path('module', 'system') . '/system.js');
 
       $form['clean_url']['#description'] .= ' <span>' . t('Before enabling clean URLs, you must perform a test to determine if your server is properly configured. If you are able to see this page again after clicking the "Run the clean URL test" link, the test has succeeded and the radio buttons above will be available. If instead you are directed to a "Page not found" error, you will need to change the configuration of your server. The <a href="@handbook">handbook page on Clean URLs</a> has additional troubleshooting information.', array('@handbook' => 'http://drupal.org/node/15365')) . '</span>';
 
Index: modules/simpletest/simpletest.module
===================================================================
RCS file: /cvs/drupal/drupal/modules/simpletest/simpletest.module,v
retrieving revision 1.3
diff -u -r1.3 simpletest.module
--- modules/simpletest/simpletest.module	10 May 2008 07:46:22 -0000	1.3
+++ modules/simpletest/simpletest.module	7 Jun 2008 04:58:24 -0000
@@ -81,7 +81,7 @@
 function simpletest_entrypoint() {
   simpletest_load();
   drupal_add_css(drupal_get_path('module', 'simpletest') . '/simpletest.css', 'module');
-  drupal_add_js(drupal_get_path('module', 'simpletest') . '/simpletest.js', 'module');
+  drupal_add_js(drupal_get_path('module', 'simpletest') . '/simpletest.js', array('weight' => JS_MODULE_WEIGHT));
   $output = drupal_get_form('simpletest_overview_form');
 
   if (simpletest_running_output()) {
Index: modules/locale/locale.module
===================================================================
RCS file: /cvs/drupal/drupal/modules/locale/locale.module,v
retrieving revision 1.217
diff -u -r1.217 locale.module
--- modules/locale/locale.module	23 Apr 2008 20:01:52 -0000	1.217
+++ modules/locale/locale.module	7 Jun 2008 04:58:23 -0000
@@ -491,6 +491,8 @@
 }
 
 /**
+ * Implementation of hook_js_alter().
+ *
  * Update JavaScript translation file, if required, and add it to the page.
  *
  * This function checks all JavaScript files currently added via drupal_add_js()
@@ -498,32 +500,33 @@
  * and Drupal.formatPlural() calls. Also refreshes the JavaScript translation
  * file if necessary, and adds it to the page.
  */
-function locale_update_js_files() {
+function locale_js_alter(&$javascript, $scope) {
   global $language;
 
+  // If there are no files set for loading, don't do anything.
+  if (!isset($javascript['file'])) {
+    return;
+  }
+
   $dir = file_create_path(variable_get('locale_js_directory', 'languages'));
   $parsed = variable_get('javascript_parsed', array());
 
   // The first three parameters are NULL in order to get an array with all
   // scopes. This is necessary to prevent recreation of JS translation files
   // when new files are added for example in the footer.
-  $javascript = drupal_add_js(NULL, NULL, NULL);
   $files = $new_files = FALSE;
 
-  foreach ($javascript as $scope) {
-    foreach ($scope as $type => $data) {
-      if ($type != 'setting' && $type != 'inline') {
-        foreach ($data as $filepath => $info) {
-          $files = TRUE;
-          if (!in_array($filepath, $parsed)) {
-            // Don't parse our own translations files.
-            if (substr($filepath, 0, strlen($dir)) != $dir) {
-              locale_inc_callback('_locale_parse_js_file', $filepath);
-              watchdog('locale', 'Parsed JavaScript file %file.', array('%file' => $filepath));
-              $parsed[] = $filepath;
-              $new_files = TRUE;
-            }
-          }
+  foreach ($javascript['file'] as $filepath => $info) {
+    // Don't parse external files.
+    if (!valid_url($filepath, TRUE)) {
+      $files = TRUE;
+      if (!in_array($filepath, $parsed)) {
+        // Don't parse our own translations files.
+        if (substr($filepath, 0, strlen($dir)) != $dir) {
+          locale_inc_callback('_locale_parse_js_file', $filepath);
+          watchdog('locale', 'Parsed JavaScript file %file.', array('%file' => $filepath));
+          $parsed[] = $filepath;
+          $new_files = TRUE;
         }
       }
     }
@@ -554,7 +557,12 @@
 
   // Add the translation JavaScript file to the page.
   if ($files && !empty($language->javascript)) {
-    drupal_add_js($dir . '/' . $language->language . '_' . $language->javascript . '.js', 'core');
+    $translation_file = $dir . '/' . $language->language . '_' . $language->javascript . '.js';
+    $javascript['file'][$translation_file] = array(
+      'scope' => $scope,
+      'cache' => FALSE,
+      'preprocess' => FALSE
+    );
   }
 }
 
Index: modules/color/color.module
===================================================================
RCS file: /cvs/drupal/drupal/modules/color/color.module,v
retrieving revision 1.41
diff -u -r1.41 color.module
--- modules/color/color.module	19 May 2008 19:36:41 -0000	1.41
+++ modules/color/color.module	7 Jun 2008 04:58:23 -0000
@@ -154,7 +154,7 @@
 
   // Add Farbtastic color picker.
   drupal_add_css('misc/farbtastic/farbtastic.css', 'module', 'all', FALSE);
-  drupal_add_js('misc/farbtastic/farbtastic.js');
+  drupal_add_js('misc/farbtastic/farbtastic.js', array('weight' => JS_CORE_WEIGHT));
 
   // Add custom CSS and JS.
   drupal_add_css($base . '/color.css', 'module', 'all', FALSE);
Index: modules/block/block-admin-display-form.tpl.php
===================================================================
RCS file: /cvs/drupal/drupal/modules/block/block-admin-display-form.tpl.php,v
retrieving revision 1.6
diff -u -r1.6 block-admin-display-form.tpl.php
--- modules/block/block-admin-display-form.tpl.php	15 May 2008 21:30:02 -0000	1.6
+++ modules/block/block-admin-display-form.tpl.php	7 Jun 2008 04:58:23 -0000
@@ -26,7 +26,7 @@
 ?>
 <?php
   // Add table javascript.
-  drupal_add_js('misc/tableheader.js');
+  drupal_add_js('misc/tableheader.js', array('weight' => JS_CORE_WEIGHT));
   drupal_add_js(drupal_get_path('module', 'block') . '/block.js');
   foreach ($block_regions as $region => $title) {
     drupal_add_tabledrag('blocks', 'match', 'sibling', 'block-region-select', 'block-region-' . $region, NULL, FALSE);
Index: modules/node/node.admin.inc
===================================================================
RCS file: /cvs/drupal/drupal/modules/node/node.admin.inc,v
retrieving revision 1.22
diff -u -r1.22 node.admin.inc
--- modules/node/node.admin.inc	14 Apr 2008 17:48:38 -0000	1.22
+++ modules/node/node.admin.inc	7 Jun 2008 04:58:24 -0000
@@ -259,7 +259,7 @@
     $form['filters']['buttons']['reset'] = array('#type' => 'submit', '#value' => t('Reset'));
   }
 
-  drupal_add_js('misc/form.js', 'core');
+  drupal_add_js('misc/form.js', array('weight' => JS_CORE_WEIGHT));
 
   return $form;
 }
Index: install.php
===================================================================
RCS file: /cvs/drupal/drupal/install.php,v
retrieving revision 1.119
diff -u -r1.119 install.php
--- install.php	26 May 2008 17:12:54 -0000	1.119
+++ install.php	7 Jun 2008 04:58:21 -0000
@@ -711,10 +711,20 @@
 
       // Add JavaScript validation.
       _user_password_dynamic_validation();
-      drupal_add_js(drupal_get_path('module', 'system') . '/system.js', 'module');
+      drupal_add_js(drupal_get_path('module', 'system') . '/system.js');
       // We add these strings as settings because JavaScript translation does not
       // work on install time.
-      drupal_add_js(array('copyFieldValue' => array('edit-site-mail' => array('edit-account-mail')), 'cleanURL' => array('success' => st('Your server has been successfully tested to support this feature.'), 'failure' => st('Your system configuration does not currently support this feature. The <a href="http://drupal.org/node/15365">handbook page on Clean URLs</a> has additional troubleshooting information.'), 'testing' => st('Testing clean URLs...'))), 'setting');
+      $js_settings = array(
+        'copyFieldValue' => array(
+          'edit-site-mail' => array('edit-account-mail')
+        ),
+        'cleanURL' => array(
+          'success' => st('Your server has been successfully tested to support this feature.'),
+          'failure' => st('Your system configuration does not currently support this feature. The <a href="http://drupal.org/node/15365">handbook page on Clean URLs</a> has additional troubleshooting information.'),
+          'testing' => st('Testing clean URLs...')
+        )
+      );
+      drupal_add_js($js_settings, 'setting');
       drupal_add_js('
 // Global Killswitch
 if (Drupal.jsEnabled) {