Remove tablePreProcessor

microsoft · Oct 26, 2024 · b8d3dcf · b8d3dcf
1 parent 09531d8
commit b8d3dcf
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 285 deletions.
diff --git a/packages/roosterjs-content-model-core/lib/editor/Editor.ts b/packages/roosterjs-content-model-core/lib/editor/Editor.ts
@@ -1,7 +1,6 @@
 import { createEditorCore } from './core/createEditorCore';
 import {
     createEmptyModel,
-    tableProcessor,
     ChangeSource,
     cloneModel,
     transformColor,
@@ -102,9 +101,6 @@ export class Editor implements IEditor {
             case 'disconnected':
                 return cloneModel(
                     core.api.createContentModel(core, {
-                        processorOverride: {
-                            table: tableProcessor,
-                        },
                         tryGetFromCache: false,
                     }),
                     {

diff --git a/packages/roosterjs-content-model-core/lib/editor/core/createEditorDefaultSettings.ts b/packages/roosterjs-content-model-core/lib/editor/core/createEditorDefaultSettings.ts
@@ -1,5 +1,4 @@
 import { createDomToModelConfig, createModelToDomConfig } from 'roosterjs-content-model-dom';
-import { tablePreProcessor } from '../../override/tablePreProcessor';
 import {
     listItemMetadataApplier,
     listLevelMetadataApplier,
@@ -21,11 +20,7 @@ import type {
 export function createDomToModelSettings(
     options: EditorOptions
 ): ContentModelSettings<DomToModelOption, DomToModelSettings> {
-    const builtIn: DomToModelOption = {
-        processorOverride: {
-            table: tablePreProcessor,
-        },
-    };
+    const builtIn: DomToModelOption = {};
     const customized: DomToModelOption = options.defaultDomToModelOptions ?? {};
 
     return {

diff --git a/packages/roosterjs-content-model-core/lib/override/tablePreProcessor.ts b/packages/roosterjs-content-model-core/lib/override/tablePreProcessor.ts
diff --git a/packages/roosterjs-content-model-core/test/editor/EditorTest.ts b/packages/roosterjs-content-model-core/test/editor/EditorTest.ts
@@ -4,9 +4,9 @@ import * as createEditorCore from '../../lib/editor/core/createEditorCore';
 import * as createEmptyModel from 'roosterjs-content-model-dom/lib/modelApi/creators/createEmptyModel';
 import * as domToContentModel from 'roosterjs-content-model-dom/lib/domToModel/domToContentModel';
 import * as transformColor from 'roosterjs-content-model-dom/lib/domUtils/style/transformColor';
-import { ChangeSource, tableProcessor } from 'roosterjs-content-model-dom';
 import { Editor } from '../../lib/editor/Editor';
 import { expectHtml } from 'roosterjs-content-model-dom/test/testUtils';
+import { ChangeSource } from 'roosterjs-content-model-dom';
 import {
     CachedElementHandler,
     ContentModelDocument,
@@ -198,9 +198,6 @@ describe('Editor', () => {
 
         expect(model).toBe(mockedClonedModel);
         expect(createContentModelSpy).toHaveBeenCalledWith(mockedCore, {
-            processorOverride: {
-                table: tableProcessor,
-            },
             tryGetFromCache: false,
         });
         expect(transformColorSpy).not.toHaveBeenCalled();
@@ -212,9 +209,6 @@ describe('Editor', () => {
 
         expect(model).toBe(mockedClonedModel);
         expect(createContentModelSpy).toHaveBeenCalledWith(mockedCore, {
-            processorOverride: {
-                table: tableProcessor,
-            },
             tryGetFromCache: false,
         });
         expect(transformColorSpy).toHaveBeenCalledWith(

diff --git a/packages/roosterjs-content-model-core/test/editor/core/createEditorDefaultSettingsTest.ts b/packages/roosterjs-content-model-core/test/editor/core/createEditorDefaultSettingsTest.ts
@@ -1,6 +1,5 @@
 import * as createDomToModelContext from 'roosterjs-content-model-dom/lib/domToModel/context/createDomToModelContext';
 import * as createModelToDomContext from 'roosterjs-content-model-dom/lib/modelToDom/context/createModelToDomContext';
-import { tablePreProcessor } from '../../../lib/override/tablePreProcessor';
 import {
     listItemMetadataApplier,
     listLevelMetadataApplier,
@@ -23,22 +22,11 @@ describe('createDomToModelSettings', () => {
         const settings = createDomToModelSettings({});
 
         expect(settings).toEqual({
-            builtIn: {
-                processorOverride: {
-                    table: tablePreProcessor,
-                },
-            },
+            builtIn: {},
             customized: {},
             calculated: mockedCalculatedConfig,
         });
-        expect(createDomToModelContext.createDomToModelConfig).toHaveBeenCalledWith([
-            {
-                processorOverride: {
-                    table: tablePreProcessor,
-                },
-            },
-            {},
-        ]);
+        expect(createDomToModelContext.createDomToModelConfig).toHaveBeenCalledWith([{}, {}]);
     });
 
     it('Has options', () => {
@@ -48,20 +36,12 @@ describe('createDomToModelSettings', () => {
         });
 
         expect(settings).toEqual({
-            builtIn: {
-                processorOverride: {
-                    table: tablePreProcessor,
-                },
-            },
+            builtIn: {},
             customized: defaultDomToModelOptions,
             calculated: mockedCalculatedConfig,
         });
         expect(createDomToModelContext.createDomToModelConfig).toHaveBeenCalledWith([
-            {
-                processorOverride: {
-                    table: tablePreProcessor,
-                },
-            },
+            {},
             defaultDomToModelOptions,
         ]);
     });

diff --git a/packages/roosterjs-content-model-core/test/overrides/tablePreProcessorTest.ts b/packages/roosterjs-content-model-core/test/overrides/tablePreProcessorTest.ts