Defect #10721 » 10721-fixed.patch
app/models/mail_handler.rb | ||
---|---|---|
209 | 209 | |
210 | 210 |
# add To and Cc as watchers before saving so the watchers can reply to Redmine |
211 | 211 |
add_watchers(issue) |
212 |
issue.save! |
|
213 | 212 |
add_attachments(issue) |
213 |
issue.save! |
|
214 | 214 |
logger&.info "MailHandler: issue ##{issue.id} created by #{user}" |
215 | 215 |
issue |
216 | 216 |
end |
... | ... | |
245 | 245 | |
246 | 246 |
# add To and Cc as watchers before saving so the watchers can reply to Redmine |
247 | 247 |
add_watchers(issue) |
248 |
issue.save! |
|
249 | 248 |
add_attachments(issue) |
249 |
issue.save! |
|
250 | 250 |
logger&.info "MailHandler: issue ##{issue.id} updated by #{user}" |
251 | 251 |
journal |
252 | 252 |
end |
... | ... | |
284 | 284 |
:content => cleaned_up_text_body) |
285 | 285 |
reply.author = user |
286 | 286 |
reply.board = message.board |
287 |
message.children << reply |
|
288 | 287 |
add_attachments(reply) |
288 |
message.children << reply |
|
289 | 289 |
reply |
290 | 290 |
else |
291 | 291 |
logger&.info "MailHandler: ignoring reply from [#{email.from.first}] to a locked topic" |
... | ... | |
297 | 297 |
email.attachments.each do |attachment| |
298 | 298 |
next unless accept_attachment?(attachment) |
299 | 299 |
next unless attachment.body.decoded.size > 0 |
300 |
obj.attachments << Attachment.create(:container => obj,
|
|
300 |
obj.saved_attachments << Attachment.create(
|
|
301 | 301 |
:file => attachment.body.decoded, |
302 | 302 |
:filename => attachment.filename, |
303 | 303 |
:author => user, |
test/unit/mail_handler_test.rb | ||
---|---|---|
906 | 906 |
def test_update_issue_with_attachment |
907 | 907 |
assert_difference 'Journal.count' do |
908 | 908 |
assert_difference 'JournalDetail.count' do |
909 |
assert_difference 'Attachment.count' do
|
|
909 |
assert_difference ->{Attachment.where(:container_type => 'Issue', :container_id => 2).count} do
|
|
910 | 910 |
assert_no_difference 'Issue.count' do |
911 | 911 |
journal = submit_email('ticket_with_attachment.eml') do |raw| |
912 | 912 |
raw.gsub! /^Subject: .*$/, 'Subject: Re: [Cookbook - Feature #2] (New) Add ingredients categories' |
... | ... | |
928 | 928 |
Issue.any_instance.stubs(:valid?).returns(false) |
929 | 929 |
assert_no_difference 'Journal.count' do |
930 | 930 |
assert_no_difference 'JournalDetail.count' do |
931 |
assert_no_difference 'Attachment.count' do
|
|
931 |
assert_no_difference ->{Attachment.where(:container_type => 'Issue', :container_id => 2).count} do
|
|
932 | 932 |
assert_no_difference 'Issue.count' do |
933 | 933 |
journal = submit_email('ticket_with_attachment.eml') do |raw| |
934 | 934 |
raw.gsub! /^Subject: .*$/, 'Subject: Re: [Cookbook - Feature #2] (New) Add ingredients categories' |
- « Previous
- 1
- 2
- 3
- Next »