##// END OF EJS Templates
jro_apps » redmine
RSS
Fixed: Issue status in the notify email's subject is the issue's old status, should be its new status (#3194)....
Jean-Philippe Lang -
r2581:2a3fe1604a6e
parent child
Show More
Side by Side Unified
Context file:
r2581:2a3fe1604a6e Loading diff...:
Show file before | Show file after | Hide comments
@@ -53,7 +53,7 class Mailer < ActionMailer::Base
53 # issue_edit(journal) => tmail object
53 # issue_edit(journal) => tmail object
54 # Mailer.deliver_issue_edit(journal) => sends an email to issue recipients
54 # Mailer.deliver_issue_edit(journal) => sends an email to issue recipients
55 def issue_edit(journal)
55 def issue_edit(journal)
56 issue = journal.journalized
56 issue = journal.journalized.reload
57 redmine_headers 'Project' => issue.project.identifier,
57 redmine_headers 'Project' => issue.project.identifier,
58 'Issue-Id' => issue.id,
58 'Issue-Id' => issue.id,
59 'Issue-Author' => issue.author.login
59 'Issue-Author' => issue.author.login
Show file before | Show file after | Hide comments
@@ -696,6 +696,8 class IssuesControllerTest < Test::Unit::TestCase
696
696
697 mail = ActionMailer::Base.deliveries.last
697 mail = ActionMailer::Base.deliveries.last
698 assert mail.body.include?("Status changed from New to Assigned")
698 assert mail.body.include?("Status changed from New to Assigned")
699 # subject should contain the new status
700 assert mail.subject.include?("(#{ IssueStatus.find(2).name })")
699 end
701 end
700
702
701 def test_post_edit_with_note_only
703 def test_post_edit_with_note_only
General Comments 0
You need to be logged in to leave comments. Login now