Skip to content

Commit fd1c92f

Browse files
authored
Add a null check in getListAnnounceData (#2964)
* Add a null check in getListAnnounceData * improve
1 parent e092921 commit fd1c92f

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

packages/roosterjs-content-model-api/lib/modelApi/list/getListAnnounceData.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ export function getListAnnounceData(path: ReadonlyContentModelBlockGroup[]): Ann
2424
const listItem = path[index] as ContentModelListItem;
2525
const level = listItem.levels[listItem.levels.length - 1];
2626

27-
if (level.format.displayForDummyItem) {
27+
if (!level || level.format.displayForDummyItem) {
2828
return null;
2929
} else if (level.listType == 'OL') {
3030
const listNumber = getListNumber(path, listItem);

packages/roosterjs-content-model-api/test/modelApi/list/getListAnnounceDataTest.ts

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,18 @@ describe('getListAnnounceData', () => {
7171
expect(getAutoListStyleTypeSpy).not.toHaveBeenCalled();
7272
});
7373

74+
it('path has list item without list levels', () => {
75+
const doc = createContentModelDocument();
76+
const listItem = createListItem([]);
77+
78+
doc.blocks.push(listItem);
79+
80+
const result = getListAnnounceData([listItem, doc]);
81+
82+
expect(result).toEqual(null);
83+
expect(getAutoListStyleTypeSpy).not.toHaveBeenCalled();
84+
});
85+
7486
it('path with bullet list', () => {
7587
const doc = createContentModelDocument();
7688
const listItem = createListItem([createListLevel('UL')]);

0 commit comments

Comments
 (0)