$destinataires est un array + pb from sans from

Dans la fonction facteur_envoyer_mail, il y a $destinataires = facteur_preparer_liste_emails($destinataires); et donc $destinataires est ensuite un array car facteur_preparer_liste_emails est typée.

Peu après vient le code suivant https://git.spip.net/spip-contrib-extensions/facteur/-/edit/master/inc/facteur.php?ref_type=heads#L125

// On ajoute le courriel de l'envoyeur s'il est fournit par la fonction
if (empty($from) and empty($facteur->From)) {
  $from = $GLOBALS['meta']['email_envoi'];
  if (empty($from) or !email_valide($from)) {
    spip_log('Meta email_envoi invalide. Le mail sera probablement vu comme spam.', 'mail.' . _LOG_ERREUR);
    if (is_array($destinataires) && count($destinataires) > 0) {
      $from = $destinataires[0];
    } else {
      $from = $destinataires;
    }
  }
}

Comme $destinataires est un array,

• il est inutile et trompeur de tester is_array.
• le else est inadapté. Peut être faut il mettre '' à la place ? Loger une erreur ?