Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Published vs Updated #360

Open
Ginko-Aloe opened this issue Jul 5, 2015 · 2 comments
Open

Published vs Updated #360

Ginko-Aloe opened this issue Jul 5, 2015 · 2 comments

Comments

@Ginko-Aloe
Copy link

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
Copy link
Owner

tontof commented Jul 6, 2015

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
Copy link
Author

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 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants