Fix #3474: Spurious email notifications.

This commit is contained in:
evazion
2017-12-28 18:50:08 -06:00
parent 857fa24697
commit 1ca2497015
2 changed files with 4 additions and 2 deletions

View File

@@ -23,7 +23,7 @@ class Dmail < ApplicationRecord
after_initialize :initialize_attributes, if: :new_record?
before_create :auto_read_if_filtered
after_create :update_recipient
after_create :send_dmail
after_commit :send_email, on: :create
rakismet_attrs author: :from_name, author_email: :from_email, content: :title_and_body, user_ip: :creator_ip_addr_str
@@ -247,7 +247,7 @@ class Dmail < ApplicationRecord
"[quote]\n#{from_name} said:\n\n#{body}\n[/quote]\n\n"
end
def send_dmail
def send_email
if !is_spam? && to.receive_email_notifications? && to.email =~ /@/ && owner_id == to.id
UserMailer.dmail_notice(self).deliver_now
end

View File

@@ -10,10 +10,12 @@ class DmailTest < ActiveSupport::TestCase
ActionMailer::Base.delivery_method = :test
ActionMailer::Base.perform_deliveries = true
ActionMailer::Base.deliveries = []
TestAfterCommit.enabled = true
end
teardown do
CurrentUser.user = nil
TestAfterCommit.enabled = false
end
context "spam" do