bbalet / jorani

Leave and Overtime Management System
http://jorani.org/
GNU Affero General Public License v3.0
389 stars 276 forks source link

/leaves/{start_date}/{end_date} : Problème SQL #389

Closed troturier closed 1 year ago

troturier commented 1 year ago

Version de Jorani :

1.0.0

Description du problème :

Bonjour, J'étais en train de tester l'API proposée par Jorani et lorsque j'ai voulu tester le endpoint "/leaves/{start_date}/{end_date}" je me suis retrouvé face à une erreur 422.

Après vérification dans le code de Jorani, il semblerait que cette erreur survienne si aucun résultat n'a pu être trouvé pour la période donnée.

En sachant pertinemment que j'avais bien des demandes de congés pour la période que j'avais fournie, j'ai commencé à regarder la requête SQL qui était appelée :

https://github.com/bbalet/jorani/blob/150a28d1dc0c2ad2ddf13f44d2e4ab0e264de978/application/models/Leaves_model.php#L1305-L1312

Sauf erreur de ma part, j'ai l'impression que la condition WHERE utilisée ici ligne 1308 est inversée et ne me retourne que des demandes en dehors de ma période.

En effet, en inversant ma date de début et ma date de fin dans ma requête, j'obtiens bien les données souhaitées.

Est-ce donc un problème de documentation de l'API / code ? Ou est-ce que je n'utilise pas le endpoint de la bonne façon ?

Merci de votre compréhension.

bbalet commented 1 year ago

Bonjour. Merci pour le rapport. effectivement la requête était inversée.