Ordre des pages KO

[benoitdequick]

L'ordre des pages des paramètres de l'audit ne correspond plus à l'ordre des onglets de l'audit dès que l'on ajoute, supprime ou modifie la position d'une page.

L'ordre des pages doit être le même :

• dans les paramètres de l'audit
• dans les onglets de l'audit
• dans la liste des pages des résultats de l'audit
• dans la liste des pages du détails des nc
• dans la liste des pages du détails des points d'améliorations

• Reprendre URL Benoît

[bellangerq]

Les endroits où les pages sont listées :

• Création étape 2
• Audit
• Paramètres
• Rapport "Résultats"
• Rapport "Contexte"
• Rapport "Déclaration d'accessibilité"
• Rapport "Détail des non-conformités"
• Rapport "Points d'amélioration"

Règles :

• Les pages doivent toujours être triées et rendues par order.
• La page des transverses a un order de -1.
• Les pages ont un order de i + 1.

Côté back :

• Fetcher l'audit, RAS.
• Updater l'audit : pas sûr mais a priori RAS (vérifier audit.service lignes 310 - 330).
• Fetcher le rapport (pas "pages" mais "samples"), RAS.

Côté front :

• AuditSettingsForm : les data sont ref()'isées sur un getter du store (pas sûr qu'il y ait un lien avec l'ordre mais c'est sus').
• AuditGenerationPage : on sortBy() les pages récupérées du back par order pour les onglets.
• getReportErrors et getReportImprovements, turbo fonctions de la mort mais a priori trié avec sortBy() par order.

Autre :

• Il y a une vieille propriété "number" sur les pages côté back, à supprimer pour moins de confusion.

[hissalht]

Problème récurent sur la gestion des données dans les composants de formulaire :

Lorsqu'un composant a besoin de la valeur actuel d'un champs de formulaire, on copie la prop ou la valeur du store dans une ref.

Problème : quand cette valeur est un objet, c'est sa référence qui est copiée. Donc lorsque le composant modifie l'objet référencé localement, ça modifie aussi l'original.

// props.pages est une string, copie litérale
const auditName = ref(props.audit.name);

// props.pages est un objet, copie de la référence à l'objet
const pages = ref(props.pages);

function handleSomething() {
  // modifie en réalité `props.pages`
  pages.value[0].foo = "bar";

  // ne modifie bien que auditName, props.audit.name n'est pas modifié
  auditName.value = "blahblah";
}

Forcément, ça ne peut être que source de bugs. Pas sûr que ça soit lié spécifiquement à cette issue mais à creuser...

[AdrienMuzyczka]

Ordre des pages incohérent dans le rapport : je viens de reproduire un problème en prod

Je viens de me lancer dans l’audit de l’observatoire des forêts (IGN).

1. je me suis connecté à mon compte sur Ara
2. j’ai créé un nouvel audit (106 critères) depuis mon tableau de bord
3. j’ai fait mon échantillon de pages
4. j’ai finalisé : je me retrouve sur la page d’audit, avec l’onglet "transverses" sélectionné
5. j’ai cliqué sur "Consulté le rapport"
6. j’ai vu que je m’étais planté sur une page (mauvais nom)
7. je suis retourné sur l’onglet de l’audit, puis j’ai cliqué sur Paramètres
8. Là j’ai modifié le nom d’une page + supprimé une page doublon ==> nickel tout fonctionne correctement, sur l’audit et le rapport
9. Puis je décide de finalement ajouter une page à mon audit : le page "Portail géographique (carte)". J’ai envie qu’elle soit en position 15 sur 18 : je change donc grace au select.
   ==> ✅ là ça fonctionne correctement pour la page paramètres et pour la page d’audit
   ==> ❌ l’ordre des pages est complètement changé dans le rapport, au niveau de "Répartition des critères par pages"

[AdrienMuzyczka]

Tout semble fonctionner ! 🎉