tontof / kriss_feed

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

Unsupported operand types #283

Closed Nodd closed 11 years ago

Nodd commented 11 years ago

En mettant à jour j'obtiens sur un flux : Fatal error: Unsupported operand types in /homez.722/kontreda/www/kriss/index.php on line 3297

Le flux est valide : http://validator.w3.org/feed/check.cgi?url=http%3A%2F%2Fblog.zanorg.com%2Frss%2Ffil_rss.xml

Au passage, cette erreur bloque totalement les mises à jour des flux suivants. C'est possible d'ignorer les erreurs et de passer au suivant, ou bien une Fatal error est définitivement fatale ?

tontof commented 11 years ago

Arf je n'ai pas de problème avec ce flux : http://tontof.net/feed/?currentHash=01hj1g Et j'ai

.chez kek.  0   0.0s    Successfully updated

Par contre je pensais que ça continuais en cas d'erreur fatale, mais c'est pas sûr faudra que je regarde Pour le flux je sais pas trop. T'arrives pas du tout à l'avoir ? Si tu l'enlèves et l'ajoutes à nouveau ?

Nodd commented 11 years ago

En fait, ça continue pour certaines erreurs (par exemple celle de #285), mais celle-là bloquait. Et c'est bon si je l'enlève et le rajoute, donc tout va bien.

Du coup en vérifiant, j'ai vu un autre truc : quand une mise à jour d'un flux plante, il est quand même marqué comme mis à jour. Donc si on refait une mise à jour de tous les flux, il seront tous marqués comme déjà à jour, sans aucune erreur. C'est voulu ?

tontof commented 11 years ago

Oui l'erreur n'est pas considérée comme une non mise à jour. Donc ça, ça ne me surprend pas trop. Par contre les erreurs bloquantes, c'est plus surprenant

tontof commented 11 years ago

J'avais pas réagit mais quelqu'un d'autre m'a parlé de ce problème et je l'ai corrigé dans le dernier commit : https://github.com/tontof/kriss_feed/commit/775451f80cbf9a27e405339169f341635a37bddc Du coup ça ne devrait plus poser problème.

En fait pour que les choses soient claires. Ce problème n'était pas lié à un problème de mise à jour mais c'était une erreur php. C'est pour ça que c'était bloquant. Les erreurs fatales de mises à jour ne sont pas bloquantes.