Skip to content

fix: loader temp footer and header loader #2798

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

AdersonSilva18
Copy link

@AdersonSilva18 AdersonSilva18 commented Jul 9, 2025

When a document starts its header or footer numbering at a value greater than 1, the library throws an error. It also fails when the indices are non-sequential—for example: header2, header6.

Fixes # (issue)

Checklist:

  • My CI is 🟢
  • I have covered my new code with unit tests (check build/coverage for coverage report)
  • I have updated the documentation to describe the changes
  • I have updated the changelog

An example document exhibiting the problem is attached.
documento_teste_error.docx

Aderson Silva added 4 commits July 9, 2025 13:26
When a document started its header or footer numbering at a value greater than 1, it caused an error, and it also failed when the indices were non-sequential—for example:

header2, header6
@coveralls
Copy link

coveralls commented Jul 9, 2025

Coverage Status

coverage: 96.758% (+0.001%) from 96.757%
when pulling 46c786a on AdersonSilva18:fix-load-footer-header-index
into 0ab0b49 on PHPOffice:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants