From c7bdbb3f76fcc9af6cd0fd4aaaf9a2f250edcf4f Mon Sep 17 00:00:00 2001 From: Carlos Antonio da Silva Date: Thu, 26 Jan 2012 13:27:50 -0200 Subject: [PATCH] Use Hash#fetch instead --- lib/simple_form/action_view_extensions/builder.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/simple_form/action_view_extensions/builder.rb b/lib/simple_form/action_view_extensions/builder.rb index 6241d620..ad1342a4 100644 --- a/lib/simple_form/action_view_extensions/builder.rb +++ b/lib/simple_form/action_view_extensions/builder.rb @@ -145,7 +145,7 @@ module SimpleForm end def render_collection(attribute, collection, value_method, text_method, options={}, html_options={}) #:nodoc: - item_wrapper_tag = options.key?(:item_wrapper_tag) ? options[:item_wrapper_tag] : SimpleForm.item_wrapper_tag + item_wrapper_tag = options.fetch(:item_wrapper_tag, SimpleForm.item_wrapper_tag) rendered_collection = collection.map do |item| value = value_for_collection(item, value_method) @@ -164,8 +164,8 @@ module SimpleForm value.respond_to?(:call) ? value.call(item) : item.send(value) end - def wrap_rendered_collection(collection, options) - wrapper_tag = options.key?(:collection_wrapper_tag) ? options[:collection_wrapper_tag] : SimpleForm.collection_wrapper_tag + def wrap_rendered_collection(collection, options) #:nodoc: + wrapper_tag = options.fetch(:collection_wrapper_tag, SimpleForm.collection_wrapper_tag) if wrapper_tag wrapper_class = [SimpleForm.collection_wrapper_class, options[:collection_wrapper_class]].compact.presence