Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Creating Topic via email - Reformating the body part of a message #3398

Closed
JoergPalmer opened this issue Mar 2, 2020 · 0 comments
Closed
Labels
Milestone

Comments

@JoergPalmer
Copy link

File / function:
EmailFormat.class.php:365 / _clean_up()

Problem:
The charset parameter is not processed correctly. Towards the end of the function a couple of special characters are replaced in a dedicated way. This must not be done if the text is UTF-8 encoded. Unfortunately this is checked for the capital letter variant 'UTF-8'. In my case the character set came in as 'utf-8' which consequently destroyed the content of the body text.

Solution:
Modify the check to be case insensitive:

// And its 1252 variants
if (strcasecmp($charset, 'UTF-8') !== 0)

@Spuds Spuds added this to the 1.1.7 milestone Mar 24, 2020
@Spuds Spuds added the bug label Dec 10, 2020
Spuds added a commit to Spuds/Elkarte that referenced this issue Dec 11, 2020
@Spuds Spuds closed this as completed Dec 17, 2020
Spuds added a commit to Spuds/Elkarte that referenced this issue Dec 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants