Produits Entr'ouvert » w.c.s.

La trace, comme demandée de vive voix au bureau :

[2019-09-11 14:37:41] exception caught
Exception:
  type = '<type 'exceptions.AttributeError'>', value = ''NoneType' object has no attribute 'startswith''

Stack trace (most recent call first):
  File "/home/paul/src/wcs/wcs/data_sources.py", line 245, in get_object
   243         named_data_source.data_source = data_source
   244         return named_data_source
>  245     if ds_type.startswith('carddef:'):
   246         named_data_source = NamedDataSource()
   247         named_data_source.data_source = data_source

  locals: 
     data_source = {'value': '{{ passerelle_url }}csvdatasource/liste-des-ecoles-primaires-triees/query/type/?type={% if form_var_niveau_ecole == "Toute petite section" or form_var_niveau_ecole == "Petite section" or form_var_niveau_ecole == "Moyenne section" or form_var_niveau_ecole == "Grande section" %}maternelle{% elif form_var_niveau_ecole == "CP" or form_var_niveau_ecole == "CE1" or form_var_niveau_ecole == "CE2" or form_var_niveau_ecole == "CM1" or form_var_niveau_ecole == "CM2" or form_var_niveau_ecole == "ULIS" %}\xc3\xa9l\xc3\xa9mentaire{% endif %}'}
     ds_type = None

  File "/home/paul/src/wcs/wcs/fields.py", line 1264, in perform_more_widget_changes
  1262 
  1263     def perform_more_widget_changes(self, form, kwargs, edit=True):
> 1264         data_source = data_sources.get_object(self.data_source)
  1265 
  1266         if data_source and data_source.type == 'jsonp':

  locals: 
     edit = True
     form = <qommon.form.Form object at 0x7f5adc83df10>
     kwargs = {'required': True}
     self = <ItemField 18 '\xc3\x89cole'>

[...]

Le gros pâté du patch modifie cette ligne :

    if data_source.get('type') and data_source.get('type').startswith('carddef:'):

Qui assure déjà que le second terme ne sera pas évalué si le premier est None.

Pâté à dégager.

En fait la partie problématique est le bloc d'en dessous

if data_source.get('type') not in ('json', 'jsonp', 'formula'):

Je pensais, plutôt que d'inclure un if data_source.get('type') and … (voire une truc plus crade genre un ajout de None dans le tuple des not in (…)), faire remonter cette condition pour la cerner dans les deux blocs consécutifs. D'où le gros paté. Je me plante ?

(Et ceci, sous l'hypothèse que la lisibilité finale du code une fois patché est plus importante que la lisibilité (et la taille) du patch. Et non pas l'inverse. Je me plante aussi ici ?)

Je pensais, plutôt que d'inclure un if data_source.get('type') and … (voire une truc plus crade genre un ajout de None dans le tuple des not in (…)), faire remonter cette condition pour la cerner dans les deux blocs consécutifs. D'où le gros paté. Je me plante ?

Je ne vois pas pourquoi tu continues à parler d'une partie du code qui n'a aucune raison de changer.

datasource = {'value': "https://whatever.com/json"}

Ok via ce test je vois donc ce qu'il y avait là, mais du coup ça me semble plutôt devoir se corriger ailleurs, plutôt qu'avoir à surveiller quantité d'endroits sur des données malformées, genre :

@@ -90,7 +90,7 @@ class DataSourceSelectionWidget(CompositeWidget):
             value = self.get(name)
             if value:
                 values[name] = value
-        if values.get('type', '') == 'none':
+        if not values.get('type') or values.get('type') == 'none':
             values = None
         self.value = values or None