Produits Entr'ouvert » w.c.s.

Le worflow est accessible sur la prod d'Arles.

A mon avis, les items vides correspondent aux expéditeurs des SMS : aux champs 'À' vides des actions 'SMS (condition)' des statuts 'Votre rendez-vous de demain' et 'Votre rendez-vous est annulé'.

Je confirme, ici j'ai laissé un champ vide.
A l'export j’obtiens :

<?xml version="1.0" encoding="utf-8"?>
<workflow id="2">
  <name>test SMS</name>
  <roles>
    <role id="_receiver">Destinataire</role>
  </roles><last_modification user_id="57">2019-08-22 10:43:24</last_modification>
  <possible_status>
    <status>
      <id>1</id>
      <name>&#233;tat1</name>
      <colour>FFFFFF</colour>
      <visibility />
      <items>
        <item id="1" type="sendsms">
          <to>
            <item>nicolas</item>
            <item />
          </to>
        </item>
      </items>
    </status>
  </possible_status>
</workflow>

Voici une proposition de patch.
Je me doute bien que je passe à côté d'une solution plus générique, mais là je sèche.
Dans le même register, et bien que ce soit un peu hors sujet, j'aurais souhaité ajouter des tests sur l'envoi d'informations via le formulaire. J'ai essayé la syntaxe suivante mais rien ne remonte :

resp.form.fields['to$element0$type'] = 'text'
resp.form.fields['to$element0$value_text'] = '0123456789'
resp.form.fields['body'] = 'my body'

Ok mais il faudrait aussi voir à corriger l'origine qui fait qu'un tel export est produit.

non, en effet j'avais zappé le "aussi".

En fait, pour plus de clarté je préférerai livrer 2 patchs distincts.

Désolé, j'ai accumulé 3 bugs dans ce ticket.
Du coup je précise un peu les patchs si dessous.

• workflow: ignore empty items on workflow import (#31823)
  Pour importer du XML avec des items vides. Erreur 500 décrite au début du ticket :
  

  AttributeError: 'NoneType' object has no attribute 'encode'
  

• tests: assert xml import-export works (#31823)
  Une mise à niveau des tests afin de faire faire en sorte que les autres tests puissent aussi détecter le bug corrigé ci-dessus. (dites-moi si ce patch est de trop)

• workflow: allow empty base64 content to be exported as xml (#31823)
  En faisant la mise à niveau expliquée ci-dessus, je suis tombé sur un bug similaire à l'import des adresses vides ci-dessus :
  

  TypeError: a2b_base64() argument 1 must be string or buffer, not None
  

• sms: ignore empty sms addresses submited by admin page (#31823)
  Pour que lorsque l'on enregistre une action SMS, on n'ait pas des adresses vides qui s'accumulent.
  (patch indépendant des précédents)

Dites-moi si ça reste compréhensible.

• dans 0001

assert_import_export_works s'occupe déjà de l'export/import XML

oui, mais sans aller jusqu'à la sérialisation XML,
or c'est lors du chargement du XML par ElementTree que les chaînes vides sont transformées en None.

• dans 0002
  j'ai choisi de renommer la fonction parce que la fonction d'origine reste requise pour 3 tests qui vérifient que les id n'ont pas changé.
  • test_export_to_model_action
  • test_global_timeout_trigger
  • test_global_webservice_trigger

• pour 0004,
  merci pour la correction générique,
  par contre je n'arrive pas à écrire de test pour la mettre en évidence.
  Le test ci-dessous fonctionne avant l'application de la correction.
  

  def test_widgetlist_widget():
      widget = WidgetList('test', value=['a', 'b', 'c'])
      mock_form_submission(req, widget, {'test$element0': 'value-a', 'test$element1': ''})
      assert widget.parse() == ['value-a', 'c']
  

Mea-cupla, j'avais effectivement inversé l'ordre des commits.
Idem pour les identifiants, ma faute : j'ai zappé le paramètre include_id=.
Merci pour les indications, ça m'a beaucoup aidé (sinon j'y serais encore).

Pour le dernier patch :
Le job était déjà fait par la superclasse Widget::_parse()

        if isinstance(value, basestring) and value.strip():
            self.value = value
        else:
            self.value = None  # ('' -> None)

        value_content = self.get('value_%s' % value_type) or ''  # (None -> '')

J'ai laissé mon premier test via les SMS pour faire ceinture et bretelles.

J'ai légèrement adapté et poussé,

commit e4a16a37c47b9ab182ea3ad6ffd774046b2a861f
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Aug 29 16:03:53 2019 +0200

    forms: do not keep empty values when parsing ComputedExpressionWidget (#31823)

commit 0cf37e198d4b5cb8f6064fce4968cf479896aead
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Aug 29 11:59:36 2019 +0200

    workflow: ignore empty items on workflow import (#31823)

commit 7ad20714aab1ee09692cb7c66878b2645173cc59
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Aug 29 11:39:09 2019 +0200

    workflow: correctly import empty model file (#31823)

commit a86a837bb44e0febae28d584af3eea54b9997e65
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Aug 29 11:17:50 2019 +0200

    tests: add full string (de)serialization to import/export tests (#31823)