From da7a82502f215491ad8323d64226f8903b522f55 Mon Sep 17 00:00:00 2001
From: snlavaise <snlavaise@gmail.com>
Date: Tue, 11 Aug 2026 23:47:45 +0000
Subject: [PATCH] Restringir operaciones de grupos al administrador y permitir
 a usuarios regulares solo modificar el Tipo de camas de su grupo de trabajo

---
 src/hooks/useHospitalStore.ts | 42 +++++++++++++++++++++------
 src/sections/MapaCamas.tsx    | 54 +++++++++++++++++++++--------------
 2 files changed, 66 insertions(+), 30 deletions(-)

diff --git a/src/hooks/useHospitalStore.ts b/src/hooks/useHospitalStore.ts
index 6186817..bfe8abc 100644
--- a/src/hooks/useHospitalStore.ts
+++ b/src/hooks/useHospitalStore.ts
@@ -305,10 +305,14 @@ export function useHospitalStore() {
     const cama = state.camas.find(c => c.id === id);
     if (!cama) return;
     const user = state.currentUser;
-    const grupoId = cama.grupoId || null;
-    if (user && user.rol !== 'admin' && grupoId && user.grupoId !== grupoId) {
-      console.warn('No tiene permisos para actualizar cama en esta área');
-      return;
+    if (user && user.rol !== 'admin') {
+      // Must have a working group and the bed must belong to it
+      if (!user.grupoId || cama.grupoId !== user.grupoId) {
+        toast.error('No tiene permisos para modificar camas fuera de su grupo de trabajo');
+        return;
+      }
+      // A regular user can ONLY modify the "Tipo" of the bed.
+      datos = { tipo: datos.tipo };
     }
     try {
       if (datos.numero !== undefined) {
@@ -326,6 +330,10 @@ export function useHospitalStore() {
   }, [state, apiCall]);
 
 const agregarCama = useCallback(async (cama: Omit<Cama, 'id'>) => {
+    if (state.currentUser?.rol !== 'admin') {
+      toast.error('Solo el usuario administrador puede agregar camas');
+      return;
+    }
     const nuevaCama: Cama = {
       ...cama,
       id: generateUUID(),
@@ -342,9 +350,13 @@ const agregarCama = useCallback(async (cama: Omit<Cama, 'id'>) => {
       console.error('Error al agregar cama:', err);
       throw err;
     }
-  }, [apiCall]);
+  }, [state.currentUser, apiCall]);
 
   const eliminarCama = useCallback(async (id: string) => {
+    if (state.currentUser?.rol !== 'admin') {
+      toast.error('Solo el usuario administrador puede eliminar camas');
+      return;
+    }
     try {
       await apiCall('DELETE', `/camas/${id}`);
       setState(prev => ({
@@ -356,7 +368,7 @@ const agregarCama = useCallback(async (cama: Omit<Cama, 'id'>) => {
       console.error('Error al eliminar cama:', err);
       throw err;
     }
-  }, [apiCall]);
+  }, [state.currentUser, apiCall]);
 
 // Acciones de internaciones
   const iniciarInternacion = useCallback(async (internacion: Omit<Internacion, 'id' | 'activa'>) => {
@@ -1211,6 +1223,10 @@ const actualizarInternacion = useCallback(async (internacionId: string, datos: P
 
   // Acciones de grupos
   const agregarGrupo = useCallback(async (grupo: Omit<Grupo, 'id'>) => {
+    if (state.currentUser?.rol !== 'admin') {
+      toast.error('Solo el usuario administrador puede agregar grupos de trabajo');
+      return;
+    }
     const nuevaGrupo: Grupo = { ...grupo, id: generateUUID() };
     try {
       await apiCall('POST', '/grupos', nuevaGrupo);
@@ -1223,9 +1239,13 @@ const actualizarInternacion = useCallback(async (internacionId: string, datos: P
       console.error('Error al agregar área:', err);
       throw err;
     }
-  }, [apiCall]);
+  }, [state.currentUser, apiCall]);
 
   const actualizarGrupo = useCallback(async (id: string, datos: Partial<Grupo>) => {
+    if (state.currentUser?.rol !== 'admin') {
+      toast.error('Solo el usuario administrador puede modificar grupos de trabajo');
+      return;
+    }
     try {
       await apiCall('PUT', `/grupos/${id}`, datos);
       setState(prev => ({
@@ -1236,9 +1256,13 @@ const actualizarInternacion = useCallback(async (internacionId: string, datos: P
       console.error('Error al actualizar área:', err);
       throw err;
     }
-  }, [apiCall]);
+  }, [state.currentUser, apiCall]);
 
   const eliminarGrupo = useCallback(async (id: string) => {
+    if (state.currentUser?.rol !== 'admin') {
+      toast.error('Solo el usuario administrador puede eliminar grupos de trabajo');
+      return;
+    }
     try {
       await apiCall('DELETE', `/grupos/${id}`);
       setState(prev => ({
@@ -1250,7 +1274,7 @@ const actualizarInternacion = useCallback(async (internacionId: string, datos: P
       console.error('Error al eliminar área:', err);
       throw err;
     }
-  }, [apiCall]);
+  }, [state.currentUser, apiCall]);
 
   // Funciones de utilidad para consultas
   const getPacienteById = useCallback((id: string) => {
diff --git a/src/sections/MapaCamas.tsx b/src/sections/MapaCamas.tsx
index 346a693..a433c93 100644
--- a/src/sections/MapaCamas.tsx
+++ b/src/sections/MapaCamas.tsx
@@ -157,23 +157,27 @@ export function MapaCamas({
             {camas.filter(c => isCamaFueraDeGrupo(c)).length} Fuera de Área
           </Badge>
           <div className="ml-0 sm:ml-4 flex items-center gap-2 flex-wrap sm:flex-nowrap">
-            <Button variant="outline" onClick={() => {
-              setEditingGrupoId(null);
-              setGrupoNombre('');
-              setGrupoDialogOpen(true);
-            }} className="text-xs px-2 py-1 sm:text-sm sm:px-4 sm:py-2">
-              Administrar Grupos
-            </Button>
-            <Button variant="outline" onClick={() => {
-              setEditingBed(null);
-              setBedNumero('');
-              setBedTipo('General');
-              setBedGrupoId(grupos?.[0]?.id);
-              setBedDialogOpen(true);
-            }} className="text-xs px-2 py-1 sm:text-sm sm:px-4 sm:py-2">
-              <Plus className="h-3 w-3 sm:h-4 sm:w-4 sm:mr-1" />
-              <span className="hidden sm:inline">Agregar Cama</span>
-            </Button>
+            {currentUser?.rol === 'admin' && (
+              <Button variant="outline" onClick={() => {
+                setEditingGrupoId(null);
+                setGrupoNombre('');
+                setGrupoDialogOpen(true);
+              }} className="text-xs px-2 py-1 sm:text-sm sm:px-4 sm:py-2">
+                Administrar Grupos
+              </Button>
+            )}
+            {currentUser?.rol === 'admin' && (
+              <Button variant="outline" onClick={() => {
+                setEditingBed(null);
+                setBedNumero('');
+                setBedTipo('General');
+                setBedGrupoId(grupos?.[0]?.id);
+                setBedDialogOpen(true);
+              }} className="text-xs px-2 py-1 sm:text-sm sm:px-4 sm:py-2">
+                <Plus className="h-3 w-3 sm:h-4 sm:w-4 sm:mr-1" />
+                <span className="hidden sm:inline">Agregar Cama</span>
+              </Button>
+            )}
           </div>
         </div>
       </div>
@@ -441,7 +445,7 @@ export function MapaCamas({
                           <Edit2 className="h-4 w-4 mr-1" />Editar
                         </Button>
                       )}
-                      {canEditCama(cama.id) && (
+                      {canEditCama(cama.id) && currentUser?.rol === 'admin' && (
                         <Button variant="destructive" onClick={() => {
                           onEliminarCama(cama.id);
                         }}>
@@ -518,7 +522,11 @@ export function MapaCamas({
           <div className="space-y-4">
             <div>
               <Label>Número</Label>
-              <Input value={bedNumero} onChange={(e) => setBedNumero(e.target.value)} />
+              <Input 
+                value={bedNumero} 
+                onChange={(e) => setBedNumero(e.target.value)} 
+                disabled={currentUser?.rol !== 'admin'}
+              />
             </div>
             <div>
               <Label>Tipo</Label>
@@ -539,7 +547,11 @@ export function MapaCamas({
             </div>
             <div>
               <Label>Grupo</Label>
-              <Select value={bedGrupoId ?? '__none'} onValueChange={(v) => setBedGrupoId(v === '__none' ? undefined : v)}>
+              <Select 
+                value={bedGrupoId ?? '__none'} 
+                onValueChange={(v) => setBedGrupoId(v === '__none' ? undefined : v)}
+                disabled={currentUser?.rol !== 'admin'}
+              >
                 <SelectTrigger>
                   <SelectValue placeholder="Seleccione grupo" />
                 </SelectTrigger>
@@ -552,7 +564,7 @@ export function MapaCamas({
               </Select>
             </div>
             <div className="flex gap-2 justify-end">
-              {editingBed && (
+              {editingBed && currentUser?.rol === 'admin' && (
                 <Button variant="destructive" onClick={() => { onEliminarCama(editingBed.id); setBedDialogOpen(false); }}>
                   Eliminar
                 </Button>