From 1dca0eaaeda6f364a4db47db50a5edb8394c8b40 Mon Sep 17 00:00:00 2001
From: Ruofan Xu <u7660525@anu.edu.au>
Date: Fri, 25 Oct 2024 14:20:36 +1100
Subject: [PATCH] fixed syntax issue that caused failing tests

---
 .../copy-session-modal/copy-session-modal.component.html        | 2 +-
 .../copy-session-modal/copy-session-modal.component.ts          | 2 +-
 2 files changed, 2 insertions(+), 2 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 3e0fabdf899..835428290d1 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
@@ -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">Copy</button>
+          [disabled]="!newFeedbackSessionName || copyToCourseSet.size < 1 || !(validateName(newSessionName.value))">Copy</button>
 </div>
\ No newline at end of file
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 53efc1e3b65..e0bb8670eb8 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
@@ -39,7 +39,7 @@ constructor(public activeModal: NgbActiveModal) {}
   }
 
   validateName(newFeedbackSessionName: string): boolean {
-    return (!(newFeedbackSessionName.trim().length === 0 && newFeedbackSessionName !== ''));
+    return (!(newFeedbackSessionName.trim().length === 0 && newFeedbackSessionName !== null));
   }
 
   /**