< Summary

Information
Class: FAU.Logica.Services.PersonalService
Assembly: FAU.Logica
File(s): /home/runner/work/Sistema.Liquidacion.BE/Sistema.Liquidacion.BE/FAU.Logica/Services/PersonalService.cs
Line coverage
75%
Covered lines: 570
Uncovered lines: 187
Coverable lines: 757
Total lines: 1152
Line coverage: 75.2%
Branch coverage
49%
Covered branches: 240
Total branches: 482
Branch coverage: 49.7%
Method coverage

Feature is only available for sponsors

Upgrade to PRO version

Metrics

MethodBranch coverage Crap Score Cyclomatic complexity Line coverage
.ctor(...)100%11100%
GetPersonalListAsync()50%201468.29%
GetDetalleAsync()80%1010100%
CreatePersonalAsync()51.78%965676.61%
UpdatePersonalAsync()56.91%41324685.96%
RegistrarBajaAsync()62.5%252486.44%
RegistrarAscensoAsync()75%333291.58%
ApplyOrdering(...)27.27%2262225%
DeterminarRegimenIdAsync()100%11100%
EsSituacionSuperior(...)50%1212100%
EsSituacionSubalterno(...)50%1010100%
ObtenerReporteMovimientosAsync()0%2162460%

File(s)

/home/runner/work/Sistema.Liquidacion.BE/Sistema.Liquidacion.BE/FAU.Logica/Services/PersonalService.cs

