Produits Entr'ouvert » Passerelle

Je changerais bien

            cod_appli = demande.get('COD_APPLI', '')

            cod_appli = demande.pop('COD_APPLI', '')

Et d'ailleurs si cod_appli vaut une chaîne vide, cette chaîne va se retrouver en tant que clé de l'un des deux dico demandes et droits. Je ne suis pas certain que ce soit le comportement attendu ici.

Dernière chose, il y a une variable demande qui s'est glissée dans la deuxième boucle, c'est droit qu'on veut ici, pas demande.

Paul Marillonnet a écrit :

Dernière chose, il y a une variable demande qui s'est glissée dans la deuxième boucle, c'est droit qu'on veut ici, pas demande.

Et donc sans doute une modif du xml de tests, pour ne pas avoir deux fois le même cod_appli ? (si ça avait été le cas, les tests auraient échoué).

Paul Marillonnet a écrit :

Je changerais bien
[...]
en
[...]
pour ne pas redonder l'information.

Non ce n'est point pratique, ça ne permet plus de différencier une demande ou un droit sans remonter à la clé du dico d'où elle vient, c'est pas parce qu'on indexe qu'il faut supprimer les colonnes d'une table.

Et d'ailleurs si cod_appli vaut une chaîne vide, cette chaîne va se retrouver en tant que clé de l'un des deux dico demandes et droits. Je ne suis pas certain que ce soit le comportement attendu ici.

L'absence de cod_appli est une erreur mais je n'ai pas envie de planter dans ce cas là.

Dernière chose, il y a une variable demande qui s'est glissée dans la deuxième boucle, c'est droit qu'on veut ici, pas demande.

Yep corrigé. Je pose la correction, avec modification aux données des tests pour que ça saute aux yeux.