From 9e68dc499fdc5f86f2363ceda4cc230267409c9c Mon Sep 17 00:00:00 2001
From: NeoHW <neohaowei@gmail.com>
Date: Sat, 13 Jul 2024 18:29:37 +0800
Subject: [PATCH 1/5] Add copy student data btn , but not functional

---
 .../instructor-course-enroll-page.component.html  | 15 ++++++++++++++-
 .../instructor-course-enroll-page.component.scss  |  3 ++-
 2 files changed, 16 insertions(+), 2 deletions(-)

diff --git a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.html b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.html
index c73db728e8f..a0c7ec1dbc3 100644
--- a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.html
+++ b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.html
@@ -101,7 +101,20 @@ <h1 id="enroll-header" class="text-break">Enroll Students for {{ courseId }}</h1
                     title="Enroll"
                     id="btn-enroll"
                     name="button-enroll"
-                    class="btn btn-success float-end"
+                    class="btn btn-info"
+                    [disabled]="isEnrolling"
+                    (click)="submitEnrollData()">
+                  <tm-progress-bar *ngIf="isEnrolling"></tm-progress-bar>
+                  <tm-ajax-loading *ngIf="isEnrolling"></tm-ajax-loading>
+                  Copy students
+                </button>
+                <button
+                    [ngClass]="{'w-50': isEnrolling}"
+                    type="submit"
+                    title="Enroll"
+                    id="btn-enroll"
+                    name="button-enroll"
+                    class="btn btn-success"
                     [disabled]="isEnrolling"
                     (click)="submitEnrollData()">
                   <tm-progress-bar *ngIf="isEnrolling"></tm-progress-bar>
diff --git a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.scss b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.scss
index 1b1895076b0..69e5c07e2b4 100644
--- a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.scss
+++ b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.scss
@@ -69,8 +69,9 @@
 }
 
 .enroll-button-group {
+  display: flex;
+  justify-content: space-between;
   width: 50%;
-  display: inline-block;
 }
 
 .enroll-button-group * {

From 11f269bf3dc85b3e07a707ca6d76a7f50c015a82 Mon Sep 17 00:00:00 2001
From: NeoHW <neohaowei@gmail.com>
Date: Sun, 14 Jul 2024 02:44:50 +0800
Subject: [PATCH 2/5] Copy Student button uses enroll student logic

---
 ...structor-course-enroll-page.component.html |  18 +--
 ...instructor-course-enroll-page.component.ts | 129 ++++++++++++++++++
 2 files changed, 138 insertions(+), 9 deletions(-)

diff --git a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.html b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.html
index a0c7ec1dbc3..0043fb79351 100644
--- a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.html
+++ b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.html
@@ -96,16 +96,16 @@ <h1 id="enroll-header" class="text-break">Enroll Students for {{ courseId }}</h1
               </div>
               <div class="enroll-button-group">
                 <button
-                    [ngClass]="{'w-50': isEnrolling}"
-                    type="submit"
-                    title="Enroll"
-                    id="btn-enroll"
-                    name="button-enroll"
+                    [ngClass]="{'w-50': isCopying}"
+                    type="button"
+                    title="Copy"
+                    id="btn-copy"
+                    name="button-copy"
                     class="btn btn-info"
-                    [disabled]="isEnrolling"
-                    (click)="submitEnrollData()">
-                  <tm-progress-bar *ngIf="isEnrolling"></tm-progress-bar>
-                  <tm-ajax-loading *ngIf="isEnrolling"></tm-ajax-loading>
+                    [disabled]="isCopying"
+                    (click)="copyStudents()">
+                  <tm-progress-bar *ngIf="isCopying"></tm-progress-bar>
+                  <tm-ajax-loading *ngIf="isCopying"></tm-ajax-loading>
                   Copy students
                 </button>
                 <button
diff --git a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
index a7344b7a82a..8dfbdef1efb 100644
--- a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
+++ b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
@@ -51,6 +51,7 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
   isLoadingCourseEnrollPage: boolean = false;
   showEnrollResults?: boolean = false;
   enrollErrorMessage: string = '';
+  copyErrorMessage: string = '';
   statusMessage: StatusMessage[] = [];
   unsuccessfulEnrolls: { [email: string]: string } = {};
 
@@ -89,6 +90,7 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
   isLoadingExistingStudents: boolean = false;
   isAjaxSuccess: boolean = true;
   isEnrolling: boolean = false;
+  isCopying: boolean = false;
 
   allStudentChunks: StudentEnrollRequest[][] = [];
   invalidRowsIndex: Set<number> = new Set();
@@ -244,6 +246,133 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
     });
   }
 
