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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to