racacax / XML-TV-Fr

45 stars 30 forks source link

Questions diverses #65

Open Aymkdn opened 2 months ago

Aymkdn commented 2 months ago

Bonjour,

Tout d'abord, bravo pour ce super projet !

Je viens juste de tester avec la config suivante :

{
  "days" : 1,
  "cache_max_days": 8,
  "output_path": "output/",
  "time_limit": 0,
  "memory_limit": -1,
  "delete_raw_xml": false,
  "enable_gz": false,
  "enable_zip": true,
  "enable_xz": false,
  "enable_dummy" : false,
  "custom_priority_orders" : {},
  "guides_to_generate" : [{"channels": "./channels.json", "filename": "xmltv.xml"}]
}

Et je vois que le programme récupère les données du 14 avril, et celle du 15 avril :

[EPG GRAB] TF1.fr (107/121) : 2024-04-14 | OK - Orange
[EPG GRAB] TF1.fr (107/121) : 2024-04-15 | OK - Orange   Et au final le fichier XML me fournit les programmes du 14 avril au matin, jusqu'au 16 avril vers 6h du matin. Le fait que je le lance le 15 avril vers 18:00, aura quel impact sur le paramètre days ? Au final mon but sera de récupérer les 12 prochains jours.

Pour franceinfo j'ai eu l'erreur HS : j'avais cru comprendre que si un service ne fonctionnait pas, il en essayait un autre, et j'ai vu que franceinfo a plusieurs services disponibles, donc comment se fait-il qu'on ait cette erreur ?

À quoi sert exactement cache_max_days ? C'est lorsqu'on relance le PHP, pour qu'il conserve dans le fichier XML les données des 8 derniers jours, sinon il les supprime ?

Merci

Benoit382 commented 2 months ago

Bonjour, "cache_max_days" n'est utile que si vous utilisez : $generator->clearCache($configurator->getCacheMaxDays()); La fonction clearCache permet de supprimer tout les fichiers de cache qui ont été créé il y a X jour (8 dans votre cas). Il y a un exemple ici : https://github.com/racacax/XML-TV-Fr/blob/master/example/script_all.php#L25, le nettoyage se fait donc à la fin.

Je ne me souvient plus trop, mais je crois que dans tout les cas, ça prend les infos de la veille pour avoir les programmes qui aurais commencé hier et fini aujourd'hui.

Je laisse @racacax confirmer et répondre pour FranceInfo