dmolinari/GestionIntegralWeb
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore(contables): CC002 deja de figurar en gral_Permisos

Browse Source 
La reapertura de cierres ya se validaba con User.IsInRole('SuperAdmin') sin
chequear CC002. Mantenerlo en gral_Permisos solo agregaba ruido — un permiso
filtrado intencionalmente de la UI de asignación. Se quita del catálogo y se
simplifica el set PERMISOS_SOLO_SUPERADMIN del frontend a {CS002} (legacy).

CS002 se mantiene en el filtro porque ya existía en producción antes del feature.
This commit is contained in:
dmolinari
2026-05-13 10:18:40 -03:00
parent 24eaf18fd9
commit 37c16686b7
3 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
Backend/GestionIntegral.Api/Controllers/Contables/CierresCuentaCorrienteController.cs
View File

@@ -19,7 +19,8 @@ namespace GestionIntegral.Api.Controllers.Contables
private readonly ICierreCuentaCorrienteService _cierreService; private readonly ICierreCuentaCorrienteService _cierreService;
private readonly ILogger<CierresCuentaCorrienteController> _logger; private readonly ILogger<CierresCuentaCorrienteController> _logger;
// Permisos asignables a perfiles. La reapertura (CC002) NO se valida acá: es exclusiva de SuperAdmin. // Permisos asignables a perfiles. La reapertura es exclusiva de SuperAdmin
// y se valida con User.IsInRole('SuperAdmin') — no tiene código en gral_Permisos.
private const string PermisoCrear = "CC001"; private const string PermisoCrear = "CC001";
private const string PermisoVer = "CC003"; private const string PermisoVer = "CC003";

2
Frontend/src/pages/Contables/CierresCuentaCorrientePage.tsx
View File

@@ -67,7 +67,7 @@ const CierresCuentaCorrientePage: React.FC = () => {
const { tienePermiso, isSuperAdmin } = usePermissions(); const { tienePermiso, isSuperAdmin } = usePermissions();
const puedeVer = isSuperAdmin || tienePermiso('CC003'); const puedeVer = isSuperAdmin || tienePermiso('CC003');
const puedeCrear = isSuperAdmin || tienePermiso('CC001'); const puedeCrear = isSuperAdmin || tienePermiso('CC001');
// Reapertura: exclusivo SuperAdmin (CC002 no se valida — no es asignable a perfiles). // Reapertura: exclusivo SuperAdmin — sin código de permiso asignable.
const puedeReabrir = isSuperAdmin; const puedeReabrir = isSuperAdmin;
const [filtroIdDistribuidor, setFiltroIdDistribuidor] = useState<number | string>(''); const [filtroIdDistribuidor, setFiltroIdDistribuidor] = useState<number | string>('');

6
Frontend/src/pages/Usuarios/AsignarPermisosAPerfilPage.tsx
View File

@@ -15,8 +15,10 @@ import PermisosChecklist from '../../components/Modals/Usuarios/PermisosChecklis
const SECCION_PERMISSIONS_PREFIX = "SS"; const SECCION_PERMISSIONS_PREFIX = "SS";
// Permisos exclusivos de SuperAdmin: no se asignan a perfiles ni se muestran en la UI de asignación. // Permisos exclusivos de SuperAdmin: no se asignan a perfiles ni se muestran en la UI de asignación.
// CS002 = Ajuste manual de saldo. CC002 = Reapertura de cierres de cuenta corriente. // CS002 = Ajuste manual de saldo (legacy, ya existía en gral_Permisos antes del filtro).
const PERMISOS_SOLO_SUPERADMIN: ReadonlySet<string> = new Set(["CS002", "CC002"]); // La reapertura de cierres también es exclusiva SuperAdmin pero no se insertó en gral_Permisos
// porque sería ruido — el backend valida directamente con User.IsInRole('SuperAdmin').
const PERMISOS_SOLO_SUPERADMIN: ReadonlySet<string> = new Set(["CS002"]);
const getModuloFromSeccionCodAcc = (codAcc: string): string | null => { const getModuloFromSeccionCodAcc = (codAcc: string): string | null => {
if (codAcc === "SS001") return "Distribución"; if (codAcc === "SS001") return "Distribución";