Amélioration du système de vérification géographique #4780
Conversation
jenovateurs
changed the title
| @@ -31,12 +31,34 @@ function testGeographicalRelevancy(benefit) { | |||
| ) | |||
| }) | |||
| if (conditionGeo) { | |||
| if ( | |||
| conditionGeo.type === "attached_to_institution" || | |||
| conditionGeo.values.length > 1 | |||
There was a problem hiding this comment.
J'ai peut-être mal saisi l'objectif de cette modification mais cette condition ne me semble pas logique car dans le cas où il y a plusieurs values dans conditionGeo, alors tu quittes la méthode en retournant isValid: true et la vérification avec conditionGeo.values.every n'est pas traitée par la suite.
Ça voudrait dire qu'une aide avec plusieurs values est automatiquement valide et que seules les aides avec une seule value sont vérifiées ?
There was a problem hiding this comment.
Très bonne question.
Après avoir réanalyser le code et peut-être que ma réponse devrait être ajouté dans le wiki. On ne peux pas faire de vérification sur les listes d'epci, region, ...
On peux uniquement vérifier la cohérence entre un élément et une institution.
Car pour les listes d'éléments, c'est plus complexe. Par exemple une aide d'une métropole qui prend en conditions certaines communes et d'autres non inclus dans la métropole.
There was a problem hiding this comment.
pour le every on pourrait le modifier pour gérer directement le premier et seul élément.
There was a problem hiding this comment.
Cela répond à ta question ?
There was a problem hiding this comment.
pour le every on pourrait le modifier pour gérer directement le premier et seul élément.
Je pense que ce serait plus compréhensible de le refacto ainsi oui
Cela répond à ta question ?
Oui parfait merci (:
|
@jenovateurs j'ai testé en local avec le code postal d'Arzon et j'ai bien la tarification solidaire de l'agglo de Vannes qui s'affiche. J'ai testé avec le code postal de Bischheim et j'ai bien le FAJ de l'eurométropole de Strasbourg. |
Amélioration du système de vérification de cohérence des points géographiques.
attached_to_institution des éléments liés à une epci.
A contrôler que cela fonctionne bien. Besoin d'un petit coup de main @yasmine-glitch si possible.
Merci d'avance.