@gabrieljenik please update the manual as well, since there is nothing in the manual about the fact that you can create participants with multiple email addresses.

there is nothing in the manual about the fact that you can create participants with multiple email addresses.

I'm sure it was in previous Notification & data manual page .
https://web.archive.org/web/20240413062200/https://manual.limesurvey.org/Notifications_%26_data#Basic_and_Detailed_Admin_Notifications_Email "yourname@test.com; myname@test.com;hisname@test.com"

@Mazi : did you try with ; ? (it must work with both)

@DenisChenu: When adding a new token, I think you HAVE to use comma (",") as a separator for multiple email addresses. Otherwise the syntax check will fail.
But you are right, for the admin notifications you can use a semicolon (";"). That actually works fine.

I suggest that we should support both separators, comma and semicolon at both fields.

I suggest that we should support both separators, comma and semicolon at both fields.

+1

https://github.com/LimeSurvey/LimeSurvey/pull/4282

Move to LimeMailer::validateAddresses when possible, no ?

Where you already update code

And maybe add deprecated https://github.com/LimeSurvey/LimeSurvey/blob/a7aaf89092a36e98b045344730315144f425cec0/application/helpers/common_helper.php#L1172 ?

I think this can be moved to ready to test (PR already approaved), right?

Just update the comment please : https://github.com/LimeSurvey/LimeSurvey/pull/4282/files#r2098444068

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=38850

I tested and merged the PR, thought this was approved, since the tag "Approved" was on the PR :/
@gabrieljenik can you please make a small PR to update the comment as Denis proposed?
Also, update the manual please as well.

Fixed in Release 6.14.1+250527

https://github.com/LimeSurvey/LimeSurvey/pull/4296