diff options
author | Josh Kalderimis <josh.kalderimis@gmail.com> | 2010-11-15 18:20:37 +0100 |
---|---|---|
committer | José Valim <jose.valim@gmail.com> | 2010-11-16 00:04:37 +0100 |
commit | ce1f87673c17338617222a4e4e0971cfb9f0655f (patch) | |
tree | eca4e638cb16ba2d3d3e1fb41e56b730a7ea3cf9 /actionpack/lib/action_view/helpers/asset_tag_helpers | |
parent | 0ff1c5935f84f86a8b88802078b115e9964c2249 (diff) | |
download | rails-ce1f87673c17338617222a4e4e0971cfb9f0655f.tar.gz rails-ce1f87673c17338617222a4e4e0971cfb9f0655f.tar.bz2 rails-ce1f87673c17338617222a4e4e0971cfb9f0655f.zip |
corrected the AV railtie to use the new home for cache_asset_timestamps, and merged asset id caching and asset paths together.
Diffstat (limited to 'actionpack/lib/action_view/helpers/asset_tag_helpers')
3 files changed, 46 insertions, 74 deletions
diff --git a/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_id_caching.rb b/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_id_caching.rb deleted file mode 100644 index 64a193e4be..0000000000 --- a/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_id_caching.rb +++ /dev/null @@ -1,69 +0,0 @@ -require 'thread' -require 'active_support/core_ext/file' -require 'active_support/concern' - -module ActionView - module Helpers - module AssetTagHelper - - module AssetIdCaching - extend ActiveSupport::Concern - - included do - # You can enable or disable the asset tag timestamps cache. - # With the cache enabled, the asset tag helper methods will make fewer - # expensive file system calls. However this prevents you from modifying - # any asset files while the server is running. - # - # ActionView::Helpers::AssetTagHelper.cache_asset_timestamps = false - mattr_accessor :cache_asset_timestamps - - private - mattr_accessor :asset_timestamps_cache - self.asset_timestamps_cache = {} - - mattr_accessor :asset_timestamps_cache_guard - self.asset_timestamps_cache_guard = Mutex.new - end - - private - - # Use the RAILS_ASSET_ID environment variable or the source's - # modification time as its cache-busting asset id. - def rails_asset_id(source) - if asset_id = ENV["RAILS_ASSET_ID"] - asset_id - else - if self.cache_asset_timestamps && (asset_id = self.asset_timestamps_cache[source]) - asset_id - else - path = File.join(config.assets_dir, source) - asset_id = File.exist?(path) ? File.mtime(path).to_i.to_s : '' - - if self.cache_asset_timestamps - self.asset_timestamps_cache_guard.synchronize do - self.asset_timestamps_cache[source] = asset_id - end - end - - asset_id - end - end - end - - # Break out the asset path rewrite in case plugins wish to put the asset id - # someplace other than the query string. - # This is the default implementation - def handle_asset_id(source) - asset_id = rails_asset_id(source) - if asset_id.empty? - source - else - "#{source}?#{asset_id}" - end - end - end - - end - end -end
\ No newline at end of file diff --git a/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_include_tag.rb b/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_include_tag.rb index e99026fb36..3bc81ae068 100644 --- a/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_include_tag.rb +++ b/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_include_tag.rb @@ -2,8 +2,6 @@ require 'active_support/core_ext/class/attribute' require 'active_support/core_ext/string/inflections' require 'active_support/core_ext/file' require 'action_view/helpers/tag_helper' -require 'action_view/helpers/asset_tag_helpers/asset_id_caching' - module ActionView module Helpers diff --git a/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_paths.rb b/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_paths.rb index 5f884e391a..7a00c8e69d 100644 --- a/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_paths.rb +++ b/actionpack/lib/action_view/helpers/asset_tag_helpers/asset_paths.rb @@ -1,12 +1,17 @@ require 'active_support/core_ext/file' -require 'action_view/helpers/asset_tag_helpers/asset_id_caching' module ActionView module Helpers module AssetTagHelper class AssetPaths - include AssetIdCaching + # You can enable or disable the asset tag timestamps cache. + # With the cache enabled, the asset tag helper methods will make fewer + # expensive file system calls. However this prevents you from modifying + # any asset files while the server is running. + # + # ActionView::Helpers::AssetTagHelper.cache_asset_timestamps = false + mattr_accessor :cache_asset_timestamps attr_reader :config, :controller @@ -36,6 +41,12 @@ module ActionView source end + def add_to_asset_timestamp_cache(source, asset_id) + self.asset_timestamps_cache_guard.synchronize do + self.asset_timestamps_cache[source] = asset_id + end + end + def is_uri?(path) path =~ %r{^[-a-z]+://|^cid:} end @@ -62,8 +73,40 @@ module ActionView return path.call(source) elsif path && path.is_a?(String) return path % [source] + end + + asset_id = rails_asset_id(source) + if asset_id.empty? + source + else + "#{source}?#{asset_id}" + end + end + + mattr_accessor :asset_timestamps_cache + self.asset_timestamps_cache = {} + + mattr_accessor :asset_timestamps_cache_guard + self.asset_timestamps_cache_guard = Mutex.new + + # Use the RAILS_ASSET_ID environment variable or the source's + # modification time as its cache-busting asset id. + def rails_asset_id(source) + if asset_id = ENV["RAILS_ASSET_ID"] + asset_id else - handle_asset_id(source) + if self.cache_asset_timestamps && (asset_id = self.asset_timestamps_cache[source]) + asset_id + else + path = File.join(config.assets_dir, source) + asset_id = File.exist?(path) ? File.mtime(path).to_i.to_s : '' + + if self.cache_asset_timestamps + add_to_asset_timestamp_cache(source, asset_id) + end + + asset_id + end end end |