asset tag timestamps about jammit HOT 8 CLOSED

Ignore that!

I do see asset tag timestamps on stylesheet and javascript links, just not on the first page load. That makes sense to me now.

I'm still not seeing them on the image urls in CSS, but those do not pass through asset_url()...

from jammit.

This works:

diff --git a/lib/jammit/compressor.rb b/lib/jammit/compressor.rb
index 0b46b95..3ed161b 100644
--- a/lib/jammit/compressor.rb
+++ b/lib/jammit/compressor.rb
@@ -7,6 +7,8 @@ module Jammit
   # all stylesheets, with all enabled assets inlined into the css.
   class Compressor
 
+    include ActionView::Helpers::AssetTagHelper
+
     # Mapping from extension to mime-type of all embeddable assets.
     EMBED_MIME_TYPES = {
       '.png'  => 'image/png',
@@ -110,7 +112,7 @@ module Jammit
         File.read(css_path).gsub(EMBED_DETECTOR) do |url|
           ipath, cpath = Pathname.new($1), Pathname.new(File.expand_path(css_path))
           is_url = URI.parse($1).absolute?
-          is_url ? url : "url(#{rewrite_asset_path(ipath, cpath, variant)})"
+          is_url ? url : "url(#{construct_asset_path(ipath, cpath, variant)})"
         end
       end
       stylesheets.join("\n")
@@ -143,10 +145,11 @@ module Jammit
     # Return a rewritten asset URL for a new stylesheet -- the asset should
     # be tagged for embedding if embeddable, and referenced at the correct level
     # if relative.
-    def rewrite_asset_path(asset_path, css_path, variant)
+    def construct_asset_path(asset_path, css_path, variant)
       public_path = absolute_path(asset_path, css_path)
       return "__EMBED__#{public_path}" if embeddable?(public_path, variant)
-      asset_path.absolute? ? asset_path.to_s : relative_path(public_path)
+      source = asset_path.absolute? ? asset_path.to_s : relative_path(public_path)
+      rewrite_asset_path(source)
     end
 
     # Get the site-absolute public path for an asset file path that may or may

from jammit.

Alright -- thanks for spearheading this one. I had mixed feelings on whether or not we should be messing with your CSS contents by adding in the rails timestamp -- since that's not what RAILS_ASSET_ID does by default. But I think it is a pretty nice feature, so thanks for making it happen.

I've merged your branch and made some tweaks:

• Removed the dependency on action_view, in favor of DIY generation of the asset_id. We don't need all the fancy caching and mutex locking that Rails needs to do.
• Fixed the path that was getting passed into rewrite_asset_path -- we need to keep the literal path in the CSS distinct from the actual path on disk.
• Added and adjusted all the tests to expect a RAILS_ASSET_ID of "101".

Take a look at the changes, and let me know if you're happy with where things stand, and we can push out an update to the gem.

from jammit.

Quick ping -- I'd still like to push out a new version of the gem. Have you had a chance to take a look?

from jammit.

Wow, somehow I completely missed your response. Checking it out now...

from jammit.

Looks good!

from jammit.

Alright, this is now out with Jammit 0.4.4 -- closing the ticket.

from jammit.

Thanks!

from jammit.