Fix Error in Ods.php Reader for some files

[cqix]

#804 Catch Uncaught Error Call to undefined method DOMText::hasAttributeNS() in Ods.php Reader

This is:

- [x] a bugfix
- [ ] a new feature

Checklist:

• Changes are covered by unit tests
• Code style is respected
• Commit message explains why the change is made (see https://github.com/erlang/otp/wiki/Writing-good-commit-messages)
• CHANGELOG.md contains a short summary of the change
• Documentation is updated as necessary

Why this change is needed?

Some ODS files have a childNode of type DOMText, instead of DOMElement only. This tiny fix allows to read these files.

[cqix]

Code Style only

[PowerKiKi]

Instead of loosely testing the existence of a method, it would be best to accurately test the expected type of child, and also extract DomElement specific code into a private method, to make it easier to understand what's going on.

Also tests and changelog would be nice too.

[cqix]

Instead of loosely testing the existence of a method, it would be best to accurately test the expected type of child, and also extract DomElement specific code into a private method, to make it easier to understand what's going on.

Also tests and changelog would be nice too.

Yes, that whole code is a little complicated to read. I did not event find out what the rest of the code was dooing, so I catched for the obvious.
Anyways, I also want to push some other stuff soon (NamedRanges for ODS Writer), but I need to find some time for tests etc before.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If this is still an issue for you, please try to help by debugging it further and sharing your results.
Thank you for your contributions.

[rsevero]

Faced the same Uncaught Error Call to undefined method DOMText::hasAttributeNS() in Ods.php Reader during some imports.

Applied the fix provided at #1137 (comment) and my imports are working again.

[oleibman]

Superseded by PR #4087.