Projet

Général

Profil

Development #52127

appels data_class() depuis un objet snapshot

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
17 mars 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Idée du moment, en explorant une version passée via l'historique, on pourrait arriver dans du code qui appelle data_class() ce qui pourrait amener les colonnes de la base de données à être adaptées pour matcher.

La plupart des appels vont être lors de la validation d'actions, ce qui n'aura pas lieu vu que ces boutons n'auront pas été mis parce que is_readonly() mais certains appels ont lieu dès l'affichage, je vois :

        disabled_url_name = bool(self.formdef.data_class().count())

et il y en a peut-être d'autres.


Fichiers

Révisions associées

Révision 68537fdb (diff)
Ajouté par Frédéric Péters il y a environ 3 ans

sql: don't alter tables for snapshot objects (#52127)

Révision 5b54f6ed (diff)
Ajouté par Frédéric Péters il y a environ 3 ans

sql: don't alter tables for snapshot objects (#52127)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 3 ans

(je ne peux pas creuser aujourd'hui mais patch posé comme ça et à mon avis déjà utile ainsi).

#2

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

Je suis allé voir sur l'instance HDS de prod si ça pourrait expliquer la problèmes au CD13, mais les logs du conteneur w.c.s. ne vont pas assez loin (9 mars, ticket du 19 février https://dev.entrouvert.org/issues/51278#change-288848) et je n'ai pas trouvé de haproxy pour en voir les logs que normalement on doit garder 1 an.

#3

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

  • Statut changé de Solution proposée à Solution validée
#5

Mis à jour par Thomas Noël il y a environ 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 68537fdbad058f5f0b5db1009c52622ee558a2be
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Mar 17 10:39:48 2021 +0100

    sql: don't alter tables for snapshot objects (#52127)

#6

Mis à jour par Thomas Noël il y a environ 3 ans

+ hotfix/v6.18

#7

Mis à jour par Frédéric Péters il y a environ 3 ans

merci.

#8

Mis à jour par Frédéric Péters il y a environ 3 ans

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

Formats disponibles : Atom PDF