Skip to content
Snippets Groups Projects
Commit a7a409f7 authored by Uma's avatar Uma
Browse files

Fixes #1227 phpMailer updated for UTF-8 default encoding

parent 98d060f0
No related branches found
No related tags found
1 merge request!537Fixes #1227 phpMailer updated for UTF-8 default encoding
Loading
  • Contributor

    @uma.s please don't forget there is a second phpmailer class under cron/class.phpmailer.php. This would be a good opportunity to get rid of that duplication. If not, at least do the same fix over there.

  • Uma @uma.s ·
    Author Guest

    @ruben.estrada Yes! Even i noticed this and made relevant changes on Charset, But the workflows emails didn't capture the follow. Even workflows are handled through Emails class.phpmailer.php. So reverted the changes made on cron.

  • Contributor

    @uma.s so deleting cron/class.phpmailer.php wouldn't be an option? why have a dead class in the code base?

  • Uma @uma.s ·
    Author Guest

    @ruben.estrada That's right! But we need to verify again all the possible case scenario to check if it's been used else where before deletion, will do it.

  • Contributor

    Great @uma.s , thanks!

0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment