use simple_form and hash updates

app/controllers/merged/cards_controller.rb

@@ -36,15 +36,8 @@ module Merged
     end
 
     def update
-      @card.allowed_fields.each do |key|
-        if( params.has_key?(key) )
-          @card.update(key, params[key])
-        end
-      end
-      options = params[:option]
-      @card.option_definitions.each do |option|
-        @card.set_option(option.name,  options[option.name])
-      end if options
+      @card.update(params[:card])
+      @card.update_options( params[:options])
       @card.edit_save(current_member.email)
       redirect_to section_cards_url(@card.section.id) , notice: "Updated #{@card.header}"
     end

app/controllers/merged/sections_controller.rb

@@ -66,10 +66,7 @@ module Merged
 
     def update
       @section.update(params[:section])
-      options = params[:option]
-      @section.option_definitions.each do |option|
-        @section.set_option(option.name,  options[option.name])
-      end if options
+      @section.update_options( params[:options])
       @section.edit_save(current_member.email)
       redirect_to :section , notice: "Update ok"
     end

app/models/merged/view_base.rb

@@ -59,6 +59,7 @@ module Merged
     end
 
     def update(hash)
+      return unless hash
       hash.each do |key , value|
         raise "unsuported field #{key} for #{template}:#{allowed_fields}" unless allowed_fields.include?(key.to_sym)
         if(! attributes[key].nil? ) # first setting ok, types not (yet?) specified
@@ -70,6 +71,12 @@ module Merged
       end
     end
 
+    def update_options( options )
+      return unless options
+      option_definitions.each do |option|
+        set_option(option.name,  options[option.name])
+      end
+    end
     #other may be nil
     def swap_index_with(other)
       return unless other