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

Recherche de propriétaire limitée à 25 resultats #512

Open
jusabatier opened this issue Mar 25, 2020 · 3 comments
Open

Recherche de propriétaire limitée à 25 resultats #512

jusabatier opened this issue Mar 25, 2020 · 3 comments

Comments

@jusabatier
Copy link
Collaborator

Une de nos secrétaire de mairie nous a fait remarquer que certains propriétaires n'apparaissaient pas dans l'autocompletion de la recherche, à moins d'affiner la recherche en tapant également le prénom.

Cela leur pose problème car dans certains cas elles ont besoin de voir si le nom est assigné à un propriétaire, et n'apparaissant pas, elles croyaient que si les propriétaires n'apparaissaient pas, c'est qu'ils n'existaient pas. Cela a pu résulter sur de mauvaises informations données aux citoyens.

Y aurait-il un moyen pour afficher tous les propriétaires, ou alors pouvoir configurer cette limite de 25 pour pouvoir l'augmenter ?

@pierrejego
Copy link
Member

Pardon je n'avais pas été notifié de ton message, je regarde comment c'est fait dans le code, je te tiens au courant

@pierrejego
Copy link
Member

Il y a effectivement un limite à 25 en dur dans le code.
https://github.com/georchestra/cadastrapp/blob/master/cadastrapp/src/main/java/org/georchestra/cadastrapp/service/ProprietaireController.java#L168

On doit pouvoir la mettre en paramètre assez facilement, par exemple dans le cadastrapp.properties
Par contre, ça va risque de surcharger le temps de réponse pour l'autocomplétion pour les grandes villes si on n'en met pas du tout, a tester.

@jusabatier
Copy link
Collaborator Author

Je me doutais que pour de grandes villes, cette limite était nécessaire.

Je pense que la meilleure solution serait effectivement de pouvoir paramétrer ceci dans le fichier properties.

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

No branches or pull requests

3 participants