Produits Entr'ouvert » w.c.s.

Tu parles d'identifiant ("identifiant est nouveau") les sources de données ne se basent pas sur les slugs ? (je vérifie et elles le font).

Le soucis ici c'est qu'un export/import ne conserverait pas les slugs ?

Je ne sais pas trop de qul identifiant on parle finalement, car dans l'import_json d'un agenda je vois bien :

        agenda, created = cls.objects.get_or_create(slug=data['slug'], defaults=data)

donc a priori effectivement les slugs ne devraient pas changer.

Je vais demander des détails au client.

dans wcs:

        datasource = existing_datasources.get(url)
        if datasource is None:
            datasource = NamedDataSource()
            datasource.external = 'agenda'
            datasource.data_source = {'type': 'json', 'value': url}
        datasource.external_status = None  # reset
        datasource.name = agenda['text']
        datasource.store()

On ne précise pas le slug de la datasource, il est généré à partir du name.
Si l'agenda change de label, agenda['text'] aussi, on se retrouve avec une nouvelle source de donnée.

=> il faudrait prendre agenda['id'] (== slug de l'agenda côté chrono) pour en faire un slug de datasource ?

note: comment ne pas casser les sources de données agenda existantes lors d'un refresh, si on change ça ?

« datasource3.slug == 'chrono-ds-slug-A' » il ne faut pas avoir de "-" sinon on ne peut pas utiliser ces slug dans des expressions du genre data_source.agenda_truc (rarement utilisé, mais.). Ajoute une passe de "misc.simplify(..., space='_')" si tu veux t'assurer le truc.

(Je constate d'ailleurs un bogue dans get_new_slug qui fait des xxx-1 / xxxx-2 quand le slug existe déjà, zou, #54979)

Aussi, quand un slug "chrono_ds_slug_A" est déjà pris on pourrait chercher à avoir "chrono_ds_slug_A_1" et donc faire que get_new_slug puisse partir d'une base « def get_new_slug(self, base=None) » -- tu pourrais alors t'éviter le try/except dans la création d'une nouvelle source et faire juste « datasource.slug = datasource.get_new_slug(base='chrono_ds_%s' % agenda['slug']) »