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

Error in mailing module #31766

Open
hasse-basse opened this issue Nov 11, 2024 · 5 comments
Open

Error in mailing module #31766

hasse-basse opened this issue Nov 11, 2024 · 5 comments
Assignees
Labels
Bug This is a bug (something does not work as expected) Event: Devcamp 2024 Nancy

Comments

@hasse-basse
Copy link

Bug

When sending a mass mailing, the following error appears for each email sent. Despite this, the emails are delivered successfully, and I haven’t found any consequences, except that the log fills up."

PHP Warning: Invalid argument supplied for foreach() in /home/XXX/public_html/azert/htdocs/includes/swiftmailer/lib/classes/Swift/Mime/Headers/AbstractHeader.php on line 312
PHP Warning: preg_split() expects parameter 2 to be string, array given in /home/XXX/public_html/azert/htdocs/includes/swiftmailer/lib/classes/Swift/Mime/Headers/AbstractHeader.php on line 312

Dolibarr Version

19.0.3

Environment PHP

7.4

Environment Database

Maria

Steps to reproduce the behavior and expected behavior

No response

Attached files

No response

@hasse-basse hasse-basse added the Bug This is a bug (something does not work as expected) label Nov 11, 2024
@marc-dll
Copy link
Contributor

Hello,

Can you please test with the lastest version including the FIX done in PR #31826 ?

Greetings

@hasse-basse
Copy link
Author

I works just fine. Great job, thanks. Now I can run on PHP8.2!

@hasse-basse
Copy link
Author

I'm not sure what I answered. I had 2 issues and it seem like there has been a c confusion by either of us. What work fine for me is

Emailing module throws error in PHP8.2 but not in PHP7.4 #31741

I changed the this code

FIX: swiftmailer: correctly set errors-to header #31826

with the result that

Emailing module throws error in PHP8.2 but not in PHP7.4 #31741

started to work

@hasse-basse hasse-basse reopened this Nov 15, 2024
@marc-dll
Copy link
Contributor

Sorry for being pointy, so you still have the problem reported on the first post of this issue ?

PHP Warning: Invalid argument supplied for foreach() in /home/XXX/public_html/azert/htdocs/includes/swiftmailer/lib/classes/Swift/Mime/Headers/AbstractHeader.php on line 312
PHP Warning: preg_split() expects parameter 2 to be string, array given in /home/XXX/public_html/azert/htdocs/includes/swiftmailer/lib/classes/Swift/Mime/Headers/AbstractHeader.php on line 312

@hasse-basse
Copy link
Author

I have no issue in PHP8.2 neither with fatal error or with errors in the log after manually having implemented

3791066

and moved to PHP8.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug (something does not work as expected) Event: Devcamp 2024 Nancy
Projects
None yet
Development

No branches or pull requests

2 participants