Replace Hash#merge! with Hash#[]= (#34160).

git-svn-id: http://svn.redmine.org/redmine/trunk@20175 e93f8b46-1217-0410-a6f0-8f06a7374b81

Author: vividtone

.rubocop_todo.yml

@@ -564,22 +564,6 @@ Performance/RedundantMatch:
     - 'lib/redmine/wiki_formatting/textile/formatter.rb'
     - 'lib/redmine/wiki_formatting/textile/redcloth3.rb'
 
-# Cop supports --auto-correct.
-# Configuration parameters: MaxKeyValuePairs.
-Performance/RedundantMerge:
-  Exclude:
-    - 'app/controllers/issues_controller.rb'
-    - 'app/helpers/application_helper.rb'
-    - 'app/helpers/avatars_helper.rb'
-    - 'app/helpers/custom_fields_helper.rb'
-    - 'app/helpers/workflows_helper.rb'
-    - 'app/models/auth_source_ldap.rb'
-    - 'app/models/principal.rb'
-    - 'lib/redmine/access_control.rb'
-    - 'test/functional/imports_controller_test.rb'
-    - 'test/unit/issue_import_test.rb'
-    - 'test/unit/time_entry_import_test.rb'
-
 # Cop supports --auto-correct.
 Performance/Squeeze:
   Exclude:

app/controllers/issues_controller.rb

@@ -666,7 +666,7 @@ def redirect_after_create
       if params[:project_id]
         redirect_to new_project_issue_path(@issue.project, url_params)
       else
-        url_params[:issue].merge! :project_id => @issue.project_id
+        url_params[:issue][:project_id] = @issue.project_id
         redirect_to new_issue_path(url_params)
       end
     else

app/helpers/application_helper.rb

@@ -1401,16 +1401,16 @@ def labelled_form_for(*args, &proc)
     args << {} unless args.last.is_a?(Hash)
     options = args.last
     if args.first.is_a?(Symbol)
-      options.merge!(:as => args.shift)
+      options[:as] = args.shift
     end
-    options.merge!({:builder => Redmine::Views::LabelledFormBuilder})
+    options[:builder] = Redmine::Views::LabelledFormBuilder
     form_for(*args, &proc)
   end
 
   def labelled_fields_for(*args, &proc)
     args << {} unless args.last.is_a?(Hash)
     options = args.last
-    options.merge!({:builder => Redmine::Views::LabelledFormBuilder})
+    options[:builder] = Redmine::Views::LabelledFormBuilder
     fields_for(*args, &proc)
   end
 

app/helpers/avatars_helper.rb

@@ -23,22 +23,22 @@ module AvatarsHelper
   def assignee_avatar(user, options={})
     return '' unless user
 
-    options.merge!(:title => l(:field_assigned_to) + ": " + user.name)
+    options[:title] = l(:field_assigned_to) + ": " + user.name
     avatar(user, options).to_s.html_safe
   end
 
   def author_avatar(user, options={})
     return '' unless user
 
-    options.merge!(:title => l(:field_author) + ": " + user.name)
+    options[:title] = l(:field_author) + ": " + user.name
     avatar(user, options).to_s.html_safe
   end
 
   # Returns the avatar image tag for the given +user+ if avatars are enabled
   # +user+ can be a User or a string that will be scanned for an email address (eg. 'joe <[email protected]>')
   def avatar(user, options = { })
     if Setting.gravatar_enabled?
-      options.merge!(:default => Setting.gravatar_default)
+      options[:default] = Setting.gravatar_default
       options[:class] = GravatarHelper::DEFAULT_OPTIONS[:class] + " " + options[:class] if options[:class]
       email = nil
       if user.respond_to?(:mail)

app/helpers/custom_fields_helper.rb

@@ -190,7 +190,7 @@ def render_api_custom_values(custom_values, api)
     api.array :custom_fields do
       custom_values.each do |custom_value|
         attrs = {:id => custom_value.custom_field_id, :name => custom_value.custom_field.name}
-        attrs.merge!(:multiple => true) if custom_value.custom_field.multiple?
+        attrs[:multiple] = true if custom_value.custom_field.multiple?
         api.custom_field attrs do
           if custom_value.value.is_a?(Array)
             api.array :value do

app/helpers/workflows_helper.rb

@@ -35,7 +35,7 @@ def options_for_workflow_select(name, objects, selected, options={})
     end
     all_tag_options = {:value => 'all', :selected => (selected == 'all')}
     if multiple
-      all_tag_options.merge!(:style => "display:none;")
+      all_tag_options[:style] = "display:none;"
     end
     option_tags << content_tag('option', l(:label_all), all_tag_options)
     option_tags << options_from_collection_for_select(objects, "id", "name", selected)

app/models/auth_source_ldap.rb

@@ -187,7 +187,7 @@ def initialize_ldap_con(ldap_user, ldap_password)
       }
     end
     unless ldap_user.blank? && ldap_password.blank?
-      options.merge!(:auth => {:method => :simple, :username => ldap_user, :password => ldap_password})
+      options[:auth] = {:method => :simple, :username => ldap_user, :password => ldap_password}
     end
     Net::LDAP.new options
   end

app/models/principal.rb

@@ -80,7 +80,7 @@ class Principal < ActiveRecord::Base
       if tokens.present?
         sql << ' OR ('
         sql << tokens.map.with_index do |token, index|
-          params.merge!(:"token_#{index}" => token)
+          params[:"token_#{index}"] = token
           "(LOWER(#{table_name}.firstname) LIKE LOWER(:token_#{index}) OR LOWER(#{table_name}.lastname) LIKE LOWER(:token_#{index}))"
         end.join(' AND ')
         sql << ')'

lib/redmine/access_control.rb

@@ -84,7 +84,7 @@ def initialize
 
       def permission(name, hash, options={})
         @permissions ||= []
-        options.merge!(:project_module => @project_module)
+        options[:project_module] = @project_module
         @permissions << Permission.new(name, hash, options)
       end
 

test/functional/imports_controller_test.rb

@@ -405,7 +405,7 @@ def test_show_without_errors
 
   def test_show_with_errors_should_show_unsaved_items
     import = generate_import_with_mapping
-    import.mapping.merge! 'subject' => 20
+    import.mapping['subject'] = 20
     import.run
     assert_not_equal 0, import.unsaved_items.count