From 2ce48006e4c924f8cca00a1e2b07454bc9f5a49b Mon Sep 17 00:00:00 2001
From: Ruofan Xu <u7660525@anu.edu.au>
Date: Fri, 25 Oct 2024 13:36:11 +1100
Subject: [PATCH] test

---
 .../copy-session-modal.component.html         |  4 ++--
 .../copy-session-modal.component.ts           | 20 ++++++++-----------
 2 files changed, 10 insertions(+), 14 deletions(-)

diff --git a/src/web/app/components/copy-session-modal/copy-session-modal.component.html b/src/web/app/components/copy-session-modal/copy-session-modal.component.html
index f66ccf42813..6863e2ca3a7 100644
--- a/src/web/app/components/copy-session-modal/copy-session-modal.component.html
+++ b/src/web/app/components/copy-session-modal/copy-session-modal.component.html
@@ -12,7 +12,7 @@ <h5 class="modal-title">
         <label><b>Name for copied session*</b></label>
         <input id="copy-session-name" type="text" class="form-control" [(ngModel)]="newFeedbackSessionName" [maxlength]="FEEDBACK_SESSION_NAME_MAX_LENGTH"
                required #newSessionName="ngModel">
-        <div [hidden]="validateName(newSessionName.value)" class="invalid-field">
+        <div [hidden]="(validName)" class="invalid-field">
             <i class="fa fa-exclamation-circle" aria-hidden="true"></i>
             The field "Name for copied session" should not be empty.
         </div>
@@ -35,5 +35,5 @@ <h5 class="modal-title">
 <div class="modal-footer">
   <button type="button" class="btn btn-light" (click)="activeModal.dismiss()">Cancel</button>
   <button id="btn-confirm-copy-course" type="button" class="btn btn-primary" (click)="copy()"
-          [disabled]="!newFeedbackSessionName || copyToCourseSet.size < 1 || !validateName(newSessionName.value)">Copy</button>
+          [disabled]="!newFeedbackSessionName || copyToCourseSet.size < 1 || !(validName)">Copy</button>
 </div>
diff --git a/src/web/app/components/copy-session-modal/copy-session-modal.component.ts b/src/web/app/components/copy-session-modal/copy-session-modal.component.ts
index 2a23d20ade7..af001d8f9a8 100644
--- a/src/web/app/components/copy-session-modal/copy-session-modal.component.ts
+++ b/src/web/app/components/copy-session-modal/copy-session-modal.component.ts
@@ -30,25 +30,21 @@ export class CopySessionModalComponent {
   /**
   * boolean to store validity of name
   * /
+  validName: boolean = false;
 
 
-  validateName(sessionName: string): boolean {
-    if (this.sessionName.trim().length==0) {
-      return false;
-    }
-    else {
-      return true;
-    }
-  }
-
   /**
    * Fires the copy event.
    */
   copy(): void {
   /**
-   * check if session name is whitespace or not
-   */
-    if (validateName(this.newFeedbackSessionName)) {
+  * check if session name is whitespace or not
+  * /
+    if (this.newFeedbackSessionName.trim().length==0) {
+      this.validName = false;
+      return;
+    } else {
+        this.validName = true;
         this.activeModal.close({
           newFeedbackSessionName: this.newFeedbackSessionName,
           sessionToCopyCourseId: this.sessionToCopyCourseId,