From 23cd6bdd53c4501704313c06eb18019bcc13cf76 Mon Sep 17 00:00:00 2001
From: Felipe Barreta <fbarreta@gmail.com>
Date: Fri, 18 Oct 2024 17:31:35 -0700
Subject: [PATCH] ALCS-2229 Added validation

---
 .../add-structure-dialog.component.html       |  2 +-
 .../add-structure-dialog.component.scss       |  1 -
 .../add-structure-dialog.component.ts         |  4 +++
 .../structure-mobile-card.component.html      | 17 +++++++--
 .../structure-mobile-card.component.scss      | 36 +++++++++++++++++--
 .../structure-mobile-card.component.ts        | 14 ++++++--
 .../src/app/shared/shared.module.ts           |  2 +-
 7 files changed, 66 insertions(+), 10 deletions(-)
 rename portal-frontend/src/app/shared/{ => mobile}/structure-mobile-card/structure-mobile-card.component.html (53%)
 rename portal-frontend/src/app/shared/{ => mobile}/structure-mobile-card/structure-mobile-card.component.scss (58%)
 rename portal-frontend/src/app/shared/{ => mobile}/structure-mobile-card/structure-mobile-card.component.ts (67%)

diff --git a/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.html b/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.html
index c8837d815..45f172772 100644
--- a/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.html
+++ b/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.html
@@ -63,7 +63,7 @@ <h2 *ngIf="data.isEdit">
   </div>
   <mat-dialog-actions align="end">
   <div class="button-container">
-    <button *ngIf="this.data.structureData" mat-flat-button color="primary" (click)="onSubmit()" [disabled]="isLoading || !form.valid">
+    <button *ngIf="this.data.structureData" mat-flat-button color="primary" (click)="onSubmit()" [disabled]="isLoading">
       {{ !data.isEdit ? "ADD" : "SAVE" }}
     </button>
     <button mat-stroked-button color="primary" [mat-dialog-close]="false" (click)="onCancel()">CANCEL</button>
