TaxHub icon indicating copy to clipboard operation
TaxHub copied to clipboard

/allnamebylist/ attend un code_liste mais Occtax passe un id_liste...

Open camillemonchicourt opened this issue 1 year ago • 0 comments

Depuis la version 1.8.0 de TaxHub, un code_liste a été ajouté à la table taxonomie.bib_listes : https://github.com/PnX-SI/TaxHub/blob/master/data/update1.7.3to1.8.0.sql

En même temps la route /allnamebylist/ a été modifiée pour prendre le code_liste en paramètre au lieu du id_liste (https://github.com/PnX-SI/TaxHub/pull/223).

Mais Occtax, ou même Occtax-mobile continue à interroger un id_liste quand il appellent cette liste.

Cela peut causer des soucis si l'administrateur créé une nouvelle liste et lui attribue un code différent de l'ID, ce qui est censé être le cas avec 100% des listes nouvellement créées vu que l'on demande un code_liste à l'utilisateur et que la liste n'a pas encore d'ID_liste.

Jusqu'à présent ce n'est pas remonté car lors de l'ajout du champs code_liste celui-ci a été rempli avec les id_liste existants : https://github.com/PnX-SI/TaxHub/blob/master/data/update1.7.3to1.8.0.sql#L7

Exemple d'un retour d'un utilisateur qui a créé une liste de taxons en lui attribuant le code "orchids", mais quand Occtax interroge l'API de TaxHub il tape sur taxhub/api/taxref/allnamebylist/101?search_name=orchis&limit=20 qui renvoie logiquement une erreur 400, car il n'existe pas de liste de taxon avec le code_liste égal à 101.

On pourrait envisager que Occtax et Occtax-mobile appellent le code_liste plutôt que le id_liste, mais ça serait un peu tordu, et cela nécessiterait de stocker le code_liste au niveau de Occtax (dans la conf pour la liste globale ou dans la BDD pour les listes par JDD).

Il semble plutôt souhaitable de garder par défaut une interrogation par id_liste et d'avoir une option pour interroger par code_liste, non ?

camillemonchicourt avatar Nov 04 '22 14:11 camillemonchicourt