Development #48979
UserFieldsFormDef.store & sql
Début:
30 novembre 2020
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
dans la method store
de la class wcs.admin.settings.UserFieldsFormDef
on crée des tables SQL:
if self.publisher.is_using_postgresql(): from wcs import sql sql.do_user_table() sql.do_tracking_code_table() sql.do_session_table() sql.do_meta_table()
il en manque pas mal par rapport à ce qui est joué dans WcsPublisher.initialize_sql
origine: #24014
remplacer ce block par un appel à initialize_sql
?
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 3 ans
Je pense que plutôt on devrait réduire et uniquement avoir l'appel de sql.do_user_table()
, qui servira à ajouter/retirer les colonnes de la table.
Les autres viennent de #24014 mais je trouve ça bien bizarre, et à lire le ticket le bon paramétrage SQL serait via l'écran idoine et il appelle bien get_publisher().initialize_sql(). (ou lors du déploiement, mais certainement pas via l'écran de configuration des usagers).
Mis à jour par Lauréline Guérin il y a plus de 3 ans
- Fichier 0001-misc-update-only-user-table-on-user-fields-change-48.patch 0001-misc-update-only-user-table-on-user-fields-change-48.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 acefe0c4938a07472ac91c544a8f8e71bb6d2772 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Dec 4 16:33:29 2020 +0100 misc: update only user table on user fields change (#48979)
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
misc: update only user table on user fields change (#48979)