georchestra / cadastrapp

Cadastre application for geOrchestra
GNU General Public License v3.0
10 stars 19 forks source link

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

Open landryb opened 2 years ago

landryb commented 2 years ago

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:

@MaelREBOUX @jusabatier vos opinions ?

jusabatier commented 2 years ago

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 commented 2 years ago

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.