Projet

Général

Profil

Bug #17291

import d'un formdef avec une balise <description> vide

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
30 juin 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Exception:
  type = '<type 'exceptions.AttributeError'>', value = ''NoneType' object has no attribute 'encode''

Stack trace (most recent call first):
  File "/home/fred/src/eo/wcs/wcs/formdef.py", line 899, in import_from_xml_tree
   897             if value is None:
   898                 continue
>  899             setattr(formdef, text_attribute, value.text.encode(charset))
   900
   901         for boolean_attribute in cls.BOOLEAN_ATTRIBUTES:

  locals:
     text_attribute = 'description'
     formdef = <FormDef "..." id:None>
     charset = 'utf-8'
     tree = <Element 'formdef' at 0x7fc85db93a50>
     value = <Element 'description' at 0x7fc86405b610>
     include_id = False
     cls = <class 'wcs.formdef.FormDef'>

Fichiers

0001-formdef-allow-empty-description-tags-on-import-17291.patch (1,72 ko) 0001-formdef-allow-empty-description-tags-on-import-17291.patch Frédéric Péters, 30 juin 2017 15:40
0001-formdef-allow-empty-description-tags-on-import-17291.patch (1,72 ko) 0001-formdef-allow-empty-description-tags-on-import-17291.patch Frédéric Péters, 29 décembre 2019 12:42

Révisions associées

Révision 1668b6b7 (diff)
Ajouté par Frédéric Péters il y a plus de 4 ans

formdef: allow empty <description/> tags on import (#17291)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 7 ans

#2

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

Patch rebasé et adapté pour la compatibilité Python 3.

#3

Mis à jour par Nicolas Roche il y a plus de 4 ans

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

J'ai l'impression que depuis ElementTree gère lui même le problème (plus d'exception).
En testant l'import (avec un export bidouillé à la main) j'ai récupéré un champs description contenant la chaîne 'None'.
Et puis le test dit pareil :

E       AssertionError: assert not 'None'
E        +  where 'None' = <FormDef 'empty' id:None>.description

Du coup, je veux bien valider (c'est tout bon et ça importe un champ description vraiment vide),
mais c'est juste que je me demande si c'est utile.

#4

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

Ton édition de commentaire pointe l'affaire :

→ contenant la chaîne 'None'.

alors qu'on veut None.

#5

Mis à jour par Nicolas Roche il y a plus de 4 ans

C'est tout bon alors.

#6

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 1668b6b7ccd4c044d57bd7834ccd34c544bbba7e
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Jun 30 15:39:58 2017 +0200

    formdef: allow empty <description/> tags on import (#17291)
#7

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

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

Formats disponibles : Atom PDF