tontof / kriss_feed

A simple and smart (or stupid) feed reader
281 stars 53 forks source link

Published vs Updated #360

Open Ginko-Aloe opened 9 years ago

Ginko-Aloe commented 9 years ago

Hello,

est-ce que vous pensez qu'il serait possible de pouvoir choisir quelle date Kriss Feed utilise pour les entrées ?

Je m'explique : chaque "provider" semble utiliser les "published date" et les "Updated date" à sa sauce. Le cas qui m'intéresse est celui de youtube où la published date semble être la date réelle de publication voire de modification de la video, mais la updated correspond j'ai l'impression à la date du dernier commentaire sur la video. Évidemment ce qui m'intéresse c'est bien la date de publication de la video ("published", donc). Or il me semble que Kriss Feed utilise la "updated".

Bon, ça c'est pour tutube, mais pour pleins d'autres flux, la updated peut être plus pertinente. Du coup la seule façon que je vois de pouvoir régler mon problème ce serait d'avoir le choix, dans la configuration du flux. Genre "updated" par défaut, mais avec un radio button permettant de passer à la "published".

Voila, je ne sais pas ce que vous en pensez, mais ça serait génial pour moi d'avoir les bonnes dates ! (Et je suis trop naze en php pour réussir à patcher le bidule dans un temps raisonnable)

tontof commented 9 years ago

KrISS feed ne gère qu'une seule date. Par contre, il cherche cette dernière dans un ordre particulier. https://github.com/tontof/kriss_feed/blob/master/index.php#L4737 Je n'ai pas le temps de regarder pour voir si ça pourrait se paramétrer facilement mais il te suffit peut-être de juste inverser l'ordre de recherche. Par exemple:

'time' => array('>pubDate', '>published', '>updated', '>lastBuildDate', '>dc:date', '>date', '>created', '>modified'),
Ginko-Aloe commented 9 years ago

Aaah, c'est donc comme ça que ça marche ! (J'avais recherché "published" dans index.php et était tombé sur cette ligne, mais mes rudiments de base de php m'ont laissé sur le carreau pour comprendre de quoi il retournait.)

Quoiqu'il en soit, je peux peut-être réussir à faire un fix "quick n dirty" chez moi (genre en vérifiant si le domaine du XMLUrl correspond à youtube.com), mais j'ai aucun espoir d'arriver à ajouter un radio button dans la GUI dans un délai raisonnable. Donc si quelqu'un a une envie soudaine et généreuse (j'ai cru voir que tu n'étais pas seul à pusher sur le repo), je suis toujours preneur.

Je vous tient au jus sur ma tentative pour moduler l'ordre de recherche :)