diff --git a/src/institucion-infraccion/institucion-infraccion.service.ts b/src/institucion-infraccion/institucion-infraccion.service.ts index e53fbe8..1907cf6 100644 --- a/src/institucion-infraccion/institucion-infraccion.service.ts +++ b/src/institucion-infraccion/institucion-infraccion.service.ts @@ -22,19 +22,22 @@ export class InstitucionInfraccionService { ) {} create(infraccion: string) { + // Busco una infracción con ese nombre return this.infraccionRepository .findOne({ where: { infraccion } }) .then((existeInfraccion) => { + // Saco error si existe if (existeInfraccion) throw new ConflictException('Ya existe esta infracción.'); + // Creo y guardo el registro return this.infraccionRepository.save( this.infraccionRepository.create({ infraccion }), ); }) .then(async (infraccion) => { - const instituciones = - await this.institucionService.findAll(); + const instituciones = await this.institucionService.findAll(); + // Le asigno la nueva infracción a todas las instituciones for (let i = 0; i < instituciones.length; i++) await this.institucionInfraccionRepository.save( this.institucionInfraccionRepository.create({ @@ -102,6 +105,8 @@ export class InstitucionInfraccionService { update(admin: Operador, attrs: Partial) { return this.findById(attrs.id_institucion_infraccion) .then((institucionInfraccion) => { + // Valido que la institucionInfraccion le pertenezca al operador que + // realiza esta acción if ( admin.institucion.id_institucion != institucionInfraccion.institucion.id_institucion @@ -109,11 +114,11 @@ export class InstitucionInfraccionService { throw new ConflictException( 'No puedes actualizar la información de esta infraccion porque no le pertenece a tu institución.', ); + // Asigno valores enviados al objeto Object.assign(institucionInfraccion, attrs); + // Guardar return this.institucionInfraccionRepository.save(institucionInfraccion); }) - .then((_) => ({ - message: 'Se guardaron los cambios correctamente.', - })); + .then((_) => ({ message: 'Se guardaron los cambios correctamente.' })); } }