From a41b30b94a4dad52909fdb6d1974692ecef78701 Mon Sep 17 00:00:00 2001
From: santiago <sm12@gmail.com>
Date: Wed, 27 Aug 2025 12:44:22 -0600
Subject: [PATCH] Modificacion y refactorizacion del codigo

---
 src/almacenamiento/almacenamiento.controller.ts | 17 +++--------------
 src/almacenamiento/almacenamiento.service.ts    | 12 ++++++++----
 2 files changed, 11 insertions(+), 18 deletions(-)

diff --git a/src/almacenamiento/almacenamiento.controller.ts b/src/almacenamiento/almacenamiento.controller.ts
index 1dd4e39..0057147 100644
--- a/src/almacenamiento/almacenamiento.controller.ts
+++ b/src/almacenamiento/almacenamiento.controller.ts
@@ -1,26 +1,15 @@
-import { Controller, Post, Get, Body, HttpException, HttpStatus } from '@nestjs/common';
+import { Controller, Post, Get, Body } from '@nestjs/common';
 import { AlmacenamientoService } from './almacenamiento.service';
 
 @Controller('almacenamiento')
 export class AlmacenamientoController {
   constructor(private readonly almacenamientoService: AlmacenamientoService) {}
 
-  // almacenamiento.controller.ts
   @Post()
-  async create(@Body() body: { correo: string }) {
-      try {
-          const resultado = await this.almacenamientoService.create(body.correo);
-          return resultado; // { mensaje: 'Usuario verificado correctamente' }
-      } catch (error: any) {
-          // Aquí devolvemos un JSON con status 404 si no se encontró usuario
-          throw new HttpException(
-              { error: error.message }, 
-              HttpStatus.NOT_FOUND
-          );
-      }
+  async create(@Body() body: { correo: string; agree: boolean }) {
+    return this.almacenamientoService.create(body.correo);
   }
 
-
   @Get()
   async findAll() {
     const data = await this.almacenamientoService.findAll();
diff --git a/src/almacenamiento/almacenamiento.service.ts b/src/almacenamiento/almacenamiento.service.ts
index 7ca3bf2..f18495e 100644
--- a/src/almacenamiento/almacenamiento.service.ts
+++ b/src/almacenamiento/almacenamiento.service.ts
@@ -1,4 +1,4 @@
-import { Injectable } from '@nestjs/common';
+import { HttpException, HttpStatus, Injectable } from '@nestjs/common';
 import { InjectRepository } from '@nestjs/typeorm';
 import { Repository } from 'typeorm';
 import { Almacenamiento } from './entity/almacenamiento.entity';
@@ -15,9 +15,13 @@ export class AlmacenamientoService {
     console.log(correo)
     const existente = await this.almacenamientoRepository.findOne({ where:{correo} });
     console.log(existente)
-  if (!existente) {
-    throw new Error("Este usuario no tiene acceso") 
-  }
+    if (!existente) {
+      // Maneja el error aquí mismo
+      throw new HttpException(
+        { error: 'Este usuario no tiene acceso' },
+        HttpStatus.NOT_FOUND
+      );
+    }
 
     return { mensaje: 'Usuario verificado correctamente' };
   }