Supprimer / désactiver la page d'erreur

[Gaetanbrl]

Présentation

Bien qu'esthétique, la page d'erreur n'est pas pratique lors des développements.
Pour ces raisons, le PSC a voté pour sa suppression (ou désactivation en mode avec un paramètre ?debug=true si possible ) afin d'améliorer l'expérience développeur.

[Gaetanbrl]

Poussé sur develop.

Si url 404 :

http://localhost:5051/?config=apps/fakeedefault.xml

Si on a le paramètre debug alors on obtient code 404 ou 400 :

http://localhost:5051/?config=apps/facedefault.xml&debug=true

[Gaetanbrl]

Sinon autre alternative, c'est d'avoir un bouton de recharge :

le code pour avoir l'URL du XML et faire l'équivalent d'un F5 sur la page du XML précédente :

A rajouter dans le HTML des pages d'erreur avec un bouton

    document.querySelector("#reload").addEventListener("click", x => {
      const url = new URL(window.location.href)
      const xml = url.searchParams.get("xml");
      const newUrl = window.location.protocol + "//" + window.location.host + "?config=" + xml;
      window.location.href = newUrl;
    })

[lecault]

Bonjour Gaëtan,
Ce qui serait le top dans la seconde alternative c'est d'avoir un autre bouton qui redirige vers le XML pour voir les erreurs en live.

[Gaetanbrl]

pour voir les erreurs en live

Je ne comprend pas ce que tu souhaites ou le type d'erreurs live ?

[lecault]

Ce lien par exemple : https://kartenn.region-bretagne.fr/kartoviz/apps/region/kelenn/gestionnaire_rh.xml

Si on a une erreur de formatage XML, le navigateur nous dit où c'est

[Gaetanbrl]

Ah je vois ! C'est en effet possible c'est juste un second bouton avec un calcul du lien différent.

[Gaetanbrl]

@lecault voici un aperçu, c'est bien ca que tu souhaites ?

[lecault]

Yes super ça merci !

[Gaetanbrl]

ok je fais une PR + merge

[Gaetanbrl]

je viens de voir que je n'avais pas remis le code dans le index.html pour afficher les pages !

[lecault]

Je réouvre je viens de tester. Si l'application n'existe pas, ça mouline.
Exemple : https://kartenn.region-bretagne.fr/kartoviz-dev/?config=apps/region/contact_lycee/contact_lycees.xml

Et si l'application est en erreur aussi : https://kartenn.region-bretagne.fr/kartoviz-dev/?config=apps/region/tmp/arc.xml

J'ai essayé avec debug=true pareil : https://kartenn.region-bretagne.fr/kartoviz-dev/?config=apps/region/tmp/arc.xml&debug=true

[Gaetanbrl]

tu peux re tester @lecault je pense que c'est mieux ;)

[lecault]

C'est mieux oui mais c'est pas encore ça.

Si je prends cet exemple : https://kartenn.region-bretagne.fr/kartoviz-dev/?config=apps/region/tmp/arc.xml

Voir le XML l'url n'est pas bonne : https://kartenn.region-bretagne.fr/apps/region/tmp/arc.xml au lieu de https://kartenn.region-bretagne.fr/kartoviz-dev/apps/region/tmp/arc.xml

Et idem pour recharger la carte : https://connaissance.bretagne.bzh/cote_bretagne/?config=apps/region/tmp/arc.xml au lieu de https://connaissance.bretagne.bzh/kartoviz-dev/?config=apps/region/tmp/arc.xml (bon là il y a une histoire de redirection interne) ou https://kartenn.region-bretagne.fr/kartoviz-dev/?config=apps/region/tmp/arc.xml.

[Gaetanbrl]

l'URL n'est pas bonne en effet, je me suis basé sur des URL type https://domaine.fr/ pour retrouver le config.xml :

const newUrl = window.location.protocol + "//" + window.location.host + "?config=" + xml;

Mais dans ce cas d'URL vous avez un sous répertoire /kartoviz-dev/. Il faut retrouver le dernier path pour que ca fonctionne avec ces URLs.

[Gaetanbrl]

C'est à dire :
window.location.protocol + "//" + window.location.host + "/" + window.location.pathname

[Gaetanbrl]

je viens de pousser un test, redis moi si c'est mieux.

[lecault]

Merci Gaëtan, ça répond au besoin.

Juste un truc, ça ajoute des slashs.

Exemple : https://kartenn.region-bretagne.fr///kartoviz-dev/pages/error_400.html?xml=apps/region/tmp/arc.xml

[Gaetanbrl]

ok je regarderai (ne pas clore pour le moment)