[TYPO3-core] RFC: Bug 1268: Respect config.notification_email_charset when sending emails

[Michael Stucki]

I agree to this. New patches for TYPO3_4-0 and Trunk are attached. Still one
+1 missing though...

- michael

Martin Kutschker wrote:

> Michael Stucki schrieb:
>> This is a SVN patch request.
>> 
>> Problem:
>> When sending out emails via tslib_cObj->plainMailEncoded() the header is
>> set accoding to config.notification_email_charset, but the content is not
>> converted.
>> 
>> Solution:
>> Check if config.notification_email_charset is different from
>> config.renderCharset or if config.metaCharset differs from
>> config.renderCharset.
> 
> +1
> 
> Note: Too me $notification_email_charset seems not really necessary and
> more distracting than helpful. So I'd have written it like this.
> 
> $charset > $this->csConvObj->parse_charset($this->config['config'
['notification_email_charset']);
> if ($charset !> $convCharset > }
> 
> Masi

-- 
Use a newsreader! Check out
http://typo3.org/community/mailing-lists/use-a-news-reader/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug_1268_notification_email_charset-trunk_v4.diff
Type: text/x-diff
Size: 7743 bytes
Desc: not available
Url : http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20060903/0367f19c/attachment.bin 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug_1268_notification_email_charset-TYPO3_4-0_v4.diff
Type: text/x-diff
Size: 1654 bytes
Desc: not available
Url : http://lists.netfielders.de/pipermail/typo3-team-core/attachments/20060903/0367f19c/attachment-0001.bin