Projet

Général

Profil

Bug #74546

l'ajout de la colonne statistics_data ne se fait pas sur wcs_all_forms

Ajouté par Frédéric Péters il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
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

Révision 629fba24 (diff)
Ajouté par Frédéric Péters il y a environ un an

sql: migrate wcs_all_forms for new statistics column (#74546)

Historique

#1

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.

#2

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.

#3

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 :

#4

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 :

#5

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 :

#6

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#8

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF