Project

General

Profile

Development #73770

statistiques, ajouter une colonne commune à wcs_all_form avec les données nécessaires

Added by Valentin Deniaud over 1 year ago. Updated over 1 year ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
24 January 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No
Tags:

Description

Selon un plan évoqué ici #73174#note-2, il s'agit de prendre les données des champs marqués à inclure dans les stats pour les mettre dans une nouvelle colonne de wcs_all_form, puis d'utiliser uniquement cette colonne pour calculer les stats.

Ça permettra de faire des statistiques sur plusieurs formulaires (#73174).

Et aussi de mieux gérer les champs avec identifiants dupliqués (limitations soulevées dans #73562).

Pour le moment on a pas de wcs_all_cards, donc les fiches seront laissées de côté.


Related issues

Related to w.c.s. - Bug #75573: statistiques: erreur dans set_auto_fields, champ items (TypeError: unhashable type: 'list')Fermé17 March 2023

Actions

Associated revisions

Revision 566f465d (diff)
Added by Valentin Deniaud over 1 year ago

tests: add some new statistics tests (#73770)

Revision 04e8d8c1 (diff)
Added by Valentin Deniaud over 1 year ago

formdata: aggregate field data for statistics in new column (#73770)

Revision 72f6d8e2 (diff)
Added by Valentin Deniaud over 1 year ago

admin: update statistics data on field display location change (#73770)

Revision cfe01167 (diff)
Added by Valentin Deniaud over 1 year ago

statistics: use new formdata column to compute counts (#73770)

History

#1

Updated by Anaïs Ecuvillon over 1 year ago

  • Tags set to statistiques
#2

Updated by Valentin Deniaud over 1 year ago

J'ai un truc qui fonctionne, dernier problème : aller cocher un champ pour qu'il apparaisse dans les statistiques n'a plus d'effet immédiat, il manque un bout pour que la nouvelle colonne qui reprend les données soit actualisée (un genre de set_reindex('formdata') pour un formulaire spécifique). Je creuse dans cette direction ?

#3

Updated by Frédéric Péters over 1 year ago

À la modif d'un champ, s'il est ajouté pour être utile dans les statistiques, lancer un afterjob qui va mettre à jour les démarches concernées (la démarche dans le cas d'un champ sur un formulaire, toutes les démarches associées au workflow dans le cas d'une donnée de traitement).

Comme le UpdateDigestsAfterJob, faire ça silencieusement, pas besoin d'envoyer l'agent qui a fait cette modif sur une page de progression du job; dans le cas que je cite on fait ça ainsi :

            get_response().add_after_job(UpdateDigestAfterJob(formdefs=[self.formdef]))
            if isinstance(self.formdef, CardDef):
                get_session().message = ('info', _('Existing cards will be updated in the background.'))
#4

Updated by Robot Gitea over 1 year ago

  • Status changed from Nouveau to Solution proposée

Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :

#5

Updated by Robot Gitea over 1 year ago

  • Status changed from Solution proposée to En cours

Frédéric Péters (fpeters) a relu et demandé des modifications sur une pull request sur Gitea concernant cette demande :

#6

Updated by Robot Gitea over 1 year ago

  • Status changed from En cours to Solution proposée
#7

Updated by Robot Gitea over 1 year ago

  • Status changed from Solution proposée to Solution validée

Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :

#8

Updated by Robot Gitea over 1 year ago

  • Status changed from Solution validée to Résolu (à déployer)

Valentin Deniaud (vdeniaud) a mergé une pull request sur Gitea concernant cette demande :

#9

Updated by Transition automatique over 1 year ago

  • Status changed from Résolu (à déployer) to Solution déployée
#10

Updated by Lauréline Guérin over 1 year ago

  • Related to Bug #75573: statistiques: erreur dans set_auto_fields, champ items (TypeError: unhashable type: 'list') added
#11

Updated by Transition automatique over 1 year ago

Automatic expiration

Also available in: Atom PDF