Enable modification observer when onChange callback throws an error

Author: t.hata

src/components/modules/modificationsObserver.ts

@@ -28,7 +28,7 @@ export default class ModificationsObserver extends Module {
   /**
    * Allows to temporary disable mutations handling
    */
-  private disabled: boolean;
+  private disabled = false;
 
   /**
    * Used to prevent several mutation callback execution
@@ -37,7 +37,10 @@ export default class ModificationsObserver extends Module {
    */
   private mutationDebouncer = _.debounce(() => {
     this.updateNativeInputs();
-    this.config.onChange(this.Editor.API.methods);
+
+    if (this.config.onChange) {
+      this.config.onChange(this.Editor.API.methods);
+    }
   }, ModificationsObserver.DebounceTimer);
 
   /**
@@ -116,7 +119,7 @@ export default class ModificationsObserver extends Module {
    * @param {MutationRecord[]} mutationList - list of mutations
    * @param {MutationObserver} observer - observer instance
    */
-  private mutationHandler(mutationList: MutationRecord[], observer): void {
+  private mutationHandler(mutationList: MutationRecord[], observer: MutationObserver): void {
     /**
      * Skip mutations in stealth mode
      */

src/components/modules/saver.ts

@@ -36,16 +36,18 @@ export default class Saver extends Module {
      */
     ModificationsObserver.disable();
 
-    blocks.forEach((block: Block) => {
-      chainData.push(this.getSavedData(block));
-    });
-
-    const extractedData = await Promise.all(chainData);
-    const sanitizedData = await Sanitizer.sanitizeBlocks(extractedData);
+    try {
+      blocks.forEach((block: Block) => {
+        chainData.push(this.getSavedData(block));
+      });
 
-    ModificationsObserver.enable();
+      const extractedData = await Promise.all(chainData);
+      const sanitizedData = await Sanitizer.sanitizeBlocks(extractedData);
 
-    return this.makeOutput(sanitizedData);
+      return this.makeOutput(sanitizedData);
+    } finally {
+      ModificationsObserver.enable();
+    }
   }
 
   /**