Projet

Général

Profil

Development #78591

critères avancés, traitement, "between" et champ liste

Ajouté par Frédéric Péters il y a 11 mois. Mis à jour il y a 10 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
16 juin 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Deux champs liste sont bien affichés mais leur contenu est vide.

Révisions associées

Révision 27ebeda7 (diff)
Ajouté par Corentin Séchet il y a 10 mois

backoffice: fix autocomplete items dropdown for between, in and not_in operators (#78591)

Révision e84cb185 (diff)
Ajouté par Lauréline Guérin il y a 10 mois

backoffice: fix get_item_filter_options for Nothing criterias (#78591)

Révision 81c4890a (diff)
Ajouté par Lauréline Guérin il y a 10 mois

backoffice: just ignore between operator with wrong parameters (#78591)

Historique

#2

Mis à jour par Corentin Séchet il y a 11 mois

  • Assigné à mis à Corentin Séchet

J'attends https://dev.entrouvert.org/issues/78590 pour pouvoir tester avec et essayer de pas créer plus de bugs.

#3

Mis à jour par Robot Gitea il y a 11 mois

  • Statut changé de Nouveau à Solution proposée

Corentin Sechet (csechet) a ouvert une pull request sur Gitea concernant cette demande :

#4

Mis à jour par Corentin Séchet il y a 10 mois

Il y avait une erreur JS sur une référence inexistante au add_button, j'ai pushé un correctif. Ceci dit, il y a un autre problème, auquel je n'ai pas de solution :

  • On sélectionne l'opérateur "between" sur un champ
  • Le formulaire de filtre est soumis, avec deux valeurs vides pour le le filtre between
  • Ça crée une erreur dans le backend, aucune fiche n'est filtrée et le select du filtre est mis à jour avec une liste d'options disponibles vides

La seule piste que j'aurai serait de mettre l'attribut "disabled" à true sur tous les champs (opérateurs, valeurs) de filtres between qui ont une borne non remplie avant de submit ou sérialiser le formulaire, pour éviter que le filtre à moitié rempli soit envoyé au backend, les remettre à true dans un setTimeout. Dans ce cas on a un comportement différent puisque le champ n'est pas pris en compte tant que les deux champs ne sont pas remplis. Ça crée d'autres bugs (race conditions qui font que les champs restent désactivés), et ça ne fonctionne pas (select vides quand même sans que j'ai pu encore trouver pourquoi).

#5

Mis à jour par Lauréline Guérin il y a 10 mois

Des modifs ajoutées côté back pour
1/ ignorer le critère Nothing dans get_item_filter_options, c'est ce qui produisait des select sans options avec l'opérateur between
2/ ne pas avoir de message de warning Valeur invalide (« | ») pour l’opérateur « between » et le filtre « xxx » sur la vue de traitement

Et ça me semble bon comme ça !

#6

Mis à jour par Robot Gitea il y a 10 mois

  • Statut changé de Solution proposée à Solution validée

Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Robot Gitea il y a 10 mois

  • Statut changé de Solution validée à Résolu (à déployer)

Corentin Sechet (csechet) a mergé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Transition automatique il y a 10 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Transition automatique il y a 8 mois

Automatic expiration

Formats disponibles : Atom PDF