Merge pull request #123 from DDMAL/remove-clear-mei

fix: remove `meiData` entry after clearing cell
DDMAL · Jul 15, 2024 · b83f170 · b83f170
2 parents ca06fd8 + 3cb56a9
commit b83f170
Showing 1 changed file with 12 additions and 11 deletions.
diff --git a/src/Editor/MeiTools.ts b/src/Editor/MeiTools.ts
@@ -43,21 +43,22 @@ export class MeiTools {
   ) {
     const meiData = this.meiData.find((meiData) => meiData.row === row);
     if (meiData) {
-      if (mei !== undefined) {
-        meiData.mei = mei;
-      }
-      if (isValid !== undefined) {
-        meiData.isValid = isValid;
-      }
-      if (errorMsg !== undefined) {
-        meiData.errorMsg = errorMsg;
+      // Update this.meiData if it exists
+      if (mei === '') {
+        // if the mei cell is empty, remove the row from meiData
+        this.meiData = this.meiData.filter((data) => data.row !== row);
+        return;
       }
+      if (mei !== undefined) meiData.mei = mei;
+      if (isValid !== undefined) meiData.isValid = isValid;
+      if (errorMsg !== undefined) meiData.errorMsg = errorMsg;
     } else {
+      // Add a new entry to this.meiData if it doesn't exist
       this.meiData.push({
         row,
-        mei: mei ?? meiData.mei,
-        isValid: isValid ?? meiData.isValid,
-        errorMsg: errorMsg ?? meiData.errorMsg,
+        mei: mei ?? '',
+        isValid: isValid ?? null,
+        errorMsg: errorMsg ?? null,
       });
     }
   }