diff options
author | Aaron Patterson <aaron.patterson@gmail.com> | 2019-02-25 13:18:44 -0800 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2019-02-25 13:18:44 -0800 |
commit | ca5e23ed4d8818d81314953aadd422b2cbde63b0 (patch) | |
tree | a411836c47d0ac22af072ec580656e983954d131 /actionmailer | |
parent | d1f68b50487ea1a8fc7f822973e2cc46d1288060 (diff) | |
download | rails-ca5e23ed4d8818d81314953aadd422b2cbde63b0.tar.gz rails-ca5e23ed4d8818d81314953aadd422b2cbde63b0.tar.bz2 rails-ca5e23ed4d8818d81314953aadd422b2cbde63b0.zip |
Templates have one format
Templates only have one format. Before this commit, templates would be
constructed with a single element array that contained the format. This
commit eliminates the single element array and just implements a
`format` method. This saves one array allocation per template.
Diffstat (limited to 'actionmailer')
-rw-r--r-- | actionmailer/lib/action_mailer/base.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/actionmailer/lib/action_mailer/base.rb b/actionmailer/lib/action_mailer/base.rb index 033a7f5b9e..52360d40fe 100644 --- a/actionmailer/lib/action_mailer/base.rb +++ b/actionmailer/lib/action_mailer/base.rb @@ -973,7 +973,7 @@ module ActionMailer templates_name = headers[:template_name] || action_name each_template(Array(templates_path), templates_name).map do |template| - self.formats = template.formats + self.formats = [template.format] { body: render(template: template), content_type: template.type.to_s @@ -986,7 +986,7 @@ module ActionMailer if templates.empty? raise ActionView::MissingTemplate.new(paths, name, paths, false, "mailer") else - templates.uniq(&:formats).each(&block) + templates.uniq(&:format).each(&block) end end |