Fix recipients array when null

[immae]

Currently when the emails argument is null, the explode results in an array with an empty element:

<?php
$recipients = explode(',', null);
print_r($recipients);
?>

result:

Array
(
    [0] => 
)

This makes the job fail with Address in mailbox given [] does not comply with RFC 2822, 3.6.2.

This PR fixes that issue

[raphodn]

Salut @immae . Désolé pour le temps d'attente. tu es dans quelle coop ? vous vous servez de l'outil ?

Ta PR me semble bonne en tout cas

[immae]

Bonsoir @raphodn Je ne suis pas dans une coop mais je suis prestataire pour Nicecoop, et ils utilisent cet outil

[raphodn]

Bonsoir @raphodn Je ne suis pas dans une coop mais je suis prestataire pour Nicecoop, et ils utilisent cet outil

ca marche ! du coup vous développez l'outil, ou vous faites que de la maintenance ? quelle version de l'espace membre avez-vous actuellement ?

[raphodn]

sur la PR, j'ai trouvé ce sujet SO : https://stackoverflow.com/questions/36648491/why-does-explode-on-null-return-1-element

il y a aussi l'option de faire array_filter(explode(',', $myvar)); mais je vais prendre ta PR plutôt 👌

[immae]

ca marche ! du coup vous développez l'outil, ou vous faites que de la maintenance ? quelle version de l'espace membre avez-vous actuellement ?

Non je ne fais que le packaging et l’hébergement, et on utilise l’outil tel quel (modulo patches de fix comme celui-ci ou un autre que quelqu’un de nicecoop a soumis par le passé sur des erreurs de code postal) ; on est actuellement à la v1.32.3 (je ne mets pas à jour de moi-même, c’est à l’initiative de Nicecoop de demander une maj)

[raphodn]

ca marche ! ta PR était en fait sur develop donc j'ai refait une PR (#594) pour la mettre sur master.

et on est à la version v1.37.4 maintenant :)
n'hésites pas à refaire des PR si besoin, ou demander à NiceCoop si ils ont des besoins particuliers qu'on pourrait considérer développer

merci 👋

[immae]

C’est transmis et je vais voir pour la mise à jour, merci :)