Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Additional default OOXML namespace aliases #97

Merged
merged 1 commit into from
Sep 21, 2021
Merged

Conversation

mbali
Copy link
Contributor

@mbali mbali commented Sep 21, 2021

The http:https://schemas.microsoft.com/office/word/2010/wordprocessingGroup namespace is expected to have an alias of wpg by LO 6.4. Having a different alias caused LibreOffice to not display grouped drawings.
I added some additional default namespace aliases from sample files also, to hopefully prevent some later errors caused by this.

A previous issue and PR with the same main cause: #55, #56

The http:https://schemas.microsoft.com/office/word/2010/wordprocessingGroup namespace is expected to have an alias of "wpg" by LO 6.4. Having a different alias caused LibreOffice to not display grouped drawings.
I copied some additional default namespace aliases from sample files, to hopefully prevent such errors in the future.
@codecov
Copy link

codecov bot commented Sep 21, 2021

Codecov Report

Merging #97 (7a5e50b) into master (2327ee2) will increase coverage by 0.05%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #97      +/-   ##
==========================================
+ Coverage   92.72%   92.78%   +0.05%     
==========================================
  Files          24       24              
  Lines        1843     1843              
  Branches      110      109       -1     
==========================================
+ Hits         1709     1710       +1     
  Misses         24       24              
+ Partials      110      109       -1     
Impacted Files Coverage Δ
src/stencil/ooxml.clj 100.00% <100.00%> (ø)
src/stencil/postprocess/ignored_tag.clj 100.00% <0.00%> (+2.43%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2327ee2...7a5e50b. Read the comment docs.

@erdos
Copy link
Owner

erdos commented Sep 21, 2021

Thank you for this amazing work!

@erdos erdos merged commit 829d151 into erdos:master Sep 21, 2021
@erdos
Copy link
Owner

erdos commented Sep 21, 2021

Released in version 0.3.26.

@erdos erdos added the bug Something isn't working label Nov 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants