From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Sun, 19 Apr 2009 08:33:56 +0000 (+0000)
Subject: Fixed: Issue status in the notify email's subject is the issue's old status, should... 
X-Git-Tag: 0.9.0~520
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=2a3fe1604a6ec17febff189305649d31e2ec871e;p=redmine.git

Fixed: Issue status in the notify email's subject is the issue's old status, should be its new status (#3194).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2673 e93f8b46-1217-0410-a6f0-8f06a7374b81
---

diff --git a/app/models/mailer.rb b/app/models/mailer.rb
index c011d9d2a..992fd50c0 100644
--- a/app/models/mailer.rb
+++ b/app/models/mailer.rb
@@ -53,7 +53,7 @@ class Mailer < ActionMailer::Base
   #   issue_edit(journal) => tmail object
   #   Mailer.deliver_issue_edit(journal) => sends an email to issue recipients
   def issue_edit(journal)
-    issue = journal.journalized
+    issue = journal.journalized.reload
     redmine_headers 'Project' => issue.project.identifier,
                     'Issue-Id' => issue.id,
                     'Issue-Author' => issue.author.login
diff --git a/test/functional/issues_controller_test.rb b/test/functional/issues_controller_test.rb
index 0df66abdb..f562ac9c5 100644
--- a/test/functional/issues_controller_test.rb
+++ b/test/functional/issues_controller_test.rb
@@ -696,6 +696,8 @@ class IssuesControllerTest < Test::Unit::TestCase
     
     mail = ActionMailer::Base.deliveries.last
     assert mail.body.include?("Status changed from New to Assigned")
+    # subject should contain the new status
+    assert mail.subject.include?("(#{ IssueStatus.find(2).name })")
   end
   
   def test_post_edit_with_note_only