formatContentModel

microsoft · Jun 6, 2024 · 08c69bf · 08c69bf
1 parent bf1533e
commit 08c69bf
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 36 deletions.
diff --git a/packages/roosterjs-content-model-core/lib/coreApi/formatContentModel/formatContentModel.ts b/packages/roosterjs-content-model-core/lib/coreApi/formatContentModel/formatContentModel.ts
@@ -24,15 +24,8 @@ export const formatContentModel: FormatContentModel = (
     options,
     domToModelOptions
 ) => {
-    const {
-        apiName,
-        onNodeCreated,
-        getChangeData,
-        changeSource,
-        rawEvent,
-        selectionOverride,
-        scrollCaretIntoView,
-    } = options || {};
+    const { onNodeCreated, getChangeData, rawEvent, selectionOverride, scrollCaretIntoView } =
+        options || {};
     const model = core.api.createContentModel(core, domToModelOptions, selectionOverride);
     const context: FormatContentModelContext = {
         newEntities: [],
@@ -82,9 +75,9 @@ export const formatContentModel: FormatContentModel = (
                 eventType: 'contentChanged',
                 contentModel: clearModelCache ? undefined : model,
                 selection: clearModelCache ? undefined : selection,
-                source: changeSource || ChangeSource.Format,
+                source: options?.changeSource || ChangeSource.Format,
                 data: getChangeData?.(),
-                formatApiName: apiName,
+                formatApiName: options?.apiName,
                 changedEntities: getChangedEntities(context, rawEvent),
             };
 

diff --git a/packages/roosterjs-content-model-plugins/lib/autoFormat/AutoFormatPlugin.ts b/packages/roosterjs-content-model-plugins/lib/autoFormat/AutoFormatPlugin.ts
@@ -220,14 +220,9 @@ export class AutoFormatPlugin implements EditorPlugin {
                                 shouldFraction ||
                                 shouldOrdinals
                             );
-                        }
+                        },
+                        formatOptions
                     );
-                    if (formatOptions.changeSource) {
-                        editor.triggerEvent('contentChanged', {
-                            source: formatOptions.changeSource,
-                            formatApiName: formatOptions.apiName,
-                        });
-                    }
 
                     break;
             }

diff --git a/packages/roosterjs-content-model-plugins/test/autoFormat/AutoFormatPluginTest.ts b/packages/roosterjs-content-model-plugins/test/autoFormat/AutoFormatPluginTest.ts
@@ -108,10 +108,6 @@ describe('Content Model Auto Format Plugin Test', () => {
                     changeSource: 'AutoFormat',
                     apiName: formatOptions.apiName,
                 });
-                expect(triggerEventSpy).toHaveBeenCalledWith('contentChanged', {
-                    source: 'AutoFormat',
-                    formatApiName: formatOptions.apiName,
-                });
             });
         }
 
@@ -397,10 +393,6 @@ describe('Content Model Auto Format Plugin Test', () => {
                     changeSource: 'AutoFormat',
                     apiName: formatOption.apiName,
                 });
-                expect(triggerEventSpy).toHaveBeenCalledWith('contentChanged', {
-                    source: 'AutoFormat',
-                    formatApiName: formatOption.apiName,
-                });
             });
         }
 
@@ -481,11 +473,6 @@ describe('Content Model Auto Format Plugin Test', () => {
                     changeSource: 'AutoFormat',
                     apiName: formatOption.apiName,
                 });
-
-                expect(triggerEventSpy).toHaveBeenCalledWith('contentChanged', {
-                    source: 'AutoFormat',
-                    formatApiName: formatOption.apiName,
-                });
             });
         }
 
@@ -552,11 +539,6 @@ describe('Content Model Auto Format Plugin Test', () => {
                     changeSource: 'AutoFormat',
                     apiName: formatOption.apiName,
                 });
-
-                expect(triggerEventSpy).toHaveBeenCalledWith('contentChanged', {
-                    source: 'AutoFormat',
-                    formatApiName: formatOption.apiName,
-                });
             });
         }