Patch #13646

Fix handling multiple text parts in email

Added by Alex Shulgin over 5 years ago. Updated about 5 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:Email receiving
Target version:2.4.0

Description

When handling emails consisting of multiple text parts interspersed with attachment parts, Redmine would only consider the first text (text/plain or text/html) part and ignore the rest.

Here's a patch against 2.2-stable to fix this, test included.

fix-multiple-text-parts-email-handling.diff Magnifier (1.74 KB) Alex Shulgin, 2013-03-30 00:55

fix-multiple-text-parts-email-handling.diff Magnifier (3.51 KB) Alex Shulgin, 2013-03-30 00:57


Related issues

Related to Redmine - Defect #15684: MailHandler : text/plain attachments are added to descrip... Closed

Associated revisions

Revision 11834
Added by Jean-Philippe Lang about 5 years ago

Fix handling multiple text parts in email (#13646).

Patch by Alex Shulgin.

History

#1 Updated by Alex Shulgin over 5 years ago

Oops, forgot to include the test email fixture.

#2 Updated by Toshi MARUYAMA over 5 years ago

  • Target version set to 2.3.1

#3 Updated by Toshi MARUYAMA over 5 years ago

  • Target version changed from 2.3.1 to 2.4.0

#4 Updated by Toshi MARUYAMA over 5 years ago

  • Target version changed from 2.4.0 to 2.3.1

#5 Updated by Toshi MARUYAMA over 5 years ago

  • Target version changed from 2.3.1 to 2.4.0

#6 Updated by Jean-Philippe Lang about 5 years ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang

Patch committed in r11834, thanks. I changed the assertion because parts may not be inserted in the expected order.

#7 Updated by Toshi MARUYAMA over 4 years ago

  • Related to Defect #15684: MailHandler : text/plain attachments are added to description added

Also available in: Atom PDF