Commit 1a383427 authored by jplang's avatar jplang

Adding an issue note via email fails due to NoMethodError (#30455).

Patch by Go MAEDA.

git-svn-id: https://svn.redmine.org/redmine/trunk@17817 e93f8b46-1217-0410-a6f0-8f06a7374b81
parent e461d7fe
...@@ -237,7 +237,7 @@ class MailHandler < ActionMailer::Base ...@@ -237,7 +237,7 @@ class MailHandler < ActionMailer::Base
end end
# ignore CLI-supplied defaults for new issues # ignore CLI-supplied defaults for new issues
handler_options[:issue].clear handler_options[:issue] = {}
journal = issue.init_journal(user) journal = issue.init_journal(user)
if from_journal && from_journal.private_notes? if from_journal && from_journal.private_notes?
......
...@@ -60,6 +60,22 @@ class MailHandlerControllerTest < Redmine::ControllerTest ...@@ -60,6 +60,22 @@ class MailHandlerControllerTest < Redmine::ControllerTest
assert_equal true, issue.is_private assert_equal true, issue.is_private
end end
def test_should_update_issue
# Enable API and set a key
Setting.mail_handler_api_enabled = 1
Setting.mail_handler_api_key = 'secret'
assert_no_difference 'Issue.count' do
assert_difference 'Journal.count' do
post :index, :params => {
:key => 'secret',
:email => IO.read(File.join(FIXTURES_PATH, 'ticket_reply.eml'))
}
end
end
assert_response 201
end
def test_should_respond_with_422_if_not_created def test_should_respond_with_422_if_not_created
Project.find('onlinestore').destroy Project.find('onlinestore').destroy
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment