diff --git a/src/institucion-carrera/institucion-carrera.service.ts b/src/institucion-carrera/institucion-carrera.service.ts index b80172b..1c30e74 100644 --- a/src/institucion-carrera/institucion-carrera.service.ts +++ b/src/institucion-carrera/institucion-carrera.service.ts @@ -56,7 +56,7 @@ export class InstitucionCarreraService { findCarreraByIdCarrera(id_carrera: number) { return this.carreraRepository.findOne({ id_carrera }).then((carrera) => { - if (!carrera) throw new NotFoundException('No existe esta carrera.'); + if (!carrera) throw new NotFoundException('No existe este id carrera.'); return carrera; }); } diff --git a/src/institucion-dia/institucion-dia.service.ts b/src/institucion-dia/institucion-dia.service.ts index c1154a2..c9c8805 100644 --- a/src/institucion-dia/institucion-dia.service.ts +++ b/src/institucion-dia/institucion-dia.service.ts @@ -28,7 +28,7 @@ export class InstitucionDiaService { }); } - findDia(institucion: Institucion, id_dia: number) { + findInstitucionDia(institucion: Institucion, id_dia: number) { return this.repository.findOne({ institucion, dia: { id_dia } }); } diff --git a/src/institucion-infraccion/institucion-infraccion.service.ts b/src/institucion-infraccion/institucion-infraccion.service.ts index a52e66f..bef2c10 100644 --- a/src/institucion-infraccion/institucion-infraccion.service.ts +++ b/src/institucion-infraccion/institucion-infraccion.service.ts @@ -39,7 +39,7 @@ export class InstitucionInfraccionService { institucion: instituciones[i], }), ); - return { message: 'Se creó creo correctamente la infracción.' }; + return { message: 'Se creó creo correctamente una nueva infracción.' }; }); } @@ -72,7 +72,7 @@ export class InstitucionInfraccionService { return this.repositoryInstitucionInfraccion.save(institucionInfraccion); }) .then((_) => ({ - message: 'Se actualizó correctamente la institución infracción.', + message: 'Se guardaron los cambios correctamente.', })); } } diff --git a/src/institucion-programa/institucion-programa.controller.ts b/src/institucion-programa/institucion-programa.controller.ts index d073776..a1b898e 100644 --- a/src/institucion-programa/institucion-programa.controller.ts +++ b/src/institucion-programa/institucion-programa.controller.ts @@ -59,8 +59,9 @@ export class InstitucionProgramaController { type: 'string', }) programasMostrar(@Query() query: IdInstitucionDto) { - return this.institucionProgramaService.findAllByIdInstitucionMostrar( + return this.institucionProgramaService.findAllByIdInstitucion( parseInt(query.id_institucion), + true, ); } diff --git a/src/institucion-programa/institucion-programa.service.ts b/src/institucion-programa/institucion-programa.service.ts index 2ff1e24..f05e522 100644 --- a/src/institucion-programa/institucion-programa.service.ts +++ b/src/institucion-programa/institucion-programa.service.ts @@ -4,7 +4,8 @@ import { NotFoundException, } from '@nestjs/common'; import { InjectRepository } from '@nestjs/typeorm'; -import { Not, Repository } from 'typeorm'; +import { FindOperator, Not, Repository } from 'typeorm'; +import { Institucion } from 'src/institucion/entity/institucion.entity'; import { InstitucionPrograma } from './entity/institucion-programa.entity'; import { Programa } from './entity/programa.entity'; import { InstitucionService } from '../institucion/institucion.service'; @@ -39,7 +40,7 @@ export class InstitucionProgramaService { institucion: instituciones[i], }), ); - return { message: 'Se creó correctamente el programa.' }; + return { message: 'Se creó correctamente un nuevo programa.' }; }); } @@ -72,27 +73,19 @@ export class InstitucionProgramaService { }); } - findAllByIdInstitucion(id_institucion: number) { - return this.institucionService - .findById(id_institucion) - .then((institucion) => - this.institucionProgramaRepository.find({ - institucion, - programa: { id_programa: Not(1) }, - }), - ); - } + async findAllByIdInstitucion(id_institucion: number, mostrar = false) { + const institucion = await this.institucionService.findById(id_institucion); + const busqueda: { + institucion: Institucion; + programa: { id_programa: FindOperator }; + mostrar?: boolean; + } = { + institucion, + programa: { id_programa: Not(1) }, + }; - findAllByIdInstitucionMostrar(id_institucion: number) { - return this.institucionService - .findById(id_institucion) - .then((institucion) => - this.institucionProgramaRepository.find({ - institucion, - mostrar: true, - programa: { id_programa: Not(1) }, - }), - ); + if (mostrar) busqueda.mostrar = mostrar; + return this.institucionProgramaRepository.find(busqueda); } update(attrs: Partial) { @@ -102,8 +95,7 @@ export class InstitucionProgramaService { return this.institucionProgramaRepository.save(institucionPrograma); }) .then((_) => ({ - message: - 'Se actualizó correctamente la información de esta institución programa.', + message: 'Se guardaron los cambios correctamente.', })); } } diff --git a/src/institucion-tipo-carrito/dto/input/update.dto.ts b/src/institucion-tipo-carrito/dto/input/update.dto.ts index 0ae9807..490fc75 100644 --- a/src/institucion-tipo-carrito/dto/input/update.dto.ts +++ b/src/institucion-tipo-carrito/dto/input/update.dto.ts @@ -1,4 +1,4 @@ -import { IsBoolean, IsInt, IsNumber } from 'class-validator'; +import { IsBoolean, IsInt } from 'class-validator'; export class UpdateInstitucionTipoCarritoDto { @IsInt() diff --git a/src/institucion-tipo-carrito/entity/institucion-tipo-carrito.entity.ts b/src/institucion-tipo-carrito/entity/institucion-tipo-carrito.entity.ts index a0969ee..0ac7bff 100644 --- a/src/institucion-tipo-carrito/entity/institucion-tipo-carrito.entity.ts +++ b/src/institucion-tipo-carrito/entity/institucion-tipo-carrito.entity.ts @@ -20,11 +20,9 @@ export class InstitucionTipoCarrito { @JoinColumn({ name: 'id_institucion' }) institucion: Institucion; - @ManyToOne( - () => TipoCarrito, - (tipoCarrito) => tipoCarrito.institucionesTipoCarrito, - { eager: true }, - ) + @ManyToOne(() => TipoCarrito, (tipoCarrito) => tipoCarrito.instituciones, { + eager: true, + }) @JoinColumn({ name: 'id_tipo_carrito' }) tipoCarrito: TipoCarrito; } diff --git a/src/institucion-tipo-carrito/entity/tipo-carrito.entity.ts b/src/institucion-tipo-carrito/entity/tipo-carrito.entity.ts index 2f28aa9..d28deb8 100644 --- a/src/institucion-tipo-carrito/entity/tipo-carrito.entity.ts +++ b/src/institucion-tipo-carrito/entity/tipo-carrito.entity.ts @@ -17,5 +17,5 @@ export class TipoCarrito { carritos: Carrito[]; @OneToMany(() => InstitucionTipoCarrito, (institucionTipoCarrito) => institucionTipoCarrito.tipoCarrito) - institucionesTipoCarrito: InstitucionTipoCarrito[]; + instituciones: InstitucionTipoCarrito[]; } diff --git a/src/institucion-tipo-carrito/institucion-tipo-carrito.controller.ts b/src/institucion-tipo-carrito/institucion-tipo-carrito.controller.ts index ff6bae8..0aa3c1d 100644 --- a/src/institucion-tipo-carrito/institucion-tipo-carrito.controller.ts +++ b/src/institucion-tipo-carrito/institucion-tipo-carrito.controller.ts @@ -67,8 +67,9 @@ export class InstitucionTipoCarritoController { type: 'string', }) tiposCarritoMostrar(@Query() query: IdInstitucionDto) { - return this.institucionTipoCarritoService.findAllByIdInstitucionMostar( + return this.institucionTipoCarritoService.findAllByIdInstitucion( parseInt(query.id_institucion), + true, ); } diff --git a/src/institucion-tipo-carrito/institucion-tipo-carrito.service.ts b/src/institucion-tipo-carrito/institucion-tipo-carrito.service.ts index 249945e..f717571 100644 --- a/src/institucion-tipo-carrito/institucion-tipo-carrito.service.ts +++ b/src/institucion-tipo-carrito/institucion-tipo-carrito.service.ts @@ -44,11 +44,11 @@ export class InstitucionTipoCarritoService { for (let i = 0; i < instituciones.length; i++) await this.repositoryInstitucionTipoCarrito.save( this.repositoryInstitucionTipoCarrito.create({ - tipoCarrito, institucion: instituciones[i], + tipoCarrito, }), ); - return { message: 'Se creo correctamente el tipo carrito.' }; + return { message: 'Se creó correctamente un nuevo tipo de carrito.' }; }); } @@ -56,20 +56,14 @@ export class InstitucionTipoCarritoService { return this.repositoryTipoCarrito.find(); } - findAllByIdInstitucion(id_institucion: number, mostrar = false) { - const busqueda: { institucion?: Institucion; mostrar?: boolean } = {}; + async findAllByIdInstitucion(id_institucion: number, mostrar = false) { + const institucion = await this.institucionService.findById(id_institucion); + const busqueda: { institucion: Institucion; mostrar?: boolean } = { + institucion, + }; if (mostrar) busqueda.mostrar = mostrar; - return this.institucionService - .findById(id_institucion) - .then((institucion) => { - busqueda.institucion = institucion; - return this.repositoryInstitucionTipoCarrito.find(busqueda); - }); - } - - findAllByIdInstitucionMostar(id_institucion: number) { - return this.findAllByIdInstitucion(id_institucion, true); + return this.repositoryInstitucionTipoCarrito.find(busqueda); } findInstitucionTipoCarritoById(id_institucion_tipo_carrito) { @@ -80,7 +74,7 @@ export class InstitucionTipoCarritoService { .then((institucionTipoCarrito) => { if (!institucionTipoCarrito) throw new NotFoundException( - 'No existe esta institucion tipo carrito.', + 'No existe este id institución tipo carrito.', ); return institucionTipoCarrito; }); @@ -91,7 +85,7 @@ export class InstitucionTipoCarritoService { .findOne({ id_tipo_carrito }) .then((tipoCarrito) => { if (!tipoCarrito) - throw new NotFoundException('No existe este tipo carrito.'); + throw new NotFoundException('No existe este id tipo carrito.'); return tipoCarrito; }); } @@ -101,7 +95,7 @@ export class InstitucionTipoCarritoService { .findOne({ tipo_carrito }) .then((tipoCarrito) => { if (validarNoExiste && !tipoCarrito) - throw new NotFoundException('No existe este tipo carrito.'); + throw new NotFoundException('No existe este tipo de carrito.'); return tipoCarrito; }); } @@ -117,7 +111,7 @@ export class InstitucionTipoCarritoService { ); }) .then((_) => ({ - message: 'Se actualizó correctamente la institución tipo carrito.', + message: 'Se guardaron los cambios correctamente.', })); } } diff --git a/src/institucion/institucion.controller.ts b/src/institucion/institucion.controller.ts index 65de794..aec94e8 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.findAllActivos(); + return this.institucionService.findAll(true); } @Put() diff --git a/src/institucion/institucion.service.ts b/src/institucion/institucion.service.ts index cc30398..929d6db 100644 --- a/src/institucion/institucion.service.ts +++ b/src/institucion/institucion.service.ts @@ -9,18 +9,17 @@ export class InstitucionService { @InjectRepository(Institucion) private repository: Repository, ) {} - findAll() { - return this.repository.find(); - } + findAll(activo = false) { + const busqueda: { activo?: boolean } = {}; - findAllActivos() { - return this.repository.find({ activo: true }); + if (activo) busqueda.activo = activo; + return this.repository.find(busqueda); } findById(id_institucion: number) { return this.repository.findOne({ id_institucion }).then((institucion) => { if (!institucion) - throw new NotFoundException('No existe esta institución.'); + throw new NotFoundException('No existe este id institución.'); return institucion; }); } diff --git a/src/prestamo/prestamo.service.ts b/src/prestamo/prestamo.service.ts index 95b4bfb..f8dc988 100644 --- a/src/prestamo/prestamo.service.ts +++ b/src/prestamo/prestamo.service.ts @@ -139,7 +139,7 @@ export class PrestamoService { // 'No se puede pedir equipo de cómputo los sabados y domingos.', // ); // else { - // const institucionDia = await this.institucionDiaService.findDia( + // const institucionDia = await this.institucionDiaService.findInstitucionDia( // modulo.institucion, // ahora.weekday(), // );