Projet

Général

Profil

Bug #23855

export/import de workflows avec condition sur les actions

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
16 mai 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Éditer une action produit {} dans l'attribut condition (plutôt que None), le résultat c'est ensuite que l'export contient <condition/>, comme il n'y a pas de nœud à l'intérieur c'est le code de compatibiité qui est appelé, mais il ne trouve pas de node.text, et boom.

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/workflows.py", line 1759, in condition_init_with_xml
  1757         else:
  1758             # backward compatibility
> 1759             self.condition = {'type': 'python', 'value': node.text.encode(charset)}
  1760
  1761     def q_admin_lookup(self, workflow, status, component, html_top):

  locals:
     charset = 'utf-8'
     include_id = False
     node = <Element 'condition' at 0x7fd9c8639610>
     self = <wcs.workflows.SendmailWorkflowStatusItem object at 0x7fd9c860d210>

  File "/home/fred/src/eo/wcs/wcs/workflows.py", line 806, in init_with_xml
   804             if getattr(self, '%s_init_with_xml' % attribute, None):
   805                 getattr(self, '%s_init_with_xml' % attribute)(el, charset,
>  806                         include_id=include_id)
   807                 continue
   808             if el is None:

  locals:
     attribute = 'condition'
     charset = 'utf-8'
     el = <Element 'condition' at 0x7fd9c8639610>
     elem = <Element 'item' at 0x7fd9c8639650>
     include_id = False

Fichiers

Révisions associées

Révision 7b0b2a35 (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

workflows: handle empty <condition> tags in exports (#23855)

Révision a2fab65a (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

misc: save empty conditions as None (#23855)

Historique

#2

Mis à jour par Thomas Noël il y a presque 6 ans

Good catch.

Ack.

#3

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

  • Statut changé de En cours à Résolu (à déployer)
commit a2fab65a141bbc2293339e4b9c58da3168a3fd0a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed May 16 09:46:32 2018 +0200

    misc: save empty conditions as None (#23855)

commit 7b0b2a35e6f791094a66c8dba01663e763200381
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed May 16 09:38:24 2018 +0200

    workflows: handle empty <condition> tags in exports (#23855)
#4

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

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

Formats disponibles : Atom PDF