diff --git a/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.scss b/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.scss
index fdccb646b..f3a932574 100644
--- a/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.scss
+++ b/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.scss
@@ -7,7 +7,6 @@ mat-dialog-title {
 
 h2 {
   font-size: rem(19.5);
-  margin-left: rem(10) !important;
 }
 
 .actions {
diff --git a/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.ts b/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.ts
index 454189f3f..42df4e2fe 100644
--- a/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.ts
+++ b/portal-frontend/src/app/features/notice-of-intents/edit-submission/additional-information/add-structure-dialog/add-structure-dialog.component.ts
@@ -52,6 +52,10 @@ export class AddStructureDialogComponent {
   }
 
   async onSubmit() {
+    if (!this.form.valid) {
+      this.dialogRef.close(null);
+      return;
+    }
     this.isLoading = true;
     const dto: ProposedStructure = {
       area: Number(this.area.value!),
diff --git a/portal-frontend/src/app/shared/structure-mobile-card/structure-mobile-card.component.html b/portal-frontend/src/app/shared/mobile/structure-mobile-card/structure-mobile-card.component.html
similarity index 53%
rename from portal-frontend/src/app/shared/structure-mobile-card/structure-mobile-card.component.html
rename to portal-frontend/src/app/shared/mobile/structure-mobile-card/structure-mobile-card.component.html
index 75724ce8b..59a2760a2 100644
--- a/portal-frontend/src/app/shared/structure-mobile-card/structure-mobile-card.component.html
+++ b/portal-frontend/src/app/shared/mobile/structure-mobile-card/structure-mobile-card.component.html
@@ -1,12 +1,23 @@
-<mat-card [ngClass]="{ 'last-card': isLast, 'review-step':  isReviewStep }">
+<mat-card [ngClass]="{ 'last-card': isLast, 'review-step':  isReviewStep, 'error-card': isError && !isReviewStep }">
   <mat-card-header>
     <mat-card-title>
       <div class="line-info-floor">
-        <span class="strong">#{{ index+1 }} Total Floor Area: </span><span [class.strong]="isReviewStep">{{ structure.area }} <span matTextSuffix>(m<sup>2</sup>)</span></span> <br />
-      </div>
+        <span class="strong">#{{ index+1 }} Total Floor Area: </span>
+        <ng-container *ngIf="structure.area === 0 || !structure.area; else validFloorArea">
+          <div class="text-error">No Data</div>
+        </ng-container>
+        <ng-template #validFloorArea>
+          <span [class.strong]="isReviewStep">{{ structure.area }} <span matTextSuffix>(m<sup>2</sup>)</span></span>
+        </ng-template>
       <div class="line-info-type">
         <span class="strong">Type: </span>{{ mapStructureTypeValueToLabel(structure.type) }}
       </div>
+      <div class="field-error" *ngIf="(!structure.area || structure.area === 0 || !structure.type) && !isReviewStep">
+        <mat-icon>warning</mat-icon>
+        <div>This field is required</div>
+      </div>
+      <app-no-data [showRequired]="showErrors" *ngIf="(!structure.area || structure.area === 0 || !structure.type) && isReviewStep"></app-no-data>
+    </div>
     </mat-card-title>
     <ng-container *ngIf="!isReviewStep">
       <button mat-icon-button [mat-menu-trigger-for]="menu">
diff --git a/portal-frontend/src/app/shared/structure-mobile-card/structure-mobile-card.component.scss b/portal-frontend/src/app/shared/mobile/structure-mobile-card/structure-mobile-card.component.scss
similarity index 58%
rename from portal-frontend/src/app/shared/structure-mobile-card/structure-mobile-card.component.scss
rename to portal-frontend/src/app/shared/mobile/structure-mobile-card/structure-mobile-card.component.scss
index 810557d05..5f86a83f0 100644
--- a/portal-frontend/src/app/shared/structure-mobile-card/structure-mobile-card.component.scss
+++ b/portal-frontend/src/app/shared/mobile/structure-mobile-card/structure-mobile-card.component.scss
@@ -1,5 +1,5 @@
-@use '../../../styles/colors';
-@use '../../../styles/functions' as *;
+@use '../../../../styles/colors.scss';
+@use '../../../../styles/functions' as *;
 
 mat-card {
   width: 100%;
@@ -9,6 +9,7 @@ mat-card {
   border-radius: 0;
   word-wrap: break-word;
   white-space: normal;
+  padding-left: rem(10);
 }
 
 mat-card-header {
@@ -60,3 +61,34 @@ a {
 .line-info-type {
   padding-bottom: rem(10);
 }
+
+.text-error {
+  color: colors.$grey-dark;
+  font-weight: 400;
+  display: inline;
+  padding-bottom: 0 !important;
+}
+
+.field-error {
+  color: colors.$error-color;
+  font-size: rem(15);
+  font-weight: 700;
+  display: flex;
+  align-items: center;
+
+  .mat-icon {
+    min-width: rem(24);
+  }
+}
+
+.error-card {
+  border-radius: rem(4) !important;
+  border: rem(2) solid colors.$error-color !important;
+  padding: 0 0 rem(5) rem(10);
+  margin-top: rem(5);
+}
+
+.no-data {
+  margin-bottom: rem(-5) !important;
+  margin-top: rem(-10) !important;
+}
diff --git a/portal-frontend/src/app/shared/structure-mobile-card/structure-mobile-card.component.ts b/portal-frontend/src/app/shared/mobile/structure-mobile-card/structure-mobile-card.component.ts
similarity index 67%
rename from portal-frontend/src/app/shared/structure-mobile-card/structure-mobile-card.component.ts
rename to portal-frontend/src/app/shared/mobile/structure-mobile-card/structure-mobile-card.component.ts
index afb8a4d4f..4e67ba186 100644
--- a/portal-frontend/src/app/shared/structure-mobile-card/structure-mobile-card.component.ts
+++ b/portal-frontend/src/app/shared/mobile/structure-mobile-card/structure-mobile-card.component.ts
@@ -3,8 +3,8 @@ import {
   FormProposedStructure,
   NOI_STRUCTURE_TYPE_LABEL_MAP,
   STRUCTURE_TYPES,
-} from '../../features/notice-of-intents/edit-submission/additional-information/additional-information.component';
-import { ProposedStructure } from '../../services/notice-of-intent-submission/notice-of-intent-submission.dto';
+} from '../../../features/notice-of-intents/edit-submission/additional-information/additional-information.component';
+import { ProposedStructure } from '../../../services/notice-of-intent-submission/notice-of-intent-submission.dto';
 
 @Component({
   selector: 'app-structure-mobile-card',
@@ -16,9 +16,19 @@ export class StructureMobileCardComponent {
   @Input() isLast: boolean = false;
   @Input() index: number = 0;
   @Input() isReviewStep: boolean = false;
+  @Input() showErrors: boolean = false;
   @Output() editClicked = new EventEmitter<FormProposedStructure | ProposedStructure>();
   @Output() removeClicked = new EventEmitter<FormProposedStructure | ProposedStructure>();
 
+  isError: boolean = false;
+
+  ngOnInit(): void {
+    if (this.structure.area === null || this.structure.area === 0 || this.structure.type === null) {
+      this.isError = true;
+      this.showErrors = true;
+    }
+  }
+
   onEdit() {
     this.editClicked.emit(this.structure);
   }
diff --git a/portal-frontend/src/app/shared/shared.module.ts b/portal-frontend/src/app/shared/shared.module.ts
index fcb05619c..98b94a0bb 100644
--- a/portal-frontend/src/app/shared/shared.module.ts
+++ b/portal-frontend/src/app/shared/shared.module.ts
@@ -54,7 +54,7 @@ import { OtherAttachmentMobileCardComponent } from './other-attachment-mobile-ca
 import { ParcelOwnerMobileCardComponent } from './mobile/parcel-owner-mobile-card/parcel-owner-mobile-card.component';
 import { TransfereeMobileCardComponent } from './mobile/transferee-mobile-card/transferee-mobile-card.component';
 import { NaruResidenceMobileCardComponent } from './mobile/naru-residence-mobile-card/naru-residence-mobile-card.component';
-import { StructureMobileCardComponent } from './structure-mobile-card/structure-mobile-card.component';
+import { StructureMobileCardComponent } from './mobile/structure-mobile-card/structure-mobile-card.component';
 
 @NgModule({
   providers: [