Bug #22889
Afficher un message clair à l'import d'une page json quand une page absente doit être importée avant
0%
Description
Aujourd'hui :
DeserializationError at /manage/site-import Page matching query does not exist.
Alors que l'erreur est simplement une page absente qu'il faut importer avant
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Fichier 0001-import-do-not-fail-if-page.parent-is-not-found-22889.patch 0001-import-do-not-fail-if-page.parent-is-not-found-22889.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Lauréline Guérin il y a presque 4 ans
Mis à jour par Frédéric Péters il y a presque 4 ans
Il me semble que la partie "# 2nd pass to set parents" devrait être conservée, elle est utile en mode import global de site, il me semble que l'erreur ici était juste sur l'import d'une unique page.
Peut-être juste faire que load_serialized_pages passe un flag quand il y a une seule page à importer, et que ce nouveau comportement ne s'applique que dans ce cas ?
Mis à jour par Lauréline Guérin il y a presque 4 ans
En fait on ne passe jamais dans le block "# 2nd pass to set parents", parce que parent_slug n'est plus renseigné. C'était le cas il y a un moment, mais depuis #8598 et la réécriture de la méthode get_serialized_page ce n'est plus le cas.
Je me trompe ?
Mis à jour par Frédéric Péters il y a presque 4 ans
J'ai l'impression en effet que c'est bon on ne passe jamais par là, parce que la sérialisation des pages se fait dans l'ordre d'apparition, que les parents arrivent toujours avant les enfants.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Yes, c'est ok. (confirmé par le coverage, on ne passait plus jamais dans ce "2nd pass to set parents").
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 91b89e6ed5116b278aa6e358aadc9e669730134e
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Mon May 11 15:34:40 2020 +0200
import: do not fail if page.parent is not found (#22889)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
import: do not fail if page.parent is not found (#22889)