Skip to content
Snippets Groups Projects
Commit 5c158cee authored by Claudiu Cristea's avatar Claudiu Cristea
Browse files

Merge branch 'ISAICP-8709' into 'develop'

ISAICP-8709: PHP error from joinup_notification (JoinupMessageDelivery::sendMessage)

See merge request digit/digit-joinup-dev!1345
parents eb6d4094 72f77084
No related branches found
No related tags found
1 merge request!170Release v1.105.0
......@@ -20,6 +20,7 @@ Feature: Solution notifications
And users:
| Username | Roles | First name | Family name | E-mail |
| Pat Harper | moderator | Pat | Harper | pat.harper@example.com |
| Jack Harper | moderator | Jack | Harper | jack.harper@example.com |
| Ramiro Myers | | Ramiro | Myers | ramiro.myers@example.com |
| Edith Poole | | Edith | Poole | edith.poole@example.com |
And the following solutions:
......@@ -76,7 +77,7 @@ Feature: Solution notifications
| recipient | Ramiro Myers |
| subject | Joinup: You are requested to update your solution |
| body | the moderator has requested you to modify the interoperability solution - Solution notification to request changes following the following advises: Can you change this?. |
| bcc | pat.harper@example.com |
| bcc | pat.harper@example.com,jack.harper@example.com |
And the email sent to "Ramiro Myers" with subject "Joinup: You are requested to update your solution" contains the following lines of text:
| the moderator has requested you to modify the interoperability solution - Solution notification to request changes following the following advises: Can you change this?. |
| If you think this action is not clear or not due, please contact Joinup Support at |
......
......@@ -231,7 +231,7 @@ function joinup_notification_mail_alter(array &$message): void {
$bcc = array_map(function ($value) {
return $value['value'];
}, $message_entity->get('field_message_bcc')->getValue());
$message['headers']['Bcc'] = implode(';', $bcc);
$message['headers']['Bcc'] = implode(',', $bcc);
}
}
......
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