Project

General

Profile

Bug #8862

préremplissage vs brouillons vs SQL

Added by Frédéric Péters over 7 years ago. Updated over 7 years ago.

Status:
Fermé
Priority:
Normal
Target version:
Start date:
04 November 2015
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

On décide de préremplir un champ quand aucune valeur n'est stockée (data.has_key...) mais si on a un brouillon qu'on restaure depuis SQL, tous les champs existent, à None.


Files

Associated revisions

Revision fe7f2a1e (diff)
Added by Frédéric Péters over 7 years ago

forms: store empty non required strings as empty strings (#8862)

Revision 7f6d4a35 (diff)
Added by Frédéric Péters over 7 years ago

Revert "forms: store empty non required strings as empty strings (#8862)"

This reverts commit fe7f2a1ee28d7dcdfe5700de2cdc584d227a8bd6.

History

#1

Updated by Frédéric Péters over 7 years ago

C'est un peu flippant d'aller jouer là-dedans, heureusement il y a des tests.

#2

Updated by Benjamin Dauvergne over 7 years ago

Je dirai ack mais je veux bien un deuxième coup d’œil de Thomas.

#3

Updated by Thomas Noël over 7 years ago

Ack.

Ca va modifier la sortie json ('' au lieu de None), mais c'est une amélioration, une mise en cohérence, la fin d'un bogue : je acke donc.

#4

Updated by Frédéric Péters over 7 years ago

  • Status changed from En cours to Résolu (à déployer)
commit fe7f2a1ee28d7dcdfe5700de2cdc584d227a8bd6
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Nov 4 12:47:28 2015 +0100

    forms: store empty non required strings as empty strings (#8862)
#5

Updated by Frédéric Péters over 7 years ago

  • Target version set to v1.20
#6

Updated by Frédéric Péters over 7 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF