From 7fce543b92613a8eba5fef40e347da4aae36fd10 Mon Sep 17 00:00:00 2001 From: xXpuma99Xx <51341582+xXpuma99Xx@users.noreply.github.com> Date: Sun, 10 Jul 2022 23:41:32 -0500 Subject: [PATCH] =?UTF-8?q?mejora=20de=20c=C3=B3digo?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../institucion-usuario.service.ts | 2 +- .../{prestamos-activos.dto.ts => activos.dto.ts} | 2 +- src/prestamo/dto/output/entrega.dto.ts | 11 +++++++++++ src/prestamo/prestamo.controller.ts | 9 +++++---- src/prestamo/prestamo.service.ts | 2 +- src/usuario/usuario.service.ts | 4 ++-- 6 files changed, 21 insertions(+), 9 deletions(-) rename src/prestamo/dto/output/{prestamos-activos.dto.ts => activos.dto.ts} (91%) create mode 100644 src/prestamo/dto/output/entrega.dto.ts diff --git a/src/institucion-usuario/institucion-usuario.service.ts b/src/institucion-usuario/institucion-usuario.service.ts index b9694af..35c3e2d 100644 --- a/src/institucion-usuario/institucion-usuario.service.ts +++ b/src/institucion-usuario/institucion-usuario.service.ts @@ -31,7 +31,7 @@ export class InstitucionUsuarioService { }); } - findByUsuarioIdInstitucionCarrera( + findByUsuarioInstitucionCarrera( usuario: Usuario, institucionCarrera: InstitucionCarrera, campo: string, diff --git a/src/prestamo/dto/output/prestamos-activos.dto.ts b/src/prestamo/dto/output/activos.dto.ts similarity index 91% rename from src/prestamo/dto/output/prestamos-activos.dto.ts rename to src/prestamo/dto/output/activos.dto.ts index ca5c256..f26d017 100644 --- a/src/prestamo/dto/output/prestamos-activos.dto.ts +++ b/src/prestamo/dto/output/activos.dto.ts @@ -2,7 +2,7 @@ import { Expose, Type } from 'class-transformer'; import { EquiposOutputDto } from '../../../equipo/dto/output/equipos.dto'; import { UsuariosOutputDto } from '../../../usuario/dto/output/usuarios.dto'; -export class PrestamosActivosOutputDto { +export class ActivosOutputDto { @Expose() id_prestamo; diff --git a/src/prestamo/dto/output/entrega.dto.ts b/src/prestamo/dto/output/entrega.dto.ts new file mode 100644 index 0000000..19c4c42 --- /dev/null +++ b/src/prestamo/dto/output/entrega.dto.ts @@ -0,0 +1,11 @@ +import { Expose, Type } from 'class-transformer'; +import { EquipoMinOutputDto } from '../../../equipo/dto/output/equipo-min.dto'; + +export class EntregaOutputDto { + @Expose() + message; + + @Expose() + @Type(() => EquipoMinOutputDto) + equipo; +} diff --git a/src/prestamo/prestamo.controller.ts b/src/prestamo/prestamo.controller.ts index fd38618..035ea92 100644 --- a/src/prestamo/prestamo.controller.ts +++ b/src/prestamo/prestamo.controller.ts @@ -2,7 +2,6 @@ import { Body, Controller, Get, Post, Put, Query } from '@nestjs/common'; import { ApiBody, ApiOperation, ApiQuery, ApiTags } from '@nestjs/swagger'; import { Serealize } from '../interceptors/serialize.interceptor'; import { PrestamoService } from './prestamo.service'; - import { IdEquipoPaginaDto } from '../dto/id-equipo-pagina.dto'; import { IdUsuarioDto } from '../dto/id-usuario.dto'; import { IdUsuarioPaginaDto } from '../dto/id-usuario-pagina.dto'; @@ -16,10 +15,11 @@ import { PedirDto } from './dto/input/pedir.dto'; import { EntregarDto } from './dto/input/entregar.dto'; import { RegresarIdPrestamoDto } from './dto/input/regresar-id-prestamo.dto'; import { RegresarNumeroInventarioDto } from './dto/input/regresar-numero-inventario.dto'; +import { ActivosOutputDto } from './dto/output/activos.dto'; +import { EntregaOutputDto } from './dto/output/entrega.dto'; import { PrestamoOutputDto } from './dto/output/prestamo.dto'; import { PrestamoEquipoOutputDto } from './dto/output/prestamo-equipo.dto'; import { PrestamosOutputDto } from './dto/output/prestamos.dto'; -import { PrestamosActivosOutputDto } from './dto/output/prestamos-activos.dto'; import { PrestamosEquipoOutputDto } from './dto/output/prestamos-equipo.dto'; import { PrestamosUsuarioOutputDto } from './dto/output/prestamos-usuario.dto'; @@ -28,7 +28,7 @@ import { PrestamosUsuarioOutputDto } from './dto/output/prestamos-usuario.dto'; export class PrestamoController { constructor(private prestamoService: PrestamoService) {} - @Serealize(PrestamosActivosOutputDto) + @Serealize(ActivosOutputDto) @Get('activos') @ApiOperation({ description: @@ -121,6 +121,7 @@ export class PrestamoController { return this.prestamoService.cancelarUsuario(body.id_prestamo); } + @Serealize(EntregaOutputDto) @Put('entregar') @ApiOperation({ description: 'Endpoint que actualiza el status del equipo a "En uso".', @@ -254,7 +255,7 @@ export class PrestamoController { ); } - @Serealize(PrestamosUsuarioOutputDto) + @Serealize(PrestamoOutputDto) @Post() @ApiOperation({ description: 'Endpoint que crea un prestamo nuevo.' }) @ApiBody({ diff --git a/src/prestamo/prestamo.service.ts b/src/prestamo/prestamo.service.ts index 46a16b0..2cba986 100644 --- a/src/prestamo/prestamo.service.ts +++ b/src/prestamo/prestamo.service.ts @@ -102,7 +102,7 @@ export class PrestamoService { id_programa?: number, id_tipo_entrada?: number, ) { - const ahora = moment().add(1, 'd'); + const ahora = moment(); const sistema = await this.operadorService.findById(1); const modulo = await this.moduloService.findById(id_modulo); const usuario = await this.usuarioService.findById(id_usuario, true, true); diff --git a/src/usuario/usuario.service.ts b/src/usuario/usuario.service.ts index dc901d2..c8724b6 100644 --- a/src/usuario/usuario.service.ts +++ b/src/usuario/usuario.service.ts @@ -83,7 +83,7 @@ export class UsuarioService { throw new ConflictException( 'No se puede registar con este número de cuenta por que se encuentra desactivado.', ); - return this.institucionUsuarioService.findByUsuarioIdInstitucionCarrera( + return this.institucionUsuarioService.findByUsuarioInstitucionCarrera( existeUsuario, institucionCarrera, 'cuenta', @@ -108,7 +108,7 @@ export class UsuarioService { throw new ConflictException( 'No se puede registar con este número de trabajador por que se encuentra desactivado.', ); - return this.institucionUsuarioService.findByUsuarioIdInstitucionCarrera( + return this.institucionUsuarioService.findByUsuarioInstitucionCarrera( existeUsuario, institucionCarrera, 'trabajador',