Skip to content

fix(docx-template): corruption of libreoffice-originated templates - #1061

Merged
winged merged 1 commit into
mainfrom
fix-libreoffice-bug
Sep 26, 2025
Merged

fix(docx-template): corruption of libreoffice-originated templates#1061
winged merged 1 commit into
mainfrom
fix-libreoffice-bug

Conversation

@czosel

@czosel czosel commented Sep 22, 2025

Copy link
Copy Markdown
Contributor

Certain versions of python-docx and python-docxtemplate cause corruption
of files that were originally created with LibreOffice. One effect of that
corruption is a duplicate entry in the document-internal files; there
are two docProps/core.xml files in the resulting document.

@winged
winged force-pushed the fix-libreoffice-bug branch 4 times, most recently from e9ad96f to 8b4267a Compare September 22, 2025 15:34
@winged winged changed the title fix(lo): WIP fix(docx-template): corruption of libreoffice-originated templates Sep 26, 2025
@czosel
czosel marked this pull request as ready for review September 26, 2025 14:52
Certain versions of python-docx and python-docxtemplate cause corruption
of files that were originally created with LibreOffice. One effect of that
corruption is a duplicate entry in the document-internal files; there
are two docProps/core.xml files in the resulting document.

Co-Authored-By: David Vogt <david.vogt@adfinis.com>
@winged
winged force-pushed the fix-libreoffice-bug branch from 8b4267a to d7b3cec Compare September 26, 2025 14:57
@winged

winged commented Sep 26, 2025

Copy link
Copy Markdown
Member

Note: This is basically monkey-patching this PR: python-openxml/python-docx#1436

@winged winged left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Co-authored (and offline-reviewed with) @czosel

@winged
winged merged commit 2b1bd8d into main Sep 26, 2025
12 checks passed
@anehx
anehx deleted the fix-libreoffice-bug branch January 15, 2026 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants