[ http://issues.apache.org/jira/browse/XALANJ-1706?page=all ]
     
Brian Minchau resolved XALANJ-1706:
-----------------------------------

    Fix Version: Latest Development Code
     Resolution: Fixed

I reviewed the patch and approve it. I have applied it to CVS head branch.

The fix only impacts DOM2DTM, and only its dispatchToEvents() method with is 
called by
SerializerUtils.outputResultTreeFragment(), which is used in ElemCopyOf.  So 
all in all
it looks like the startDocument() endDocument() is not correct for outputting a 
fragment.

It also passes the smoke test and conformance tests.

> [PATCH] DocumentFragment returned by extension element causes multiple SAX 
> endDocument() events
> -----------------------------------------------------------------------------------------------
>
>          Key: XALANJ-1706
>          URL: http://issues.apache.org/jira/browse/XALANJ-1706
>      Project: XalanJ2
>         Type: Bug
>   Components: Other
>     Versions: Latest Development Code
>  Environment: Operating System: All
> Platform: All
>     Reporter: Jeremias Maerki
>     Assignee: Brian Minchau
>      Fix For: Latest Development Code
>  Attachments: BugzillaExtElemSAXendDocumentCheck.zip, 
> BugzillaExtElemSAXendDocumentCheck.zip, XALANJ-1706-patch.diff, 
> endDocumentPatch.diff
>
> At Krysalis Barcode (http://www.krysalis.org/barcode) I have a Xalan 
> extension 
> which creates barcodes in SVG format [1]. Until recently I've used an 
> extension function for that. As I was able to create an extension element for 
> SAXON I wanted to do the same for the Xalan-extension. But here's the problem:
> The extension element (like the extension function where it works correctly) 
> returns a DocumentFragment containing the generated barcode in SVG format. In 
> the case of the extension element the DocumentFragment is processed by a 
> TreeWalker which issues start/endDocument() SAX events although it shouldn't 
> IMO. This behaviour causes Apache FOP to crash with an NPE because more than 
> one endDocument() SAX event is sent to FOP by Xalan.
> I will attach a test case for the bugzilla test directory. Current CVS (HEAD) 
> and 2.5.1 fail in this test. I will try to find a bugfix but if someone more 
> intimate with the code beats me to it, so much better.
> [1] http://cvs.sourceforge.net/viewcvs.py/krysalis/krysalis-
> barcode/src/java/org/krysalis/barcode/xalan/BarcodeExt.java

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to