fixed syntax issue that caused failing tests

TEAMMATES · Oct 25, 2024 · 1dca0ea · 1dca0ea
1 parent cad7302
commit 1dca0ea
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 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
@@ -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>
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
@@ -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));
   }
 
   /**
-Original file line number
+Diff line change
@@ Expand Up / @@ -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));
       }
       /**
@@ Expand Down @@