Projet

Général

Profil

Bug #19798

variable de workflow vs génération de document

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
30 octobre 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Il peut arriver à l'action ExportToModel de ne pas fournir de widget pour son champ "backoffice_filefield_id" (s'il n'y a pas de champ backoffice de type fichier), et cette situation n'est pas gérée pour la construction du formulaire d'édition de workflow et boum.

  type = '<type 'exceptions.IndexError'>', value = 'list index out of range'

Stack trace (most recent call first):
  File "/usr/lib/python2.7/dist-packages/wcs/admin/workflows.py", line 797, in __init__
   795                         fake_form = Form()
   796                         item.add_parameters_widgets(fake_form, [parameter])
>  797                         parameter_label = fake_form.widgets[0].title
   798                         option_value = '%s / %s / %s' % (status.name, label, parameter_label)
   799                         options.append((key, option_value, key))

Fichiers

Révisions associées

Révision f258de57 (diff)
Ajouté par Frédéric Péters il y a plus de 6 ans

workflows: allow parameters not to add any widget (#19798)

Historique

#3

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

--- a/wcs/admin/workflows.py
+++ b/wcs/admin/workflows.py
@@ -799,6 +799,8 @@ class WorkflowVariableWidget(CompositeWidget):
                         key = prefix + parameter
                         fake_form = Form()
                         item.add_parameters_widgets(fake_form, [parameter])
+                        if not fake_form.widgets:
+                            continue
                         parameter_label = fake_form.widgets[0].title
                         option_value = '%s / %s / %s' % (status.name, label, parameter_label)
                         options.append((key, option_value, key))

tests laissés aux relecteurs.

#4

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

Zou.

#5

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

Avec une ligne blanche en moins parce que ça a beau être congé, on ne va pas se permettre n'importe quoi.

#6

Mis à jour par Christophe Boulanger il y a plus de 6 ans

C'est fait, ça fonctionne, merci bien :-)

#7

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

  • Statut changé de En cours à Résolu (à déployer)
commit f258de57eb260437aa8de4a102faa59db992817c
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Oct 30 13:21:02 2017 +0100

    workflows: allow parameters not to add any widget (#19798)

#8

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