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

php mail module #52

Closed
sendtogeo opened this issue Jul 6, 2016 · 6 comments · May be fixed by sendtogeo/Seo-Panel#10
Closed

php mail module #52

sendtogeo opened this issue Jul 6, 2016 · 6 comments · May be fixed by sendtogeo/Seo-Panel#10
Labels

Comments

@sendtogeo
Copy link

The daily report that arrives by email displays Greek Chars like "¿Ï�θοπαιδικα" if i go to email and set UTF-8 in the settings the chars appear good though.
I guess it has something to do with the php mail module update.

@sendtogeo sendtogeo added the bug label Jul 6, 2016
@AlexanderSk
Copy link

Is there something i can do to speed it up? Would you like me to check this issue and post a solution for the next version perhaps?

@sendtogeo
Copy link
Author

Yes sure. That will really help seo panel project. Thanks for the support.

@AlexanderSk
Copy link

AlexanderSk commented Oct 12, 2016

Hello again, i tried to Pull a request but i do not have permissions.
Anyway the fix is easy to perform:

On file: includes/sp-common.php

function sendMail($from, $fromName, $to ,$subject,$content, $attachment = ''){
$mail = new PHPMailer();
//add this line after new php mailer
$mail->CharSet = 'UTF-8';

I did test it on yesterdays cron and the email appeared fine.

@sendtogeo
Copy link
Author

Great to know that you found the fix. Greta work.

Can you please create a pull request to => https://github.com/seopanel/Seo-Panel/tree/develop

This was referenced Oct 12, 2016
@AlexanderSk
Copy link

I did create this sendtogeo#10
but do you want a pull from master into the Develop tree perhaps?

sendtogeo added a commit that referenced this issue Oct 13, 2016
@sendtogeo
Copy link
Author

Please check above commit. I have added it to the develop branch.

Thanks a lot Alex for fixing it.

Please keep supporting seo panel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants