Bug #74546
l'ajout de la colonne statistics_data ne se fait pas sur wcs_all_forms
Début:
15 février 2023
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Non
Planning:
Non
Description
https://sentry.entrouvert.org/entrouvert/publik/issues/106387/
Le service est démarré mais les opérations sur des demandes vont échouer comme ça :
SyntaxError: INSERT has more expressions than target columns LINE 25: NEW.statistics_data) ^ QUERY: INSERT INTO wcs_all_forms VALUES ( 23, 183, NEW.id, NEW.user_id, NEW.receipt_time, NEW.status, NEW.id_display, NEW.submission_agent_id, NEW.submission_channel, NEW.backoffice_submission, NEW.last_update_time, NEW.digests, NEW.user_l... File "wcs/compat.py", line 219, in process_request output = self.try_publish(request) File "quixote/publish.py", line 241, in try_publish output = self.root_directory._q_traverse(components) File "wcs/root.py", line 306, in _q_traverse return Directory._q_traverse(self, path) File "quixote/directory.py", line 88, in _q_traverse return obj._q_traverse(path) File "quixote/directory.py", line 116, in _q_traverse return super(AccessControlled, self)._q_traverse(path) File "quixote/directory.py", line 88, in _q_traverse return obj._q_traverse(path) File "quixote/directory.py", line 92, in _q_traverse return obj() File "wcs/forms/root.py", line 1312, in _q_index filled = self.save_draft(form_data, page_no) File "wcs/forms/root.py", line 1578, in save_draft filled.store(where=where) File "wcs/sql.py", line 714, in f return func(*args, **kwargs) File "wcs/qommon/substitution.py", line 27, in f return func(*args, **kwargs) File "wcs/sql.py", line 2691, in store cur.execute(sql_statement, sql_dict)
Il faudrait passer une partie de la migration en synchrone (la création de la colonne, pas son alimentation).
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ un an
Peut-être autre chose, ça parle de wcs_all_forms qui n'a pas la colonne.
Mis à jour par Frédéric Péters il y a environ un an
- Sujet changé de l'ajout de la colonne statistics_data se fait en arrière-plan (mais le code ne gère pas ça) à l'ajout de la colonne statistics_data ne se fait pas sur wcs_all_forms
- Assigné à changé de Valentin Deniaud à Frédéric Péters
Je prends vu que j'ai commencé à réparer.
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Nouveau à Solution proposée
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/111
- Titre : sql: migrate wcs_all_forms for new statistics column (#74546)
- Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/111/files
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution proposée à Solution validée
Pierre Ducroquet (pducroquet) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/111
- Titre : sql: migrate wcs_all_forms for new statistics column (#74546)
- Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/111/files
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
sql: migrate wcs_all_forms for new statistics column (#74546)