+  /**
+   * Copies data from existing students HOT to new students HOT
+   */
+  copyStudents(): void {
+    this.isCopying = true;
+    this.copyErrorMessage = '';
+    this.allStudentChunks = [];
+
+    const lastColIndex: number = 4;
+    const newStudentsHOTInstance: Handsontable =
+        this.hotRegisterer.getInstance(this.newStudentsHOT);
+    const hotInstanceColHeaders: string[] = (newStudentsHOTInstance.getColHeader() as string[]);
+
+    // Reset error highlighting on a new submission
+    this.resetTableStyle(newStudentsHOTInstance, 0,
+        newStudentsHOTInstance.getData().length - 1,
+        0,
+        hotInstanceColHeaders.indexOf(this.colHeaders[lastColIndex]));
+
+    // Remove error highlight on click
+    newStudentsHOTInstance.addHook('afterSelectionEnd', (row: number, column: number,
+                                                          row2: number, column2: number) => {
+      this.resetTableStyle(newStudentsHOTInstance, row, row2, column, column2);
+    });
+
+    // Record the row with its index on the table
+    const studentEnrollRequests: Map<number, StudentEnrollRequest> = new Map();
+
+    // Parse the user input to be requests.
+    // Handsontable contains null value initially,
+    // see https://github.com/handsontable/handsontable/issues/3927
+    newStudentsHOTInstance.getData()
+        .forEach((row: string[], index: number) => {
+          if (!row.every((cell: string) => cell === null || cell === '')) {
+            studentEnrollRequests.set(index, {
+              section: row[hotInstanceColHeaders.indexOf(this.colHeaders[0])] === null
+                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[0])].trim(),
+              team: row[hotInstanceColHeaders.indexOf(this.colHeaders[1])] === null
+                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[1])].trim(),
+              name: row[hotInstanceColHeaders.indexOf(this.colHeaders[2])] === null
+                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[2])].trim(),
+              email: row[hotInstanceColHeaders.indexOf(this.colHeaders[3])] === null
+                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[3])].trim(),
+              comments: row[hotInstanceColHeaders.indexOf(this.colHeaders[4])] === null
+                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[4])].trim(),
+            });
+          }
+        });
+
+    if (studentEnrollRequests.size === 0) {
+      this.enrollErrorMessage = 'Empty table';
+      this.isEnrolling = false;
+      return;
+    }
+
+    this.checkCompulsoryFields(studentEnrollRequests);
+    this.checkEmailNotRepeated(studentEnrollRequests);
+    this.checkTeamsValid(studentEnrollRequests);
+
+    if (this.invalidRowsIndex.size > 0) {
+      this.setTableStyleBasedOnFieldChecks(newStudentsHOTInstance, hotInstanceColHeaders);
+      this.isEnrolling = false;
+      return;
+    }
+
+    this.partitionStudentEnrollRequests(Array.from(studentEnrollRequests.values()));
+    const enrolledStudents: Student[] = [];
+
+    // Use concat because we cannot afford to parallelize with forkJoin when there's data dependency
+    const enrollRequests: Observable<EnrollStudents> = concat(
+        ...this.allStudentChunks.map((studentChunk: StudentEnrollRequest[]) => {
+          const request: StudentsEnrollRequest = {
+            studentEnrollRequests: studentChunk,
+          };
+          return this.studentService.enrollStudents(
+              this.courseId, request,
+          );
+        }),
+    );
+
+    this.progressBarService.updateProgress(0);
+    enrollRequests.pipe(finalize(() => {
+      this.isEnrolling = false;
+    })).subscribe({
+      next: (resp: EnrollStudents) => {
+        enrolledStudents.push(...resp.studentsData.students);
+
+        if (resp.unsuccessfulEnrolls != null) {
+          for (const unsuccessfulEnroll of resp.unsuccessfulEnrolls) {
+            this.unsuccessfulEnrolls[unsuccessfulEnroll.studentEmail] = unsuccessfulEnroll.errorMessage;
+
+            for (const index of studentEnrollRequests.keys()) {
+              if (studentEnrollRequests.get(index)?.email === unsuccessfulEnroll.studentEmail) {
+                this.invalidRowsIndex.add(index);
+                break;
+              }
+            }
+          }
+        }
+        const percentage: number = Math.round(100 * enrolledStudents.length / studentEnrollRequests.size);
+        this.progressBarService.updateProgress(percentage);
+      },
+      complete: () => {
+        this.showEnrollResults = true;
+        this.statusMessage.pop(); // removes any existing error status message
+        this.statusMessageService.showSuccessToast('Enrollment successful. Summary given below.');
+        this.prepareEnrollmentResults(enrolledStudents, studentEnrollRequests);
+
+        if (this.invalidRowsIndex.size > 0
+          || this.newStudentRowsIndex.size > 0
+          || this.modifiedStudentRowsIndex.size > 0
+          || this.unchangedStudentRowsIndex.size > 0) {
+          this.setTableStyleBasedOnFieldChecks(newStudentsHOTInstance, hotInstanceColHeaders);
+        }
+      },
+      error: (resp: ErrorMessageOutput) => {
+        if (enrolledStudents.length > 0) {
+          this.showEnrollResults = true;
+          this.prepareEnrollmentResults(enrolledStudents, studentEnrollRequests);
+        }
+
+        // Set error message after populating result panels to avoid it being overridden
+        this.enrollErrorMessage = resp.error.message;
+      },
+    });
+  }
+
   private prepareEnrollmentResults(enrolledStudents: Student[],
                                    studentEnrollRequests: Map<number, StudentEnrollRequest>): void {
     this.enrollResultPanelList = this.populateEnrollResultPanelList(this.existingStudents,

From 85e3c87155bb735d9f173c0321b16c798ace90f5 Mon Sep 17 00:00:00 2001
From: NeoHW <neohaowei@gmail.com>
Date: Sun, 14 Jul 2024 03:00:54 +0800
Subject: [PATCH 3/5] Copy Student button works as expected

---
 ...instructor-course-enroll-page.component.ts | 121 +++---------------
 1 file changed, 15 insertions(+), 106 deletions(-)

diff --git a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
index 8dfbdef1efb..8a3306224af 100644
--- a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
+++ b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
@@ -248,6 +248,7 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
 
   /**
    * Copies data from existing students HOT to new students HOT
+   * The data is copied only if the existing students tab is expanded
    */
   copyStudents(): void {
     this.isCopying = true;
@@ -255,122 +256,30 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
     this.allStudentChunks = [];
 
     const lastColIndex: number = 4;
+    const existingStudentsHOTInstance: Handsontable =
+        this.hotRegisterer.getInstance(this.existingStudentsHOT);
     const newStudentsHOTInstance: Handsontable =
         this.hotRegisterer.getInstance(this.newStudentsHOT);
+    const existingStudentsData: Handsontable.CellValue[] = existingStudentsHOTInstance.getData();
     const hotInstanceColHeaders: string[] = (newStudentsHOTInstance.getColHeader() as string[]);
 
-    // Reset error highlighting on a new submission
-    this.resetTableStyle(newStudentsHOTInstance, 0,
-        newStudentsHOTInstance.getData().length - 1,
-        0,
-        hotInstanceColHeaders.indexOf(this.colHeaders[lastColIndex]));
-
-    // Remove error highlight on click
-    newStudentsHOTInstance.addHook('afterSelectionEnd', (row: number, column: number,
-                                                          row2: number, column2: number) => {
-      this.resetTableStyle(newStudentsHOTInstance, row, row2, column, column2);
-    });
-
-    // Record the row with its index on the table
-    const studentEnrollRequests: Map<number, StudentEnrollRequest> = new Map();
-
-    // Parse the user input to be requests.
-    // Handsontable contains null value initially,
-    // see https://github.com/handsontable/handsontable/issues/3927
-    newStudentsHOTInstance.getData()
-        .forEach((row: string[], index: number) => {
-          if (!row.every((cell: string) => cell === null || cell === '')) {
-            studentEnrollRequests.set(index, {
-              section: row[hotInstanceColHeaders.indexOf(this.colHeaders[0])] === null
-                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[0])].trim(),
-              team: row[hotInstanceColHeaders.indexOf(this.colHeaders[1])] === null
-                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[1])].trim(),
-              name: row[hotInstanceColHeaders.indexOf(this.colHeaders[2])] === null
-                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[2])].trim(),
-              email: row[hotInstanceColHeaders.indexOf(this.colHeaders[3])] === null
-                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[3])].trim(),
-              comments: row[hotInstanceColHeaders.indexOf(this.colHeaders[4])] === null
-                  ? '' : row[hotInstanceColHeaders.indexOf(this.colHeaders[4])].trim(),
-            });
-          }
-        });
-
-    if (studentEnrollRequests.size === 0) {
-      this.enrollErrorMessage = 'Empty table';
-      this.isEnrolling = false;
-      return;
-    }
-
-    this.checkCompulsoryFields(studentEnrollRequests);
-    this.checkEmailNotRepeated(studentEnrollRequests);
-    this.checkTeamsValid(studentEnrollRequests);
 
