If a user requests this feature, we can enable in a later issue. But then also xinclude should be enabled. Any other opinions?
Nope, like i said: we can worry about the detials of making it configurable if/when anyone asks for it.
I can extend the patch to also take care of DIH (
i figured maybe we could - but we can also worry about it in that issue after this one gets commited, whatever you think is easier.
We should add the same one for XSLReqHandler (because it uses other parser).
good call – i updated the patch to test that named entities still work in XSL transformed docs & stylesheets, and they do. But this lead me to discover that SYSTEM entities in the xsl docs (aparently) aren't being ignored, so with the patch attached (trying to refer to a bogs file fro mthe XSL) you get a stylesheet compilation error – so i think maybe there is still a code path missing the use of your new EmptyEntityResolver? (either that or i've just got a silly bug in the stylesheet that isn't obvious to me)