Bug #39865
IntegrityError sur snapshot
0%
Description
null value in column "creation_timestamp" violates not-null constraint
DETAIL: Failing row contains (32, Cantine, cantine, standard-sidebar, 6, t, , t, null, 2020-02-13 10:29:21.265+00,
page-pictures/picto_cantine.png, Consultez les menus des cantines scolaires et des centres de loi..., 219, {}, , null).
Request Method: GET
Request URL: https://.../snapshot/219/
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a environ 4 ans
ça doit être résolu par #39768
je vais rajouter un test pour être sûre
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Fichier 0001-pages-add-a-default-on-Page.creation_timestamp-field.patch 0001-pages-add-a-default-on-Page.creation_timestamp-field.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En fait le patch de #39768 n'est pas suffisant pour ce cas précis: on a un champ manquant, et non un champ supplémentaire inconnu dans la serialization.
J'ai remplacé le auto_now_add par un default, le default résout le pb
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
(pour ma gouverne, ce que fait auto_now_add en plus c'est mettre editable et blank à True; comme pour les pages on n'expose pas de formulaire avec ce champ, c'est ok).
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
Frédéric Péters a écrit :
(pour ma gouverne, ce que fait auto_now_add en plus c'est mettre editable et blank à True; comme pour les pages on n'expose pas de formulaire avec ce champ, c'est ok).
Je n'ai pas bien compris ce que ça change de mon coté, auto_now_add ça ne fait pas une sorte de default=timezone.now() à l'insertion ? Ou bien on conserve l'id ?
Ok j'ai vu, c'est du à une utilisation créative de deserialize.
Mis à jour par Lauréline Guérin il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit fa17bffadfa73c7f6a38cf4c4e869aa8ebd142b0
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri Feb 14 14:24:20 2020 +0100
pages: add a default on Page.creation_timestamp field (#39865)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
pages: add a default on Page.creation_timestamp field (#39865)