search

tontof / kriss_feed

A simple and smart (or stupid) feed reader
282 stars 54 forks source link

Enclosure #334

Closed ghost closed 8 years ago

ghost commented 10 years ago

Yop,

J'utilise kriss_feed depuis quelques jours et j'ai remarqué qu'il ne gérait pas les pièces jointes (enclosure) dans les podcasts audio ou vidéo, donc j'ai fait l'ajout.

C'est pas parfait (il m'a fallu du temps pour comprendre comment fonctionnait la traduction tout en étant complètement incapable de la faire fonctionner), mais ça marche plutôt bien sur tous les flux que j'ai essayés.

Je ne sais pas si tu comptais l'intégrer à un moment, mais en tout cas, j'espère que ça pourra te permettre de faire une partie du travail.

tontof commented 10 years ago

J'avais pensé ma bibliothèque Rss.php pour justement gérer les enclosure plus tard. Je ne sais pas si tu avais vu pour t'aider : http://tontof.net/kriss/php5/rss/#advanced-demo

J'ai regardé vite fait ton code, ça ne gère qu'un seul enclosure ? https://github.com/tontof/kriss_feed/pull/334/files#diff-53f962c40fd5c9adb774d76b0cfe252bR30 Par rapport à ma démo qui utilise l'étoile >enclosure*[url] avec l'exemple : http://www.rssboard.org/files/example-multiple-enclosures.xml

En tout cas, c'est très chouette comme modif.

Pour la trad, j'ai vu que tu n'avais pas modifié https://github.com/tontof/kriss_feed/blob/master/src/po/messages/fr.po C'est peut-être juste ça ? En plus, il y a des problèmes avec la compil, c'est en train de changer et pour le moment, c'est un peu compliqué : https://github.com/tontof/kriss_feed/issues/330#issuecomment-30912964

Merci encore pour cet ajout qui était attendu par certains : https://github.com/tontof/kriss_feed/issues/190

Pour la traduction, c'est pas simple avec

ghost commented 10 years ago

J'avais fait la modif dans le fr.po mais apparemment, j'ai oublié de la pusher (ou de la compiler, je ne sais pas exactement).

Et effectivement, j'ai pas pensé qu'il pouvait y avoir plusieurs enclosures (pour un podcast, c'est pas fréquent néanmoins, mais comme c'est prévu pourquoi pas). D'ailleurs, j'ai pas non plus pensé que l'author pouvait se surajouter sous forme channel>itunes:author.

Bref, content que ça aide :).

tontof commented 8 years ago

J'ai enfin ajouté les enclosure https://github.com/tontof/kriss_feed/commit/3fc964453c6d6c17aed7056ebd9cc80e23281c28 Ton approche était beaucoup plus propre, mais je n'avais pas le temps de le faire proprement en ajoutant les enclosure multiple dans un template.