New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

L'attribut 'next' est toujours généré même sans page suivante #15

Open
dchatry opened this Issue Dec 12, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@dchatry
Copy link

dchatry commented Dec 12, 2018

Bonjour,

Selon la documentation :

Les attributs prev et next (définis à null si il n'y a pas de page précédente et/ou suivante) sont
disponibles dans l'URL de la réponse pour éviter d'avoir à calculer les pages précédentes et/ou 
suivantes.

Or, même quand l'attribut data est vide l'URL next est générée, voir par exemple :
http://hubeau.eaufrance.fr/api/v1/qualite_rivieres/analyse_pc?code_parametre=1340&date_debut_prelevement=2017-01-01&date_fin_prelevement=2017-02-20&page=1000&size=20

ou

http://hubeau.eaufrance.fr/api/v1/qualite_nappes/analyses?date_debut_prelevement=2017-01-01&date_fin_prelevement=2017-12-31&code_param=1340&page=902&size=20

@tvilmus

This comment has been minimized.

Copy link
Collaborator

tvilmus commented Dec 18, 2018

Bonjour,

Ce problème est lié à l'autre problème que vous avez signalé (counts inconsistants).
Quand on arrive à la dernière page de résultats et que le count change et est inférieur, Hub'Eau est perdu : "last" devient Null et "next" augmente indéfiniment.
La résolution du bug précédent permettra de résoudre celui là également.
En attendant, vous pouvez contourner ce problème en testant que "page" * "size" ne dépasse pas le count obtenu lors de l'appel de la première page.
Une solution peut être aussi d'utiliser des "size" très grandes (5000, 10000 voire 20000) afin que le nombre de pages soit le même quelque soient les différences de counts renvoyées par Hub'Eau. Par contre, on peut peut-être arriver à des timeouts avec de trop grosses pages...

Nous vous tenons au courant dès que ces problèmes sont résolus !

Cordialement,
Thierry

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment