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

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

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

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