Projet

Général

Profil

Development #48671

Cardef & get_data_source_custom_view

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:
19 novembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Fred, il manquerait pas un critère formdef_id dans la méthode CardDef.get_data_source_custom_view ?

    @classmethod
    def get_data_source_custom_view(cls, data_source_id):
        parts = data_source_id.split(':')
        if len(parts) != 3:
            return None
        lookup_criterias = [
            Equal('formdef_type', 'carddef'),
            Equal('formdef_id', parts[1]),  # <= ici
            Equal('visibility', 'datasource'),
            Equal('slug', parts[2]),
        ]
        try:
            return get_publisher().custom_view_class.select(lookup_criterias)[0]
        except IndexError:
            return None

Fichiers

Révisions associées

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

cards: check carddef in get_data_source_custom_view (#48671)

Historique

#1

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

Oui tout à fait sauf que dans parts[1] on va avoir le slug et pas l'id; il faudrait je pense ajouter un paramètre carddef à get_data_source_custom_view, les deux méthodes qui l'appellent dans carddef.py ont ça à disposition, et pour l'appel depuis data_sources.py, on laisserait None et à get_data_source_custom_view dans ce cas de faire le lookup. (lecture très rapide)

#2

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

  • Assigné à changé de Frédéric Péters à 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

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

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 93972fe1556fcc5f88f62037bdd8fafc4c071bf9
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu Nov 19 14:19:23 2020 +0100

    cards: check carddef in get_data_source_custom_view (#48671)
#6

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