Skip to content
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

Filtre de date absent sur les services géo descendants #155

Open
ChristopheYon opened this issue Jan 28, 2025 · 0 comments
Open

Filtre de date absent sur les services géo descendants #155

ChristopheYon opened this issue Jan 28, 2025 · 0 comments
Assignees

Comments

@ChristopheYon
Copy link

Les services géo de type "descendants" renvoient des résultats incorrects. Pour la date donnée (la date courante par défaut), ils doivent renvoyer les territoires actifs à cette date qui sont subdivision du territoire indiqué, lui-même actif à cette date.

Par exemple l'appel à https://api.insee.fr/metadonnees/geo/arrondissement/9741/descendants?type=Commune&date=2024-12-31 doit renvoyer les communes qui sont subdivision de l'arrondissement 9741 (Saint-Denis) le 31 décembre 2024.
Il devrait renvoyer trois communes : 97441 (Saint-Denis), 97418 (Sainte-Marie) et 97420 (Sainte-Suzanne).
Mais il renvoie actuellement deux communes supplémentaires 97407 (Le Port) et 97408 (La Possession). Ces deux communes étaient bien subdivision de l'arrondissement de Saint-Denis mais dans sa version terminée en 2006 (http://id.insee.fr/geo/arrondissement/27c0f4fa-16db-4b99-88d6-dfe239f6643d) et pas dans sa version active le 31/12/2024 (http://id.insee.fr/geo/arrondissement/8df3dad6-5219-4bd1-ba08-b87ab71cb672).

Bref, il manque un filtre sur le territoire d'origine dans la requête.
En regardant rapidement dans la requête FreeMarker getAscendantsOrDescendantsByCodeTypeDate, il y a des "optional" qui me semblent suspects autour des filtres sur les dates ?dateCreationOrigine et ?dateSuppressionOrigine (lignes 55 et 58). Et dont la suppression semble donner un meilleur résultat pour l'exemple donné (mais je n'ai pas creusé plus loin).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants