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

Prevented fatal errors when opening corrupt files or "doc" files #2626

Merged
merged 5 commits into from
Aug 8, 2024

Conversation

mmcev106
Copy link
Contributor

@mmcev106 mmcev106 commented Jun 19, 2024

Description

This prevents fatal exceptions when trying to load corrupt files or old "doc" files, allowing applications to catch the exception and fail gracefully.

Fixes # (issue) - None. I just fixed it without reporting an issue.

Checklist:

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

@mmcev106
Copy link
Contributor Author

I'm looking into the coding standard failures.

@coveralls
Copy link

Coverage Status

coverage: 97.209% (+0.001%) from 97.208%
when pulling d6a2106 on mmcev106:master
into 2daa50c on PHPOffice:master.

@coveralls
Copy link

Coverage Status

coverage: 97.209% (+0.001%) from 97.208%
when pulling d6a2106 on mmcev106:master
into 2daa50c on PHPOffice:master.

@coveralls
Copy link

Coverage Status

coverage: 97.209% (+0.001%) from 97.208%
when pulling d6a2106 on mmcev106:master
into 2daa50c on PHPOffice:master.

@mmcev106
Copy link
Contributor Author

I think we're good to go now (guessing the deploy failure is not something I should look into).

@mmcev106
Copy link
Contributor Author

@everaldofilho, thanks for the approval. Any idea when this might get merged? If there are any further changes you'd like me to make, please ask.

@Progi1984
Copy link
Member

@mmcev106 Could you update the changelog ?

@Progi1984 Progi1984 added the Status: Waiting for feedback Question has been asked, waiting for response from PR author label Aug 7, 2024
@mmcev106
Copy link
Contributor Author

mmcev106 commented Aug 8, 2024

@Progi1984, the changelog has been updated.

@coveralls
Copy link

coveralls commented Aug 8, 2024

Coverage Status

coverage: 97.211%. remained the same
when pulling 7f52b99 on mmcev106:master
into b99230a on PHPOffice:master.

@Progi1984 Progi1984 merged commit 761280b into PHPOffice:master Aug 8, 2024
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Waiting for feedback Question has been asked, waiting for response from PR author
Development

Successfully merging this pull request may close these issues.

4 participants