Produits Entr'ouvert » w.c.s.

Il y a très longtemps il y a eu confusion sur les brouillons et ils ont parfois été enregistrés avec un statut "wf-draft" plutôt que "draft", plutôt que corriger les formdata existant, le code s'est mis à gérer les deux situations. (#2758)

    def is_draft(self, status=None):
        if status is None:
            status = self.status
        return status in ('draft', 'wf-draft')

Mais pas partout et récemment il y a eu traces avec

  type = '<class 'TypeError'>', value = ''NoneType' object is not subscriptable'

Stack trace (most recent call first):
  File "/usr/lib/python3/dist-packages/wcs/formdata.py", line 1063, in get_json_export_dict
  1061         data['display_name'] = self.get_display_name()
  1062         data['text'] = self.get_display_label()
> 1063         data['receipt_time'] = datetime.datetime(*self.receipt_time[:6])
  1064         data['last_update_time'] = datetime.datetime(*self.last_update_time[:6])
  1065         data['criticality_level'] = self.criticality_level

parce que des demandes en "wf-draft" sont passées, parce qu'ailleurs seulement le statut "draft" est considéré :

        if selected_filter == 'all':
            item_ids = formdata_class.keys(**clause_kwargs)
            drafts = {x: True for x in formdata_class.get_ids_with_indexed_value('status', 'draft')}
            item_ids = [x for x in item_ids if x not in drafts]

Du coup je suis maintenant allé corriger les quelques vieux formdata concernés, et la gestion "wf-draft" peut être dégagée.