Development #48671
Cardef & get_data_source_custom_view
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
Historique
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)
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
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0001-cards-check-carddef-in-get_data_source_custom_view-4.patch 0001-cards-check-carddef-in-get_data_source_custom_view-4.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
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)
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
cards: check carddef in get_data_source_custom_view (#48671)