-    if (this.invalidRowsIndex.size > 0) {
-      this.setTableStyleBasedOnFieldChecks(newStudentsHOTInstance, hotInstanceColHeaders);
-      this.isEnrolling = false;
+    if (existingStudentsData.length == 0) {
+      this.copyErrorMessage = 'No data to copy from existing students.';
+      this.isCopying = false;
       return;
     }
 
-    this.partitionStudentEnrollRequests(Array.from(studentEnrollRequests.values()));
-    const enrolledStudents: Student[] = [];
+    newStudentsHOTInstance.loadData(existingStudentsData);
 
-    // Use concat because we cannot afford to parallelize with forkJoin when there's data dependency
-    const enrollRequests: Observable<EnrollStudents> = concat(
-        ...this.allStudentChunks.map((studentChunk: StudentEnrollRequest[]) => {
-          const request: StudentsEnrollRequest = {
-            studentEnrollRequests: studentChunk,
-          };
-          return this.studentService.enrollStudents(
-              this.courseId, request,
-          );
-        }),
-    );
-
-    this.progressBarService.updateProgress(0);
-    enrollRequests.pipe(finalize(() => {
-      this.isEnrolling = false;
-    })).subscribe({
-      next: (resp: EnrollStudents) => {
-        enrolledStudents.push(...resp.studentsData.students);
-
-        if (resp.unsuccessfulEnrolls != null) {
-          for (const unsuccessfulEnroll of resp.unsuccessfulEnrolls) {
-            this.unsuccessfulEnrolls[unsuccessfulEnroll.studentEmail] = unsuccessfulEnroll.errorMessage;
-
-            for (const index of studentEnrollRequests.keys()) {
-              if (studentEnrollRequests.get(index)?.email === unsuccessfulEnroll.studentEmail) {
-                this.invalidRowsIndex.add(index);
-                break;
-              }
-            }
-          }
-        }
-        const percentage: number = Math.round(100 * enrolledStudents.length / studentEnrollRequests.size);
-        this.progressBarService.updateProgress(percentage);
-      },
-      complete: () => {
-        this.showEnrollResults = true;
-        this.statusMessage.pop(); // removes any existing error status message
-        this.statusMessageService.showSuccessToast('Enrollment successful. Summary given below.');
-        this.prepareEnrollmentResults(enrolledStudents, studentEnrollRequests);
-
-        if (this.invalidRowsIndex.size > 0
-          || this.newStudentRowsIndex.size > 0
-          || this.modifiedStudentRowsIndex.size > 0
-          || this.unchangedStudentRowsIndex.size > 0) {
-          this.setTableStyleBasedOnFieldChecks(newStudentsHOTInstance, hotInstanceColHeaders);
-        }
-      },
-      error: (resp: ErrorMessageOutput) => {
-        if (enrolledStudents.length > 0) {
-          this.showEnrollResults = true;
-          this.prepareEnrollmentResults(enrolledStudents, studentEnrollRequests);
-        }
+    // Reset styles on new students HOT
+    this.resetTableStyle(newStudentsHOTInstance, 0,
+        newStudentsHOTInstance.getData().length - 1,
+        0,
+        hotInstanceColHeaders.indexOf(this.colHeaders[lastColIndex]));
 
-        // Set error message after populating result panels to avoid it being overridden
-        this.enrollErrorMessage = resp.error.message;
-      },
-    });
+    this.isCopying = false;
+    this.statusMessageService.showSuccessToast('Students copied successfully.');
   }
 
   private prepareEnrollmentResults(enrolledStudents: Student[],

From fad85dae87587314f73eedef5a827939430727dc Mon Sep 17 00:00:00 2001
From: NeoHW <neohaowei@gmail.com>
Date: Sun, 14 Jul 2024 04:01:35 +0800
Subject: [PATCH 4/5] Allow copy student to be used when existing students HOT
 is not expanded

---
 ...instructor-course-enroll-page.component.ts | 44 ++++++++++++-------
 1 file changed, 27 insertions(+), 17 deletions(-)

diff --git a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
index 8a3306224af..df1d937e91e 100644
--- a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
+++ b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
@@ -260,26 +260,34 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
         this.hotRegisterer.getInstance(this.existingStudentsHOT);
     const newStudentsHOTInstance: Handsontable =
         this.hotRegisterer.getInstance(this.newStudentsHOT);
-    const existingStudentsData: Handsontable.CellValue[] = existingStudentsHOTInstance.getData();
     const hotInstanceColHeaders: string[] = (newStudentsHOTInstance.getColHeader() as string[]);
 
-
-    if (existingStudentsData.length == 0) {
-      this.copyErrorMessage = 'No data to copy from existing students.';
+    const copyData = () => {
+      const existingStudentsData: Handsontable.CellValue[] = existingStudentsHOTInstance.getData();
+  
+      if (existingStudentsData.length === 0) {
+        this.copyErrorMessage = 'No data to copy from existing students.';
+        this.isCopying = false;
+        return;
+      }
+  
+      newStudentsHOTInstance.loadData(existingStudentsData);
+  
+      this.resetTableStyle(newStudentsHOTInstance, 0,
+          newStudentsHOTInstance.getData().length - 1,
+          0,
+          hotInstanceColHeaders.indexOf(this.colHeaders[lastColIndex]));
+  
       this.isCopying = false;
-      return;
-    }
-
-    newStudentsHOTInstance.loadData(existingStudentsData);
-
-    // Reset styles on new students HOT
-    this.resetTableStyle(newStudentsHOTInstance, 0,
-        newStudentsHOTInstance.getData().length - 1,
-        0,
-        hotInstanceColHeaders.indexOf(this.colHeaders[lastColIndex]));
+      this.statusMessageService.showSuccessToast('Students copied successfully.');
+    };
 
-    this.isCopying = false;
-    this.statusMessageService.showSuccessToast('Students copied successfully.');
+    if (this.isExistingStudentsPanelCollapsed) {
+      this.toggleExistingStudentsPanel(copyData);
+    }
+    else {
+      copyData();
+    }
   }
 
   private prepareEnrollmentResults(enrolledStudents: Student[],
@@ -584,7 +592,7 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
   /**
    * Toggles the view of 'Existing Students' spreadsheet interface
    */
-  toggleExistingStudentsPanel(): void {
+  toggleExistingStudentsPanel(callback?: () => void): void {
     // Has to be done before the API call is made so that HOT is available for data population
     this.isExistingStudentsPanelCollapsed = !this.isExistingStudentsPanelCollapsed;
     this.isLoadingExistingStudents = true;
@@ -594,6 +602,7 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
     // Calling REST API only the first time when spreadsheet has no data
     if (this.getSpreadsheetLength(existingStudentsHOTInstance.getData()) !== 0) {
       this.isLoadingExistingStudents = false;
+      if (callback) callback();
       return;
     }
 
@@ -614,6 +623,7 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
       },
       complete: () => {
         this.isLoadingExistingStudents = false;
+        if (callback) callback();
       },
     });
   }

