Use Hash#fetch instead of testing for #key?

rails · Feb 13, 2012 · 6960e48 · 6960e48
1 parent 9b9838e
commit 6960e48
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/actionpack/lib/sprockets/railtie.rb b/actionpack/lib/sprockets/railtie.rb
@@ -24,7 +24,7 @@ class Railtie < ::Rails::Railtie
         env.version = ::Rails.env + "-#{config.assets.version}"
 
         if config.assets.logger != false
-          env.logger  = config.assets.logger || ::Rails.logger
+          env.logger = config.assets.logger || ::Rails.logger
         end
 
         if config.assets.cache_store != false

diff --git a/actionpack/lib/sprockets/static_compiler.rb b/actionpack/lib/sprockets/static_compiler.rb
@@ -8,8 +8,8 @@ def initialize(env, target, paths, options = {})
       @env = env
       @target = target
       @paths = paths
-      @digest = options.key?(:digest) ? options.delete(:digest) : true
-      @manifest = options.key?(:manifest) ? options.delete(:manifest) : true
+      @digest = options.fetch(:digest, true)
+      @manifest = options.fetch(:manifest, true)
       @manifest_path = options.delete(:manifest_path) || target
       @zip_files = options.delete(:zip_files) || /\.(?:css|html|js|svg|txt|xml)$/
     end