From 96e7290fb7afc251369824402fe93dbee8900e4a Mon Sep 17 00:00:00 2001 From: dmolinari Date: Wed, 15 Apr 2026 16:49:21 -0300 Subject: [PATCH] =?UTF-8?q?refactor(web):=20eliminar=20guards=20inline=20r?= =?UTF-8?q?ol=20admin=20en=20p=C3=A1ginas=20de=20roles/permisos=20[UDT-006?= =?UTF-8?q?]?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../src/features/permisos/pages/RolPermisosPage.tsx | 9 --------- src/web/src/features/roles/pages/EditRolPage.tsx | 7 ------- src/web/src/features/roles/pages/NewRolPage.tsx | 7 ------- src/web/src/features/roles/pages/RolesPage.tsx | 11 +---------- 4 files changed, 1 insertion(+), 33 deletions(-) diff --git a/src/web/src/features/permisos/pages/RolPermisosPage.tsx b/src/web/src/features/permisos/pages/RolPermisosPage.tsx index 19dd823..7c79c8d 100644 --- a/src/web/src/features/permisos/pages/RolPermisosPage.tsx +++ b/src/web/src/features/permisos/pages/RolPermisosPage.tsx @@ -1,6 +1,4 @@ import { useState } from 'react' -import { useNavigate } from 'react-router-dom' -import { useAuthStore } from '@/stores/authStore' import { Card, CardContent, @@ -12,17 +10,10 @@ import { useRoles } from '../../roles/hooks/useRoles' import { RolPermisosEditor } from '../components/RolPermisosEditor' export function RolPermisosPage() { - const navigate = useNavigate() - const user = useAuthStore((s) => s.user) const [selectedRol, setSelectedRol] = useState(null) const { data: roles, isLoading: loadingRoles } = useRoles() - if (!user || user.rol !== 'admin') { - void navigate('/', { replace: true }) - return null - } - const rolesActivos = roles?.filter((r) => r.activo) ?? [] return ( diff --git a/src/web/src/features/roles/pages/EditRolPage.tsx b/src/web/src/features/roles/pages/EditRolPage.tsx index b5e7973..bc62937 100644 --- a/src/web/src/features/roles/pages/EditRolPage.tsx +++ b/src/web/src/features/roles/pages/EditRolPage.tsx @@ -1,5 +1,4 @@ import { useNavigate, useParams } from 'react-router-dom' -import { useAuthStore } from '@/stores/authStore' import { Card, CardContent, @@ -15,14 +14,8 @@ import { EditRolForm } from '../components/RolForm' export function EditRolPage() { const navigate = useNavigate() const { codigo } = useParams<{ codigo: string }>() - const user = useAuthStore((s) => s.user) const { data: rol, isLoading, isError } = useRol(codigo) - if (!user || user.rol !== 'admin') { - void navigate('/', { replace: true }) - return null - } - return (
diff --git a/src/web/src/features/roles/pages/NewRolPage.tsx b/src/web/src/features/roles/pages/NewRolPage.tsx index 3bcd68e..0c7ff15 100644 --- a/src/web/src/features/roles/pages/NewRolPage.tsx +++ b/src/web/src/features/roles/pages/NewRolPage.tsx @@ -1,5 +1,4 @@ import { useNavigate } from 'react-router-dom' -import { useAuthStore } from '@/stores/authStore' import { Card, CardContent, @@ -11,12 +10,6 @@ import { CreateRolForm } from '../components/RolForm' export function NewRolPage() { const navigate = useNavigate() - const user = useAuthStore((s) => s.user) - - if (!user || user.rol !== 'admin') { - void navigate('/', { replace: true }) - return null - } return (
diff --git a/src/web/src/features/roles/pages/RolesPage.tsx b/src/web/src/features/roles/pages/RolesPage.tsx index 3741377..d13e8a7 100644 --- a/src/web/src/features/roles/pages/RolesPage.tsx +++ b/src/web/src/features/roles/pages/RolesPage.tsx @@ -1,5 +1,4 @@ -import { Link, useNavigate } from 'react-router-dom' -import { useAuthStore } from '@/stores/authStore' +import { Link } from 'react-router-dom' import { Button } from '@/components/ui/button' import { Card, @@ -11,14 +10,6 @@ import { import { RolesList } from '../components/RolesList' export function RolesPage() { - const navigate = useNavigate() - const user = useAuthStore((s) => s.user) - - if (!user || user.rol !== 'admin') { - void navigate('/', { replace: true }) - return null - } - return (