From ee43b56ec89e17a872913ad01a20c8662609a0f9 Mon Sep 17 00:00:00 2001
From: NeoHW <neohaowei@gmail.com>
Date: Tue, 16 Jul 2024 03:05:20 +0800
Subject: [PATCH 5/5] Fix linting issues

---
 package.json                                        |  2 +-
 .../instructor-course-enroll-page.component.ts      | 13 ++++++-------
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/package.json b/package.json
index cbb8c5b3689..37dc75bc558 100644
--- a/package.json
+++ b/package.json
@@ -10,7 +10,7 @@
     "lint:css:syntax": "stylelint \"src/web/**/*.css\" \"src/web/**/*.scss\" \"src/web/**/*.html\" --ignore-pattern \"src/web/dist/*.css\" --cache --config static-analysis/teammates-stylelint.yml",
     "lint:css:styles": "prettier --print-width=120 --single-quote=true \"src/web/**/*.css\" \"src/web/**/*.scss\"",
     "lint:windows:spaces": "lintspaces -n -t -d spaces -l 1 -. \"src/main/**/*.html\" \"src/web/**/*.html\" \"src/**/*.xml\" \"src/**/*.json\" \"src/**/*.properties\" \"*.yml\" \"*.json\" \"*.gradle\" \"static-analysis/*.*ml\" \".gitattributes\"",
