Projet

Général

Profil

Development #79804

Avoir une erreur explicite quand on veut importer une page qui référence une page inexistante

Ajouté par Nicolas Roche il y a 10 mois. Mis à jour il y a 8 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
19 juillet 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Lorsqu'on essaye d'importer une cellule menu qui référence une page qui n'existe pas,
on a un message d'erreur qui mentionne uniquement l'uuid de la page inexistante.

Ça pourrait être plus parlant de retoucher le message d'erreur du deserializer django
pour mentionner la cellule sur laquelle l'import n'a pas fonctionné.

> /home/nroche/src/combo/combo/data/models.py(635)
-> cell = list(serializers.deserialize('json', json.dumps([cell_data]), ignorenonexistent=True))[0]
(Pdb) json.dumps([cell_data])
'[{"model": "data.menucell", "fields": {"placeholder": "content", "order": 1, "slug": "", "extra_css_class": "", "template_name": null, "condition": "", "public": true, "restricted_to_unlogged": false, "last_update_timestamp": "2023-07-19T08:38:55.719Z", "depth": 1, "initial_level": -1, "root_page": ["a4e58739-ab3d-4b5d-b4eb-47b23e366a89"], "groups": [], "page": 85}}]'
(Pdb) n
django.core.serializers.base.DeserializationError: Page matching query does not exist.: (data.menucell:pk=None) field_value was '['a4e58739-ab3d-4b5d-b4eb-47b23e366a89']'

Fichiers

Révisions associées

Révision da721b70 (diff)
Ajouté par Valentin Deniaud il y a 8 mois

import: mention cell in error message if related page not found (#79804)

Révision e3832178 (diff)
Ajouté par Valentin Deniaud il y a 8 mois

fix quote style error (#79804)

Historique

#2

Mis à jour par Robot Gitea il y a 9 mois

  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Valentin Deniaud

Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :

#3

Mis à jour par Robot Gitea il y a 8 mois

  • Statut changé de Solution proposée à Solution validée

Nicolas Roche (nroche) a approuvé une pull request sur Gitea concernant cette demande :

#4

Mis à jour par Robot Gitea il y a 8 mois

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

Valentin Deniaud (vdeniaud) a mergé une pull request sur Gitea concernant cette demande :

#5

Mis à jour par Transition automatique il y a 8 mois

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

Mis à jour par Transition automatique il y a 6 mois

Automatic expiration

Formats disponibles : Atom PDF