diff --git a/src/Controller/UserController.php b/src/Controller/UserController.php index 389aa52..b33737f 100644 --- a/src/Controller/UserController.php +++ b/src/Controller/UserController.php @@ -62,13 +62,11 @@ final class UserController extends AbstractController #[Route('/user/{id}/edit', name: 'app_user_edit', methods: ['GET', 'POST'])] public function edit(Request $request, Utilisateur $user, EntityManagerInterface $entityManager): Response { - // Si l'utilisateur est un secrétaire et qu'il essaie de modifier un autre secrétaire if ($this->isGranted('ROLE_SECRETAIRE') && $user->hasRole('ROLE_SECRETAIRE')) { throw $this->createAccessDeniedException('Vous ne pouvez pas modifier un autre secrétaire.'); } - // Si l'utilisateur est un chauffagiste et qu'il essaie de modifier un admin - if ($this->isGranted('ROLE_CHAUFFAGISTE') && $user->hasRole('ROLE_ADMIN')) { + if ($this->isGranted('ROLE_CHAUFFAGISTE') && ($user->hasRole('ROLE_CHAUFFAGISTE') || $user->hasRole('ROLE_SECRETAIRE'))) { throw $this->createAccessDeniedException('Vous ne pouvez pas modifier un admin.'); } @@ -95,17 +93,17 @@ final class UserController extends AbstractController public function delete(Request $request, Utilisateur $user, EntityManagerInterface $entityManager): Response { // Si l'utilisateur est un secrétaire et qu'il essaie de supprimer un autre secrétaire - if ($this->isGranted('ROLE_SECRETAIRE') && ($user->hasRole('ROLE_SECRETAIRE') || $user->hasRole('ROLE_ADMIN'))) { + if ($this->isGranted('ROLE_SECRETAIRE') && $user->hasRole('ROLE_SECRETAIRE')) { throw $this->createAccessDeniedException('Vous ne pouvez pas supprimer un autre secrétaire ou un administrateur.'); } // Si l'utilisateur est un chauffagiste et qu'il essaie de supprimer un admin - if ($this->isGranted('ROLE_CHAUFFAGISTE') && ($user->hasRole('ROLE_SECRETAIRE') || $user->hasRole('ROLE_ADMIN'))) { + if ($this->isGranted('ROLE_CHAUFFAGISTE') && ($user->hasRole('ROLE_SECRETAIRE') || $user->hasRole('ROLE_CHAUFFAGISTE'))) { throw $this->createAccessDeniedException('Vous ne pouvez pas supprimer un utilisateur.'); } // On s'assure que seul un admin peut supprimer un autre admin - $this->denyAccessUnlessGranted('ROLE_ADMIN'); + $this->denyAccessUnlessGranted("ROLE_ADMIN"); if ($this->isCsrfTokenValid('delete' . $user->getId(), $request->get('csrf_token'))) { $entityManager->remove($user);