Projet

Général

Profil

Bug #42585

import de site, ignorer les erreurs de sources/fiches manquantes

Ajouté par Frédéric Péters il y a environ 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
06 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Il faudrait charger les modèles de fiches dans un ordre précis et interdire les cycles, ça ne me semble ni facile ni adéquat, on doit plutôt valider ça uniquement derrière.

Exception:
  type = '<class 'wcs.formdef.FormdefImportError'>', value = 'Unknown datasources'

Stack trace (most recent call first):
  File "/usr/lib/python3/dist-packages/wcs/formdef.py", line 1039, in import_from_xml
  1037         if unknown_datasources:
  1038             raise FormdefImportError(N_('Unknown datasources'),
> 1039                     details=', '.join(sorted(unknown_datasources)))
  1040
  1041         # check if all field id are unique

  locals:
     charset = None
     cls = <class 'wcs.carddef.CardDef'>
     data_source = {'type': 'carddef:parametrage-reponses-standards-rejet-demande'}
     fd = <zipfile.ZipExtFile name='carddefs_xml/1' mode='r'>
     field = <ItemField 10 'Type de réponse Rejet'>
     fix_on_error = False
     formdef = <CardDef 'Paramétrage - Pilotage Signalements' id:1>
     include_id = True
     max_field_id = 10
     tree = <xml.etree.ElementTree.ElementTree object at 0x7fba08ae9550>
     unknown_datasources = {'signalements_sur_toodego'}

  File "/usr/lib/python3/dist-packages/wcs/publisher.py", line 246, in import_zip
   244                 results['formdefs'] += 1
   245             if os.path.dirname(f) == 'carddefs_xml' and os.path.basename(f):
>  246                 carddef = CardDef.import_from_xml(z.open(f), include_id=True)
   247                 carddef.store()
   248                 carddefs.append(carddef)

Fichiers

0001-misc-don-t-check-datasources-during-bulk-import-4258.patch (3,35 ko) 0001-misc-don-t-check-datasources-during-bulk-import-4258.patch Frédéric Péters, 06 mai 2020 18:30

Révisions associées

Révision eeedc29c (diff)
Ajouté par Frédéric Péters il y a environ 4 ans

misc: don't check datasources during bulk import (#42585)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Description mis à jour (diff)
  • Assigné à mis à Frédéric Péters
#2

Mis à jour par Frédéric Péters il y a environ 4 ans

#3

Mis à jour par Thomas Noël il y a environ 4 ans

Ce que tu imagines ici c'est que lors de l'import, on suppose que toutes les sources de données sont bien incluses dans l'export ? ie "à la fin ça devrait aller" mais on ne teste pas au fur et à mesure à cause des dépendances. C'est ça ? (et a priori "à la fin", on ne teste rien du tout en réalité -- ça me pose pas tant de soucis que ça, note).

#4

Mis à jour par Frédéric Péters il y a environ 4 ans

Oui c'est ça, à l'import "global" on considère que tout va être présent.

(le seul cas particulier étant les rôles, qui ne peuvent pas manquer et doivent arriver avant via authentic, mais pour eux il y avait déjà #41252).

#5

Mis à jour par Thomas Noël il y a environ 4 ans

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

Let's go.

(Je me disais que sur /backoffice/settings/import on devrait signaler ça, que l'import doit être cohérent, mais comme on n'a rien pour tester encore, c'est un message inutile, personne ne va le lire)

#6

Mis à jour par Frédéric Péters il y a environ 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit eeedc29c4466f988295d8f31a880bcea656b7b35
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed May 6 18:29:41 2020 +0200

    misc: don't check datasources during bulk import (#42585)
#7

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

Formats disponibles : Atom PDF