Projet

Général

Profil

Development #15665

Étendre import/export _site comme les autres applis Publik

Ajouté par Frédéric Péters il y a presque 7 ans. Mis à jour il y a plus de 5 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Sur de la réplication j'étais parti sur l'idée de faire tenant_command export_site -d ... | tenant_command import_site -d ... - et là j'ai découvert qu'import_site ne pouvait pas utiliser l'entrée standard.


Fichiers


Demandes liées

Lié à Hobo - Development #14630: Avoir un import-template génériqueFermé17 janvier 2017

Actions

Révisions associées

Révision 48065634 (diff)
Ajouté par Frédéric Péters il y a presque 7 ans

general: enhance import/export cmdline commands (#15665)

This unifies behaviour with other publik components, it adds support for
stdin/stdout using '-' as filename, and adds if-empty and clean flags to
the import command.

Historique

#1

Mis à jour par Frédéric Péters il y a presque 7 ans

#2

Mis à jour par Frédéric Péters il y a presque 7 ans

Au-delà de la prise en charge du -, il y a d'autres bouts à unifier avec les autres applications, voici un patch qui ajoute également la prise en charge de --clean et --if-empty. (ce patch intègre le précédent)

#3

Mis à jour par Frédéric Péters il y a presque 7 ans

#4

Mis à jour par Thomas Noël il y a presque 7 ans

  • indent=4, je pensais que pour json c'était 2.
  • pour le clean « Page.objects.all().delete() » : on est sûr que ça efface les cellules (par cascade ?) ?

Voilà, c'était juste histoire de faire croire que j'ai relu :)

Ack.

#5

Mis à jour par Frédéric Péters il y a presque 7 ans

  • indent=4, je pensais que pour json c'était 2.

Je préfère 2 mais par uniformité avec Passerelle qui est à 4 j'ai tapé 4.

  • pour le clean « Page.objects.all().delete() » : on est sûr que ça efface les cellules (par cascade ?) ?

J'allais dire que oui mais je n'ai honnêtement pas vérifié. Je regarde ça demain.

#6

Mis à jour par Frédéric Péters il y a presque 7 ans

  • Statut changé de En cours à Résolu (à déployer)

pour le clean « Page.objects.all().delete() » : on est sûr que ça efface les cellules (par cascade ?) ?

J'allais dire que oui mais je n'ai honnêtement pas vérifié. Je regarde ça demain.

Du coup ça m'a fait voir que j'avais oublié le git add de tests/test_import_export.py, qui vérifie justement ça.

commit 48065634d287992a36eead17b634b0c2dcb18d4b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sun Apr 2 14:29:10 2017 +0200

    general: enhance import/export cmdline commands (#15665)

    This unifies behaviour with other publik components, it adds support for
    stdin/stdout using '-' as filename, and adds if-empty and clean flags to
    the import command.

(et indentation remise à 2, parce que c'est plus commun que 4).

#7

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

Formats disponibles : Atom PDF