Skip to content

Commit b456f34

Browse files
committed
fix: multiple OLEs, same ext type, same slide.
1 parent c772f40 commit b456f34

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/shapes/ole.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -141,13 +141,14 @@ export class OLEObject extends Shape {
141141

142142
const types = contentTypesXml.getElementsByTagName('Types')[0];
143143
const fileExtension = this.getFileExtension(this.oleObjectPath);
144+
const partName = `/ppt/embeddings/${this.createdRid}${fileExtension}`;
144145
const existingOverride = Array.from(types.getElementsByTagName('Override')).find(
145-
(override) => override.getAttribute('PartName').endsWith(fileExtension)
146+
(override) => override.getAttribute('PartName') === partName
146147
);
147148

148149
if (!existingOverride) {
149150
const newOverride = contentTypesXml.createElement('Override');
150-
newOverride.setAttribute('PartName', `/ppt/embeddings/${this.createdRid}${fileExtension}`);
151+
newOverride.setAttribute('PartName', partName);
151152
newOverride.setAttribute('ContentType', this.getContentType(fileExtension));
152153
types.appendChild(newOverride);
153154

0 commit comments

Comments
 (0)