Produits Entr'ouvert » w.c.s.

Frédéric Péters a écrit :

gérer lors de l'affichage des champs, champ par champ, la possibilité d'erreur et dans ce cas remplacer le champ en erreur par un message disant ouille ?

Je verrais bien cela, sur le champ en erreur (et éventuellemnt en gros warning en haut de la page formulaire). Mais j'ai du mal à imaginer comment ça pourrait se faire (ou alors à travers une exception dédiée "HoustonOnAUnProbleme(message)" qui s'afficherait en rouge ?).

En tout cas, ne pas silencieusement ignorer l'erreur.

(Voir aussi #15163, c'est pas la même chose mais c'est lié, pour éviter de se retrouver dans certaines de ces situations)

Tu ne devrais a priori pas avoir à introduire deux nouvelles méthodes et modifier tout un existant pour appeler celles-ci. Et ça ne devrait pas introduire de jeu de piste de méthode qui appelle des méthodes passées en paramètre.

En gros tu as fait le tour, il faut que data_sources.get_real, data_sources.get_items, data_sources.get_object, data_sources.get_structured_items n'échouent jamais sur une source de données manquantes, et pour arriver à ça, en regardant vite fait, ça doit pouvoir se passer en ne touchant pas à ces méthodes même mais en remontant d'un cran, sur NamedDataSource.get_by_slug.

Sans doute pas été assez clair, ce patch ne passera pas.

Compris, je retente ma chance.

Toujours pas bien fan, toujours l'idée qu'il est possible d'avoir moins d'impact,

Si tout passe au final bien par get_by_slug, plutôt que retourner None et gérer le cas particulier à n endroits (ce qui est somme toute pareil que laisser le KeyError se lever et le gérer à n endroits), je lui ferais retourner une datasource qui fonctionne,

    ...
    get_logger().warn("data source '%s' does not exist" % slug)
    stub = NamedDataSource()
    stub.data_source = {'type': 'formula', 'value': '[]'}
    return stub

ou une variation de l'idée, genre

class StubDataSource(object):
    data_source = {'type': 'formula', 'value': []}

...
    get_logger().warn("data source '%s' does not exist" % slug)
    return StubDataSource()

+                        'This datasource is still used in some formularies.')))

(on n'a pas "formularies" dans notre vocabulaire), mais passons, mettons plutôt : "This datasource is still used, it cannot be deleted."

Plus important, get_formdefs_of_all_kinds peut retourner des None dans sa liste (cas d'une action "formulaire" pas configurée), et ça plante du coup derrière ainsi,

Exception:
  type = '<type 'exceptions.AttributeError'>', value = ''NoneType' object has no attribute 'fields''

Stack trace (most recent call first):
  File "/home/fred/src/eo/wcs/wcs/data_sources.py", line 424, in is_used_in_formdef
   422     def is_used_in_formdef(self, formdef):
   423         from .fields import WidgetField
>  424         for field in formdef.fields or []:
   425             data_source = getattr(field, 'data_source', None)
   426             if not data_source:

  locals:
     WidgetField = <class 'wcs.fields.WidgetField'>
     formdef = None
     self = <NamedDataSource 'xxxxxx' id:8>

  File "/home/fred/src/eo/wcs/wcs/admin/data_sources.py", line 38, in is_used
    36     def is_used(self):
    37         for formdef in get_formdefs_of_all_kinds():
>   38             if self.datasource.is_used_in_formdef(formdef):
    39                 return True
    40         return False

Bien reçu,
j'ai pris en compte la première remarque dans #15163,
et la seconde dans #37058.