Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/hierarchy filter no rb #91

Merged

Conversation

JulienCorny
Copy link
Contributor

corrections d'une série de bugs sur les filtres de hiérarchisation :

  • veuillez indiquer le bassin versant alors que bassin versant déjà sélectionné :
    image
  • des object object dans la colonne 'connaissance' des filtres lorsqu'on rajoute des éléments à la recherche par hierarchisation suivant
  • des incohérences ou comportements inattendus dans les colonnes 'attributs' ou 'connaissance' selon la sélection de l'élément (s'ils s'agit d'une sous-rubrique ayant des valeurs de connaissance existant ou rubrique sans valeur de connaissance existantes)

image

  • correction du reset des incomplets des filtres : disable le toggle des filtres avancés et des filtres de hiérarchisation : pouvait amener des confusions et erreurs parce que lorsqu'on cliquait sur 'réinitialiser', le bassin versant était supprimé dans les filtres mais la recherche pas hiérarchisation n'était pas disable

image

// avoid object object in 'connaissance' column of the frontend table
knowledges_array.push(element['knowledges']);
element['knowledges'] = knowledges_array;
knowledges_array = [];
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c'est assez bizarre ça.

@edelclaux
Copy link
Contributor

Le cas de élection d'un RB, puis de sa déselection, était pas couvert.
Je me suis permis de faire un check un peu moins strict, et maintenant c'est ok. Il devait y avoir une autre valeur possible que undefined

3aa918a
image

@JulienCorny
Copy link
Contributor Author

Les rubriques qui n'ont pas de notes de hiérarchisation (juste présente pour ranger les sous-catégories) sont maintenant en disable dans les filtres de hiérarchisation dans le select "Rubriques". Les rubriques qui ont une note de hierarchisation restent enable (par exemple la rubrique "Fonctions écologiques" dans le cas des règles de hiérarchisation du Calavon).
Avant cette correction, il était possible de sélectionner un attribut pour une rubrique sans note ce qui donnait des résultats incohérents.

@JulienCorny JulienCorny merged commit 20a5e41 into PnX-SI:develop Dec 12, 2024
2 checks passed
@JulienCorny JulienCorny deleted the fix/hierarchy_filter_no_rb branch December 12, 2024 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants