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

Filtre dynamique non fonctionnel #291

Closed
lvinsonneau opened this issue Mar 6, 2024 · 10 comments
Closed

Filtre dynamique non fonctionnel #291

lvinsonneau opened this issue Mar 6, 2024 · 10 comments
Labels

Comments

@lvinsonneau
Copy link
Collaborator

Bonjour,
je tente de mettre un filtre dynamique sur une couche de données Mégalis présente sur GéoBretagne (en l'occurrence celle-ci).
Le filtre souhaité est le suivant : service_coffre_fort=OUI
Or dans la fenêtre, il ne me propose que la valeur "NON" alors qu'il y a bien des valeurs à "OUI" dans la BDD :
image

Il y a peut-être qqchose que je n'ai pas capté ?

@Gaetanbrl
Copy link
Member

Gaetanbrl commented Mar 6, 2024

Lorsque l'IHM des filtres dynamiques est construite, ces étapes sont réalisées :

  • appel de toutes les features de la couche
  • lecture de l'attribut sélectionné (ici service_coffre_fort)
  • création d'une liste de valeurs uniques pour ce champ

Mais je vois qu'il y a une limite de 100 (maxFeatures) dans l'appels de la requête de toutes les features. Si les 100 premières features n'ont qu'une valeur NON, alors la valeur OUI ne sera pas identifiées et non visible.

On peut considérer que c'est un bug @lvinsonneau car il ne devrait pas y avoir de limite pour lister toutes les valeurs. Il est aussi possible que ce comportement était un choix délibéré (ca ne me dit rien mais on ne sait jamais) car un appel sur une couche large va ralentir le fonctionnement le temps de récupérer et traiter toutes les features (comme ici car cette couche à + de 6 000 entités).

Dans ce cas de figure, je conseil de saisir manuellement une valeur dans le champs Valeur(s) sélectionnée(s).
En premier comme alternative à ce comportement et pour éviter des appels larges :

valider la saisie avec un appuie sur la touche "entrée" du clavier ou en cliquant en dehors du champ

dynFilter

On reste d'accord que ce comportement n'est pas tellement souhaitable car cela oblige l'utilisateur à connaître toutes les valeurs possibles comme lors de la saisie dans le XML... ce qui n'a pas dans sens dans studio car les valeurs ne sont que partiellement complète dans ce cas (pas pratique).

@Gaetanbrl Gaetanbrl added the bug label Mar 6, 2024
@Gaetanbrl
Copy link
Member

En complément de mon dernier commentaire, voici le résultat :

image

@Gaetanbrl
Copy link
Member

La valeur maxfeature=100 semble présente depuis le début.

Voir premier commit :
0ffba92#diff-fbbdf45d15fe208d2883f9619e0098453949e5c3c338cc91532959b79b26554d

@lvinsonneau
Copy link
Collaborator Author

lvinsonneau commented Mar 6, 2024

@Gaetanbrl : j'avais essayé de saisir manuellement la valeur OUI, mais chez moi, ça ne marche pas... bizarre, je ne dois pas m'y prendre comme il faut
ça n'empêche que mon besoin reste de filtrer et ce, même s'il y a 6000 entités

@Gaetanbrl
Copy link
Member

ça n'empêche que mon besoin reste de filtrer et ce, même s'il y a 6000 entités

Comme indiqué en bas de mon commentaire :

On reste d'accord que ce comportement n'est pas tellement souhaitable car cela oblige l'utilisateur à connaître toutes les valeurs possibles comme lors de la saisie dans le XML... ce qui n'a pas dans sens dans studio car les valeurs ne sont que partiellement complète dans ce cas (pas pratique).

@lvinsonneau
Copy link
Collaborator Author

OKèche, mais moi pas réussir avec solution de contournement... alors moi retourne au mviewer en attendant

Gaetanbrl added a commit that referenced this issue Mar 6, 2024
Gaetanbrl added a commit that referenced this issue Mar 6, 2024
@Gaetanbrl
Copy link
Member

Un correctif a été apporté dans la branche develop. La prochaine release bénéficiera de ce correctif.

@Gaetanbrl
Copy link
Member

@lvinsonneau

mais moi pas réussir avec solution de contournement... alors moi retourne au mviewer en attendant

Je comprend qu'il y a un bug sur l'utilisation de ce champ en plus d'une limite MAX qui est un autre sujet.

Est-ce que tu peux m'en dire plus stp sur ce que tu vois stp ?

@lvinsonneau
Copy link
Collaborator Author

lvinsonneau commented Mar 6, 2024

yes, on peut se faire un partage d'écran via element, ce sera + simple non ?
(si c'est possible?)

@Gaetanbrl
Copy link
Member

vu avec @lvinsonneau, l'alternative fonctionne bien.
Le ticket reste ouvert et sera à fermer après vérification du fonctionnement lié à la suppression du MAXFEATURES.

@lecault lecault closed this as completed Jul 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants