Development #54570
export_site : arrêter d'échapper les caractères non ascii
0%
Description
Parce que ce n’est pas nécessaire pour la commande d’import, et que ça pose inutilement des soucis genre #54180-29.
Fichiers
Historique
Mis à jour par Paul Marillonnet il y a presque 3 ans
- Fichier 0001-export_site-do-not-enfore-ascii-escaping-in-json-out.patch 0001-export_site-do-not-enfore-ascii-escaping-in-json-out.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a presque 3 ans
Ce n'est pas lié à l'import d'un json, mais pour être d'équerre peut-être aussi appliquer la modification au fichier json inclus dans l'export TAR :
add_tar_content(tar, '_site.json', json.dumps(data, indent=2)edit:
et aussi sur l'export via l'IHM,
- du site /combo/manager/views.py::SiteExportView
json.dump(export_site(**form.cleaned_data), response, indent=2)
- et d'une page /combo/manager/views.py::PageExportView
json.dump({'pages': [p.get_serialized_page() for p in ordered_pages]}, response, indent=2)
- il y aurait encore ici (mais ça ne semble pas possible d'avoir de l'utf8 l'identifiant d'une ressource) : combo/apps/assets/utils.py::tar_assets_files
add_tar_content(tar, '_assets.json', json.dumps(export, indent=2))
Mis à jour par Paul Marillonnet il y a presque 3 ans
Je voudrais que le patch se concentre sur le souci actuel, qui est que la procédure de déploiement d’une nouvelle instance est boguée, elle plante lorsque le nom de la commune source n’est pas de l'ascii.
Que les bouts de code qui effectuent des exports ailleurs dans combo procèdent à cet échappement ensure_ascii n’est à mon avis pas un problème.
Mis à jour par Nicolas Roche il y a presque 3 ans
Ok, c'était pour continuer à avoir un comportement cohérent, peut être introduire une option alors ?
Mais bon, j'ai énoncé ce que j'avais en tête, je passe la main ici.
Mis à jour par Frédéric Péters il y a presque 3 ans
On ne va pas taper des options à la con pour un non-sujet (les scripts de déploiement bidons de toodego, qu'il faut appeler avec comme source une commune qui n'a pas d'accent dans son nom).
Mis à jour par Paul Marillonnet il y a presque 3 ans
- Statut changé de Solution proposée à Rejeté
Allez oui ok c’est un non-problème.