This breaks the specs but the performance was hard to justify. The in…

…sert images dialogue was taking 10+ seconds to render because job.name calls the Process step in dragonfly not the Fetch step. We need to access the name at the Fetch step for performance to remain good.
gunn · Jan 14, 2011 · 2db2ab9 · 2db2ab9
1 parent b240b79
commit 2db2ab9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/images/lib/images.rb b/images/lib/images.rb
@@ -28,7 +28,7 @@ class Engine < Rails::Engine
         # /system/images/BAhbB1sHOgZmIiMyMDEwLzA5LzAxL1NTQ19DbGllbnRfQ29uZi5qcGdbCDoGcDoKdGh1bWIiDjk0MngzNjAjYw/refinery_is_awesome.jpg
         # Officially the way to do it, from: http://markevans.github.com/dragonfly/file.URLs.html
         app_images.url_suffix = proc{|job|
-          "/#{job.name}"
+          "/#{job.uid_basename}#{job.encoded_extname || job.uid_extname}"
         }
 
         ### Extend active record ###

diff --git a/resources/lib/resources.rb b/resources/lib/resources.rb
@@ -26,7 +26,7 @@ class Engine < Rails::Engine
         # /system/images/BAhbB1sHOgZmIiMyMDEwLzA5LzAxL1NTQ19DbGllbnRfQ29uZi5qcGdbCDoGcDoKdGh1bWIiDjk0MngzNjAjYw/refinery_is_awesome.pdf
         # Officially the way to do it, from: http://markevans.github.com/dragonfly/file.URLs.html
         app_resources.url_suffix = proc{|job|
-          "/#{job.name}"
+          "/#{job.uid_basename}#{job.encoded_extname || job.uid_extname}"
         }
 
         ### Extend active record ###