Project

General

Profile

Development #36711

"écraser avec un nouvel import", "sans danger"

Added by Frédéric Péters 8 months ago. Updated 7 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
06 Oct 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Reprendre cette fonctionnalité pour ne plus avoir ces moments avec des données incompatibles en base, ça se règle "simplement" en faisant une première passe où les colonnes de type incompatible sont supprimées. Cela corrigera directement #15379.

Dans le même temps, aussi, traiter #13599 et #32177.

0001-admin-revamp-overriding-forms-36711.patch View (17.6 KB) Frédéric Péters, 06 Oct 2019 06:42 PM


Related issues

Related to w.c.s. - Development #13599: Ne pas afficher l'écran d'avertissement de champs incompatbiles sur un "écraser avec un nouvel import" quand il n'y a aucune demande Fermé 14 Oct 2016
Related to w.c.s. - Bug #15379: crash suite à écrasement d'un formulaire, crash lié au format SQL Fermé 10 Mar 2017
Related to w.c.s. - Development #32177: à l'écrasement d'un formulaire, ne pas mettre de barre de défilement quand il y a des modifs dangereuses Fermé 10 Apr 2019

Associated revisions

Revision 2eea3725 (diff)
Added by Frédéric Péters 7 months ago

admin: revamp overriding forms (#36711)

  • always display summary of changes if there are data
  • -> skip summary of changes if there are no data (#13599)
  • do not use scrolling in summary (#32177)
  • ignore no-data fields when comparing types
  • warn about data loss instead of data corruption/bugs
  • remove data from incompatible columns (in SQL) (#15379)

History

#1 Updated by Frédéric Péters 8 months ago

#2 Updated by Frédéric Péters 8 months ago

  • Related to Development #13599: Ne pas afficher l'écran d'avertissement de champs incompatbiles sur un "écraser avec un nouvel import" quand il n'y a aucune demande added

#3 Updated by Frédéric Péters 8 months ago

  • Related to Bug #15379: crash suite à écrasement d'un formulaire, crash lié au format SQL added

#4 Updated by Frédéric Péters 8 months ago

  • Related to Development #32177: à l'écrasement d'un formulaire, ne pas mettre de barre de défilement quand il y a des modifs dangereuses added

#5 Updated by Benjamin Dauvergne 8 months ago

  • Status changed from Solution proposée to Solution validée

Ça m'a l'air tout bon; pour mon édification personnelle quelle est la raison qui fait qu'on peut se passer de use_tokens=True sur Form() et ne pas être vulnérable à une attaque CSRF ?.

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

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

Je ne saurais plus dire pourquoi ça a été fait comme ça, il faudrait chercher dans l'historique etc.

commit 2eea37253eff757d902dcf30ca31025afabe7cae
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Oct 6 18:37:41 2019 +0200

    admin: revamp overriding forms (#36711)

    * always display summary of changes if there are data
    * -> skip summary of changes if there are no data (#13599)
    * do not use scrolling in summary (#32177)
    * ignore no-data fields when comparing types
    * warn about data loss instead of data corruption/bugs
    * remove data from incompatible columns (in SQL) (#15379)

#7 Updated by Frédéric Péters 7 months ago

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

Also available in: Atom PDF