Merge remote-tracking branch 'origin/Lino' into development

src/Profesor/dto/profesorDto.dto.ts

@@ -121,9 +121,4 @@ export class LineasInvestigacion{
     @IsOptional()
     @Length(0,300)
     lineas_inv_html: string;
-}
-
-export class UpdateProfesorImageDto {
-  @IsString()
-  fotografia: string;
 }

src/Profesor/profesor.controller.ts

@@ -3,7 +3,6 @@ import {
   Controller,
   Delete,
   Get,
-  HttpException,
   Logger,
   Param,
   ParseIntPipe,
@@ -13,11 +12,10 @@ import {
   UseGuards
 } from "@nestjs/common";
 import { ProfesorService } from "./profesor.service";
-import { profesorDto, UpdateProfesorImageDto } from "./dto/profesorDto.dto";
+import { profesorDto } from "./dto/profesorDto.dto";
 import { Roles } from "../permissions/roles.decorator";
 import { RolesGuard } from "../permissions/roles.guard";
 import { Role } from "../permissions/role.enum";
-import { Profesor } from "./entities/profesor.entity";
 
 @Controller("profesor")
 @UseGuards(RolesGuard)
@@ -32,24 +30,6 @@ export class ProfesorController {
     return this.profesorService.register(data);
   }
 
-  @Put(':id_profesor/image')
-  async updateImage(
-    @Param('id_profesor') id_profesor: number,
-    @Body() updateProfesorImageDto: UpdateProfesorImageDto,
-  ): Promise<Profesor> {
-    const { fotografia } = updateProfesorImageDto;
-
-    const profesor = await this.profesorService.findOne(id_profesor);
-
-    if (!profesor) {
-      throw new HttpException('Profesor not found', 404);
-    }
-
-    profesor.fotografia = fotografia;
-
-    return this.profesorService.update(profesor);
-  }
-
   @Get()
   findAll() {
     Logger.debug("findAllProfesors");

src/Profesor/profesor.module.ts

@@ -10,7 +10,12 @@ import { ProyectosAcademicos } from 'src/proyectos_academicos/entities/proyectos
 import { DatosAcademicos } from 'src/datos_academicos/entities/datos_academicos.entity'
 import { LineasInvestigacion } from 'src/lineas_investigacion/entities/lineas_investigacion.entity'
 
+import { ProyectosAcademicosService } from 'src/proyectos_academicos/proyectos_academicos.service';
+import { DatosAcademicosService } from 'src/datos_academicos/datos_academicos.service';
+import { LineasInvestigacionService } from 'src/lineas_investigacion/lineas_investigacion.service';
+
 import { JwtModule } from '@nestjs/jwt';
+import { RolesGuard } from '../permissions/roles.guard';
 
 @Module({
   imports: [

src/Profesor/profesor.service.ts

@@ -27,14 +27,6 @@ export class ProfesorService {
   ) {
   }
 
-  async findOne(id_profesor: number): Promise<Profesor | undefined> {
-    return this.profesorRepository.findOne({ where: { id_profesor } });
-  }
-
-  async update(profesor: Profesor): Promise<Profesor> {
-    return this.profesorRepository.save(profesor);
-  }
-
   private async findAvailableId(): Promise<number> {
     let id = 1;
     let exists = true;

src/auth/auth.controller.ts

@@ -13,7 +13,6 @@ import {
   ParseIntPipe,
   Query,
   Logger,
-  Req,
 } from '@nestjs/common';
 import { AuthGuard } from './auth.guard';
 import { AuthService } from './auth.service';
@@ -37,7 +36,7 @@ export class AuthController {
 
   //@UseGuards()
   @Post('register')
-  @Roles(Role.Admin)
+  //@Roles(Role.Admin)
   async register(@Body() data: registerDto) {
     Logger.debug('register user');
     return this.authService.register(data);
@@ -74,12 +73,9 @@ export class AuthController {
   async update(
     @Param('id_usuario') id_usuario: number,
     @Body() data: registerDto,
-    @Req() req: any
   ) {
-    const authHeader = req.headers['authorization'];
-    const [, token] = authHeader.split(' ');
-
-    return this.authService.update(id_usuario, data, token);
+    Logger.debug('update user');
+    return this.authService.update(id_usuario, data);
   }
 
   //@UseGuards(AuthGuard)

src/auth/auth.service.ts

@@ -26,7 +26,7 @@ export class AuthService {
 
   //register users
   async register(data: registerDto) {
-    const { usuario, contraseña } = data;
+    const { usuario, contraseña} = data;
 
     if (await this.usersService.findOne(usuario)) {
       throw new HttpException('User already exist', 403);
@@ -74,29 +74,17 @@ export class AuthService {
   }
 
   //update user
-  async update(
-    id_usuario: number,
-    data: registerDto,
-    token: string,
-  ): Promise<any> {
-    const { contraseña, id_tipo_usuario } = data;
-
-    const tokenUserId = this.getTokenUserId(token);
-
-    if (tokenUserId == id_usuario && id_tipo_usuario !== undefined) {
-      Logger.debug('Cannot modify id_tipo_usuario')
-      delete data.id_tipo_usuario;
-    }
-
-    const user = await this.userRepository.findOne({ where: { id_usuario } });
-    if (!user) {
-      throw new HttpException('User not found', 404);
-    }
+  async update(id_usuario: number, data: registerDto) {
+    const { contraseña } = data;
 
     if (contraseña) {
       const hashedpassword = await hash(contraseña, 10);
       data = { ...data, contraseña: hashedpassword };
     } else {
+      const user = await this.userRepository.findOne({ where: { id_usuario } });
+      if (!user) {
+        throw new HttpException('User not found', 404);
+      }
       data = { ...data, contraseña: user.contraseña };
     }
 
@@ -126,7 +114,7 @@ export class AuthService {
         throw new UnauthorizedException('Token has expired');
       }
 
-      return;
+      return ;
     } catch (error) {
       throw new UnauthorizedException('validation token failed');
     }
@@ -139,22 +127,18 @@ export class AuthService {
   async findAllPaginated(
     page: number,
     limit: number,
-    filters: any,
-  ): Promise<{ usuarios: User[]; total: number; totalPages: number }> {
+    filters: any
+  ): Promise<{ usuarios: User[], total: number, totalPages: number }> {
     const query = this.userRepository.createQueryBuilder('usuario');
 
     if (filters.nombre) {
       Logger.debug('fiter by name user');
-      query.andWhere('usuario.nombre LIKE :nombre', {
-        nombre: `%${filters.nombre}%`,
-      });
+      query.andWhere('usuario.nombre LIKE :nombre', { nombre: `%${filters.nombre}%` });
     }
 
     if (filters.id_tipo_usuario) {
       Logger.debug('fiter by id user');
-      query.andWhere('usuario.id_tipo_usuario = :id_tipo_usuario', {
-        id_tipo_usuario: filters.id_tipo_usuario,
-      });
+      query.andWhere('usuario.id_tipo_usuario = :id_tipo_usuario', { id_tipo_usuario: filters.id_tipo_usuario });
     }
 
     const [usuarios, total] = await query
@@ -172,7 +156,7 @@ export class AuthService {
     const user = await this.userRepository.findOne({
       where: { id_usuario },
     });
-
+    
     if (!user) {
       Logger.debug('user not found');
       throw new HttpException('user not found', 404);
@@ -180,13 +164,4 @@ export class AuthService {
 
     return user;
   }
-
-  getTokenUserId(token: string): number {
-    try {
-      const decoded = this.jwtService.verify(token);
-      return decoded['id_usuario'];
-    } catch (error) {
-      throw new UnauthorizedException('Invalid token');
-    }
-  }
 }

src/images/images.service.ts

@@ -1,10 +1,11 @@
-import { Injectable, BadRequestException, Logger } from '@nestjs/common';
+import { Injectable, BadRequestException, NotFoundException, Logger } from '@nestjs/common';
 import * as fs from 'fs';
 import * as path from 'path';
+import * as sharp from 'sharp';
 
 @Injectable()
 export class ImagenService {
-  async saveImage(fotografia: string, id: string): Promise<string> {
+  async saveImage(fotografia: string, nombre: string): Promise<string> {
     if (!fotografia) {
       throw new BadRequestException('No image provided');
     }
@@ -18,7 +19,8 @@ export class ImagenService {
 
     // Decode base64 string and save the image
     const buffer = Buffer.from(fotografia.split(',')[1], 'base64');
-    const imageName = `${id}.${ext}`;
+    const sanitizedFileName = nombre.replace(/\s+/g, '_').toLowerCase();
+    const imageName = `${sanitizedFileName}.${ext}`;
 
     // Save image to the 'imagenes' directory in the root of the project
     const imagePath = path.join(process.cwd(), 'imagenes', imageName);
@@ -29,15 +31,16 @@ export class ImagenService {
       fs.mkdirSync(dirPath, { recursive: true });
     }
 
-    fs.writeFileSync(imagePath, buffer);
+    await sharp(buffer).toFile(imagePath);
 
-    return imageName;
+    return `/imagenes/${imageName}`;
   }
 
-  async deleteImage(id: string): Promise<void> {
+  async deleteImage(nombre: string): Promise<void> {
     Logger.debug('delete image');
+    const sanitizedFileName = nombre.replace(/\s+/g, '_').toLowerCase();
     const imageDir = path.join(process.cwd(), 'imagenes');
-    const imagePath = path.join(imageDir, id);
+    const imagePath = path.join(imageDir, sanitizedFileName);
   
     const extensions = ['jpg', 'jpeg', 'png', 'gif', 'bmp', 'webp', 'avif'];
   
@@ -50,18 +53,19 @@ export class ImagenService {
         break;
       }
     }
-
+  
     if (!fileFound) {
       console.log('Image not found, but continuing to save the new image.');
+      // No se lanza excepción
     }
   }
 
-  async modify(fotografia: string, id: string): Promise<string> {
+  async modify(fotografia: string, nombre: string): Promise<string> {
     if (!fotografia) {
       throw new BadRequestException('No image provided');
     }
 
-    await this.deleteImage(id);
+    await this.deleteImage(nombre);
 
     // Extract the image extension
     const matches = fotografia.match(/^data:image\/([a-zA-Z]+);base64,/);
@@ -72,7 +76,8 @@ export class ImagenService {
 
     // Decode base64 string and save the image
     const buffer = Buffer.from(fotografia.split(',')[1], 'base64');
-    const imageName = `${id}.${ext}`;
+    const sanitizedFileName = nombre.replace(/\s+/g, '_').toLowerCase();
+    const imageName = `${sanitizedFileName}.${ext}`;
 
     // Save image to the 'imagenes' directory in the root of the project
     const imagePath = path.join(process.cwd(), 'imagenes', imageName);
@@ -83,8 +88,8 @@ export class ImagenService {
       fs.mkdirSync(dirPath, { recursive: true });
     }
 
-    fs.writeFileSync(imagePath, buffer);
+    await sharp(buffer).toFile(imagePath);
 
-    return imageName;
+    return `/imagenes/${imageName}`;
   }
 }

src/main.ts

@@ -2,19 +2,14 @@ import { NestFactory } from '@nestjs/core';
 import { AppModule } from './app.module';
 import { NestExpressApplication } from '@nestjs/platform-express';
 import * as path from 'path';
-import * as bodyParser from 'body-parser';
 
 async function bootstrap() {
   const app = await NestFactory.create<NestExpressApplication>(AppModule);
   app.enableCors();
 
-  app.use(bodyParser.json({ limit: '1mb' }));
-  app.use(bodyParser.urlencoded({ limit: '1mb', extended: true }));
-
   app.useStaticAssets(path.join(__dirname, '..', 'imagenes'), {
     prefix: '/imagenes/',
   });
-
   await app.listen(3000);
 }
 bootstrap();