Update endpoints to allow forest data posting and fetching

Endpoints, DTOs updated with 'type' parameter, to specify what kind of reports is being created/fetched
AplinkosMinisterija · Feb 2, 2024 · 82cddae · 82cddae
1 parent 6382bc0
commit 82cddae
Show file tree

Hide file tree

Showing 6 changed files with 25 additions and 17 deletions.
diff --git a/src/admin/admin.controller.ts b/src/admin/admin.controller.ts
@@ -47,10 +47,11 @@ export class AdminController {
   @Get('/reports')
   async getAllReports(
     @Query('isDeleted', ParseBoolPipe) isDeleted: boolean,
+    @Query('type') type: string,
   ): Promise<FullReportDto[]> {
     return isDeleted
       ? await this.adminService.getAllDeletedReports()
-      : await this.adminService.getAllReports();
+      : await this.adminService.getAllReports(type);
   }
 
   @ApiOkResponse({

diff --git a/src/admin/admin.service.ts b/src/admin/admin.service.ts
@@ -22,8 +22,8 @@ export class AdminService {
     private readonly reportRepository: ReportRepository,
   ) {}
 
-  async getAllReports(): Promise<FullReportDto[]> {
-    const reports = await this.reportRepository.getAllReports();
+  async getAllReports(type: string): Promise<FullReportDto[]> {
+    const reports = await this.reportRepository.getAllReports(type);
     return reports.map(AdminService.docToFullReport);
   }
 

diff --git a/src/report/dto/create-report.dto.ts b/src/report/dto/create-report.dto.ts
@@ -11,6 +11,9 @@ export class CreateReportDto {
   @IsLatitude()
   latitude: number;
 
+  @IsNotEmpty()
+  type: string;
+
   @IsEmail()
   email: string;
 

diff --git a/src/report/report.controller.ts b/src/report/report.controller.ts
@@ -6,6 +6,7 @@ import {
   Param,
   ParseIntPipe,
   Post,
+  Query,
   UploadedFiles,
   UseInterceptors,
 } from '@nestjs/common';
@@ -46,17 +47,19 @@ export class ReportController {
     type: [PublicReportDto],
   })
   @Get()
-  getAllPublicReports(): Promise<PublicReportDto[]> {
-    return this.reportService.getAllVisibleReports();
+  getAllPublicReports(@Query('type') type: string): Promise<PublicReportDto[]> {
+    return this.reportService.getAllVisibleReports(type);
   }
 
   @ApiOkResponse({
     description: 'Report statistics have been successfully fetched',
     type: ReportStatisticsDto,
   })
   @Get('/statistics')
-  getReportStatistics(): Promise<ReportStatisticsDto> {
-    return this.reportService.getReportStatistics();
+  getReportStatistics(
+    @Query('type') type: string,
+  ): Promise<ReportStatisticsDto> {
+    return this.reportService.getReportStatistics(type);
   }
 
   @Get('/:refId')

diff --git a/src/report/report.service.ts b/src/report/report.service.ts
@@ -8,8 +8,8 @@ import { ReportStatisticsDto } from './dto/report-statistics.dto';
 export class ReportService {
   constructor(private readonly reportRepository: ReportRepository) {}
 
-  async getAllVisibleReports(): Promise<PublicReportDto[]> {
-    const reports = await this.reportRepository.getVisibleReports();
+  async getAllVisibleReports(type: string): Promise<PublicReportDto[]> {
+    const reports = await this.reportRepository.getVisibleReports(type);
     return reports.map(ReportService.docToPublicReport);
   }
 
@@ -30,8 +30,8 @@ export class ReportService {
     return ReportService.docToPublicReport(report);
   }
 
-  async getReportStatistics(): Promise<ReportStatisticsDto> {
-    const stats = await this.reportRepository.getVisibleStatusCounts();
+  async getReportStatistics(type: string): Promise<ReportStatisticsDto> {
+    const stats = await this.reportRepository.getVisibleStatusCounts(type);
     return ReportService.docToReportStatistics(stats);
   }
 

diff --git a/src/repositories/reports/report.repository.ts b/src/repositories/reports/report.repository.ts
@@ -14,16 +14,16 @@ export class ReportRepository {
     private cloudinary: CloudinaryService,
   ) {}
 
-  async getVisibleReports(): Promise<Report[]> {
+  async getVisibleReports(type: string): Promise<Report[]> {
     return await this.reportModel
-      .find({ isVisible: true, isDeleted: false })
+      .find({ isVisible: true, isDeleted: false, type: type })
       .sort({ reportDate: -1 })
       .exec();
   }
 
-  async getAllReports(): Promise<Report[]> {
+  async getAllReports(type: string): Promise<Report[]> {
     return await this.reportModel
-      .find({ isDeleted: false })
+      .find({ isDeleted: false, type: type })
       .sort({ reportDate: -1 })
       .exec();
   }
@@ -39,13 +39,14 @@ export class ReportRepository {
     return await this.reportModel.findOne({ refId: refId }).exec();
   }
 
-  async getVisibleStatusCounts(): Promise<any[]> {
+  async getVisibleStatusCounts(type: string): Promise<any[]> {
     return await this.reportModel
       .aggregate([
         {
           $match: {
             isDeleted: false,
             isVisible: true,
+            type: type,
           },
         },
         {
@@ -72,7 +73,7 @@ export class ReportRepository {
     const reportCount = reports.length;
     const newReport = new this.reportModel({
       name: createReport.name,
-      type: 'trash',
+      type: createReport.type,
       refId: reportCount + 1,
       comment: ' ',
       reportLong: createReport.longitude,