diff options
| author | Steve Klabnik <steve@steveklabnik.com> | 2012-06-16 12:02:35 +0200 |
|---|---|---|
| committer | Steve Klabnik <steve@steveklabnik.com> | 2012-06-16 18:06:15 +0200 |
| commit | afb053b4d388137830320ed8dd49a40dc29a962c (patch) | |
| tree | b8f90fd3b0eabf92df3ef6887b1c151983cfb91f /actionpack/lib/action_view | |
| parent | c1b1956a15d3d38d0a4504e168bb69638d71e536 (diff) | |
| download | rails-afb053b4d388137830320ed8dd49a40dc29a962c.tar.gz rails-afb053b4d388137830320ed8dd49a40dc29a962c.tar.bz2 rails-afb053b4d388137830320ed8dd49a40dc29a962c.zip | |
Respect absolute paths in compute_source_path.
When using compute_source_path to determine the full path of an
asset, if our source begins with '/', we don't want to include
the directory. Examples are illustrative:
> compute_source_path("foo", "stylesheets", "css")
=> "/Users/steve/src/my_app/public/stylesheets/foo.css"
> compute_source_path("/foo", "stylesheets", "css")
=> "/Users/steve/src/my_app/public/foo.css"
Before this patch, the second example would return the same as the
first.
Fixes #5680.
Diffstat (limited to 'actionpack/lib/action_view')
| -rw-r--r-- | actionpack/lib/action_view/asset_paths.rb | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/actionpack/lib/action_view/asset_paths.rb b/actionpack/lib/action_view/asset_paths.rb index 4ce41d51f1..81880d17ea 100644 --- a/actionpack/lib/action_view/asset_paths.rb +++ b/actionpack/lib/action_view/asset_paths.rb @@ -35,7 +35,13 @@ module ActionView # Return the filesystem path for the source def compute_source_path(source, dir, ext) source = rewrite_extension(source, dir, ext) if ext - File.join(config.assets_dir, dir, source) + + sources = [] + sources << config.assets_dir + sources << dir unless source[0] == ?/ + sources << source + + File.join(sources) end def is_uri?(path) |