Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #8038 (new defect)

Opened 1 year ago

Last modified 3 weeks ago

[PATCH] ActionMailer Does not Set Body Charset When there are Attachments

Reported by: Theory Assigned to: bitsweat
Priority: normal Milestone: 2.1
Component: ActionMailer Version: edge
Severity: normal Keywords: charset, encoding, character+set
Cc:

Description

When you create an ActionMailer email with attachments, the character set does not get properly set for the body. The attached patch adds a test for this and fixes it.

Attachments

body_charset.patch (1.3 kB) - added by Theory on 04/09/07 22:22:58.
Use default charset when no content type specified

Change History

04/09/07 22:22:58 changed by Theory

  • attachment body_charset.patch added.

Use default charset when no content type specified

04/09/07 22:24:09 changed by Theory

  • summary changed from ActionMailer Does not Set Body Charset When there are Attachments to [PATCH] ActionMailer Does not Set Body Charset When there are Attachments.

04/09/08 16:50:27 changed by pkondzior

  • owner changed from core to pkondzior.
  • status changed from new to assigned.

After 1 year patch is still not commited into to trunk ? This is very very sad.

http://github.com/pkondzior/rails/commit/ea8b5f2d0d32a2260f2888d5979f7ed52a530c21 here it is my rails clone with this path to pull.

(follow-up: ↓ 4 ) 04/09/08 17:03:49 changed by Radarek

+1

(in reply to: ↑ 3 ) 04/09/08 19:47:58 changed by bober0

+1

04/27/08 13:01:19 changed by pkondzior

  • owner changed from pkondzior to bitsweat.
  • status changed from assigned to new.

04/27/08 19:52:03 changed by bitsweat

  • milestone changed from 2.x to 2.1.

Sad indeed :) Do you have a new link to that commit? The one above fails

04/28/08 08:04:06 changed by pkondzior

Something bad happend with my fork, i creted another one again from today edge. http://github.com/pkondzior/rails/commit/1905352909d38c16bccec9f24bd81ed33ba854b5 checkout this.