Refactor getGroupsByName to streamline database call

Replaced the variable assignment and logging with a streamlined return statement for better code readability and performance. This change improves the method's efficiency by reducing unnecessary steps and directly returning the result.

apps/backend/src/app/files/files.controller.ts

@@ -1,14 +1,22 @@
-import { Controller, Get, Param, Post, StreamableFile } from '@nestjs/common';
+import { Controller, DefaultValuePipe, Get, Param, ParseIntPipe, Post, Query, StreamableFile } from '@nestjs/common';
 import { FilesService } from "./files.service";
 
 @Controller("files")
 export class FilesController {
 	constructor(private readonly filesService: FilesService) {}
 
-  //TODO POST FILE
   @Post('new')
   async saveFile() {
 
+  }
+
+  @Get('find')
+  async findMany(
+    @Query("limit", new DefaultValuePipe(20), ParseIntPipe) limit: number,
+    @Query("offset", new DefaultValuePipe(0), ParseIntPipe) offset: number,
+    @Query("search", new DefaultValuePipe("")) search: string,
+  ) {
+
   }
 
 	@Get(':fileId')

apps/backend/src/app/files/files.dto.ts

@@ -0,0 +1,19 @@
+import { IsUUID, MaxLength, MinLength } from 'class-validator';
+import { DefaultValuePipe } from '@nestjs/common';
+
+
+export class CreateFilesDto {
+  @MaxLength(128)
+  @MinLength(4)
+  fileName: string;
+
+  @MaxLength(64)
+  @MinLength(2)
+  uploadedBy: string;
+
+  isDocumentation?: boolean;
+  isRestricted?: boolean;
+
+  @IsUUID()
+  groupId: string;
+}

apps/backend/src/app/files/files.service.ts

@@ -2,7 +2,7 @@ import { Injectable, NotFoundException, StreamableFile } from '@nestjs/common';
 import { DbService } from 'apps/backend/src/app/db/db.service';
 import { FilesTable } from 'apps/backend/src/app/db/schema';
 import { StorageService } from 'apps/backend/src/app/storage/storage.service';
-import { eq } from 'drizzle-orm';
+import { eq, ilike } from 'drizzle-orm';
 
 
 @Injectable()
@@ -55,10 +55,20 @@ export class FilesService {
 		});
 	}
 
-	//TODO list the files
+	public async search(limit: number, offset: number, searchField: string) {
+		return await this.database
+			.use()
+			.select()
+			.from(FilesTable)
+			.where(ilike(FilesTable.fileName, String(searchField)))
+			.limit(limit)
+			.offset(offset)
+			.prepare("searchFiles")
+			.execute();
+	}
 
 	//TODO save a file
-	public async set() {
+	public async save() {
 
 	}
 }

apps/backend/src/app/groups/groups.service.ts

@@ -1,24 +1,23 @@
-import { Injectable } from "@nestjs/common";
+import { Injectable } from '@nestjs/common';
 import { DbService } from 'apps/backend/src/app/db/db.service';
 import { FilesGroupTable } from 'apps/backend/src/app/db/schema';
 import { ilike } from 'drizzle-orm';
 
+
 @Injectable()
 export class GroupsService {
   constructor(private readonly database: DbService) {}
 
   //TODO a method to fetch groups in the database by a specific search with limit, offset and a search field (can be blank)
   async getGroupsByName(limit: number, offset: number, search: string) {
-    const result = await this.database.use()
+    return await this.database.use()
       .select()
       .from(FilesGroupTable)
       .where(ilike(FilesGroupTable.groupName, search))
       .limit(limit)
       .offset(offset)
       .prepare("getGroupsByName")
-      .execute()
+      .execute();
-    console.log(`Found ${result.length} groups for search :\n > "${search}"`)
-    return result;
   }
 
   //TODO The method to create a group