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

autoriser la recherche de propriétaires 'dans toute la base' ? #682

Open
landryb opened this issue Oct 6, 2022 · 2 comments
Open

autoriser la recherche de propriétaires 'dans toute la base' ? #682

landryb opened this issue Oct 6, 2022 · 2 comments

Comments

@landryb
Copy link
Member

landryb commented Oct 6, 2022

nous avons des demandes de collectivités voulant faire des recherche de propriétaires 'sur toutes les communes du département'. Il est vrai qu'actuellement on limite les recherches sur un code insee, et meme en dehors du cas 'département entier' il peut etre interessant d'avoir une recherche sur l'ensemble d'une agglo (qui renverrait X comptes propriétaires, un par commune ayant des résultats?)

C'est techniquement faisable dans le code, mais:

  • faut-il le limiter d'une manière ou d'une autre? le restreindre aux roles pour lesquels il y'a une limitation départementale (ccodep?)
  • pour quelles api le faire ? uniquement /cadastrapp/services/getProprietaire, ou le meme genre de recherche 'globale' peut etre interessant pour d'autres APIs du service ?

@MaelREBOUX @jusabatier vos opinions ?

@jusabatier
Copy link
Collaborator

Effectivement, ça peut être intéressant pour les agents de collectivités.

Je pense que ça devrait être ajouté comme une option avec la possibilité de chercher sur une commune, ou bien de chercher sur toute l'emprise sur laquelle on a les droits.

Se baser sur le ccodep ne me parait pas pertinent car sauf pour les départements, les collectivités sont configurés via les code commune.

Je pense que seules les API Propriétaire/Copropriétaire sont pertinente pour cette modif.

@landryb
Copy link
Member Author

landryb commented Oct 10, 2022

Je pense que ça devrait être ajouté comme une option avec la possibilité de chercher sur une commune, ou bien de chercher sur toute l'emprise sur laquelle on a les droits.

donc en terme d'API, rendre le parametre cgocommune optionnel, et si non passé, utiliser le filtre correspondant a l'emprise de compétence du demandeur (qui est déjà ajouté ici https://github.com/georchestra/cadastrapp/blob/master/cadastrapp/src/main/java/org/georchestra/cadastrapp/service/ProprietaireController.java#L162) ?

jevois que sur https://github.com/georchestra/cadastrapp/blob/master/cadastrapp/src/main/java/org/georchestra/cadastrapp/service/ProprietaireController.java#L52 le commentaire et le code ont l'air de dire qu'il y'a un mode d'appel ou cgocommune n'est pas obligatoire, alors que tous les appels a getProprietaire listés sur https://docs.georchestra.org/cadastrapp/latest/guide_developpeur/matrice_fonctionnalites.html l'utilisent.

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