Replying to ayeshrajans:
Bleeding-edge PHPMailer :) Thank you.
#57874 has a way better description. I don't think I can edit the original description; I'm happy with however you suggest we proceed. We could copy-paste but perhaps someone with permissions update the ticket?
I have updated the description of this ticket now.
I took the liberty of deciding to not include the new DSN class because we initialize PHPMailer on our own, use no autoloader, and because we don't include the whole PHPMailer library anyway.
That makes perfect sense to me.
Also: the Hindi translation has been updated - do we need to ping the Polyglots team to get the translation used by WP updated as well ?
From what I see, we catch PHPMailer exceptions and throw a WP_Error
with wp_mail_failed
, containing the Exception message thrown by PHPMailer. However, we don't use PHPMailer's localization, so the Exception messages would be the same in v6.8.0 too. My guess is that we don't need to bother with the translations, because as far as WordPress is concerned, we don't change any of the source text strings. I don't understand Hindi to know if the updated translations are dramatically different, but pinging them could prove useful.
Want to send out a ping in Slack ? There must be a polyglots channel there, I suppose ?