diff --git a/app/mailers/teacher_mailer.rb b/app/mailers/teacher_mailer.rb index 4c035ff5..7f2c581a 100644 --- a/app/mailers/teacher_mailer.rb +++ b/app/mailers/teacher_mailer.rb @@ -81,9 +81,8 @@ def set_body # Must be called after @teacher is set. def set_recipients @recipients = Liquid::Template.parse(email_template.to).render(liquid_assigns).html_safe - @recipients = @recipients.split(','). + @recipients = @recipients.split(","). map { |addr| addr.strip }. - filter { |addr| addr != '(blank)' || addr.blank? }.compact - + filter { |addr| addr != "(blank)" || addr.blank? }.compact end end diff --git a/spec/models/email_template_spec.rb b/spec/models/email_template_spec.rb index bb38d8b1..c75c2ac7 100644 --- a/spec/models/email_template_spec.rb +++ b/spec/models/email_template_spec.rb @@ -15,7 +15,7 @@ end it "returns false if no reason is in the body" do - template = EmailTemplate.new(body: '') + template = EmailTemplate.new(body: "") expect(template.accepts_custom_reason?).to be false end end