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

Simplifier le CRUVED minimum pour accéder à GeoNature et aux modules #1622

Closed
camillemonchicourt opened this issue Jan 17, 2022 · 3 comments

Comments

@camillemonchicourt
Copy link
Member

Jusqu'à la 2.9.0, la page d'accueil et le menu listant les modules vérifiait obligatoirement le R du CRUVED de l'utilisateur connecté sur "GeoNature" devant être supérieur à 0.

Donc si on voulait créer un compte simple avec accès à seulement un ou 2 modules (compte associé à l'accès public par exemple), il fallait obligatoirement lui attribuer un CRUVED sur GeoNature dont héritait alors tous les modules, créer un CRUVED sur les modules auxquels l'utilisateur devait avoir accès, mais aussi sur tous les autres modules pour spécifier explicitement que l'utilisateur n'y a pas accès, car sinon il héritait de l'accès à tous les modules lié au CRUVED de GeoNature.

Il serait préférable de ne pas imposer un CRUVED sur "GeoNature" mais de pouvoir uniquement spécifier un CRUVED sur le module auquel on souhaite qu'il puisse accéder.

@DonovanMaillard
Copy link
Contributor

D'une manière générale, je me pose de plus en plus la question personnellement sur cette notion d'héritage

Dans la pratique je me dis que ca apporte finalement plus de flou dans la lisibilité des droits des uns et des autres, que de bénéfices.

Devoir expliciter les droits de chaque groupe sur chaque module permettrait dy voir plus clair, et qu'un nouveau module ke soit accessible par personne par défaut aussi, que d'installer un module et devoir reprendre son cruved pour savoir qui peut y faire quelle action. Quitte à avoir un bloc "tous les modules" explicite. Qui permettent d'accorder les mêmes droits par defaut sur tous les modules.

@camillemonchicourt
Copy link
Member Author

camillemonchicourt commented Jan 17, 2022

Oui à l'usage il y a un consensus sur le fait de supprimer l'héritage des permissions dans GeoNature.
On avait fait ça pour simplifier la gestion des permissions mais en fait ça la complexifie et la rend moins lisible.

Je pense qu'on l'a évoqué dans d'autres tickets mais en effet on aimerait supprimer cet héritage.

Là c'est déjà un premier pas en n'imposant plus un CRUVED sur GeoNature pour pouvoir accéder à GeoNature et sa page d'accueil.

@camillemonchicourt
Copy link
Member Author

Fait avec les PR et commits :

Désormais on ne vérifie plus le R du CRUVED de l'utilisateur sur le module "GeoNature" pour l'accès à la page d'accueil ni à la liste des modules.
On vérifie uniquement que l'utilisateur peut accéder à GeoNature avec un login et mot de passe corrects.

Sur la page d'accueil, on vérifie si l'utilisateur a un R>0 sur Synthèse (ou GeoNature par héritage si pas de CRUVED défini sur Synthèse) pour lui afficher ou non la carte des 100 dernières observations et les statistiques globales de la page d'accueil. Et désormais les statistiques globales se basent sur le R de Synthèse (ou le R de GeoNature par héritage si l'utilisateur n'a pas de CRUVED sur Synthèse mais en a un sur GeoNature).

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

2 participants