[Back - Signalement] Mettre à jour les qualifications du signalement

[hmeneuvrier]

Ticket

#1547

Description

Mettre à jour les qualifications et tags des signalements en fonction du score et des désordres et précisions.
Etendre les qualifications NDE sur tous les territoires
Algo pour les qualifications :
Algo conditions.docx

Changements apportés

• Suppression de la contrainte sur les territoires d'expérimentation pour afficher la partie NDE dans le back
• Ajout des qualifications et qualificationsstatus manquantes dans les enum
• Mise à jour du service SignalementQualificationUpdater pour les signalements créés avec le nouveau formulaire
• Appel de ce service depuis le SignalementBuilder
• Mise à jour de l'affichage des qualifications dans le back
• mise à jour des tests

Pré-requis

Tests

• Faire des signalements différents en prêtant attention aux données pour vérifier si la qualification danger ou suroccupation s'affichent bien, si toutes les qualifications des désordres sont appliquées aux signalements sans doublon et selon l'algo défini
• Faire un signalement avec des critères NDE
• Vérifier l'affichage avec des signalements créés avec l'ancien formulaire

[numew]

2 petits retours mais les tests sont ok

[emilschn]

J'ai mis des commentaires pour l'instant. J'attends les mises à jour suite échange avec Mathilde pour faire des tests.

[hmeneuvrier]

TODO : regarder si il reste des erreurs de mapping https://mattermost.incubateur.net/betagouv/pl/4b386rqksiruiq3cj4e6yqxgkh

[hmeneuvrier]

Critères NDE (réponse de Mathilde) :

Alors lignes 68 à 71 puis 76 à 83 (c'est ajouté dans le tableau)

Par contre y a 2 autres critères que je ne sais pas où mettre :

Quand on indique "Non" à Avez-vous reçu le DPE du logement ? (diagnostic performance énergie) ou à Avez-vous réalisé un DPE pour le logement ? (diagnostic performance énergie)
Quand on indique une conso d'énergie finale > 450 kWh/m²/an

[emilschn]

Mes tests sont ok sur les signalements que j'ai pu faire.
Au niveau relecture, l'ensemble est ok.
Mais je pense qu'il faut modifier la table signalement_qualification dont la colonne criticites peut renvoyer à l'id de 2 tables différentes.
Donc soit avoir 2 colonnes différentes, soit avoir une nouvelle colonne qui indique à quelle table on fait référence.

[numew]

Test et relecture ok pour moi

[sfinx13]

Début de réflexion design SignalementQualificationUpdater (A creuser) @sfinx13

Je me demande si y'a moyen de mettre toutes ces règles associés à chaque score dans un fichier de conf.

On aurait donc une boucle qui parcours le yaml afin de savoir s'il y'a une règle à appliquer selon le score, faudrait pousser la réflexion mais je voulais te la partager.

Il resterait des cas comme l'insalubrité qui n'est pas évident à première vue à configurer mais l'idée serait de s’abstenir de toutes les méthodes privées selon la plage de score.

je traduis le score 0 en yaml

scores_qualifications:
  - score_range: 0
    qualifications:
      - qualification: NON_DECENCE
        status: NON_DECENCE_CHECK
      - qualification: RSD,
         status: RSD_CHECK

En m'appuyant sur l'example yaml un exemple de traitement php

// $qualificationRules tableau PHP après avoir parsé le YAML
foreach ($qualificationRules as $rule) {
    if ($this->isScoreInRange($score, $rule['score_range'])) {
        $this->applyQualifications($signalement, $rule['qualifications']);
    }
}

https://mattermost.incubateur.net/betagouv/pl/kdg4qsfh3jd67rs8cfe9sqh3dc

[sfinx13]

Test OK nouveau et ancien signalement

[sonarcloud]

Quality Gate failed

Failed conditions

17.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud