Merge pull request #12884 from akshay-vishnoi/refactor

Ruby's new Hash syntax applied in actionmailer

actionmailer/lib/action_mailer/base.rb

@@ -517,8 +517,8 @@ module ActionMailer
 
     def process(method_name, *args) #:nodoc:
       payload = {
-        :mailer => self.class.name,
-        :action => method_name
+        mailer: self.class.name,
+        action: method_name
       }
 
       ActiveSupport::Notifications.instrument("process.action_mailer", payload) do

actionmailer/test/fixtures/base_mailer/email_with_translations.html.erb

@@ -1 +1 @@
-<%= t('.greet_user', :name => 'lifo') %>
+<%= t('.greet_user', name: 'lifo') %>

actionmailer/test/fixtures/test_mailer/included_subtemplate.text.erb

@@ -1 +1 @@
-Hey Ho, <%= render :partial => "subtemplate" %>
+Hey Ho, <%= render partial: "subtemplate" %>

actionmailer/test/mailers/base_mailer.rb

@@ -120,7 +120,7 @@ class BaseMailer < ActionMailer::Base
   end
 
   def with_nil_as_return_value
-    mail(:template_name => "welcome")
+    mail(template_name: "welcome")
     nil
   end