Development #65539
arrêter les tentatives d'autosave après quelques erreurs
0%
Description
On fait un autosave quand des données ont changé et logiquement si l'autosave échoue on n'enregistre pas le fait que les données transférées ont été sauvegardées. Mais quand ça tombe en erreur de manière systématique (typiquement sur dépassement du délai autorisé de 200ms pour l'autosave), ça va retaper, et retaper, et retaper, et ok peut-être qu'il y a juste eu un moment pas de chance une erreur mais si ça persiste, faudrait arrêter.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 2 ans
- Fichier 0001-misc-stop-calling-autosave-after-5-errors-65539.patch 0001-misc-stop-calling-autosave-after-5-errors-65539.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
(aussi dans le patch je mets autosave_timeout_id à null après les appels à window.clearTimeout parce que je trouve ça plus net).
Mis à jour par Emmanuel Cazenave il y a presque 2 ans
J'imagine que je loupe quelque chose mais je ne comprends le error_counter++ appelé sur un évènement complete, on s'attendrait à voir ça après un évènement error.
Mis à jour par Frédéric Péters il y a presque 2 ans
complete est appelé dans tous les cas ("A function to be called when the request finishes (after success and error callbacks are executed)"), ça permet d'avoir ainsi l'incrément fait quand on a un 200 qui dit autre chose que json.result == 'success' et également fait sur une erreur réseau ou autre.
Mis à jour par Emmanuel Cazenave il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit efef11fa2ff3859911e0d431f72a822a28cbc50b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri May 20 19:25:52 2022 +0200
misc: stop calling autosave after 5 errors (#65539)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: stop calling autosave after 5 errors (#65539)