#LineLine coverage
 1using FAU.DataAccess;
 2using FAU.DataAccess.Repositories;
 3using FAU.Entidades;
 4using FAU.Logica.DTOs;
 5using FAU.Logica.Validadores;
 6using Microsoft.EntityFrameworkCore;
 7using Microsoft.Extensions.Logging;
 8
 9namespace FAU.Logica.Services;
 10
 11/// <summary>
 12/// Servicio de gestión de Personal
 13/// </summary>
 14public class PersonalService : IPersonalService
 15{
 16    private readonly IPersonalRepository _personalRepository;
 17    private readonly IMotivoBajaRepository _motivoBajaRepository;
 18    private readonly ApplicationDbContext _context;
 19    private readonly ILogger<PersonalService> _logger;
 20    private readonly IAuditoriaService _auditoriaService;
 21    private readonly ICurrentUserContext _currentUser;
 22
 3323    public PersonalService(
 3324        IPersonalRepository personalRepository,
 3325        IMotivoBajaRepository motivoBajaRepository,
 3326        ApplicationDbContext context,
 3327        ILogger<PersonalService> logger,
 3328        IAuditoriaService auditoriaService,
 3329        ICurrentUserContext currentUser)
 30    {
 3331        _personalRepository = personalRepository;
 3332        _motivoBajaRepository = motivoBajaRepository;
 3333        _context = context;
 3334        _logger = logger;
 3335        _auditoriaService = auditoriaService;
 3336        _currentUser = currentUser;
 3337    }
 38
 39    public async Task<object> GetPersonalListAsync(
 40        string? buscar,
 41        long? cargoId,
 42        string? estado,
 43        string? ley,
 44        int page,
 45        int pageSize,
 46        string? orderBy,
 47        string sort)
 48    {
 149        var query = _context.RelacionesLaborales
 150            .Include(r => r.Persona)
 151            .Include(r => r.Regimen)
 152            .Include(r => r.Escalafon)
 153            .Include(r => r.Grado)
 154            .Include(r => r.Situacion)
 155            .AsQueryable();
 56
 57        // Filtro por búsqueda (CI, nombre o apellido)
 158        if (!string.IsNullOrWhiteSpace(buscar))
 59        {
 060            var buscarLower = buscar.ToLower();
 061            query = query.Where(r =>
 062                r.Persona.Cedula.Contains(buscarLower) ||
 063                r.Persona.PrimerNombre.ToLower().Contains(buscarLower) ||
 064                r.Persona.PrimerApellido.ToLower().Contains(buscarLower) ||
 065                (r.Persona.SegundoNombre != null && r.Persona.SegundoNombre.ToLower().Contains(buscarLower)) ||
 066                (r.Persona.SegundoApellido != null && r.Persona.SegundoApellido.ToLower().Contains(buscarLower)));
 67        }
 68
 69        // Filtro por cargo (grado)
 170        if (cargoId.HasValue)
 71        {
 072            query = query.Where(r => r.GradoId == cargoId.Value);
 73        }
 74
 75        // Filtro por estado (parse string to enum)
 176        if (!string.IsNullOrWhiteSpace(estado))
 77        {
 178            if (Enum.TryParse<EstadoRelacion>(estado, true, out var estadoEnum))
 79            {
 180                query = query.Where(r => r.Estado == estadoEnum);
 81            }
 82            else
 83            {
 84                // Si no se puede parsear, intentar aceptar valores comunes
 085                var estLower = estado.Trim().ToLower();
 086                if (estLower == "activo") query = query.Where(r => r.Estado == EstadoRelacion.Activo);
 087                else if (estLower == "inactivo") query = query.Where(r => r.Estado == EstadoRelacion.Inactivo);
 88            }
 89        }
 90
 91        // Filtro por ley (régimen)
 192        if (!string.IsNullOrWhiteSpace(ley))
 93        {
 094            var leyLower = ley.ToLower();
 095            query = query.Where(r => r.Regimen.NumeroLey.ToLower().Contains(leyLower));
 96        }
 97
 198        var total = await query.CountAsync();
 99
 1100        query = ApplyOrdering(query, orderBy, sort);
 101
 102        // Paginación: obtener entidades y luego proyectar a DTO en memoria para poder convertir enums a string
 1103        var relacionesPage = await query
 1104            .Skip((page - 1) * pageSize)
 1105            .Take(pageSize)
 1106            .ToListAsync();
 107
 2108        var items = relacionesPage.Select(r => new PersonalListDto
 2109        {
 2110            Id = r.Id,
 2111            Ci = r.Persona.Cedula,
 2112            personalId = r.PersonaId,
 2113            PrimerNombre = r.Persona.PrimerNombre,
 2114            SegundoNombre = r.Persona.SegundoNombre,
 2115            PrimerApellido = r.Persona.PrimerApellido,
 2116            SegundoApellido = r.Persona.SegundoApellido,
 2117            Grado = r.Grado.Denominacion,
 2118            Ley = r.Regimen.NumeroLey,
 2119            Estado = r.Estado.ToString(),
 2120            FechaIngreso = r.FechaInicio
 2121        }).ToList();
 122
 1123        return new
 1124        {
 1125            items,
 1126            total,
 1127            page,
 1128            pageSize,
 1129            totalPages = (int)Math.Ceiling((double)total / pageSize)
 1130        };
 1131    }
 132
 133    public async Task<PersonalDetalleDto?> GetDetalleAsync(long id)
 134    {
 135        // id is PersonaId: obtener la relación laboral activa o la última relación laboral
 5136        var relacionLaboral = await _context.RelacionesLaborales
 5137            .Include(r => r.Persona)
 5138            .Include(r => r.Regimen)
 5139            .Include(r => r.Escalafon)
 5140            .Include(r => r.Grado)
 5141            .Include(r => r.Situacion)
 5142            .Include(r => r.Programa)
 5143            .Include(r => r.Unidad)
 5144            .Include(r => r.SubUnidad)
 5145            .Include(r => r.MotivoBaja)
 5146            .FirstOrDefaultAsync(r => r.PersonaId == id && r.Estado == EstadoRelacion.Activo);
 147
 5148        if (relacionLaboral == null)
 149        {
 2150            relacionLaboral = await _context.RelacionesLaborales
 2151                .Include(r => r.Persona)
 2152                .Include(r => r.Regimen)
 2153                .Include(r => r.Escalafon)
 2154                .Include(r => r.Grado)
 2155                .Include(r => r.Situacion)
 2156                .Include(r => r.Programa)
 2157                .Include(r => r.Unidad)
 2158                .Include(r => r.SubUnidad)
 2159                .Include(r => r.MotivoBaja)
 2160                .Where(r => r.PersonaId == id)
 2161                .OrderByDescending(r => r.FechaInicio)
 2162                .FirstOrDefaultAsync();
 163        }
 164
 5165        if (relacionLaboral == null)
 1166            return null;
 167
 4168        var cuentaBancaria = await _context.CuentasBancarias
 4169            .Include(c => c.Banco)
 4170            .FirstOrDefaultAsync(c => c.PersonaId == relacionLaboral.PersonaId && c.Estado == "activa");
 171
 172        // Obtener todos los ids de relación laboral de la persona y luego el historial de movimientos para cualquiera d
 4173        var relacionIds = await _context.RelacionesLaborales
 4174            .Where(r => r.PersonaId == relacionLaboral.PersonaId)
 4175            .Select(r => r.Id)
 4176            .ToListAsync();
 177
 4178        var historial = await _context.MovimientosLaborales
 4179            .Include(m => m.TipoMovimiento)
 4180            .Include(m => m.Usuario)
 4181                .ThenInclude(u => u!.Persona)
 4182            .Where(m => relacionIds.Contains(m.RelacionLaboralId))
 4183            .OrderByDescending(m => m.Id)
 4184            .Select(m => new HistorialMovimientoDto
 4185            {
 4186                Id = m.Id,
 4187                TipoMovimiento = m.TipoMovimiento.Nombre,
 4188                EsAlta = m.TipoMovimiento.EsAlta,
 4189                FechaMovimiento = m.FechaMovimiento,
 4190                Observaciones = m.Observaciones,
 4191                RegistradoPor = m.Usuario != null ? m.Usuario.Username ?? string.Empty : string.Empty,
 4192                FechaRegistro = m.FechaMovimiento
 4193            })
 4194            .ToListAsync();
 195
 4196        return new PersonalDetalleDto
 4197        {
 4198            Id = relacionLaboral.Id,
 4199            DatosPersonales = new DatosPersonalesDto
 4200            {
 4201                PersonaId = relacionLaboral.PersonaId,
 4202                Ci = relacionLaboral.Persona.Cedula,
 4203                PrimerNombre = relacionLaboral.Persona.PrimerNombre,
 4204                SegundoNombre = relacionLaboral.Persona.SegundoNombre,
 4205                PrimerApellido = relacionLaboral.Persona.PrimerApellido,
 4206                SegundoApellido = relacionLaboral.Persona.SegundoApellido,
 4207                FechaNacimiento = relacionLaboral.Persona.FechaNacimiento,
 4208                Email = relacionLaboral.Persona.Email,
 4209                Telefono = relacionLaboral.Persona.Telefono,
 4210                Direccion = relacionLaboral.Persona.Direccion,
 4211                CodigoPostal = relacionLaboral.Persona.CodigoPostal,
 4212                Departamento = relacionLaboral.Persona.Departamento,
 4213                Localidad = relacionLaboral.Persona.Localidad,
 4214                CuentaBancaria = cuentaBancaria == null ? null : new CuentaBancariaDto
 4215                {
 4216                    Id = cuentaBancaria.Id,
 4217                    BancoId = cuentaBancaria.BancoId,
 4218                    Banco = cuentaBancaria.Banco.Nombre,
 4219                    NumeroCuenta = cuentaBancaria.NumeroCuenta
 4220                }
 4221            },
 4222            DatosLaborales = new DatosLaboralesDto
 4223            {
 4224                RelacionLaboralId = relacionLaboral.Id,
 4225                Estado = relacionLaboral.Estado.ToString(),
 4226                FechaInicio = relacionLaboral.FechaInicio,
 4227                FechaFin = relacionLaboral.FechaFin,
 4228                RegimenId = relacionLaboral.RegimenId,
 4229                Regimen = relacionLaboral.Regimen.Denominacion,
 4230                Ley = relacionLaboral.Regimen.NumeroLey,
 4231                EscalafonId = relacionLaboral.EscalafonId,
 4232                Escalafon = relacionLaboral.Escalafon.Denominacion,
 4233                GradoId = relacionLaboral.GradoId,
 4234                Grado = relacionLaboral.Grado.Denominacion,
 4235                SituacionId = relacionLaboral.SituacionId,
 4236                Situacion = relacionLaboral.Situacion.Denominacion,
 4237                ProgramaId = relacionLaboral.ProgramaId,
 4238                Programa = relacionLaboral.Programa.Denominacion,
 4239                UnidadId = relacionLaboral.UnidadId,
 4240                Unidad = relacionLaboral.Unidad.Denominacion,
 4241                SubUnidadId = relacionLaboral.SubUnidadId,
 4242                SubUnidad = relacionLaboral.SubUnidad?.Denominacion,
 4243                MotivoBajaId = relacionLaboral.MotivoBajaId,
 4244                MotivoBaja = relacionLaboral.MotivoBaja?.Denominacion,
 4245                RiesgoVuelo = relacionLaboral.RiesgoVuelo,
 4246                CategoriaViatico = relacionLaboral.CategoriaViatico,
 4247                UsaFonasa = relacionLaboral.UsaFonasa,
 4248                AniosServicioAnterior = relacionLaboral.AniosServicioAnterior,
 4249                PrimaTecnica = relacionLaboral.PrimaTecnica,
 4250                PrimaSolidariaFamiliar = relacionLaboral.PrimaSolidariaFamiliar,
 4251                AniosInactivos = relacionLaboral.AniosInactivos,
 4252                Observaciones = relacionLaboral.Observaciones
 4253            },
 4254            Historial = historial
 4255        };
 5256    }
 257
 258    public async Task<(bool success, long? personalId, string? error)> CreatePersonalAsync(CreatePersonalRequest request
 259    {
 260        try
 261        {
 262            // Validar y limpiar CI
 7263            var cedulaLimpia = CedulaValidator.Limpiar(request.Ci);
 7264            if (!CedulaValidator.Validar(cedulaLimpia))
 265            {
 1266                return (false, null, "La cédula ingresada no es válida.");
 267            }
 268
 269            // Validar que la fecha de ingreso no sea futura
 6270            var errorFechaIngreso = FechaIngresoValidator.ObtenerErrorSiEsFutura(request.FechaIngreso);
 6271            if (errorFechaIngreso != null)
 272            {
 1273                return (false, null, errorFechaIngreso);
 274            }
 275
 276            // Validar que no exista una relación laboral activa para esa cédula
 5277            var validadorRelacion = new RelacionLaboralValidator(_context);
 5278            var errorRelacionActiva = await validadorRelacion.ObtenerErrorSiExisteRelacionActiva(cedulaLimpia);
 5279            if (errorRelacionActiva != null)
 280            {
 1281                return (false, null, errorRelacionActiva);
 282            }
 283
 284            // Validar que el grado pertenece al escalafón indicado
 4285            var escalafonId = request.EscalafonId ?? 1;
 4286            var grado = await _context.Grados.FindAsync(request.GradoId);
 4287            if (grado == null)
 0288                return (false, null, "El grado seleccionado no existe.");
 289            // TODO: reactivar validación cuando el frontend envíe EscalafonId correctamente
 290            // if (grado.EscalafonId != escalafonId)
 291            //     return (false, null, "El grado seleccionado no pertenece al escalafón indicado.");
 292
 293            // Validar fecha de nacimiento
 4294            if (request.FechaNacimiento != default)
 295            {
 4296                var fechaNac = request.FechaNacimiento.Date;
 4297                if (fechaNac > DateTime.Now.Date)
 0298                    return (false, null, "La fecha de nacimiento no puede ser futura.");
 4299                if (fechaNac < new DateTime(1940, 1, 1))
 0300                    return (false, null, "La fecha de nacimiento no puede ser anterior a 01/01/1940.");
 301            }
 302
 303            // Buscar o crear persona
 4304            Persona? persona = await _context.Personas.FirstOrDefaultAsync(p => p.Cedula == cedulaLimpia);
 305
 4306            if (persona == null)
 307            {
 308                // Crear persona - Convertir fechas a UTC
 4309                var fechaNacimientoUtc = request.FechaNacimiento.Kind == DateTimeKind.Unspecified
 4310                    ? DateTime.SpecifyKind(request.FechaNacimiento.Date, DateTimeKind.Utc)
 4311                    : request.FechaNacimiento.Date.ToUniversalTime();
 312
 4313                persona = new Persona
 4314                {
 4315                    Cedula = cedulaLimpia,
 4316                    PrimerNombre = request.PrimerNombre,
 4317                    SegundoNombre = request.SegundoNombre,
 4318                    PrimerApellido = request.PrimerApellido,
 4319                    SegundoApellido = request.SegundoApellido,
 4320                    FechaNacimiento = fechaNacimientoUtc,
 4321                    Email = request.Email,
 4322                    Telefono = request.Telefono,
 4323                    Direccion = request.Direccion,
 4324                    CodigoPostal = request.CodigoPostal,
 4325                    Departamento = request.Departamento,
 4326                    Localidad = request.Localidad
 4327                };
 328
 4329                _context.Personas.Add(persona);
 4330                await _context.SaveChangesAsync();
 331            }
 332            else
 333            {
 334                // Persona ya existe: actualizar campos de dirección si vienen en el request
 0335                bool personaModificada = false;
 0336                if (request.CodigoPostal != null) { persona.CodigoPostal = request.CodigoPostal; personaModificada = tru
 0337                if (request.Departamento != null) { persona.Departamento = request.Departamento; personaModificada = tru
 0338                if (request.Localidad != null) { persona.Localidad = request.Localidad; personaModificada = true; }
 0339                if (personaModificada)
 340                {
 0341                    persona.FechaActualizacion = DateTime.Now;
 0342                    _context.Personas.Update(persona);
 0343                    await _context.SaveChangesAsync();
 344                }
 345            }
 346
 347            // Determinar régimen: explícito si se pasa, sino auto-calcular
 348            long regimenId;
 4349            if (!string.IsNullOrWhiteSpace(request.Ley))
 350            {
 0351                var regimenExplicito = await _context.Regimenes
 0352                    .FirstOrDefaultAsync(r => r.NumeroLey == request.Ley && r.Vigente);
 0353                if (regimenExplicito == null)
 0354                    return (false, null, $"No se encontró el régimen con ley {request.Ley}");
 0355                regimenId = regimenExplicito.Id;
 356            }
 357            else
 358            {
 4359                regimenId = await DeterminarRegimenIdAsync(request.FechaIngreso, request.EscalafonId);
 360            }
 361
 362            // Validar que la situación fue explícitamente indicada
 4363            if (!request.SituacionId.HasValue)
 0364                return (false, null, "La situación es requerida.");
 365
 366            // Validar coherencia entre grado y situación
 4367            var situacion = await _context.Situaciones.FindAsync(request.SituacionId.Value);
 4368            if (situacion == null)
 0369                return (false, null, "La situación seleccionada no existe.");
 370
 4371            if (EsSituacionSuperior(situacion.Codigo) && grado.EsSubalterno)
 0372                return (false, null, $"El grado '{grado.Codigo}' corresponde a personal subalterno; no puede asignarse a
 4373            if (EsSituacionSubalterno(situacion.Codigo) && grado.EsOficial)
 0374                return (false, null, $"El grado '{grado.Codigo}' corresponde a personal superior/oficial; no puede asign
 375
 376            // Crear relación laboral - Convertir fechas a UTC
 4377            var fechaIngresoUtc = request.FechaIngreso.Kind == DateTimeKind.Unspecified
 4378                ? DateTime.SpecifyKind(request.FechaIngreso.Date, DateTimeKind.Utc)
 4379                : request.FechaIngreso.Date.ToUniversalTime();
 380
 4381            var relacionLaboral = new RelacionLaboral
 4382            {
 4383                PersonaId = persona.Id,
 4384                RegimenId = regimenId,
 4385                EscalafonId = request.EscalafonId ?? 1,
 4386                GradoId = request.GradoId,
 4387                SituacionId = request.SituacionId.Value,
 4388                ProgramaId = request.ProgramaId ?? 1,
 4389                UnidadId = request.UnidadId ?? 1,
 4390                SubUnidadId = request.SubUnidadId,
 4391                Estado = EstadoRelacion.Activo,
 4392                FechaInicio = fechaIngresoUtc,
 4393                Observaciones = request.Observaciones,
 4394                // Campos de compensación desde el alta
 4395                RiesgoVuelo = request.RiesgoVuelo,
 4396                PrimaTecnica = string.IsNullOrWhiteSpace(request.PrimaTecnica) ? null : request.PrimaTecnica.Trim(),
 4397                CategoriaViatico = request.CategoriaViatico ?? 0,
 4398                UsaFonasa = request.UsaFonasa ?? false,
 4399                AniosServicioAnterior = request.AniosServicioAnterior ?? 0,
 4400            };
 401
 4402            _context.RelacionesLaborales.Add(relacionLaboral);
 4403            await _context.SaveChangesAsync();
 404
 4405            if (request.BancoId.HasValue)
 406            {
 2407                var banco = await _context.Bancos.FindAsync(request.BancoId.Value);
 2408                if (banco == null)
 1409                    return (false, null, "El banco seleccionado no existe.");
 410
 1411                var errorCuenta = NumeroCuentaValidator.Validar(banco.Codigo, request.NumeroCuenta);
 1412                if (errorCuenta != null)
 0413                    return (false, null, errorCuenta);
 414
 1415                _context.CuentasBancarias.Add(new CuentaBancaria
 1416                {
 1417                    PersonaId = persona.Id,
 1418                    BancoId = request.BancoId.Value,
 1419                    NumeroCuenta = request.NumeroCuenta != null ? NumeroCuentaValidator.FormatearParaGuardar(banco.Codig
 1420                    Estado = "activa"
 1421                });
 1422                await _context.SaveChangesAsync();
 423            }
 424
 425            // Prima Solidaria Familiar NO se asigna automáticamente — no aplica a todos.
 426            // Se carga manualmente desde Beneficios Sociales cuando corresponde.
 427
 3428            await _auditoriaService.LogAuditoriaAsync(
 3429                usuarioId: _currentUser.UserId ?? 0,
 3430                AccionEnum.CrearPersonal,
 3431                ContextoEnum.Personal,
 3432                host: _currentUser.Host,
 3433                entidad: nameof(Persona),
 3434                entidadId: persona.Id,
 3435                detalle: new { relacionLaboralId = relacionLaboral.Id, personaId = persona.Id, cedula = persona.Cedula }
 436
 3437            return (true, persona.Id, null);
 438        }
 0439        catch (Exception ex)
 440        {
 0441            _logger.LogError(ex, "Error al crear personal");
 0442            var errorMessage = ex.Message;
 0443            var innerEx = ex.InnerException;
 0444            while (innerEx != null)
 445            {
 0446                errorMessage += $" | {innerEx.Message}";
 0447                innerEx = innerEx.InnerException;
 448            }
 0449            return (false, null, $"Error al crear personal: {errorMessage}");
 450        }
 7451    }
 452
 453    public async Task<(bool success, string? error)> UpdatePersonalAsync(long id, UpdatePersonalRequest request)
 454    {
 455        try
 456        {
 11457            var persona = await _context.Personas.FindAsync(id);
 11458            if (persona == null)
 1459                return (false, "Persona no encontrada");
 460
 461            // Validar campos de compensación si se proporcionan
 10462            var errorCompensacion = CompensacionValidator.ValidarCompensaciones(
 10463                request.RiesgoVuelo,
 10464                request.PrimaTecnica,
 10465                request.PrimaSolidariaFamiliar,
 10466                request.AniosInactivos);
 467
 10468            if (errorCompensacion != null)
 469            {
 0470                return (false, errorCompensacion);
 471            }
 472
 473            // Actualizar datos personales
 10474            if (!string.IsNullOrWhiteSpace(request.PrimerNombre))
 3475                persona.PrimerNombre = request.PrimerNombre.Trim();
 476
 10477            persona.SegundoNombre = string.IsNullOrWhiteSpace(request.SegundoNombre) ? null : request.SegundoNombre.Trim
 478
 10479            if (!string.IsNullOrWhiteSpace(request.PrimerApellido))
 1480                persona.PrimerApellido = request.PrimerApellido.Trim();
 481
 10482            persona.SegundoApellido = string.IsNullOrWhiteSpace(request.SegundoApellido) ? null : request.SegundoApellid
 483
 10484            if (request.FechaNacimiento.HasValue)
 485            {
 0486                var fechaNac = request.FechaNacimiento.Value;
 0487                persona.FechaNacimiento = fechaNac.Kind == DateTimeKind.Unspecified
 0488                    ? DateTime.SpecifyKind(fechaNac.Date, DateTimeKind.Utc)
 0489                    : fechaNac.Date.ToUniversalTime();
 490            }
 491
 10492            if (!string.IsNullOrWhiteSpace(request.Email))
 1493                persona.Email = request.Email.Trim();
 494
 10495            persona.Telefono = string.IsNullOrWhiteSpace(request.Telefono) ? null : request.Telefono.Trim();
 10496            persona.Direccion = string.IsNullOrWhiteSpace(request.Direccion) ? null : request.Direccion.Trim();
 10497            persona.CodigoPostal = string.IsNullOrWhiteSpace(request.CodigoPostal) ? null : request.CodigoPostal.Trim();
 10498            persona.Departamento = string.IsNullOrWhiteSpace(request.Departamento) ? null : request.Departamento.Trim();
 10499            persona.Localidad = string.IsNullOrWhiteSpace(request.Localidad) ? null : request.Localidad.Trim();
 500
 10501            if (persona.FechaNacimiento.HasValue)
 502            {
 1503                var fn = persona.FechaNacimiento.Value;
 1504                persona.FechaNacimiento = fn.Kind == DateTimeKind.Unspecified
 1505                    ? DateTime.SpecifyKind(fn.Date, DateTimeKind.Utc)
 1506                    : fn.ToUniversalTime();
 507            }
 508
 10509            persona.FechaActualizacion = DateTime.Now;
 10510            _context.Personas.Update(persona);
 511
 512            // Buscar relación activa para la persona
 10513            var relacionActiva = await _context.RelacionesLaborales
 10514                .FirstOrDefaultAsync(r => r.PersonaId == persona.Id && r.Estado == EstadoRelacion.Activo);
 515
 516            // Obtener la última relación (puede ser null)
 10517            var ultimaRelacion = await _context.RelacionesLaborales
 10518                .Where(r => r.PersonaId == persona.Id)
 10519                .OrderByDescending(r => r.FechaInicio)
 10520                .FirstOrDefaultAsync();
 521
 522            // Validar transición de estado usando la relación más relevante (activa si existe, sino la última)
 10523            var relacionParaValidacion = relacionActiva ?? ultimaRelacion;
 524
 10525            if (!string.IsNullOrWhiteSpace(request.Estado) && relacionParaValidacion != null)
 526            {
 0527                var errorTransicion = EstadoTransicionValidator.ObtenerErrorSiTransicionInvalida(
 0528                    relacionParaValidacion.Estado.ToString(),
 0529                    request.Estado);
 0530                if (errorTransicion != null)
 0531                    return (false, errorTransicion);
 532            }
 533
 534            // Determinar si se solicitaron cambios laborales
 10535            bool cambioLaboralSolicitado = request.GradoId.HasValue || request.RegimenId.HasValue || request.EscalafonId
 10536                request.SituacionId.HasValue || request.ProgramaId.HasValue || request.UnidadId.HasValue || request.SubU
 537
 10538            if (relacionActiva != null)
 539            {
 540                // Si hay relación activa, verificar si los cambios requieren crear una nueva relación
 6541                bool requiereNueva = false;
 7542                if (request.GradoId.HasValue && request.GradoId.Value != relacionActiva.GradoId) requiereNueva = true;
 6543                if (request.RegimenId.HasValue && request.RegimenId.Value != relacionActiva.RegimenId) requiereNueva = t
 6544                if (request.EscalafonId.HasValue && request.EscalafonId.Value != relacionActiva.EscalafonId) requiereNue
 6545                if (request.SituacionId.HasValue && request.SituacionId.Value != relacionActiva.SituacionId) requiereNue
 6546                if (request.ProgramaId.HasValue && request.ProgramaId.Value != relacionActiva.ProgramaId) requiereNueva 
 6547                if (request.UnidadId.HasValue && request.UnidadId.Value != relacionActiva.UnidadId) requiereNueva = true
 6548                if (request.SubUnidadId.HasValue && request.SubUnidadId.Value != relacionActiva.SubUnidadId) requiereNue
 549
 6550                if (requiereNueva)
 551                {
 552                    // Cerrar relación activa
 1553                    relacionActiva.FechaFin = DateTime.Now;
 1554                    relacionActiva.Estado = EstadoRelacion.Inactivo;
 1555                    relacionActiva.FechaActualizacion = DateTime.Now;
 556
 557                    // Crear nueva relación activa con los valores actualizados
 1558                    var nuevaRelacion = new RelacionLaboral
 1559                    {
 1560                        PersonaId = persona.Id,
 1561                        RegimenId = request.RegimenId ?? relacionActiva.RegimenId,
 1562                        EscalafonId = request.EscalafonId ?? relacionActiva.EscalafonId,
 1563                        GradoId = request.GradoId ?? relacionActiva.GradoId,
 1564                        SituacionId = request.SituacionId ?? relacionActiva.SituacionId,
 1565                        ProgramaId = request.ProgramaId ?? relacionActiva.ProgramaId,
 1566                        UnidadId = request.UnidadId ?? relacionActiva.UnidadId,
 1567                        SubUnidadId = request.SubUnidadId ?? relacionActiva.SubUnidadId,
 1568                        Estado = EstadoRelacion.Activo,
 1569                        FechaInicio = DateTime.Now,
 1570                        Observaciones = request.Observaciones ?? relacionActiva.Observaciones,
 1571                        FechaActualizacion = DateTime.Now,
 1572                        RiesgoVuelo = request.RiesgoVuelo ?? relacionActiva.RiesgoVuelo,
 1573                        PrimaTecnica = string.IsNullOrWhiteSpace(request.PrimaTecnica) ? relacionActiva.PrimaTecnica : r
 1574                        PrimaSolidariaFamiliar = string.IsNullOrWhiteSpace(request.PrimaSolidariaFamiliar) ? relacionAct
 1575                        AniosInactivos = request.AniosInactivos ?? relacionActiva.AniosInactivos
 1576                    };
 577
 1578                    _context.RelacionesLaborales.Add(nuevaRelacion);
 579                }
 580                else
 581                {
 582                    // Actualizar en sitio la relación activa
 5583                    if (!string.IsNullOrWhiteSpace(request.Estado))
 584                    {
 0585                        if (Enum.TryParse<EstadoRelacion>(request.Estado, true, out var est))
 0586                            relacionActiva.Estado = est;
 587                        else
 588                        {
 0589                            var estLower = request.Estado.Trim().ToLower();
 0590                            if (estLower == "activo") relacionActiva.Estado = EstadoRelacion.Activo;
 0591                            else if (estLower == "inactivo") relacionActiva.Estado = EstadoRelacion.Inactivo;
 592                        }
 593                    }
 594
 5595                    if (request.RegimenId.HasValue) relacionActiva.RegimenId = request.RegimenId.Value;
 5596                    if (request.EscalafonId.HasValue) relacionActiva.EscalafonId = request.EscalafonId.Value;
 5597                    if (request.GradoId.HasValue) relacionActiva.GradoId = request.GradoId.Value;
 5598                    if (request.SituacionId.HasValue) relacionActiva.SituacionId = request.SituacionId.Value;
 5599                    if (request.ProgramaId.HasValue) relacionActiva.ProgramaId = request.ProgramaId.Value;
 5600                    if (request.UnidadId.HasValue) relacionActiva.UnidadId = request.UnidadId.Value;
 5601                    if (request.SubUnidadId.HasValue) relacionActiva.SubUnidadId = request.SubUnidadId.Value;
 602
 5603                    if (request.RiesgoVuelo.HasValue) relacionActiva.RiesgoVuelo = request.RiesgoVuelo;
 5604                    if (request.CategoriaViatico.HasValue) relacionActiva.CategoriaViatico = request.CategoriaViatico.Va
 6605                    if (request.UsaFonasa.HasValue) relacionActiva.UsaFonasa = request.UsaFonasa.Value;
 6606                    if (request.AniosServicioAnterior.HasValue) relacionActiva.AniosServicioAnterior = request.AniosServ
 5607                    if (request.PrimaTecnica != null) relacionActiva.PrimaTecnica = string.IsNullOrWhiteSpace(request.Pr
 5608                    if (request.PrimaSolidariaFamiliar != null) relacionActiva.PrimaSolidariaFamiliar = string.IsNullOrW
 5609                    if (request.AniosInactivos.HasValue) relacionActiva.AniosInactivos = request.AniosInactivos;
 6610                    if (request.Observaciones != null) relacionActiva.Observaciones = request.Observaciones;
 611
 5612                    relacionActiva.FechaActualizacion = DateTime.Now;
 613                }
 614            }
 615            else
 616            {
 617                // No hay relación activa
 4618                if (cambioLaboralSolicitado)
 619                {
 620                    // Crear nueva relación activa basándose en la última relación si existe
 1621                    var plantilla = ultimaRelacion;
 622
 1623                    var nuevaRelacion = new RelacionLaboral
 1624                    {
 1625                        PersonaId = persona.Id,
 1626                        RegimenId = request.RegimenId ?? plantilla?.RegimenId ?? 0,
 1627                        EscalafonId = request.EscalafonId ?? plantilla?.EscalafonId ?? 0,
 1628                        GradoId = request.GradoId ?? plantilla?.GradoId ?? 0,
 1629                        SituacionId = request.SituacionId ?? plantilla?.SituacionId ?? 0,
 1630                        ProgramaId = request.ProgramaId ?? plantilla?.ProgramaId ?? 0,
 1631                        UnidadId = request.UnidadId ?? plantilla?.UnidadId ?? 0,
 1632                        SubUnidadId = request.SubUnidadId ?? plantilla?.SubUnidadId,
 1633                        Estado = EstadoRelacion.Activo,
 1634                        FechaInicio = DateTime.Now,
 1635                        Observaciones = request.Observaciones ?? plantilla?.Observaciones,
 1636                        FechaActualizacion = DateTime.Now,
 1637                        RiesgoVuelo = request.RiesgoVuelo ?? plantilla?.RiesgoVuelo,
 1638                        CategoriaViatico = request.CategoriaViatico ?? plantilla?.CategoriaViatico ?? 0,
 1639                        UsaFonasa = request.UsaFonasa ?? plantilla?.UsaFonasa ?? false,
 1640                        AniosServicioAnterior = request.AniosServicioAnterior ?? plantilla?.AniosServicioAnterior ?? 0,
 1641                        PrimaTecnica = string.IsNullOrWhiteSpace(request.PrimaTecnica) ? plantilla?.PrimaTecnica : reque
 1642                        PrimaSolidariaFamiliar = string.IsNullOrWhiteSpace(request.PrimaSolidariaFamiliar) ? plantilla?.
 1643                        AniosInactivos = request.AniosInactivos ?? plantilla?.AniosInactivos
 1644                    };
 645
 1646                    _context.RelacionesLaborales.Add(nuevaRelacion);
 647                }
 648                else
 649                {
 650                    // Solo actualizar estado u observaciones de la última relación si existe
 3651                    if (ultimaRelacion != null)
 652                    {
 1653                        if (!string.IsNullOrWhiteSpace(request.Estado))
 654                        {
 0655                            if (Enum.TryParse<EstadoRelacion>(request.Estado, true, out var est))
 0656                                ultimaRelacion.Estado = est;
 657                            else
 658                            {
 0659                                var estLower = request.Estado.Trim().ToLower();
 0660                                if (estLower == "activo") ultimaRelacion.Estado = EstadoRelacion.Activo;
 0661                                else if (estLower == "inactivo") ultimaRelacion.Estado = EstadoRelacion.Inactivo;
 662                            }
 663                        }
 664
 2665                        if (request.Observaciones != null) ultimaRelacion.Observaciones = request.Observaciones;
 1666                        ultimaRelacion.FechaActualizacion = DateTime.Now;
 667                    }
 668                }
 669            }
 670
 10671            if (request.BancoId.HasValue)
 672            {
 2673                var banco = await _context.Bancos.FindAsync(request.BancoId.Value);
 2674                if (banco == null)
 1675                    return (false, "El banco seleccionado no existe.");
 676
 1677                var errorCuenta = NumeroCuentaValidator.Validar(banco.Codigo, request.NumeroCuenta);
 1678                if (errorCuenta != null)
 0679                    return (false, errorCuenta);
 680
 1681                var cuentasActivas = await _context.CuentasBancarias
 1682                    .Where(c => c.PersonaId == persona.Id && c.Estado == "activa")
 1683                    .ToListAsync();
 4684                foreach (var c in cuentasActivas)
 1685                    c.Estado = "inactiva";
 686
 1687                _context.CuentasBancarias.Add(new CuentaBancaria
 1688                {
 1689                    PersonaId = persona.Id,
 1690                    BancoId = request.BancoId.Value,
 1691                    NumeroCuenta = request.NumeroCuenta != null ? NumeroCuentaValidator.FormatearParaGuardar(banco.Codig
 1692                    Estado = "activa"
 1693                });
 1694            }
 8695            else if (request.LimpiarCuentaBancaria)
 696            {
 1697                var cuentasActivas = await _context.CuentasBancarias
 1698                    .Where(c => c.PersonaId == persona.Id && c.Estado == "activa")
 1699                    .ToListAsync();
 4700                foreach (var c in cuentasActivas)
 1701                    c.Estado = "inactiva";
 702            }
 703
 9704            await _context.SaveChangesAsync();
 705
 9706            return (true, null);
 707        }
 0708        catch (Exception ex)
 709        {
 0710            _logger.LogError(ex, "Error al actualizar personal");
 0711            return (false, $"Error al actualizar personal: {ex.Message}");
 712        }
 11713    }
 714
 715    public async Task<(bool success, string? error)> RegistrarBajaAsync(long id, RegistrarBajaRequest request)
 716    {
 717        try
 718        {
 719            // id is PersonaId: buscar la relación laboral activa de la persona
 4720            var relacionLaboral = await _context.RelacionesLaborales
 4721                .FirstOrDefaultAsync(r => r.PersonaId == id && r.Estado == EstadoRelacion.Activo);
 722
 4723            if (relacionLaboral == null)
 724            {
 1725                return (false, "Relación laboral no encontrada");
 726            }
 727
 728            // Validar y obtener el MotivoBaja (catálogo UCU)
 3729            if (request.MotivoBajaId <= 0)
 1730                return (false, "El motivo de baja (catálogo) es requerido");
 731
 2732            var motivoBaja = await _motivoBajaRepository.GetByIdAsync(request.MotivoBajaId);
 2733            if (motivoBaja == null)
 0734                return (false, "Motivo de baja (catálogo) no encontrado o no vigente");
 735
 736            // Determinar TipoMovimiento automáticamente según el motivo:
 737            //   RETIRO_OBL / RETIRO_VOL → "Retiro" | resto → "Baja"
 2738            var nombreTipo = motivoBaja.Codigo is "RETIRO_OBL" or "RETIRO_VOL" ? "Retiro" : "Baja";
 2739            var tipoMovimiento = await _context.TiposMovimiento
 2740                .FirstOrDefaultAsync(t => t.Nombre == nombreTipo && !t.EsAlta);
 741
 2742            if (tipoMovimiento == null)
 1743                return (false, $"Tipo de movimiento '{nombreTipo}' no encontrado en el catálogo");
 744
 745            // Validar que la fecha de baja no sea futura
 1746            var errorFechaBaja = FechaIngresoValidator.ObtenerErrorSiEsFutura(request.FechaBaja);
 1747            if (errorFechaBaja != null)
 748            {
 0749                return (false, errorFechaBaja);
 750            }
 751
 752            // Validar fecha de baja vs último movimiento y fecha de inicio
 1753            var validadorMovimiento = new MovimientoLaboralValidator(_context);
 1754            var errorFechaMovimiento = await validadorMovimiento.ValidarFechaMovimiento(relacionLaboral.Id, request.Fech
 1755            if (errorFechaMovimiento != null)
 756            {
 0757                return (false, errorFechaMovimiento);
 758            }
 759
 1760            var errorFechaBajaVsMovimiento = await validadorMovimiento.ValidarFechaBajaVsUltimoMovimiento(relacionLabora
 1761            if (errorFechaBajaVsMovimiento != null)
 762            {
 0763                return (false, errorFechaBajaVsMovimiento);
 764            }
 765
 766            // Validar transición de estado: debe ir de activo a inactivo
 1767            var errorTransicion = EstadoTransicionValidator.ObtenerErrorSiTransicionInvalida(
 1768                relacionLaboral.Estado.ToString(),
 1769                "Inactivo");
 770
 1771            if (errorTransicion != null)
 772            {
 0773                return (false, errorTransicion);
 774            }
 775
 776            // Cambiar estado a inactivo
 1777            relacionLaboral.Estado = EstadoRelacion.Inactivo;
 1778            var fechaBajaUtc = request.FechaBaja.Kind == DateTimeKind.Unspecified
 1779                ? DateTime.SpecifyKind(request.FechaBaja.Date, DateTimeKind.Utc)
 1780                : request.FechaBaja.Date.ToUniversalTime();
 1781            relacionLaboral.FechaFin = fechaBajaUtc;
 1782            relacionLaboral.MotivoBajaId = request.MotivoBajaId;
 1783            relacionLaboral.FechaActualizacion = DateTime.Now;
 784
 785            // Registrar movimiento de baja
 1786            var movimiento = new MovimientoLaboral
 1787            {
 1788                RelacionLaboralId = relacionLaboral.Id,
 1789                TipoMovimientoId = tipoMovimiento.Id,
 1790                FechaMovimiento = fechaBajaUtc,
 1791                UsuarioId = _currentUser.UserId ?? 0,
 1792                Observaciones = request.Observaciones
 1793            };
 794
 1795            _context.MovimientosLaborales.Add(movimiento);
 1796            await _context.SaveChangesAsync();
 797
 1798            await _auditoriaService.LogAuditoriaAsync(
 1799                usuarioId: _currentUser.UserId ?? 0,
 1800                AccionEnum.BajaPersonal,
 1801                ContextoEnum.Personal,
 1802                host: _currentUser.Host,
 1803                entidad: nameof(RelacionLaboral),
 1804                entidadId: relacionLaboral.Id,
 1805                detalle: new { personaId = id, relacionLaboralId = relacionLaboral.Id, request });
 806
 1807            return (true, null);
 808        }
 0809        catch (Exception ex)
 810        {
 0811            _logger.LogError(ex, "Error al registrar baja");
 0812            return (false, $"Error al registrar baja: {ex.Message}");
 813        }
 4814    }
 815
 816    public async Task<(bool success, string? error)> RegistrarAscensoAsync(long id, AscensoRequest request)
 817    {
 818        try
 819        {
 5820            var relacionLaboral = await _context.RelacionesLaborales
 5821                .FirstOrDefaultAsync(r => r.PersonaId == id && r.Estado == EstadoRelacion.Activo);
 822
 5823            if (relacionLaboral == null)
 1824                return (false, "Relación laboral no encontrada");
 825
 826            // Validar grado objetivo
 4827            var grado = await _context.Grados.FirstOrDefaultAsync(g => g.Id == request.NuevoGradoId && g.Vigente);
 4828            if (grado == null)
 1829                return (false, "Grado no encontrado");
 830
 831            // Calcular régimen automáticamente según fecha de ingreso y escalafón (igual que en el Alta)
 3832            var regimenAscensoId = await DeterminarRegimenIdAsync(relacionLaboral.FechaInicio, relacionLaboral.Escalafon
 3833            var regimenAscenso = await _context.Regimenes.FindAsync(regimenAscensoId);
 3834            if (regimenAscenso == null)
 0835                return (false, "No se pudo determinar el régimen para el ascenso");
 836
 837            // Validar que el grado objetivo
 3838            if (relacionLaboral.GradoId != 0)
 839            {
 3840                var gradoActual = await _context.Grados.FirstOrDefaultAsync(g => g.Id == relacionLaboral.GradoId);
 3841                if (gradoActual != null)
 842                {
 3843                    if (grado.Orden <= gradoActual.Orden)
 844                    {
 1845                        return (false, "El grado objetivo debe ser superior al grado actual");
 846                    }
 847                }
 848            }
 849
 2850            var fechaAscensoUtc = request.FechaAscenso.Kind == DateTimeKind.Unspecified
 2851                ? DateTime.SpecifyKind(request.FechaAscenso.Date, DateTimeKind.Utc)
 2852                : request.FechaAscenso.Date.ToUniversalTime();
 853
 854            // Validar unidad objetivo si se proporciona
 855            Unidad? unidadObj = null;
 2856            if (request.UnidadId.HasValue)
 857            {
 0858                unidadObj = await _context.Unidades.FirstOrDefaultAsync(u => u.Id == request.UnidadId.Value && u.Vigente
 0859                if (unidadObj == null)
 860                {
 0861                    return (false, "Unidad no válida o no vigente");
 862                }
 863            }
 864
 865            // Cerrar la relación laboral actual
 866            // Validar transición: debe estar activo para poder ascender
 2867            var errorTransicion = EstadoTransicionValidator.ObtenerErrorSiTransicionInvalida(
 2868                relacionLaboral.Estado.ToString(),
 2869                "Inactivo");
 870
 2871            if (errorTransicion != null)
 872            {
 0873                return (false, errorTransicion);
 874            }
 875
 2876            relacionLaboral.FechaFin = fechaAscensoUtc;
 2877            relacionLaboral.FechaActualizacion = DateTime.Now;
 2878            relacionLaboral.Estado = EstadoRelacion.Inactivo;
 2879            relacionLaboral.Observaciones = request.Observacion;
 880
 881            // Validar programa objetivo
 2882            var programaAscenso = await _context.Programas.FirstOrDefaultAsync(p => p.Id == request.ProgramaId && p.Vige
 2883            if (programaAscenso == null)
 1884                return (false, "Programa no válido o no vigente");
 885
 886            // Determinar prima técnica: al ascender a Sgto. 1° se desactiva la prima técnica variable
 1887            var primaTecnicaNueva = relacionLaboral.PrimaTecnica;
 1888            if (grado.Codigo == "SGTO_1RO")
 0889                primaTecnicaNueva = "VACIO";
 890
 891            // Crear nueva relación laboral con el nuevo grado
 1892            var nuevaRelacion = new RelacionLaboral
 1893            {
 1894                PersonaId = relacionLaboral.PersonaId,
 1895                RegimenId = regimenAscenso.Id,
 1896                EscalafonId = relacionLaboral.EscalafonId,
 1897                GradoId = request.NuevoGradoId,
 1898                SituacionId = relacionLaboral.SituacionId,
 1899                ProgramaId = request.ProgramaId,
 1900                UnidadId = request.UnidadId ?? relacionLaboral.UnidadId,
 1901                SubUnidadId = request.SubUnidadId ?? relacionLaboral.SubUnidadId,
 1902                Estado = EstadoRelacion.Activo,
 1903                FechaInicio = fechaAscensoUtc,
 1904                PrimaTecnica = primaTecnicaNueva,
 1905                PrimaSolidariaFamiliar = relacionLaboral.PrimaSolidariaFamiliar,
 1906                RiesgoVuelo = relacionLaboral.RiesgoVuelo,
 1907                CategoriaViatico = relacionLaboral.CategoriaViatico,
 1908                UsaFonasa = relacionLaboral.UsaFonasa,
 1909                AniosServicioAnterior = relacionLaboral.AniosServicioAnterior,
 1910                Observaciones = relacionLaboral.Observaciones,
 1911                FechaActualizacion = DateTime.Now
 1912            };
 913
 1914            _context.RelacionesLaborales.Add(nuevaRelacion);
 915
 916            // Registrar movimientos
 917            // Prefer specific tipo_movimiento ids: 6 = Cambio de Situacion (cierre), 7 = Ascenso (alta)
 1918            var tipoCierre = await _context.TiposMovimiento.FirstOrDefaultAsync(t => t.Nombre.ToLower().Contains("cambio
 1919            var tipoAlta = await _context.TiposMovimiento.FirstOrDefaultAsync(t => t.Nombre.ToLower().Contains("ascenso"
 920
 1921            if (tipoCierre == null)
 922            {
 1923                tipoCierre = new Entidades.TipoMovimiento { Nombre = "Cambio de Situacion", EsAlta = false };
 1924                _context.TiposMovimiento.Add(tipoCierre);
 1925                await _context.SaveChangesAsync();
 926            }
 927
 1928            if (tipoAlta == null)
 929            {
 1930                tipoAlta = new Entidades.TipoMovimiento { Nombre = "Ascenso", EsAlta = true };
 1931                _context.TiposMovimiento.Add(tipoAlta);
 1932                await _context.SaveChangesAsync();
 933            }
 934
 935            // Movimiento de cierre
 1936            var movimientoCierre = new Entidades.MovimientoLaboral
 1937            {
 1938                RelacionLaboralId = relacionLaboral.Id,
 1939                TipoMovimientoId = tipoCierre.Id,
 1940                FechaMovimiento = fechaAscensoUtc,
 1941                UsuarioId = _currentUser.UserId ?? 0,
 1942                Observaciones = request.Observacion,
 1943                Puesto = request.Puesto,
 1944                Plaza = request.Plaza
 1945            };
 946
 1947            _context.MovimientosLaborales.Add(movimientoCierre);
 948
 1949            await _context.SaveChangesAsync();
 950
 1951            var movimientoAlta = new Entidades.MovimientoLaboral
 1952            {
 1953                RelacionLaboralId = nuevaRelacion.Id,
 1954                TipoMovimientoId = tipoAlta.Id,
 1955                FechaMovimiento = fechaAscensoUtc,
 1956                UsuarioId = _currentUser.UserId ?? 0,
 1957                Observaciones = request.Observacion,
 1958                Puesto = request.Puesto,
 1959                Plaza = request.Plaza
 1960            };
 961
 1962            _context.MovimientosLaborales.Add(movimientoAlta);
 963
 1964            await _context.SaveChangesAsync();
 965
 1966            await _auditoriaService.LogAuditoriaAsync(
 1967                usuarioId: _currentUser.UserId ?? 0,
 1968                AccionEnum.AscensoPersonal,
 1969                ContextoEnum.Personal,
 1970                host: _currentUser.Host,
 1971                entidad: nameof(RelacionLaboral),
 1972                entidadId: nuevaRelacion.Id,
 1973                detalle: new { personaId = id, relacionAnteriorId = relacionLaboral.Id, relacionNuevaId = nuevaRelacion.
 974
 1975            return (true, null);
 976        }
 0977        catch (Exception ex)
 978        {
 0979            _logger.LogError(ex, "Error al registrar ascenso");
 0980            return (false, $"Error al registrar ascenso: {ex.Message}");
 981        }
 5982    }
 983
 984    private IQueryable<RelacionLaboral> ApplyOrdering(
 985        IQueryable<RelacionLaboral> query,
 986        string? orderBy,
 987        string sort)
 988    {
 1989        var ascending = sort.ToLower() == "asc";
 990
 1991        return orderBy?.ToLower() switch
 1992        {
 0993            "ci" => ascending
 0994                ? query.OrderBy(r => r.Persona.Cedula)
 0995                : query.OrderByDescending(r => r.Persona.Cedula),
 0996            "nombre" => ascending
 0997                ? query.OrderBy(r => r.Persona.PrimerNombre)
 0998                : query.OrderByDescending(r => r.Persona.PrimerNombre),
 0999            "apellido" => ascending
 01000                ? query.OrderBy(r => r.Persona.PrimerApellido)
 01001                : query.OrderByDescending(r => r.Persona.PrimerApellido),
 01002            "grado" => ascending
 01003                ? query.OrderBy(r => r.Grado.Orden)
 01004                : query.OrderByDescending(r => r.Grado.Orden),
 01005            "fechaingreso" => ascending
 01006                ? query.OrderBy(r => r.FechaInicio)
 01007                : query.OrderByDescending(r => r.FechaInicio),
 11008            _ => query.OrderByDescending(r => r.FechaInicio)
 11009        };
 1010    }
 1011
 1012    #region Métodos Privados
 1013
 1014    /// <summary>
 1015    /// Fallback cuando el alta no especifica la ley explícitamente.
 1016    /// En el legacy (FoxPro), PERSONAL.Ley_Vieja es un campo manual por persona —
 1017    /// no existe una regla automática confiable (depende de la situación previsional
 1018    /// real del funcionario al momento de la reforma). Por eso siempre se retorna
 1019    /// Ley Nueva (19.695) como fallback seguro, y se recomienda enviar el campo
 1020    /// Ley explícitamente en el alta.
 1021    /// </summary>
 1022    private async Task<long> DeterminarRegimenIdAsync(DateTime fechaIngreso, long? escalafonId)
 1023    {
 71024        var regimen = await _context.Regimenes
 71025            .FirstOrDefaultAsync(r => r.NumeroLey == "19.695" && r.Vigente);
 71026        return regimen!.Id;
 71027    }
 1028
 1029    private static bool EsSituacionSuperior(string codigo) =>
 41030        codigo is "SIT01" or "SIT02" or "SIT04" or "SIT05" or "SIT06" or "SIT16";
 1031
 1032    private static bool EsSituacionSubalterno(string codigo) =>
 41033        codigo is "SIT07" or "SIT08" or "SIT09" or "SIT11" or "SIT17";
 1034
 1035    #endregion
 1036
 1037    public async Task<List<MovimientoReporteDto>> ObtenerReporteMovimientosAsync(int mes, int anio, string? tipo)
 1038    {
 01039        var tipoNorm = tipo?.ToLower();
 1040
 1041        // Traer movimientos del mes/año con sus relaciones
 01042        var movimientos = await _context.MovimientosLaborales
 01043            .Include(m => m.TipoMovimiento)
 01044            .Include(m => m.RelacionLaboral)
 01045                .ThenInclude(r => r.Persona)
 01046            .Include(m => m.RelacionLaboral)
 01047                .ThenInclude(r => r.Grado)
 01048            .Include(m => m.RelacionLaboral)
 01049                .ThenInclude(r => r.Escalafon)
 01050            .Include(m => m.RelacionLaboral)
 01051                .ThenInclude(r => r.Unidad)
 01052            .Include(m => m.RelacionLaboral)
 01053                .ThenInclude(r => r.Programa)
 01054            .Include(m => m.RelacionLaboral)
 01055                .ThenInclude(r => r.Regimen)
 01056            .Where(m => m.FechaMovimiento.Month == mes && m.FechaMovimiento.Year == anio)
 01057            .OrderBy(m => m.FechaMovimiento)
 01058            .ToListAsync();
 1059
 1060        // Para ascensos: agrupar los movimientos de cierre + alta para emitir un solo renglón
 01061        var resultado = new List<MovimientoReporteDto>();
 01062        var procesados = new HashSet<long>();
 1063
 01064        foreach (var mov in movimientos)
 1065        {
 01066            if (procesados.Contains(mov.Id)) continue;
 1067
 01068            var nombreTipo = mov.TipoMovimiento.Nombre.ToLower();
 1069            string categoriaReporte;
 1070
 01071            if (nombreTipo.Contains("alta por ascenso") || nombreTipo.Contains("ascenso"))
 1072            {
 01073                categoriaReporte = "Ascenso";
 01074                if (tipoNorm != null && tipoNorm != "ascenso" && tipoNorm != "todos") continue;
 1075
 1076                // Buscar el movimiento de cierre del mismo día para obtener grado anterior
 01077                var cierreRelacionado = movimientos.FirstOrDefault(m =>
 01078                    !procesados.Contains(m.Id) &&
 01079                    m.Id != mov.Id &&
 01080                    m.FechaMovimiento.Date == mov.FechaMovimiento.Date &&
 01081                    m.RelacionLaboral.PersonaId == mov.RelacionLaboral.PersonaId &&
 01082                    (m.TipoMovimiento.Nombre.ToLower().Contains("baja por ascenso") ||
 01083                     m.TipoMovimiento.Nombre.ToLower().Contains("cambio de situaci")));
 1084
 01085                var gradoAnterior = cierreRelacionado?.RelacionLaboral.Grado.Denominacion;
 01086                if (cierreRelacionado != null) procesados.Add(cierreRelacionado.Id);
 1087
 01088                resultado.Add(new MovimientoReporteDto
 01089                {
 01090                    TipoMovimiento = categoriaReporte,
 01091                    Grado = mov.RelacionLaboral.Grado.Denominacion,
 01092                    GradoAnterior = gradoAnterior,
 01093                    Escalafon = mov.RelacionLaboral.Escalafon.Denominacion,
 01094                    Cedula = mov.RelacionLaboral.Persona.Cedula,
 01095                    Nombre = mov.RelacionLaboral.Persona.PrimerNombre,
 01096                    Apellido = mov.RelacionLaboral.Persona.PrimerApellido,
 01097                    Unidad = mov.RelacionLaboral.Unidad.Denominacion,
 01098                    Programa = mov.RelacionLaboral.Programa.Denominacion,
 01099                    Ley = mov.RelacionLaboral.Regimen.NumeroLey,
 01100                    FechaMovimiento = mov.FechaMovimiento
 01101                });
 1102            }
 01103            else if (mov.TipoMovimiento.EsAlta &&
 01104                     !nombreTipo.Contains("baja por ascenso") &&
 01105                     !nombreTipo.Contains("cambio de situaci"))
 1106            {
 01107                categoriaReporte = "Alta";
 01108                if (tipoNorm != null && tipoNorm != "alta" && tipoNorm != "todos") continue;
 1109
 01110                resultado.Add(new MovimientoReporteDto
 01111                {
 01112                    TipoMovimiento = categoriaReporte,
 01113                    Grado = mov.RelacionLaboral.Grado.Denominacion,
 01114                    Escalafon = mov.RelacionLaboral.Escalafon.Denominacion,
 01115                    Cedula = mov.RelacionLaboral.Persona.Cedula,
 01116                    Nombre = mov.RelacionLaboral.Persona.PrimerNombre,
 01117                    Apellido = mov.RelacionLaboral.Persona.PrimerApellido,
 01118                    Unidad = mov.RelacionLaboral.Unidad.Denominacion,
 01119                    Programa = mov.RelacionLaboral.Programa.Denominacion,
 01120                    Ley = mov.RelacionLaboral.Regimen.NumeroLey,
 01121                    FechaMovimiento = mov.FechaMovimiento
 01122                });
 1123            }
 01124            else if (!mov.TipoMovimiento.EsAlta &&
 01125                     !nombreTipo.Contains("baja por ascenso") &&
 01126                     !nombreTipo.Contains("cambio de situaci") &&
 01127                     !nombreTipo.Contains("ascenso"))
 1128            {
 01129                categoriaReporte = "Baja";
 01130                if (tipoNorm != null && tipoNorm != "baja" && tipoNorm != "todos") continue;
 1131
 01132                resultado.Add(new MovimientoReporteDto
 01133                {
 01134                    TipoMovimiento = categoriaReporte,
 01135                    Grado = mov.RelacionLaboral.Grado.Denominacion,
 01136                    Escalafon = mov.RelacionLaboral.Escalafon.Denominacion,
 01137                    Cedula = mov.RelacionLaboral.Persona.Cedula,
 01138                    Nombre = mov.RelacionLaboral.Persona.PrimerNombre,
 01139                    Apellido = mov.RelacionLaboral.Persona.PrimerApellido,
 01140                    Unidad = mov.RelacionLaboral.Unidad.Denominacion,
 01141                    Programa = mov.RelacionLaboral.Programa.Denominacion,
 01142                    Ley = mov.RelacionLaboral.Regimen.NumeroLey,
 01143                    FechaMovimiento = mov.FechaMovimiento
 01144                });
 1145            }
 1146
 01147            procesados.Add(mov.Id);
 1148        }
 1149
 01150        return resultado;
 01151    }
 1152}