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

Incorrect past participle agreement in phpmailer.lang-fr.php #2993

Closed
6 tasks done
Blue3957 opened this issue Dec 6, 2023 · 0 comments · Fixed by #2992
Closed
6 tasks done

Incorrect past participle agreement in phpmailer.lang-fr.php #2993

Blue3957 opened this issue Dec 6, 2023 · 0 comments · Fixed by #2992

Comments

@Blue3957
Copy link
Contributor

Blue3957 commented Dec 6, 2023

Please check these things before submitting your issue:

  • Read the error message you're seeing - it often tells you what is wrong, and may contain useful links & instructions
  • Make sure you're using the latest version of PHPMailer
  • Check that your problem is not dealt with in the troubleshooting guide, especially if you're having problems connecting to Gmail or GoDaddy
  • Include sufficient code to reproduce your problem
  • If you're having an SMTP issue, include the debug output generated with SMTPDebug = 2 set
  • If you have a question about how to use PHPMailer (rather than reporting a bug in it), tag a question on Stack Overflow with phpmailer, but search first!

Problem description

There's a grammatical error in the french localization file. It's for the error I encounter most often and it's been bugging me for some time.

As is, the message reads "La fonction SMTP connect() a échouée". "échouée" should be "échoué", because, when using the "avoir" auxiliary, the past participle agrees with the direct object, and never the subject. In the absence of a direct object (as is the case here), it shouldn't agree with anything, and the base form should be used.

Here is an english-language source for my claim.

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

Successfully merging a pull request may close this issue.

1 participant