Produits Entr'ouvert » Mandaye

Je préférerais qu'il y ait alors possibilité de repérer l'erreur, sans attendre l'événement de chargement d'une nouvelle page.

Par ailleurs, pour ne pas laisser de phantomjs trainer, #14606.

Ça serait utile d'avoir un commentaire à ce setTimeout final, qui dirait genre "en l'absence d'événement onload après un petit temps on regarde à nouveau la page actuelle qui peut avoir été modifiée pour indiquer succès ou erreur de l'authentifiction".

Aussi, le timeout s'annule tout seul du fait qu'une nouvelle page est chargée ?

Aussi(bis), quand on est dans le cas où on peut détecter à la fois le succès et l'échec, on pourrait avoir un setInterval rapproché, comme c'est le cas dans le onLoadFinished ? (histoire de pas imposer 700ms à l'usager)

Frédéric Péters a écrit :

Aussi, le timeout s'annule tout seul du fait qu'une nouvelle page est chargée ?

Le timeout ne s'annule pas, la fonction s'execute dans le contexte d'une page "vide", car elle est rechargée et c'est onLoadFinished qui prend le relai.

Aussi(bis), quand on est dans le cas où on peut détecter à la fois le succès et l'échec, on pourrait avoir un setInterval rapproché, comme c'est le cas dans le onLoadFinished ? (histoire de pas imposer 700ms à l'usager)

Etant donné que l'element de détection de l'échec est la présence du formulaire de login, lancer le check_auth dans un setInterval ne va rien donner car dès le premier appel il va detecter que l'authentification a échoué et ceci avant que la page se rafraichisse.

Le timeout ne s'annule pas, la fonction s'execute dans le contexte d'une page "vide", car elle est rechargée et c'est onLoadFinished qui prend le relai.

Alors il faut l'annuler manuellement en début de onload.

Etant donné que l'element de détection de l'échec est la présence du formulaire de login, lancer le check_auth dans un setInterval ne va rien donner car dès le premier appel il va detecter que l'authentification a échoué et ceci avant que la page se rafraichisse.

Mais c'est parce que la détection se fait pas sur un élément opportun, il n'y a pas, par exemple, un message "vous avez foiré" qui pourrait être détecté à la place ?

      if (!page)
          // when page reloads the object is undefined
          return

Si je comprends bien avec les explications précédentes, on lance le premier timer, quand une nouvelle page est chargée il va continuer à s'exécuter mais le check_auth retournera tôt dès ici ?

Je suggérerais d'ajouter un clearInterval(timer) dans le page.onLoadFinished(), pour évacuer l'ancien.

(aussi, ne nous épargnons pas les point-virgules).

Il doit y avoir micmac de commits, le

+  if (page.injectJs(input.auth_checker)) {
+      timer = setInterval(check_auth, 100, page);
+  }

de fin a disparu.

Ok, en fait la partie onLoadStarted / nettoyage du timer, de #14575, elle a plutôt sa place de ce côté.

Semble y avoir un problème d'indentation; et je suggérerais qu'on commence à utiliser des accolades de manière systématique.