Fix ALC advanced search time range bugs for inquiry and planning review

bcgov · Aug 12, 2024 · b0a9b8d · b0a9b8d
1 parent 78ee27b
commit b0a9b8d
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 7 deletions.
diff --git a/services/apps/alcs/src/alcs/search/inquiry/inquiry-advanced-search.service.ts b/services/apps/alcs/src/alcs/search/inquiry/inquiry-advanced-search.service.ts
@@ -11,6 +11,7 @@ import { LocalGovernment } from '../../local-government/local-government.entity'
 import { SEARCH_CACHE_TIME } from '../search.config';
 import { AdvancedSearchResultDto, SearchRequestDto } from '../search.dto';
 import { InquirySearchView } from './inquiry-search-view.entity';
+import { getNextDayToPacific, getStartOfDayToPacific } from '../../../utils/pacific-date-time-helper';
 
 @Injectable()
 export class InquiryAdvancedSearchService {
@@ -284,16 +285,20 @@ export class InquiryAdvancedSearchService {
       query = query.andWhere(
         'inquiry.date_submitted_to_alc >= :date_submitted_from',
         {
-          date_submitted_from: new Date(searchDto.dateSubmittedFrom),
+          date_submitted_from: getStartOfDayToPacific(
+            searchDto.dateSubmittedFrom
+          ).toISOString(),
         },
       );
     }
 
     if (searchDto.dateSubmittedTo !== undefined) {
       query = query.andWhere(
-        'inquiry.date_submitted_to_alc <= :date_submitted_to',
+        'inquiry.date_submitted_to_alc < :date_submitted_to',
         {
-          date_submitted_to: new Date(searchDto.dateSubmittedTo),
+          date_submitted_to: getNextDayToPacific(
+            searchDto.dateSubmittedTo
+          ).toISOString(),
         },
       );
     }

diff --git a/...ices/apps/alcs/src/alcs/search/planning-review/planning-review-advanced-search.service.ts b/...ices/apps/alcs/src/alcs/search/planning-review/planning-review-advanced-search.service.ts
@@ -284,13 +284,17 @@ export class PlanningReviewAdvancedSearchService {
 
     if (searchDto.dateSubmittedFrom !== undefined) {
       query.andWhere('referral.submission_date >= :date_submitted_from', {
-        date_submitted_from: new Date(searchDto.dateSubmittedFrom),
+        date_submitted_from: getStartOfDayToPacific(
+          searchDto.dateSubmittedFrom
+        ).toISOString(),
       });
     }
 
     if (searchDto.dateSubmittedTo !== undefined) {
-      query.andWhere('referral.submission_date <= :date_submitted_to', {
-        date_submitted_to: new Date(searchDto.dateSubmittedTo),
+      query.andWhere('referral.submission_date < :date_submitted_to', {
+        date_submitted_to: getNextDayToPacific(
+          searchDto.dateSubmittedTo
+        ).toISOString(),
       });
     }
     promises.push(query.getMany());
@@ -318,7 +322,7 @@ export class PlanningReviewAdvancedSearchService {
     }
 
     if (searchDto.dateDecidedTo) {
-      query.andWhere('decision.date <= :decision_date_to', {
+      query.andWhere('decision.date < :decision_date_to', {
         decision_date_to: getNextDayToPacific(
           searchDto.dateDecidedTo,
         ).toISOString(),