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

statistiques, ajouter une colonne commune à wcs_all_form avec les données nécessaires (#73770) #63

Merged
vdeniaud merged 4 commits from wip/73770-statistiques-ajouter-une-colonne into main 2023-02-14 18:16:16 +01:00
Conversation 0 Commits 4 Files Changed 11 +265 -29
vdeniaud commented 2023-01-25 10:25:56 +01:00
Owner
Copy Link

Pas mal d'essais infructueux et au final patches assez simple.

J'ajoute une colonne statistics_data qui contient un dico id du champ: liste de valeurs. La liste n'est nécessaire que pour les champs items et pour les données d'un bloc de champ, mais l'avoir aussi pour les champs simples permet une homogénéité qui simplifie pas mal de choses.

Ensuite niveau SQL on transforme juste les critères d'égalité en « ArrayContain » et hop tous les cas semblent couverts (il y a une bonne masse de tests et j'en rajoute même quelques uns).

Pas mal d'essais infructueux et au final patches assez simple. J'ajoute une colonne statistics_data qui contient un dico `id du champ: liste de valeurs`. La liste n'est nécessaire que pour les champs items et pour les données d'un bloc de champ, mais l'avoir aussi pour les champs simples permet une homogénéité qui simplifie pas mal de choses. Ensuite niveau SQL on transforme juste les critères d'égalité en « ArrayContain » et hop tous les cas semblent couverts (il y a une bonne masse de tests et j'en rajoute même quelques uns).
vdeniaud added 46 commits 2023-02-07 11:55:04 +01:00
gitea-wip/wcs/pipeline/pr-main This commit looks good Details
gitea/wcs/pipeline/head Something is wrong with the build of this commit Details
f22e7b4ad7 misc: reinforce french phonenumber validation (#73860)
gitea-wip/wcs/pipeline/pr-main This commit looks good Details
a10c3d2335 formdata: do not fail store on missing block (#73774)
gitea-wip/wcs/pipeline/pr-main This commit looks good Details
720df5f6a3 testdef: add support for blocks (#73332)
gitea-wip/wcs/pipeline/pr-main This commit looks good Details
ae1f92a257
i18n: translate page field in step list (#74035)
vdeniaud force-pushed wip/73770-statistiques-ajouter-une-colonne from 49150dad7c to 25512b5bd1 2023-02-07 12:05:01 +01:00 Compare
vdeniaud force-pushed wip/73770-statistiques-ajouter-une-colonne from 25512b5bd1 to a3a2b6ac28 2023-02-07 12:06:42 +01:00 Compare
fpeters requested changes 2023-02-10 14:38:12 +01:00
fpeters left a comment
Owner
Copy Link

À part ce point ça me semble tout ok.

À part ce point ça me semble tout ok.
wcs/admin/blocks.py Outdated
@ -37,1 +38,4 @@
def schedule_statistics_data_update(self):
get_response().add_after_job(UpdateDigestAfterJob(formdefs=self.objectdef.get_usage_formdefs()))
fpeters commented 2023-02-10 14:37:48 +01:00
Owner
Copy Link

Plutôt que tel quel le UpdateDigestAfterJob je préfererais un nouveau job (très simple, ça peut juste être une classe qui hérite de UpdateDigestAfterJob et lui met un attribut label adapté aux statistiques).

Plutôt que tel quel le UpdateDigestAfterJob je préfererais un nouveau job (très simple, ça peut juste être une classe qui hérite de UpdateDigestAfterJob et lui met un attribut label adapté aux statistiques).
vdeniaud commented 2023-02-13 12:45:13 +01:00
Author
Owner
Copy Link

Ajouté « UpdateStatisticsDataAfterJob ».

Ajouté « UpdateStatisticsDataAfterJob ».
vdeniaud added 4 commits 2023-02-13 12:19:22 +01:00
gitea-wip/wcs/pipeline/pr-main This commit looks good Details
271f874779 statistics: use new formdata column to compute counts (#73770)
vdeniaud requested review from fpeters 2023-02-14 17:44:14 +01:00
fpeters approved these changes 2023-02-14 18:10:17 +01:00
vdeniaud merged commit cfe0116741 into main 2023-02-14 18:16:16 +01:00
vdeniaud deleted branch wip/73770-statistiques-ajouter-une-colonne 2023-02-14 18:16:16 +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#63
No description provided.
Delete Branch "wip/73770-statistiques-ajouter-une-colonne"

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?