[ http://issues.apache.org/jira/browse/XALANJ-1706?page=comments#action_12332633 ]
Brian Minchau commented on XALANJ-1706: --------------------------------------- Jeremias, your "Ping!" caught my attention. For some reason I have an interest in SAX events, and that the events are, shall we say, well-formed. Thanks for the full testcase, and patch. Sorry one of the Xalan committers didn't have a look earlier. Assuming the patch is OK, and applied to CVS HEAD, is that sufficient, or are you wanting it in an official release, e.g. 2.7.1 ? One other point, since the patch was migrated from bugzilla, it is not marked as giving the rights to Apache. It seems silly, but please attach that patch yet again, with the appropriate checkbox set. > [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 > Attachments: BugzillaExtElemSAXendDocumentCheck.zip, 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]
