Development #15665
Étendre import/export _site comme les autres applis Publik
0%
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
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0001-misc-add-support-for-importing-site-from-stdin-15665.patch 0001-misc-add-support-for-importing-site-from-stdin-15665.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0001-general-enhance-import-export-cmdline-commands-15665.patch 0001-general-enhance-import-export-cmdline-commands-15665.patch ajouté
- Sujet changé de Prise en charge de '-' (sys.stdin) dans la commande import_site à Étendre import/export _site comme les autres applis Publik
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)
Mis à jour par Frédéric Péters il y a presque 7 ans
- Lié à Development #14630: Avoir un import-template générique ajouté
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.
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.
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).
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
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.