entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 29 Releases Activity

statistiques, permettre de sélectionner plusieurs formulaires (#73174) #94

Merged
fpeters merged 2 commits from wip/73174-statistiques-permettre-de-select into main 2023-03-03 10:59:09 +01:00
Conversation 0 Commits 2 Files Changed 2 +255 -99
vdeniaud commented 2023-02-07 12:28:10 +01:00
Owner
Copy Link
No description provided.
vdeniaud force-pushed wip/73174-statistiques-permettre-de-select from b15a44c400 to ce84e80a8d 2023-02-07 13:57:52 +01:00 Compare
fpeters requested changes 2023-02-10 14:46:56 +01:00
wcs/statistics/views.py Outdated
@ -278,0 +241,4 @@
criterias = [Contains('url_name', formdef_slugs), Contains('category_id', category_ids)]
formdefs = self.formdef_class.select([Or(criterias)])
if not formdefs:
fpeters commented 2023-02-10 14:46:52 +01:00
Owner
Copy Link

Il me semble que ça ne peut pas arriver parce que le regroupement sur plusieurs est possible uniquement pour les formulaires, pas les fiches, mais comme on utilise ici self.formdef_class on peut penser le contraire, et se dire que le Category au-dessus devrait varier selon qu'on est sur des formulaires ou des modèles de fiche.

Si c'est bien comme ça, je pense quand même qu'un commentaire pourrait être ajouté, pour dire qu'on est conscient que Category concerne uniquement les formulaires, pas les modèles de fiche.

Détail, sur le Category.select(), tu peux ajouter ignore_errors=True, pour s'éviter un crash inutile en cas de suppression de catégorie.

Il me semble que ça ne peut pas arriver parce que le regroupement sur plusieurs est possible uniquement pour les formulaires, pas les fiches, mais comme on utilise ici self.formdef_class on peut penser le contraire, et se dire que le Category au-dessus devrait varier selon qu'on est sur des formulaires ou des modèles de fiche. Si c'est bien comme ça, je pense quand même qu'un commentaire pourrait être ajouté, pour dire qu'on est conscient que Category concerne uniquement les formulaires, pas les modèles de fiche. Détail, sur le Category.select(), tu peux ajouter ignore_errors=True, pour s'éviter un crash inutile en cas de suppression de catégorie.
vdeniaud commented 2023-02-13 12:44:33 +01:00
Author
Owner
Copy Link

Yep les fitres « category:xxx » n'apparaissent pas pour les fiches. Pas d'inspi pour le commentaire (et il risquerait d'être oublié lors d'une mise à jours du code où on introduirait un wcs_all_cards), j'ai préféré mettre ça dans une méthode à part, méthode qu'on fait différer entre la vue formulaire et la vue fiche.

Yep les fitres « category:xxx » n'apparaissent pas pour les fiches. Pas d'inspi pour le commentaire (et il risquerait d'être oublié lors d'une mise à jours du code où on introduirait un wcs_all_cards), j'ai préféré mettre ça dans une méthode à part, méthode qu'on fait différer entre la vue formulaire et la vue fiche.
vdeniaud force-pushed wip/73174-statistiques-permettre-de-select from ce84e80a8d to 68646adb70 2023-02-13 12:41:31 +01:00 Compare
vdeniaud force-pushed wip/73174-statistiques-permettre-de-select from 68646adb70 to ce24143001 2023-02-20 10:09:25 +01:00 Compare
vdeniaud requested review from fpeters 2023-02-20 10:09:41 +01:00
vdeniaud force-pushed wip/73174-statistiques-permettre-de-select from ce24143001 to 6d6a1a6a3c 2023-02-27 09:50:53 +01:00 Compare
vdeniaud force-pushed wip/73174-statistiques-permettre-de-select from 6d6a1a6a3c to 2f8bd56a90 2023-02-28 10:52:09 +01:00 Compare
fpeters approved these changes 2023-03-02 15:07:44 +01:00
vdeniaud force-pushed wip/73174-statistiques-permettre-de-select from 2f8bd56a90 to dda87319bb 2023-03-02 15:14:40 +01:00 Compare
fpeters merged commit 02433859f1 into main 2023-03-03 10:59:09 +01:00
fpeters deleted branch wip/73174-statistiques-permettre-de-select 2023-03-03 10:59:09 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#94
No description provided.
Delete Branch "wip/73174-statistiques-permettre-de-select"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?