Dans tout le reste du projet, la convention de code est la suivante : définition des attributs d'une classe, puis constructeur éventuel de la classe, puis getter/setter de chaque attribut.

Je pense qu'il serait bien de respecter la convention existante pour une meilleur lisibilité du code.

yes

Avec cette solution, tous les prestataires devront rechoisir leur rôle après la mise en prod dans les environnements existants !

Y compris les prestaires TAV qui n'ont jamais eu à sélection de rôle avant, et qui risquent de ne pas savoir sur quoi cliquer (ce n'est pas évident quand on n'est pas habitué au vocabulaire)

Je n'ai rien dit, la fonction est paramétrée avec presta_self_init_and_eval.

le nom du champ n'est pas très parlant, je rajouterais une description dans le commentaire du champ

changed this line in version 2 of the diff

ok

added 1 commit

• 704119f7 - clean and comment code

Compare with previous version

merged

mentioned in commit 57c79fc6