tontof / kriss_feed

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

Bascule oublier/conserver cassée #307

Closed d-faure closed 11 years ago

d-faure commented 11 years ago

Dans la version précédente (7), on pouvait choisir de conserver ou non un élément de flux en sélectionnant successivement le bouton "oublier/conserver" qui marchait comme une bascule.

Dans la version courante (8), cela semble ne plus marcher: "conserver" (default) ==[clic bouton]==> "oublier" "oublier" ==[clic bouton]==> "oublier" au lieu de "conserver"

Du coup, on ne sait plus si l'élément à été conservé ou non.

tontof commented 11 years ago

Arf c'est un problème de traduction parce qu'il n'y a pas le problème en anglais. Je regarde ça en fin de semaine. Merci du retour.

d-faure commented 11 years ago

Le bug semble plus profond puisque aucun élément marqué "conservé" n'apparait dans la liste des conservés. En anglais, ça semble bien marcher.

tontof commented 11 years ago

C'est noté. j'ai juste regardé vite fait et c'est parce que l'url qui est appelée quand on clique sur conserver est traduite. Par exemple : http://tontof.net/feed/?currentHash=all&conserver=q2wOoQz761CQ&ajax au lieu de http://tontof.net/feed/?currentHash=all&star=q2wOoQz761CQ&ajax Mais du coup ça ne devrait pas être trop compliqué à corriger, il faudra juste que je trouve un petit moment pour le faire :-)

tontof commented 11 years ago

Normalement c'est corrigé dans la version de dev dans src Si tu veux tester c'est pas https://raw.github.com/tontof/kriss_feed/master/index.php mais https://raw.github.com/tontof/kriss_feed/master/src/index.php qu'il faut télécharger

d-faure commented 11 years ago

Parse error: syntax error, unexpected T_PAAMAYIM_NEKUDOTAYIM in /.../kriss_feed/kriss_feed.php on line 4556

tontof commented 11 years ago

Arf tu avais quelle version avant ? Ce n'est pas bien normal.

Il y a d'autres petits problèmes avec la version de développement mais je n'ai pas changé cette partie je crois.

d-faure commented 11 years ago

Bonne question :) Si j'en crois la date du .htaccess du dossier 'data', elle date du 19/09/13

Je me suis contenté de faire un backup de la version en cours et de copier-coller le contenu de https://raw.github.com/tontof/kriss_feed/master/src/index.php dans mon kiss_feed.php original (mon index.php perso se contente de faire un include_once du php après avoir customisé le stockage de la session) .

2013/11/7 tontof notifications@github.com

Arf tu avais quelle version avant ? Ce n'est pas bien normal.

Il y a d'autres petits problèmes avec la version de développement mais je n'ai pas changé cette partie je crois.

— Reply to this email directly or view it on GitHubhttps://github.com/tontof/kriss_feed/issues/307#issuecomment-27973912 .

Dominique

tontof commented 11 years ago

Bon j'ai dit n'imp j'ai bien changé le code à cet endroit :-( Tu peux utiliser la version stable en attendant que je corrige. C'est lié à PHP 5.2 je vais voir comment je peux faire mais comme j'arrive à reproduire l'erreur, ça devrait le faire :-)

tontof commented 11 years ago

C'est OK ça devrait marcher maintenant https://raw.github.com/tontof/kriss_feed/master/src/index.php