From 68ac05e31ce963c0c2339f435bcd789dee662ef2 Mon Sep 17 00:00:00 2001 From: albert Date: Sat, 2 Mar 2013 22:45:15 -0500 Subject: [PATCH] fixes #705 --- app/models/dmail.rb | 2 +- test/unit/dmail_test.rb | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/app/models/dmail.rb b/app/models/dmail.rb index f620c31f6..62a4b7fef 100644 --- a/app/models/dmail.rb +++ b/app/models/dmail.rb @@ -154,7 +154,7 @@ class Dmail < ActiveRecord::Base end def send_dmail - if to.receive_email_notifications? && to.email.include?("@") + if to.receive_email_notifications? && to.email.include?("@") && owner_id == CurrentUser.id UserMailer.dmail_notice(self).deliver end end diff --git a/test/unit/dmail_test.rb b/test/unit/dmail_test.rb index 734603dd2..5443bd87b 100644 --- a/test/unit/dmail_test.rb +++ b/test/unit/dmail_test.rb @@ -62,6 +62,13 @@ class DmailTest < ActiveSupport::TestCase FactoryGirl.create(:dmail, :to => user, :owner => @user) end end + + should "create only one message for a split response" do + user = FactoryGirl.create(:user, :receive_email_notifications => true) + assert_difference("ActionMailer::Base.deliveries.size", 1) do + Dmail.create_split(:to_id => user.id, :title => "foo", :body => "foo") + end + end should "be marked as read after the user reads it" do dmail = FactoryGirl.create(:dmail, :owner => @user)