-    "lint:nix:spaces": "lintspaces -n -t -d spaces -l 1 --matchdotfiles $(git diff --name-only --diff-filter=ACMRTUXB origin/master HEAD | grep -E \"src/(main|web)/.*.html|src/.*.(xml|json|properties)|*.(yml|json|gradle)|static-analysis/*.*ml|.gitattributes\") || :",
+    "lint:nix:spaces": "lintspaces -n -t -d spaces -l 1 --matchdotfiles $(git diff --name-only --diff-filter=ACMRTUXB origin/master HEAD | grep -E \"src/(main|web)/.*\\.html|src/.*\\.(xml|json|properties)|.*\\.(yml|json|gradle)|static-analysis/.*\\.(xml|yml)\") || :",
     "lint": "run-script-os",
     "lint:windows": "npm-run-all lint:ts lint:css:syntax lint:css:styles lint:windows:spaces -c",
     "lint:nix": "npm-run-all lint:ts lint:css:syntax lint:css:styles lint:nix:spaces -c"
diff --git a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
index df1d937e91e..444652aaf58 100644
--- a/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
+++ b/src/web/app/pages-instructor/instructor-course-enroll-page/instructor-course-enroll-page.component.ts
@@ -262,30 +262,29 @@ export class InstructorCourseEnrollPageComponent implements OnInit {
         this.hotRegisterer.getInstance(this.newStudentsHOT);
     const hotInstanceColHeaders: string[] = (newStudentsHOTInstance.getColHeader() as string[]);
 
-    const copyData = () => {
+    const copyData = (): void => {
       const existingStudentsData: Handsontable.CellValue[] = existingStudentsHOTInstance.getData();
-  
+
       if (existingStudentsData.length === 0) {
         this.copyErrorMessage = 'No data to copy from existing students.';
         this.isCopying = false;
         return;
       }
-  
+
       newStudentsHOTInstance.loadData(existingStudentsData);
-  
+
       this.resetTableStyle(newStudentsHOTInstance, 0,
           newStudentsHOTInstance.getData().length - 1,
           0,
           hotInstanceColHeaders.indexOf(this.colHeaders[lastColIndex]));
-  
+
       this.isCopying = false;
       this.statusMessageService.showSuccessToast('Students copied successfully.');
     };
 
     if (this.isExistingStudentsPanelCollapsed) {
       this.toggleExistingStudentsPanel(copyData);
-    }
-    else {
+    } else {
       copyData();
     }
   }