Nathaniel Alfred wrote:
Hi Hussayn,
thanks for sharing your patch.
I'll have a look at it.
IIUC, the problem with this patch is that it drops usage of Source and replaces
it with jaxp API. Which means, if bug description is correct, that Source still
is not using entity resolver as one would expect, and any other XML file (beside
XSLT) won't be able to use entity resolver as well.
Preferred bug fix IMHO should fix the original issue - source not using entity
resolver - instead of bypassing it.
Vadim
Cheers, Alfred.
-----Original Message-----
From: news [mailto:[EMAIL PROTECTED] On Behalf Of Hussayn dabbous
Sent: Mittwoch, 30. August 2006 22:48
To: [email protected]
Subject: patch for an entityResolver problem in xsl stylesheets ...
Hi;
I tried to use external entities like ü etc. in xsl stylesheets.
For this purpose in first place i added a DOCTYPE definition on top of
my stylesheet, i.e.:
<?xml version="1.0"?>
<!DOCTYPE mydoc [
<!ENTITY % ISOlat1 PUBLIC "ISO 8879:1986//ENTITIES Added Latin
1//EN//XML" "ISOlat1.pen"> %ISOlat1;
]><xsl:stylesheet version="1.0">
...
I assumed, the default entity resolver would resolve the ISOlat1.pen,
but it did not!
I tracked this down and found, that entity resolution only works
correctly when i
set the SYSTEM identifier to the absolute location of the external
entity file. I did
not like this "hack", hence i tracked the problem further down and
eventually found
one location in TraxProcessor.java which apparently needs a modification
in order to inject
the correct EntityResolver to work as expected.
I finally created a patch, which adds Entity resolving within XSLT
processing wherever
a document(), import or include is performed. i did not check whether
this patch works
recursively (follows an import within an imported xsl), but it looks
like it would do ...
The patch has been created against cocoon/branches/BRANCH_2_1_X
It works for me. Hopefully it is usefull for others and does not break
code at other
places. If anyone would like to review the patch and give some feedback,
That would be great ;-)
best regards,
Hussayn
