Project

General

Profile

Bug #39865

IntegrityError sur snapshot

Added by Frédéric Péters 11 days ago. Updated 6 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
14 Feb 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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/

0001-pages-add-a-default-on-Page.creation_timestamp-field.patch View (3.08 KB) Lauréline Guerin, 14 Feb 2020 02:29 PM

Associated revisions

Revision fa17bffa (diff)
Added by Lauréline Guérin 7 days ago

pages: add a default on Page.creation_timestamp field (#39865)

History

#1 Updated by Lauréline Guerin 11 days ago

  • Assignee set to Lauréline Guerin

#2 Updated by Lauréline Guerin 11 days ago

ça doit être résolu par #39768
je vais rajouter un test pour être sûre

#3 Updated by Lauréline Guerin 11 days ago

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

#4 Updated by Frédéric Péters 9 days ago

  • Status changed from Solution proposée to 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).

#5 Updated by Benjamin Dauvergne 7 days ago

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.

#6 Updated by Lauréline Guerin 7 days ago

  • Status changed from Solution validée to 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)

#7 Updated by Frédéric Péters 6 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF