From 8ee5410c913bb30cb82c2e282c8b95777675d095 Mon Sep 17 00:00:00 2001
From: Mathis <avnyr@yidhra.fr>
Date: Tue, 15 Oct 2024 11:27:47 +0200
Subject: [PATCH] Refactor files controller imports and rearrange decorators

Reorganized import statements for cleaner structure and added a missing import for CreateFileTypeDto. Adjusted the placement of UseGuards decorators to maintain consistency across methods.
---
 .../backend/src/app/files/files.controller.ts | 28 +++++++++++--------
 apps/backend/src/app/files/files.dto.ts       |  2 +-
 2 files changed, 17 insertions(+), 13 deletions(-)

diff --git a/apps/backend/src/app/files/files.controller.ts b/apps/backend/src/app/files/files.controller.ts
index 39b5a1a..5d585ae 100644
--- a/apps/backend/src/app/files/files.controller.ts
+++ b/apps/backend/src/app/files/files.controller.ts
@@ -1,6 +1,7 @@
 import { IncomingMessage } from "node:http";
 import {
-	BadRequestException, Body,
+	BadRequestException,
+	Body,
 	Controller,
 	DefaultValuePipe,
 	Delete,
@@ -8,7 +9,8 @@ import {
 	HttpCode,
 	HttpStatus,
 	Param,
-	ParseIntPipe, ParseUUIDPipe,
+	ParseIntPipe,
+	ParseUUIDPipe,
 	Post,
 	Query,
 	Req,
@@ -16,18 +18,18 @@ import {
 	Res,
 	Response,
 	StreamableFile,
-	UseGuards
-} from '@nestjs/common';
+	UseGuards,
+} from "@nestjs/common";
+import { CreateFileTypeDto } from "apps/backend/src/app/files/files.dto";
 import { AdminGuard, InsertAdminState } from "../auth/auth.guard";
 import { FilesService } from "./files.service";
-import { CreateFileTypeDto } from 'apps/backend/src/app/files/files.dto';
 
 @Controller("files")
 export class FilesController {
 	constructor(private readonly filesService: FilesService) {}
 
-	@UseGuards(InsertAdminState)
 	@HttpCode(HttpStatus.OK)
+	@UseGuards(InsertAdminState)
 	@Post("new")
 	async saveFile(@Req() req: IncomingMessage, @Res() res: Response) {
 		let fileBuffer: Buffer = Buffer.from([]);
@@ -124,26 +126,28 @@ export class FilesController {
 		return await this.filesService.get(fileId);
 	}
 
-	@UseGuards(AdminGuard)
 	@HttpCode(HttpStatus.OK)
+	@UseGuards(AdminGuard)
 	@Delete(":fileId")
 	async deleteFile(@Param("fileId", ParseUUIDPipe) fileId: string) {
 		return await this.filesService.deleteFile(fileId);
 	}
 
-	@HttpCode(HttpStatus.OK)
-  @Get("types")
-  async getTypes() {
+	@HttpCode(HttpStatus.FOUND)
+	@Get("types")
+	async getTypes() {
 		//TODO
-  }
+	}
 
-	@UseGuards(AdminGuard)
 	@HttpCode(HttpStatus.CREATED)
+	@UseGuards(AdminGuard)
 	@Post("types/new")
 	async newType(@Body() body: CreateFileTypeDto) {
 		//TODO
 	}
 
+	@HttpCode(HttpStatus.ACCEPTED)
+	@UseGuards(AdminGuard)
 	@Delete("types/:typeId")
 	async delType(@Param(":typeId", ParseUUIDPipe) typeId: string) {
 		//TODO
diff --git a/apps/backend/src/app/files/files.dto.ts b/apps/backend/src/app/files/files.dto.ts
index bd81b46..9fa652a 100644
--- a/apps/backend/src/app/files/files.dto.ts
+++ b/apps/backend/src/app/files/files.dto.ts
@@ -19,4 +19,4 @@ export class CreateFilesDto {
 
 export class CreateFileTypeDto {
 	//TODO
-}
\ No newline at end of file
+}