Incohérence de workflow dans l'envoi par FTP

[Wilkins]

Bonjour,

Je souhaiterai signaler un problème de workflow dans la bibliothèque.

J'utilise le module maileva pour faire un envoi par FTP.
Je créé la campagne à partir d'un code similaire à https://github.com/rdarricau/maileva/blob/master/public/scenarii/package/package.php
Sauf qu'avant d'envoyer mon package par FTP, je souhaite vérifier la cohérence du XSD généré, pour être sur que mon package est valide.

Donc juste avant la ligne :

$package->generate($campaign);

Je rajoute ce code :

$domDoc = \Maileva\Element::getDomFromCampaign($campaign);
$domDoc->preserveWhiteSpace = false;
$domDoc->formatOutput = true;
libxml_use_internal_errors(true);
$valide = $domDoc->schemaValidate(\Maileva\Element::getXmlSchema());
$error = libxml_get_last_error();
libxml_clear_errors();

if (!$valide) {
    throw new \Exception("Génération du XML non valide : ".$error->message."\n XML : ".$domDoc->saveXml());
}  

Et j'obtiens une erreur de XSD, car j'ai ajouté des documents qui contiennent des caractères invalides notamment un underscore « _ ».
Pourtant, si je ne fais pas cette vérification de XSD, le package envoyé sur le FTP est tout de même valide, car la méthode \Maileva\Ftp\Package::generate() change elle même le nom des fichiers.

Je pense donc qu'il y a une petit incohérence, et que le changement de nom des fichiers dans la méthode generate devrait être effectué dans la classe \Element\Campaign, via une méthode close(). Et on pourrait ajouter une méthode validate() dans le \Element\Campaign qui vérifierait la validité selon le XSD.

En tout cas merci pour cet excellent module.
J'ai quelques patchs à soumettre, je vous envoie une PR sous peu.

PS: Je peux faire le patch si besoin.

[rdarricau]

@Wilkins Salut, salut,
As-tu une PR à soumettre sur le sujet qui nous agite ?

A ta dispo,

Rémi

[Wilkins]

Pas dans l'immédiat, mais dans le courant de la semaine prochaine, probablement.