Merge pull request #5595 from louim/bugfix/mailer-defaults-lambda-arity

Make sure Mailer defaults :from and :reply_to are handled correctly

lib/devise/mailers/helpers.rb

@@ -33,28 +33,22 @@ module Devise
           subject: subject_for(action),
           to: resource.email,
           from: mailer_sender(devise_mapping),
-          reply_to: mailer_reply_to(devise_mapping),
+          reply_to: mailer_sender(devise_mapping),
           template_path: template_paths,
           template_name: action
-        }.merge(opts)
+        }
+        # Give priority to the mailer's default if they exists.
+        headers.delete(:from) if default_params[:from]
+        headers.delete(:reply_to) if default_params[:reply_to]
+
+        headers.merge!(opts)
 
         @email = headers[:to]
         headers
       end
 
-      def mailer_reply_to(mapping)
-        mailer_sender(mapping, :reply_to)
-      end
-
-      def mailer_from(mapping)
-        mailer_sender(mapping, :from)
-      end
-
-      def mailer_sender(mapping, sender = :from)
-        default_sender = default_params[sender]
-        if default_sender.present?
-          default_sender.respond_to?(:to_proc) ? instance_eval(&default_sender) : default_sender
-        elsif Devise.mailer_sender.is_a?(Proc)
+      def mailer_sender(mapping)
+        if Devise.mailer_sender.is_a?(Proc)
           Devise.mailer_sender.call(mapping.name)
         else
           Devise.mailer_sender

test/mailers/mailer_test.rb

@@ -17,4 +17,30 @@ class MailerTest < ActionMailer::TestCase
 
     assert mail.content_transfer_encoding, "7bit"
   end
+
+  test "default values defined as proc with different arity are handled correctly" do
+    class TestMailerWithDefault < Devise::Mailer
+      default from: -> { computed_from }
+      default reply_to: ->(_) { computed_reply_to }
+
+      def confirmation_instructions(record, token, opts = {})
+        @token = token
+        devise_mail(record, :confirmation_instructions, opts)
+      end
+
+      private
+
+      def computed_from
+        "from@example.com"
+      end
+
+      def computed_reply_to
+        "reply_to@example.com"
+      end
+    end
+
+    mail = TestMailerWithDefault.confirmation_instructions(create_user, "confirmation-token")
+    assert mail.from, "from@example.com"
+    assert mail.reply_to, "reply_to@example.com"
+  end
 end