[jira] Created: (XALANJ-2422) TreeWalker set systemId to dummy.xsl always (patch merge issue with XALANJ-1985)

["Brian McDonald (JIRA)" <xa...@xml.apache.org>]

TreeWalker set systemId to dummy.xsl always (patch merge issue with XALANJ-1985)
--------------------------------------------------------------------------------

                 Key: XALANJ-2422
                 URL: https://issues.apache.org/jira/browse/XALANJ-2422
             Project: XalanJ2
          Issue Type: Bug
          Components: Xalan
    Affects Versions: 2.7, 2.7.1
            Reporter: Brian McDonald
            Priority: Blocker


If you take a look at the merge of the TreeWalker.patch (org/apache/xml/serializer/TreeWalker.java) between versions 338117 and 338053 (for XALANJ-1985), the constructor has some remnant code at the end which breaks it's intended functionality.

This chunk should be removed (sorry for not attaching a patch):

      // Set the document locator  
                if (m_contentHandler != null)
                        m_contentHandler.setDocumentLocator(m_locator);
                try {
                   // Bug see Bugzilla  26741
                  m_locator.setSystemId(System.getProperty("user.dir") + File.separator + "dummy.xsl");
                } 
                catch (SecurityException se){// user.dir not accessible from applet

You can see that the constructor before this chunk already sets the document locator on the content handler and if the system id is null, sets the dummy value. Problem is that this extra chunk is setting the system id no matter what, which breaks all attempts to load relative files, with no workaround possible. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: xalan-dev-unsubscribe@xml.apache.org
For additional commands, e-mail: xalan-dev-help@xml.apache.org