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.

Sign up for GitHub

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

Jump to bottom

SAXReader: hardcoded namespace features #44

Closed
philippeu opened this issue Apr 29, 2018 · 1 comment
Closed

SAXReader: hardcoded namespace features #44

philippeu opened this issue Apr 29, 2018 · 1 comment
Assignees
@FilipJirsak
Labels
bug
Milestone
2.1.1

Comments

@philippeu
Copy link

It is not possible to modify the following features:

hence, calling saxReader.setFeature has no effect
@FilipJirsak FilipJirsak self-assigned this Apr 30, 2018
FilipJirsak added a commit that referenced this issue Jul 1, 2018
@FilipJirsak
#44 Default SAXParser features are set when SAXParser is created, so …
161078a 
…they can be overriden.
@FilipJirsak FilipJirsak added the bug label Jul 1, 2018
@FilipJirsak FilipJirsak modified the milestones: 2.0.3, 2.1.1 Jul 1, 2018
@FilipJirsak
Copy link
Contributor

These features are set in createXMLReader() now so you can override them.

@FilipJirsak FilipJirsak modified the milestones: 2.0.3, 2.1.1 Jul 1, 2018
@FilipJirsak FilipJirsak mentioned this issue Oct 9, 2018
Open
FilipJirsak added a commit to FilipJirsak/dom4j that referenced this issue Mar 12, 2020
@FilipJirsak
dom4j#44 Default SAXParser features are set when SAXParser is created…
cf17a45 
…, so they can be overriden.
FilipJirsak added a commit that referenced this issue Apr 11, 2020
@FilipJirsak
#44 Default SAXParser features are set when SAXParser is created, so …
a16aaa7 
…they can be overriden.

(cherry picked from commit 161078a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@FilipJirsak FilipJirsak

Labels
bug
Milestone
2.1.1
Development

No branches or pull requests
2 participants
@FilipJirsak @philippeu