GeotrekCE / Geotrek-admin

Paths management for National Parks and Tourism organizations
https://geotrek.fr
BSD 2-Clause "Simplified" License
131 stars 75 forks source link

API v2 - Problème de filtre par difficulté / Utilise les cirkwi_level #2702

Closed camillemonchicourt closed 3 years ago

camillemonchicourt commented 3 years ago

J'ai 4 niveaux de difficulté : https://geotrek-admin.ecrins-parcnational.fr/api/v2/trek_difficulty/

L'API renvoie bien les 56 randos très faciles : https://geotrek-admin.ecrins-parcnational.fr/api/v2/trek/?fields=id%2Cname%2Cdifficulty&difficulty_min=1&difficulty_max=1&page=1&page_size=150 Mais pas les randos faciles : https://geotrek-admin.ecrins-parcnational.fr/api/v2/trek/?fields=id%2Cname%2Cdifficulty&difficulty_min=2&difficulty_max=2&page=1&page_size=150

Si je ne demande que les très faciles et faciles, elle ne renvoie aussi que les très faciles : https://geotrek-admin.ecrins-parcnational.fr/api/v2/trek/?fields=id%2Cname%2Cdifficulty&difficulty_min=1&difficulty_max=2&page=1&page_size=150

Si je demande les très faciles, faciles et moyennes, elle ne renvoie que les très faciles et faciles : https://geotrek-admin.ecrins-parcnational.fr/api/v2/trek/?fields=id%2Cname%2Cdifficulty&difficulty_min=1&difficulty_max=3&page=1&page_size=150

En creusant je me suis demandé si l'API n'utilisait pas les cirkwi_level à la place des id. Et en interrogeant les difficultés de niveau 7, cela se confirme : https://geotrek-admin.ecrins-parcnational.fr/api/v2/trek/?fields=id%2Cname%2Cdifficulty&difficulty_min=7&difficulty_max=7&page=1&page_size=150

Je n'ai pas de difficulté ayant un id=7, mais ma difficulté difficile (id=4) a le cirkwi_level=7

submarcos commented 3 years ago

effectivement c'est bien l'id cirkwi qui est utilisé pour le filtre (https://github.com/GeotrekCE/Geotrek-admin/blob/master/geotrek/api/v2/filters.py#L339)

Chatewgne commented 3 years ago

PR fix et tests https://github.com/GeotrekCE/Geotrek-admin/pull/2706

camillemonchicourt commented 3 years ago

Merci.

Chatewgne commented 3 years ago

PR mergée

camillemonchicourt commented 3 years ago

Corrigé dans la 2.62.0. Merci.