Project

General

Profile

Bug #22889

Afficher un message clair à l'import d'une page json quand une page absente doit être importée avant

Added by Anonymous over 2 years ago. Updated 18 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
29 Mar 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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

0001-import-do-not-fail-if-page.parent-is-not-found-22889.patch View (9.11 KB) Lauréline Guerin, 11 May 2020 03:35 PM

0001-import-do-not-fail-if-page.parent-is-not-found-22889.patch View (9.14 KB) Lauréline Guerin, 12 May 2020 05:04 PM

Associated revisions

Revision 91b89e6e (diff)
Added by Lauréline Guérin 20 days ago

import: do not fail if page.parent is not found (#22889)

History

#1 Updated by Lauréline Guerin about 2 months ago

  • Assignee set to Lauréline Guerin

#2 Updated by Lauréline Guerin about 2 months ago

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

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 ?

#5 Updated by Lauréline Guerin about 2 months ago

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 ?

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

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.

#7 Updated by Lauréline Guerin about 2 months ago

du coup tu valides ? :)

#8 Updated by Frédéric Péters 21 days ago

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

Yes, c'est ok. (confirmé par le coverage, on ne passait plus jamais dans ce "2nd pass to set parents").

#9 Updated by Lauréline Guerin 20 days ago

  • Status changed from Solution validée to 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)

#10 Updated by Frédéric Péters 18 days ago

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

Also available in: Atom PDF