Implementación AnomalIA - Fix de dropdowns y permisos.

Frontend/src/pages/Usuarios/AsignarPermisosAPerfilPage.tsx

@@ -8,67 +8,68 @@ import SaveIcon from '@mui/icons-material/Save';
 import perfilService from '../../services/Usuarios/perfilService';
 import type { PermisoAsignadoDto } from '../../models/dtos/Usuarios/PermisoAsignadoDto';
 import type { PerfilDto } from '../../models/dtos/Usuarios/PerfilDto';
-import { usePermissions as usePagePermissions } from '../../hooks/usePermissions'; // Renombrar para evitar conflicto
+import { usePermissions as usePagePermissions } from '../../hooks/usePermissions';
 import axios from 'axios';
 import PermisosChecklist from '../../components/Modals/Usuarios/PermisosChecklist';
 
 const SECCION_PERMISSIONS_PREFIX = "SS";
 
 const getModuloFromSeccionCodAcc = (codAcc: string): string | null => {
-    if (codAcc === "SS001") return "Distribución";
-    if (codAcc === "SS002") return "Contables";
-    if (codAcc === "SS003") return "Impresión";
-    if (codAcc === "SS004") return "Reportes";
-    if (codAcc === "SS005") return "Radios";
-    if (codAcc === "SS006") return "Usuarios";
-    return null;
+  if (codAcc === "SS001") return "Distribución";
+  if (codAcc === "SS002") return "Contables";
+  if (codAcc === "SS003") return "Impresión";
+  if (codAcc === "SS004") return "Reportes";
+  if (codAcc === "SS005") return "Radios";
+  if (codAcc === "SS006") return "Usuarios";
+  return null;
 };
 
 const getModuloConceptualDelPermiso = (permisoModulo: string): string => {
-    const moduloLower = permisoModulo.toLowerCase();
-    if (moduloLower.includes("distribuidores") ||
-        moduloLower.includes("canillas") ||
-        moduloLower.includes("publicaciones distribución") ||
-        moduloLower.includes("zonas distribuidores") ||
-        moduloLower.includes("movimientos distribuidores") ||
-        moduloLower.includes("empresas") ||
-        moduloLower.includes("otros destinos") ||
-        moduloLower.includes("ctrl. devoluciones") ||
-        moduloLower.includes("movimientos canillas") ||
-        moduloLower.includes("salidas otros destinos")) {
-        return "Distribución";
-    }
-    if (moduloLower.includes("cuentas pagos") ||
-        moduloLower.includes("cuentas notas") ||
-        moduloLower.includes("cuentas tipos pagos")) {
-        return "Contables";
-    }
-    if (moduloLower.includes("impresión tiradas") ||
-        moduloLower.includes("impresión bobinas") ||
-        moduloLower.includes("impresión plantas") ||
-        moduloLower.includes("tipos bobinas")) {
-        return "Impresión";
-    }
-    if (moduloLower.includes("radios")) {
-        return "Radios";
-    }
-    if (moduloLower.includes("usuarios") ||
-        moduloLower.includes("perfiles")) {
-        return "Usuarios";
-    }
-    if (moduloLower.includes("reportes")) {
-        return "Reportes";
-    }
-     if (moduloLower.includes("permisos")) {
-      return "Permisos (Definición)";
-    }
-    return permisoModulo;
+  const moduloLower = permisoModulo.toLowerCase();
+  if (moduloLower.includes("distribuidores") ||
+    moduloLower.includes("canillas") ||
+    moduloLower.includes("publicaciones distribución") ||
+    moduloLower.includes("zonas distribuidores") ||
+    moduloLower.includes("movimientos distribuidores") ||
+    moduloLower.includes("empresas") ||
+    moduloLower.includes("otros destinos") ||
+    moduloLower.includes("ctrl. devoluciones") ||
+    moduloLower.includes("movimientos canillas") ||
+    moduloLower.includes("salidas otros destinos")) {
+    return "Distribución";
+  }
+  if (moduloLower.includes("cuentas pagos") ||
+    moduloLower.includes("cuentas notas") ||
+    moduloLower.includes("cuentas tipos pagos")) {
+    return "Contables";
+  }
+  if (moduloLower.includes("impresión tiradas") ||
+    moduloLower.includes("impresión bobinas") ||
+    moduloLower.includes("impresión plantas") ||
+    moduloLower.includes("estados bobinas") ||
+    moduloLower.includes("tipos bobinas")) {
+    return "Impresión";
+  }
+  if (moduloLower.includes("radios")) {
+    return "Radios";
+  }
+  if (moduloLower.includes("usuarios") ||
+    moduloLower.includes("perfiles")) {
+    return "Usuarios";
+  }
+  if (moduloLower.includes("reportes")) {
+    return "Reportes";
+  }
+  if (moduloLower.includes("permisos")) {
+    return "Permisos (Definición)";
+  }
+  return permisoModulo;
 };
 
 const AsignarPermisosAPerfilPage: React.FC = () => {
   const { idPerfil } = useParams<{ idPerfil: string }>();
   const navigate = useNavigate();
-  const { tienePermiso: tienePermisoPagina, isSuperAdmin } = usePagePermissions(); // Renombrado
+  const { tienePermiso: tienePermisoPagina, isSuperAdmin } = usePagePermissions();
 
   const puedeAsignar = isSuperAdmin || tienePermisoPagina("PU004");
 
@@ -124,76 +125,75 @@ const AsignarPermisosAPerfilPage: React.FC = () => {
     moduloConceptualAsociado?: string // Este es el módulo conceptual del padre SSxxx o del grupo del hijo
   ) => {
     setPermisosSeleccionados(prevSelected => {
-        const newSelected = new Set(prevSelected);
-        const permisoActual = permisosDisponibles.find(p => p.id === permisoId);
-        if (!permisoActual) return prevSelected;
+      const newSelected = new Set(prevSelected);
+      const permisoActual = permisosDisponibles.find(p => p.id === permisoId);
+      if (!permisoActual) return prevSelected;
 
-        const permisosDelModuloHijo = moduloConceptualAsociado
-            ? permisosDisponibles.filter(p => {
-                const mc = getModuloConceptualDelPermiso(p.modulo); // Usar la función helper
-                return mc === moduloConceptualAsociado && !p.codAcc.startsWith(SECCION_PERMISSIONS_PREFIX);
-            })
-            : [];
+      const permisosDelModuloHijo = moduloConceptualAsociado
+        ? permisosDisponibles.filter(p => {
+          const mc = getModuloConceptualDelPermiso(p.modulo); // Usar la función helper
+          return mc === moduloConceptualAsociado && !p.codAcc.startsWith(SECCION_PERMISSIONS_PREFIX);
+        })
+        : [];
 
-        if (esPermisoSeccionClick && moduloConceptualAsociado) {
-            const idPermisoSeccion = permisoActual.id;
-            const estabaSeccionSeleccionada = prevSelected.has(idPermisoSeccion);
-            const todosHijosEstabanSeleccionados = permisosDelModuloHijo.length > 0 && permisosDelModuloHijo.every(p => prevSelected.has(p.id));
-            const ningunHijoEstabaSeleccionado = permisosDelModuloHijo.every(p => !prevSelected.has(p.id));
+      if (esPermisoSeccionClick && moduloConceptualAsociado) {
+        const idPermisoSeccion = permisoActual.id;
+        const estabaSeccionSeleccionada = prevSelected.has(idPermisoSeccion);
+        const todosHijosEstabanSeleccionados = permisosDelModuloHijo.length > 0 && permisosDelModuloHijo.every(p => prevSelected.has(p.id));
+        const ningunHijoEstabaSeleccionado = permisosDelModuloHijo.every(p => !prevSelected.has(p.id));
 
 
-            if (!estabaSeccionSeleccionada) { // Estaba Off, pasa a "Solo Sección" (Indeterminate si hay hijos)
-                newSelected.add(idPermisoSeccion);
-                // NO se marcan los hijos
-            } else if (estabaSeccionSeleccionada && (ningunHijoEstabaSeleccionado || !todosHijosEstabanSeleccionados) && permisosDelModuloHijo.length > 0 ) {
-                // Estaba "Solo Sección" o "Parcial Hijos", pasa a "Sección + Todos los Hijos"
-                newSelected.add(idPermisoSeccion); // Asegurar
-                permisosDelModuloHijo.forEach(p => newSelected.add(p.id));
-            } else { // Estaba "Sección + Todos los Hijos" (o no había hijos), pasa a Off
-                newSelected.delete(idPermisoSeccion);
-                permisosDelModuloHijo.forEach(p => newSelected.delete(p.id));
-            }
-
-        } else if (!esPermisoSeccionClick && moduloConceptualAsociado) { // Clic en un permiso hijo
-            if (asignadoViaCheckboxHijo) {
-                newSelected.add(permisoId);
-                const permisoSeccionPadre = permisosDisponibles.find(
-                    ps => ps.codAcc.startsWith(SECCION_PERMISSIONS_PREFIX) && getModuloFromSeccionCodAcc(ps.codAcc) === moduloConceptualAsociado
-                );
-                if (permisoSeccionPadre && !newSelected.has(permisoSeccionPadre.id)) {
-                    newSelected.add(permisoSeccionPadre.id); // Marcar padre si no estaba
-                }
-            } else { // Desmarcando un hijo
-                newSelected.delete(permisoId);
-                const permisoSeccionPadre = permisosDisponibles.find(
-                    ps => ps.codAcc.startsWith(SECCION_PERMISSIONS_PREFIX) && getModuloFromSeccionCodAcc(ps.codAcc) === moduloConceptualAsociado
-                );
-                if (permisoSeccionPadre) {
-                    const algunOtroHijoSeleccionado = permisosDelModuloHijo.some(p => p.id !== permisoId && newSelected.has(p.id));
-                    if (!algunOtroHijoSeleccionado && newSelected.has(permisoSeccionPadre.id)) {
-                         // Si era el último hijo y el padre estaba marcado, NO desmarcamos el padre automáticamente.
-                         // El estado indeterminate se encargará visualmente.
-                         // Si quisiéramos que se desmarque el padre, aquí iría: newSelected.delete(permisoSeccionPadre.id);
-                    }
-                }
-            }
-        } else { // Permiso sin módulo conceptual asociado (ej: "Permisos (Definición)")
-             if (asignadoViaCheckboxHijo) {
-                newSelected.add(permisoId);
-            } else {
-                newSelected.delete(permisoId);
-            }
+        if (!estabaSeccionSeleccionada) { // Estaba Off, pasa a "Solo Sección" (Indeterminate si hay hijos)
+          newSelected.add(idPermisoSeccion);
+          // NO se marcan los hijos
+        } else if (estabaSeccionSeleccionada && (ningunHijoEstabaSeleccionado || !todosHijosEstabanSeleccionados) && permisosDelModuloHijo.length > 0) {
+          // Estaba "Solo Sección" o "Parcial Hijos", pasa a "Sección + Todos los Hijos"
+          newSelected.add(idPermisoSeccion); // Asegurar
+          permisosDelModuloHijo.forEach(p => newSelected.add(p.id));
+        } else { // Estaba "Sección + Todos los Hijos" (o no había hijos), pasa a Off
+          newSelected.delete(idPermisoSeccion);
+          permisosDelModuloHijo.forEach(p => newSelected.delete(p.id));
         }
 
-        if (successMessage) setSuccessMessage(null);
-        if (error) setError(null);
-        return newSelected;
+      } else if (!esPermisoSeccionClick && moduloConceptualAsociado) { // Clic en un permiso hijo
+        if (asignadoViaCheckboxHijo) {
+          newSelected.add(permisoId);
+          const permisoSeccionPadre = permisosDisponibles.find(
+            ps => ps.codAcc.startsWith(SECCION_PERMISSIONS_PREFIX) && getModuloFromSeccionCodAcc(ps.codAcc) === moduloConceptualAsociado
+          );
+          if (permisoSeccionPadre && !newSelected.has(permisoSeccionPadre.id)) {
+            newSelected.add(permisoSeccionPadre.id); // Marcar padre si no estaba
+          }
+        } else { // Desmarcando un hijo
+          newSelected.delete(permisoId);
+          const permisoSeccionPadre = permisosDisponibles.find(
+            ps => ps.codAcc.startsWith(SECCION_PERMISSIONS_PREFIX) && getModuloFromSeccionCodAcc(ps.codAcc) === moduloConceptualAsociado
+          );
+          if (permisoSeccionPadre) {
+            const algunOtroHijoSeleccionado = permisosDelModuloHijo.some(p => p.id !== permisoId && newSelected.has(p.id));
+            if (!algunOtroHijoSeleccionado && newSelected.has(permisoSeccionPadre.id)) {
+              // Si era el último hijo y el padre estaba marcado, NO desmarcamos el padre automáticamente.
+              // El estado indeterminate se encargará visualmente.
+              // Si quisiéramos que se desmarque el padre, aquí iría: newSelected.delete(permisoSeccionPadre.id);
+            }
+          }
+        }
+      } else { // Permiso sin módulo conceptual asociado (ej: "Permisos (Definición)")
+        if (asignadoViaCheckboxHijo) {
+          newSelected.add(permisoId);
+        } else {
+          newSelected.delete(permisoId);
+        }
+      }
+
+      if (successMessage) setSuccessMessage(null);
+      if (error) setError(null);
+      return newSelected;
     });
   }, [permisosDisponibles, successMessage, error]);
 
 
   const handleGuardarCambios = async () => {
-    // ... (sin cambios) ...
     if (!puedeAsignar || !perfil) return;
     setSaving(true); setError(null); setSuccessMessage(null);
     try {
@@ -214,54 +214,54 @@ const AsignarPermisosAPerfilPage: React.FC = () => {
   };
 
   if (loading) {
-        return <Box sx={{ display: 'flex', justifyContent: 'center', p: 3 }}><CircularProgress /></Box>;
-    }
+    return <Box sx={{ display: 'flex', justifyContent: 'center', p: 3 }}><CircularProgress /></Box>;
+  }
 
-    if (error && !perfil) {
-        return <Alert severity="error" sx={{ m: 2 }}>{error}</Alert>;
-    }
-    if (!puedeAsignar) {
-        return <Alert severity="error" sx={{ m: 2 }}>Acceso denegado.</Alert>;
-    }
-    if (!perfil && !loading) {
-        return <Alert severity="warning" sx={{ m: 2 }}>Perfil no encontrado o error al cargar.</Alert>;
-    }
+  if (error && !perfil) {
+    return <Alert severity="error" sx={{ m: 2 }}>{error}</Alert>;
+  }
+  if (!puedeAsignar) {
+    return <Alert severity="error" sx={{ m: 2 }}>Acceso denegado.</Alert>;
+  }
+  if (!perfil && !loading) {
+    return <Alert severity="warning" sx={{ m: 2 }}>Perfil no encontrado o error al cargar.</Alert>;
+  }
 
-    return (
-        <Box sx={{ p: 1 }}>
-            <Button startIcon={<ArrowBackIcon />} onClick={() => navigate('/usuarios/perfiles')} sx={{ mb: 2 }}>
-                Volver a Perfiles
-            </Button>
-            <Typography variant="h5" gutterBottom>
-                Asignar Permisos al Perfil: {perfil?.nombrePerfil || 'Cargando...'}
-            </Typography>
-            <Typography variant="body2" color="textSecondary" gutterBottom>
-                ID Perfil: {perfil?.id}
-            </Typography>
+  return (
+    <Box sx={{ p: 1 }}>
+      <Button startIcon={<ArrowBackIcon />} onClick={() => navigate('/usuarios/perfiles')} sx={{ mb: 2 }}>
+        Volver a Perfiles
+      </Button>
+      <Typography variant="h5" gutterBottom>
+        Asignar Permisos al Perfil: {perfil?.nombrePerfil || 'Cargando...'}
+      </Typography>
+      <Typography variant="body2" color="textSecondary" gutterBottom>
+        ID Perfil: {perfil?.id}
+      </Typography>
 
-            {error && !successMessage && <Alert severity="error" sx={{ mb: 2 }}>{error}</Alert>}
-            {successMessage && <Alert severity="success" sx={{ mb: 2 }}>{successMessage}</Alert>}
+      {error && !successMessage && <Alert severity="error" sx={{ mb: 2 }}>{error}</Alert>}
+      {successMessage && <Alert severity="success" sx={{ mb: 2 }}>{successMessage}</Alert>}
 
-            <Paper sx={{ p: { xs: 1, sm: 2 }, mt: 2 }}>
-                <PermisosChecklist
-                    permisosDisponibles={permisosDisponibles}
-                    permisosSeleccionados={permisosSeleccionados}
-                    onPermisoChange={handlePermisoChange}
-                    disabled={saving}
-                />
-                <Box sx={{ mt: 3, display: 'flex', justifyContent: 'flex-end' }}>
-                    <Button
-                        variant="contained"
-                        color="primary"
-                        startIcon={saving ? <CircularProgress size={20} color="inherit" /> : <SaveIcon />}
-                        onClick={handleGuardarCambios}
-                        disabled={saving || !puedeAsignar}
-                    >
-                        Guardar Cambios
-                    </Button>
-                </Box>
-            </Paper>
+      <Paper sx={{ p: { xs: 1, sm: 2 }, mt: 2 }}>
+        <PermisosChecklist
+          permisosDisponibles={permisosDisponibles}
+          permisosSeleccionados={permisosSeleccionados}
+          onPermisoChange={handlePermisoChange}
+          disabled={saving}
+        />
+        <Box sx={{ mt: 3, display: 'flex', justifyContent: 'flex-end' }}>
+          <Button
+            variant="contained"
+            color="primary"
+            startIcon={saving ? <CircularProgress size={20} color="inherit" /> : <SaveIcon />}
+            onClick={handleGuardarCambios}
+            disabled={saving || !puedeAsignar}
+          >
+            Guardar Cambios
+          </Button>
         </Box>
-    );
+      </Paper>
+    </Box>
+  );
 };
 export default AsignarPermisosAPerfilPage;