Projet

Général

Profil

Bug #47764

Carddef, custom view & datasource

Ajouté par Lauréline Guérin il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
15 octobre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Définir une custom view de Fiche, visibility datasource, avec un filtre sur le statut des fiches
Utiliser la datasource dans un formulaire (champ list)

=> le statut n'est pas pris en compte pour le filtrage des values du select


Fichiers

Révisions associées

Révision f5434332 (diff)
Ajouté par Lauréline Guérin il y a plus de 3 ans

cards: filter on status if relevant in custom view as datasource (#47764)

Historique

#2

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Assigné à mis à Lauréline Guérin
#3

Mis à jour par Lauréline Guérin il y a plus de 3 ans

#4

Mis à jour par Frédéric Péters il y a plus de 3 ans

-        criterias = [NotEqual('status', 'draft')]
+        criterias = []

J'ai un peu peur d'une situation où criterias.extend(form_page.get_view_criterias()) retournerait quelque chose mais où il n'y aurait pas de selected_filter (ou == 'all'), et que dans ce cas on ne passe pas par

        if not criterias:
            criterias = [NotEqual('status', 'draft')]

et qu'on se retrouve avec peut-être des brouillons dans la liste.

Je serais pour tout le temps avoir le NotEqual('status', 'draft'), même si ça sonne ridicule à côté d'un Equalt('status', 'autre-chose'). (je fais totale confiance à postgresql pour ne pas perdre de temps là-dessus).

                    try:
                        applied_filters = ['wf-%s' % selected_filter]
                    except KeyError:
                        applied_filters = None

je ne capte pas la possibilité de KeyError ici.

#5

Mis à jour par Lauréline Guérin il y a plus de 3 ans

j'ai rétabli le filtre sur draft
pour le try/except KeyError, erreur de copier/coller :)

#6

Mis à jour par Frédéric Péters il y a plus de 3 ans

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

J'aurais pu le voir plus haut, il existe un opérateur Contains, tu pourrais juste faire

criterias.append(Contains('status', applied_filters))

plutôt que le Or[Equal...].

(mais je valide déjà)

#7

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit f54343327d1a99b4c6ad5313102403488f03c934
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Fri Oct 16 09:59:33 2020 +0200

    cards: filter on status if relevant in custom view as datasource (#47764)

J'ai changé le Or en Contains

#8

Mis à jour par Frédéric Péters il y a plus de 3 ans

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

Formats disponibles : Atom PDF