From 28c1afa8f7ec5262888f63021ffa07e42957a24a Mon Sep 17 00:00:00 2001 From: xXpuma99Xx <51341582+xXpuma99Xx@users.noreply.github.com> Date: Mon, 24 Oct 2022 09:55:13 -0500 Subject: [PATCH] listo prestamo --- .../carrera-programa.service.ts | 2 +- src/carrito/carrito.service.ts | 4 +-- src/equipo/equipo.service.ts | 6 ++-- .../institucion-carrera.service.ts | 6 ++-- .../institucion-dia.service.ts | 4 +-- .../institucion-infraccion.service.ts | 4 +-- .../institucion-programa.service.ts | 6 ++-- .../institucion-tipo-carrito.service.ts | 6 ++-- .../institucion-tipo-entrada.service.ts | 7 ++--- src/institucion/institucion.controller.ts | 8 ++--- src/institucion/institucion.service.ts | 30 +++++++++---------- src/modulo/modulo.service.ts | 2 +- src/multa/multa.service.ts | 4 +-- src/operador/operador.service.ts | 6 ++-- src/prestamo/prestamo.service.ts | 6 ++-- src/upload-file/upload-file.service.ts | 12 ++------ src/usuario/usuario.service.ts | 4 +-- 17 files changed, 45 insertions(+), 72 deletions(-) diff --git a/src/carrera-programa/carrera-programa.service.ts b/src/carrera-programa/carrera-programa.service.ts index 4b1192a..0f6ab98 100644 --- a/src/carrera-programa/carrera-programa.service.ts +++ b/src/carrera-programa/carrera-programa.service.ts @@ -105,7 +105,7 @@ export class CarreraProgramaService { id_institucion: number, ) { return this.institucionService - .findInfoInstitucionById(id_institucion) + .findById(id_institucion) .then(async (institucion) => { // Si es un Operador if (usuarioOperador instanceof Operador) { diff --git a/src/carrito/carrito.service.ts b/src/carrito/carrito.service.ts index 9a27d5f..8bee2d2 100644 --- a/src/carrito/carrito.service.ts +++ b/src/carrito/carrito.service.ts @@ -153,9 +153,7 @@ export class CarritoService { }) { const carritos: Carrito[] = []; const institucion = filtros.id_institucion - ? await this.institucionService.findInfoInstitucionById( - parseInt(filtros.id_institucion), - ) + ? await this.institucionService.findById(parseInt(filtros.id_institucion)) : null; const marca = filtros.id_marca ? await this.marcaService.findById(parseInt(filtros.id_marca), 'c') diff --git a/src/equipo/equipo.service.ts b/src/equipo/equipo.service.ts index aa6afb9..a5fc99b 100644 --- a/src/equipo/equipo.service.ts +++ b/src/equipo/equipo.service.ts @@ -109,9 +109,7 @@ export class EquipoService { ? await this.carritoService.findById(parseInt(filtros.id_carrito)) : null; const institucion = filtros.id_institucion - ? await this.institucionService.findInfoInstitucionById( - parseInt(filtros.id_institucion), - ) + ? await this.institucionService.findById(parseInt(filtros.id_institucion)) : null; const marca = filtros.id_marca ? await this.marcaService.findById(parseInt(filtros.id_marca), 'e') @@ -385,7 +383,7 @@ export class EquipoService { numero_inventario: string, ) { return this.institucionService - .findInfoInstitucionById(id_institucion) + .findById(id_institucion) .then((institucion) => this.fullInformacionEquipoView.findOne({ where: { diff --git a/src/institucion-carrera/institucion-carrera.service.ts b/src/institucion-carrera/institucion-carrera.service.ts index f5f27c6..7f8f0aa 100644 --- a/src/institucion-carrera/institucion-carrera.service.ts +++ b/src/institucion-carrera/institucion-carrera.service.ts @@ -21,7 +21,7 @@ export class InstitucionCarreraService { findAllByIdInstitucion(id_institucion: number) { return this.institucionService - .findInfoInstitucionById(id_institucion) + .findById(id_institucion) .then((institucion) => this.institucionCarreraRepository.find({ where: { @@ -53,7 +53,7 @@ export class InstitucionCarreraService { : id_carrera; const institucion = typeof id_institucion === 'number' - ? await this.institucionService.findInfoInstitucionById(id_institucion) + ? await this.institucionService.findById(id_institucion) : id_institucion; return this.institucionCarreraRepository.findOne({ @@ -82,7 +82,7 @@ export class InstitucionCarreraService { findInstitucionProfesor(id_institucion: number) { return this.institucionService - .findInfoInstitucionById(id_institucion) + .findById(id_institucion) .then((institucion) => this.institucionCarreraRepository.findOne({ where: { carrera: { id_carrera: 1 }, institucion }, diff --git a/src/institucion-dia/institucion-dia.service.ts b/src/institucion-dia/institucion-dia.service.ts index 390dc2c..821d37c 100644 --- a/src/institucion-dia/institucion-dia.service.ts +++ b/src/institucion-dia/institucion-dia.service.ts @@ -22,7 +22,7 @@ export class InstitucionDiaService { findAllByIdInstitucion(id_institucion: number) { return this.institucionService - .findInfoInstitucionById(id_institucion) + .findById(id_institucion) .then((institucion) => this.repository.find({ where: { institucion }, @@ -60,7 +60,7 @@ export class InstitucionDiaService { if (ahora.weekday() === 0) throw new ConflictException('No hay servicio los días domingo.'); return this.institucionService - .findInfoInstitucionById(id_institucion) + .findById(id_institucion) .then((institucion) => this.findByInstitucionDia(institucion, ahora.weekday()), ); diff --git a/src/institucion-infraccion/institucion-infraccion.service.ts b/src/institucion-infraccion/institucion-infraccion.service.ts index f2205d3..e53fbe8 100644 --- a/src/institucion-infraccion/institucion-infraccion.service.ts +++ b/src/institucion-infraccion/institucion-infraccion.service.ts @@ -33,7 +33,7 @@ export class InstitucionInfraccionService { }) .then(async (infraccion) => { const instituciones = - await this.institucionService.findInfoInstitucionAll(); + await this.institucionService.findAll(); for (let i = 0; i < instituciones.length; i++) await this.institucionInfraccionRepository.save( @@ -55,7 +55,7 @@ export class InstitucionInfraccionService { findAllByIdInstitucion(id_institucion: number) { return this.institucionService - .findInfoInstitucionById(id_institucion) + .findById(id_institucion) .then((institucion) => this.institucionInfraccionRepository .createQueryBuilder('ii') diff --git a/src/institucion-programa/institucion-programa.service.ts b/src/institucion-programa/institucion-programa.service.ts index 5bf0b5c..61f1973 100644 --- a/src/institucion-programa/institucion-programa.service.ts +++ b/src/institucion-programa/institucion-programa.service.ts @@ -32,7 +32,7 @@ export class InstitucionProgramaService { }) .then(async (programa) => { const instituciones = - await this.institucionService.findInfoInstitucionAll(); + await this.institucionService.findAll(); for (let i = 0; i < instituciones.length; i++) await this.institucionProgramaRepository.save( @@ -50,9 +50,7 @@ export class InstitucionProgramaService { } async findAllByIdInstitucion(id_institucion: number, mostrar = false) { - const institucion = await this.institucionService.findInfoInstitucionById( - id_institucion, - ); + const institucion = await this.institucionService.findById(id_institucion); const query = this.institucionProgramaRepository .createQueryBuilder('ip') .innerJoinAndSelect('ip.programa', 'p', 'p.id_programa != 1') diff --git a/src/institucion-tipo-carrito/institucion-tipo-carrito.service.ts b/src/institucion-tipo-carrito/institucion-tipo-carrito.service.ts index 2fe60b8..90b7f1f 100644 --- a/src/institucion-tipo-carrito/institucion-tipo-carrito.service.ts +++ b/src/institucion-tipo-carrito/institucion-tipo-carrito.service.ts @@ -40,7 +40,7 @@ export class InstitucionTipoCarritoService { }) .then(async (tipoCarrito) => { const instituciones = - await this.institucionService.findInfoInstitucionAll(); + await this.institucionService.findAll(); for (let i = 0; i < instituciones.length; i++) await this.institucionTipoCarritoRepository.save( @@ -58,9 +58,7 @@ export class InstitucionTipoCarritoService { } async findAllByIdInstitucion(id_institucion: number, mostrar = false) { - const institucion = await this.institucionService.findInfoInstitucionById( - id_institucion, - ); + const institucion = await this.institucionService.findById(id_institucion); const query = this.institucionTipoCarritoRepository .createQueryBuilder('itc') .innerJoinAndSelect( diff --git a/src/institucion-tipo-entrada/institucion-tipo-entrada.service.ts b/src/institucion-tipo-entrada/institucion-tipo-entrada.service.ts index 6de3cb1..60f97e8 100644 --- a/src/institucion-tipo-entrada/institucion-tipo-entrada.service.ts +++ b/src/institucion-tipo-entrada/institucion-tipo-entrada.service.ts @@ -33,8 +33,7 @@ export class InstitucionTipoEntradaService { ); }) .then(async (tipoEntrada) => { - const instituciones = - await this.institucionService.findInfoInstitucionAll(); + const instituciones = await this.institucionService.findAll(); for (let i = 0; i < instituciones.length; i++) await this.institucionTipoEntradaRepository.save( @@ -54,9 +53,7 @@ export class InstitucionTipoEntradaService { } async findAllByIdInstitucion(id_institucion: number, mostrar = false) { - const institucion = await this.institucionService.findInfoInstitucionById( - id_institucion, - ); + const institucion = await this.institucionService.findById(id_institucion); const query = await this.institucionTipoEntradaRepository .createQueryBuilder('ite') .innerJoinAndSelect( diff --git a/src/institucion/institucion.controller.ts b/src/institucion/institucion.controller.ts index 1986da2..52c25eb 100644 --- a/src/institucion/institucion.controller.ts +++ b/src/institucion/institucion.controller.ts @@ -42,7 +42,7 @@ export class InstitucionController { description: 'Endpoint que retorna todas las instituciones activas.', }) activas() { - return this.institucionService.findMinInfoInstitucionAll(true); + return this.institucionService.findMinInfoAll(true); } @Serealize(InstitucionMinOutputDto) @@ -56,7 +56,7 @@ export class InstitucionController { const superAdmin: Operador = req.user.operador; this.validarUsuarioService.validarSuperAdmin(superAdmin); - return this.institucionService.findMinInfoInstitucionAll(); + return this.institucionService.findMinInfoAll(); } @Serealize(InstitucionOutputDto) @@ -76,7 +76,7 @@ export class InstitucionController { this.validarUsuarioService.validarSuperAdminAdmin(admin); return this.institucionService - .findById(parseInt(query.id_institucion)) + .findFullInfoById(parseInt(query.id_institucion)) .then((institucion) => { if ( admin.tipoUsuario.id_tipo_usuario === 3 && @@ -100,7 +100,7 @@ export class InstitucionController { const superAdmin: Operador = req.user.operador; this.validarUsuarioService.validarSuperAdmin(superAdmin); - return this.institucionService.findMinInfoInstitucionAll(false, true); + return this.institucionService.findMinInfoAll(false, true); } @Serealize(MessageOutputDto) diff --git a/src/institucion/institucion.service.ts b/src/institucion/institucion.service.ts index 6330659..c7d534d 100644 --- a/src/institucion/institucion.service.ts +++ b/src/institucion/institucion.service.ts @@ -18,17 +18,11 @@ export class InstitucionService { return this.repository.create({ id_institucion }); } - findById(id_institucion: number) { - return this.repository - .findOne({ where: { id_institucion } }) - .then((institucion) => { - if (!institucion) - throw new NotFoundException('No existe este id institución.'); - return institucion; - }); + findAll() { + return this.repository.find({ select: ['id_institucion'] }); } - findInfoInstitucionById(id_institucion: number) { + findById(id_institucion: number) { return this.repository .findOne({ select: ['id_institucion'], where: { id_institucion } }) .then((institucion) => { @@ -38,11 +32,17 @@ export class InstitucionService { }); } - findInfoInstitucionAll() { - return this.repository.find({ select: ['id_institucion'] }); + findFullInfoById(id_institucion: number) { + return this.repository + .findOne({ where: { id_institucion } }) + .then((institucion) => { + if (!institucion) + throw new NotFoundException('No existe este id institución.'); + return institucion; + }); } - findMinInfoInstitucionAll(activo = false, responsable = false) { + findMinInfoAll(activo = false, responsable = false) { const busqueda: FindOptionsWhere = {}; if (activo) busqueda.activo = activo; @@ -62,7 +62,7 @@ export class InstitucionService { } update(admin: Operador, attrs: Partial) { - return this.findInfoInstitucionById(attrs.id_institucion) + return this.findById(attrs.id_institucion) .then((institucion) => { if ( admin.tipoUsuario.id_tipo_usuario === 3 && @@ -74,8 +74,6 @@ export class InstitucionService { Object.assign(institucion, attrs); return this.repository.save(institucion); }) - .then((_) => ({ - message: 'Se guardaron los cambios correctamente.', - })); + .then((_) => ({ message: 'Se guardaron los cambios correctamente.' })); } } diff --git a/src/modulo/modulo.service.ts b/src/modulo/modulo.service.ts index aaf38d8..9c56bba 100644 --- a/src/modulo/modulo.service.ts +++ b/src/modulo/modulo.service.ts @@ -60,7 +60,7 @@ export class ModuloService { async findMinInfoModuloAll(id_institucion: number, activo = 0) { return this.institucionService - .findInfoInstitucionById(id_institucion) + .findById(id_institucion) .then((institucion) => { const busqueda: FindOptionsWhere = { id_institucion: institucion.id_institucion, diff --git a/src/multa/multa.service.ts b/src/multa/multa.service.ts index 9289c92..2769283 100644 --- a/src/multa/multa.service.ts +++ b/src/multa/multa.service.ts @@ -135,9 +135,7 @@ export class MultaService { usuario?: string; }) { const institucion = filtros.id_institucion - ? await this.institucionService.findInfoInstitucionById( - parseInt(filtros.id_institucion), - ) + ? await this.institucionService.findById(parseInt(filtros.id_institucion)) : null; const busqueda: FindOptionsWhere = {}; diff --git a/src/operador/operador.service.ts b/src/operador/operador.service.ts index 223974a..1605d9f 100644 --- a/src/operador/operador.service.ts +++ b/src/operador/operador.service.ts @@ -35,7 +35,7 @@ export class OperadorService { ) { const institucion = typeof id_institucion === 'number' - ? await this.institucionService.findInfoInstitucionById(id_institucion) + ? await this.institucionService.findById(id_institucion) : id_institucion; const tipoUsuario = await this.tipoUsuarioService.findById(id_tipo_usuario); @@ -111,9 +111,7 @@ export class OperadorService { operador?: string; }) { const institucion = filtros.id_institucion - ? await this.institucionService.findInfoInstitucionById( - parseInt(filtros.id_institucion), - ) + ? await this.institucionService.findById(parseInt(filtros.id_institucion)) : null; const tipoUsuario = filtros.id_tipo_usuario ? await this.tipoUsuarioService.findById( diff --git a/src/prestamo/prestamo.service.ts b/src/prestamo/prestamo.service.ts index e2b00d5..a6e471f 100644 --- a/src/prestamo/prestamo.service.ts +++ b/src/prestamo/prestamo.service.ts @@ -446,9 +446,7 @@ export class PrestamoService { usuario?: string; }) { const institucion = filtros.id_institucion - ? await this.institucionService.findInfoInstitucionById( - parseInt(filtros.id_institucion), - ) + ? await this.institucionService.findById(parseInt(filtros.id_institucion)) : null; const modulo = filtros.id_modulo ? await this.moduloService.findInfoModuloById(parseInt(filtros.id_modulo)) @@ -694,7 +692,7 @@ export class PrestamoService { numero_inventario: string, ) { return this.institucionService - .findInfoInstitucionById(id_institucion) + .findById(id_institucion) .then((institucion) => this.equipoService.findByNumeroInventario( institucion, diff --git a/src/upload-file/upload-file.service.ts b/src/upload-file/upload-file.service.ts index 5a108dd..bddffb5 100644 --- a/src/upload-file/upload-file.service.ts +++ b/src/upload-file/upload-file.service.ts @@ -45,9 +45,7 @@ export class UploadFileService { ) {} async createEquipos(path: string, id_institucion: number) { - const institucion = await this.institucionService.findInfoInstitucionById( - id_institucion, - ); + const institucion = await this.institucionService.findById(id_institucion); let errores: string[] = []; let mensajes: string[] = []; let equiposNuevos: Equipo[] = []; @@ -81,9 +79,7 @@ export class UploadFileService { } async createUsuarios(path: string, id_institucion: number) { - const institucion = await this.institucionService.findInfoInstitucionById( - id_institucion, - ); + const institucion = await this.institucionService.findById(id_institucion); let errores: string[] = []; let mensajes: string[] = []; let usuariosNuevos: InstitucionUsuario[] = []; @@ -484,9 +480,7 @@ export class UploadFileService { async uploadLogo(file: Express.Multer.File, id_institucion: number) { const path = file ? `${file.destination}/${file.filename}` : null; - const institucion = await this.institucionService.findInfoInstitucionById( - id_institucion, - ); + const institucion = await this.institucionService.findById(id_institucion); if (!file) throw new BadRequestException('No se mandó ningún logo.'); institucion.logo = path; diff --git a/src/usuario/usuario.service.ts b/src/usuario/usuario.service.ts index 0fac780..646f1e1 100644 --- a/src/usuario/usuario.service.ts +++ b/src/usuario/usuario.service.ts @@ -72,9 +72,7 @@ export class UsuarioService { ) : null; const institucion = filtros.id_institucion - ? await this.institucionService.findInfoInstitucionById( - parseInt(filtros.id_institucion), - ) + ? await this.institucionService.findById(parseInt(filtros.id_institucion)) : null; const tipoUsuario = filtros.id_tipo_usuario ? await this.tipoUsuarioService.findById(