Fixed: Issue status in the notify email's subject is the issue's old status,…

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

Fixed: Issue status in the notify email's subject is the issue's old status,…
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
2a3fe160 · Jean-Philippe Lang · 43200e21 · 2a3fe160 · 2a3fe160
Commit 2a3fe160 authored Apr 19, 2009 by Jean-Philippe Lang
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

mailer.rb app/models/mailer.rb +1 -1

issues_controller_test.rb test/functional/issues_controller_test.rb +2 -0

No files found.
--- 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

--- 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