Cohérence offset API / OFFSET SQL et numéro de page

[lpofredc]

Lors de l'appel à l'API d'export du module d'export, le paramètre offset correspond au numéro de page de la réponse (clé page dans la réponse d'API), différent d'un OFFSET SQL

Utils-Flask-SQLAlchemy/src/utils_flask_sqla/generic.py

Line 266 in 37e6dc0

data = q.limit(self.limit).offset(self.offset * self.limit).all()

Lorsque OFFSET vaut 0 (et donc aussi la page), alors il renvoie automatiquement les premières lignes de la réponse (OFFSET sql désactivé)

Lorsque le nombre de données est inférieur à la limite du nombre de réponse, alors la page vaut également 0.

Ne serait-ce pas plus logique d'utiliser une variable page correspondant à un vrai numéro de page plutôt qu'à un paramètre offset différent d'un OFFSET SQL, cf. PnX-SI/Utils-Flask-SQLAlchemy.

le OFFSET SQL serait alors calculé avec offset = limit * (page - 1).

[bouttier]

Flask-SQLAlchemy propose déjà un utilitaire pour gérer la pagination : https://flask-sqlalchemy.palletsprojects.com/en/2.x/api/#flask_sqlalchemy.Pagination
Il serait intéressant de voir à l’utiliser plutôt que de devoir maintenir la même chose de notre côté.

[camillemonchicourt]

L'utilisation de paginate de Falsk-SQLAlchemy est désormais bien implémenté dans GeoNature depuis sa version 2.16.0 avec l'intégration de PnX-SI/GeoNature#3463.
C'est aussi la méthode conseillée et documentée dans la doc de GeoNature. Reste à la généraliser dans GeoNature.