Produits Entr'ouvert » w.c.s.

Dans la branche j'ai tapé les deux approches mais je pense que la seconde approche est plus simple et sûre.

C'est confus d'avoir mis dans la branche les deux, j'ai réduit au seul commit tout court qui compte.

Ca va le faire, mais on a par ailleurs le bout de code ci-dessous (dans wcs/admin/settings.py) ; on ne voudrait pas le (re)prendre ici ?

        if not users_cfg.get('fullname_template') and users_cfg.get('field_name'):
            # migrate old value (list of field ids)
            field_name_values = users_cfg.get('field_name')
            fields = UserFieldsFormDef().fields
            field_varnames = [[x for x in fields if str(x.id) == str(y)] for y in field_name_values]
            field_varnames = [x[0].varname for x in field_varnames if x]
            users_cfg['fullname_template'] = ' '.join(
                '{{ user_var_%s|default:"" }}' % x for x in field_varnames
            )

Oui c'est là où j'écris : c'était fait uniquement lors d'une visite au paramétrage des settings, avec l'idée qu'ainsi ça permettait à l'admin de relire ce qui est proposé.

J'ai tapé une branche wip/71471-fullname-vs-hobo--alt avec la version alternative que j'avais, qui fait la conversion automatique, mais comme je notais je préfère la sûreté. (et jenkins peut échouer sur la branche, c'était juste un début avant de préférer l'autre piste).

commit 12b3d94932f67e7b01ad8b00b9196bb92c5d9826
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Nov 18 13:58:14 2022 +0100

    hobo: do not set fullname_template if legacy settings is already there (#71471)