Index: includes/common.inc =================================================================== RCS file: /cvs/drupal/drupal/includes/common.inc,v retrieving revision 1.756.2.74 diff -u -F '^f' -r1.756.2.74 common.inc --- includes/common.inc 16 Dec 2009 20:47:10 -0000 1.756.2.74 +++ includes/common.inc 2 Feb 2010 15:29:19 -0000 @@ -1870,15 +1870,15 @@ function drupal_get_css($css = NULL) { // If a CSS file is not to be preprocessed and it's a module CSS file, it needs to *always* appear at the *top*, // regardless of whether preprocessing is on or off. if (!$preprocess && $type == 'module') { - $no_module_preprocess .= ''."\n"; + $no_module_preprocess .= ''."\n"; } // If a CSS file is not to be preprocessed and it's a theme CSS file, it needs to *always* appear at the *bottom*, // regardless of whether preprocessing is on or off. else if (!$preprocess && $type == 'theme') { - $no_theme_preprocess .= ''."\n"; + $no_theme_preprocess .= ''."\n"; } else { - $output .= ''."\n"; + $output .= ''."\n"; } } } @@ -1890,7 +1890,7 @@ function drupal_get_css($css = NULL) { // starting with "ad*". $filename = 'css_'. md5(serialize($types) . $query_string) .'.css'; $preprocess_file = drupal_build_css_cache($types, $filename); - $output .= ''."\n"; + $output .= ''."\n"; } } @@ -2224,7 +2224,7 @@ function drupal_get_js($scope = 'header' // 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] .= '\n"; + $no_preprocess[$type] .= '\n"; } else { $files[$path] = $info; @@ -2239,7 +2239,7 @@ function drupal_get_js($scope = 'header' // starting with "ad*". $filename = 'js_'. md5(serialize($files) . $query_string) .'.js'; $preprocess_file = drupal_build_js_cache($files, $filename); - $preprocessed .= ''."\n"; + $preprocessed .= ''."\n"; } // Keep the order of JS files consistent as some are preprocessed and others are not. Index: includes/file.inc =================================================================== RCS file: /cvs/drupal/drupal/includes/file.inc,v retrieving revision 1.121.2.9 diff -u -F '^f' -r1.121.2.9 file.inc --- includes/file.inc 16 Sep 2009 19:34:14 -0000 1.121.2.9 +++ includes/file.inc 2 Feb 2010 15:29:22 -0000 @@ -35,19 +35,57 @@ /** * Create the download path to a file. * - * @param $path A string containing the path of the file to generate URL for. - * @return A string containing a URL that can be used to download the file. + * There are two kinds of local files: + * - "created files", i.e. those in the files directory (which is stored in + * the file_directory_path variable and can be retrieved using + * file_directory_path()). These are files that have either been uploaded by + * users or were generated automatically (for example through CSS + * aggregation). + * - "shipped files", i.e. those outside of the files directory, which ship as + * part of Drupal core or contributed modules or themes. + * + * @param $path + * A string containing the Drupal path (i.e. path relative to the Drupal + * root directory) of the file to generate the URL for. + * @return + * A string containing a URL that can be used to download the file. */ function file_create_url($path) { - // Strip file_directory_path from $path. We only include relative paths in urls. - if (strpos($path, file_directory_path() .'/') === 0) { - $path = trim(substr($path, strlen(file_directory_path())), '\\/'); - } - switch (variable_get('file_downloads', FILE_DOWNLOADS_PUBLIC)) { - case FILE_DOWNLOADS_PUBLIC: - return $GLOBALS['base_url'] .'/'. file_directory_path() .'/'. str_replace('\\', '/', $path); - case FILE_DOWNLOADS_PRIVATE: - return url('system/files/'. $path, array('absolute' => TRUE)); + // Clean up Windows paths. + $old_path = $path = str_replace('\\', '/', $path); + + drupal_alter('file_url', $path); + + // If any module has altered the path, then return the alteration. + if ($path != $old_path) { + return $path; + } + + // Otherwise serve the file from Drupal's web server. This point will only + // be reached when either no custom_file_url_rewrite() function has been + // defined, or when that function returned FALSE, thereby indicating that it + // cannot (or doesn't wish to) rewrite the URL. This is typically because + // the file doesn't match some conditions to be served from a CDN or static + // file server, or because the file has not yet been synced to the CDN or + // static file server. + + // Shipped files. + if (strpos($path, file_directory_path() . '/') !== 0) { + return base_path() . $path; + } + // Created files. + else { + switch (variable_get('file_downloads', FILE_DOWNLOADS_PUBLIC)) { + case FILE_DOWNLOADS_PUBLIC: + return $GLOBALS['base_url'] . '/' . $path; + case FILE_DOWNLOADS_PRIVATE: + // Strip file_directory_path from $path. Private downloads' URLs are + // rewritten to be served relatively to system/files (which is a menu + // callback that streams the file) instead of relatively to the file + // directory path. + $path = file_directory_strip($path); + return url('system/files/' . $path, array('absolute' => TRUE)); + } } } Index: includes/form.inc =================================================================== RCS file: /cvs/drupal/drupal/includes/form.inc,v retrieving revision 1.265.2.31 diff -u -F '^f' -r1.265.2.31 form.inc --- includes/form.inc 7 Dec 2009 15:54:52 -0000 1.265.2.31 +++ includes/form.inc 2 Feb 2010 15:29:24 -0000 @@ -1986,7 +1986,7 @@ function theme_image_button($element) { (!empty($element['#value']) ? ('value="'. check_plain($element['#value']) .'" ') : '') . 'id="'. $element['#id'] .'" '. drupal_attributes($element['#attributes']) . - ' src="'. base_path() . $element['#src'] .'" '. + ' src="'. file_create_url($element['#src']) .'" '. (!empty($element['#title']) ? 'alt="'. check_plain($element['#title']) .'" title="'. check_plain($element['#title']) .'" ' : '' ) . "/>\n"; } Index: includes/theme.inc =================================================================== RCS file: /cvs/drupal/drupal/includes/theme.inc,v retrieving revision 1.415.2.24 diff -u -F '^f' -r1.415.2.24 theme.inc --- includes/theme.inc 18 Jun 2009 12:04:04 -0000 1.415.2.24 +++ includes/theme.inc 2 Feb 2010 15:29:32 -0000 @@ -974,24 +974,24 @@ function theme_get_setting($setting_name if ($settings['toggle_logo']) { if ($settings['default_logo']) { - $settings['logo'] = base_path() . dirname($theme_object->filename) .'/logo.png'; + $settings['logo'] = file_create_url(dirname($theme_object->filename) .'/logo.png'); } elseif ($settings['logo_path']) { - $settings['logo'] = base_path() . $settings['logo_path']; + $settings['logo'] = file_create_url($settings['logo_path']); } } if ($settings['toggle_favicon']) { if ($settings['default_favicon']) { if (file_exists($favicon = dirname($theme_object->filename) .'/favicon.ico')) { - $settings['favicon'] = base_path() . $favicon; + $settings['favicon'] = file_create_url($favicon); } else { - $settings['favicon'] = base_path() .'misc/favicon.ico'; + $settings['favicon'] = file_create_url('misc/favicon.ico'); } } elseif ($settings['favicon_path']) { - $settings['favicon'] = base_path() . $settings['favicon_path']; + $settings['favicon'] = file_create_url($settings['favicon_path']); } else { $settings['toggle_favicon'] = FALSE; @@ -1212,7 +1212,7 @@ function theme_links($links, $attributes function theme_image($path, $alt = '', $title = '', $attributes = NULL, $getsize = TRUE) { if (!$getsize || (is_file($path) && (list($width, $height, $type, $image_attributes) = @getimagesize($path)))) { $attributes = drupal_attributes($attributes); - $url = (url($path) == $path) ? $path : (base_path() . $path); + $url = (url($path) == $path) ? $path : file_create_url($path); return ''. check_plain($alt) .''; } } Index: themes/garland/template.php =================================================================== RCS file: /cvs/drupal/drupal/themes/garland/template.php,v retrieving revision 1.16.2.2 diff -u -F '^f' -r1.16.2.2 template.php --- themes/garland/template.php 10 Aug 2009 11:32:54 -0000 1.16.2.2 +++ themes/garland/template.php 2 Feb 2010 15:30:21 -0000 @@ -90,9 +90,9 @@ function phptemplate_node_submitted($nod function phptemplate_get_ie_styles() { global $language; - $iecss = ''; + $iecss = '@import "'. base_path() . path_to_theme() .'/fix-ie-rtl.css";'; + $iecss .= ''; } return $iecss;