racacax / XML-TV-Fr

53 stars 29 forks source link

To bundle #12

Closed Benoit382 closed 2 years ago

Benoit382 commented 2 years ago

Parmis les choses qui me reste a faire (avant un possible merge) :

En bonus :

En complement Créer un dockerFile pour le dossier tool

Dans une autre (ou plusieurs) PR:

racacax commented 2 years ago

Bonjour, Merci pour cette PR. Pour l'instant ça avance bien. J'ai remarqué quelques quelques problèmes :

Le reste me semble correct 👍

Pour info, les changements de timezone existent pour se calquer sur les fuseaux horaires de certains sites (chaines outremer, Canada, ...). J'avais mis par défaut le timezone à Europe/Paris pour ne pas le redéfinir à chaque fois mais il serait peut-être judicieux en effet d'indiquer une timezone dans le Provider en lui-même qui se reset à la fin. Je ferai cette modification dans une future MAJ.

Benoit382 commented 2 years ago

Bonjour, Merci pour cette PR. Pour l'instant ça avance bien. J'ai remarqué quelques quelques problèmes :

* Les dossiers channels, logs et var sont manquant.

* Le fichier XML généré n'est pas reformaté (devrait utiliser les options du dom présentes dans la méthode reformatXML de Utils)

* La version de Symfony utilisée est compatible à partir de PHP 8.0.2. Ce serait bien de garder une compatibilité avec PHP 7.4 pour le moment (après, ceci n'est pas très grave)

* La méthode displayTextOnCurrentLine() a disparu. Impossible d'afficher le pourcentage au fur et à mesure de l'évolution de la récupuration des programmes (exemple MyCanal, ViniPF, TVHebdo, ...) pour les providers avec beaucoup de requêtes.

Le reste me semble correct +1

Pour info, les changements de timezone existent pour se calquer sur les fuseaux horaires de certains sites (chaines outremer, Canada, ...). J'avais mis par défaut le timezone à Europe/Paris pour ne pas le redéfinir à chaque fois mais il serait peut-être judicieux en effet d'indiquer une timezone dans le Provider en lui-même qui se reset à la fin. Je ferai cette modification dans une future MAJ.