[jira] [Created] (XERCESJ-1750) XInclude error output to include xpointer values

["John Poole (Jira)" <xe...@xml.apache.org>]

John Poole created XERCESJ-1750:
-----------------------------------

             Summary: XInclude error output to include xpointer values
                 Key: XERCESJ-1750
                 URL: https://issues.apache.org/jira/browse/XERCESJ-1750
             Project: Xerces2-J
          Issue Type: Wish
          Components: XInclude 1.0
    Affects Versions: 2.12.2
            Reporter: John Poole


Currently error output on failure of a resolution of an XIncludes only provides the file, i.e. *href *value.  It would be extremely helpful to have the *xpointer *value bubble up in the error output so one would see both *href *and *xpointer* values. For example, I have file A referencing file B with many XIncludes pointing to different nodes and some resolve, and other purportedly fail.  Having the XMLParseError message include the *xpointer *value would help debug problems, especially when some value resolve and others fails.  (And no, the target values are in the referenced file, I just do not know at this time why there are some failures and successes and when dealing with hundreds of files with thousands of XIncludes, it is a struggle to segregate the failures.)

I've been using Syncro Soft's Oxygen Author and made an enhancement request and was told Author simply takes what Xerces gives it as an error message.  So here I am requesting Xerces provide one critical bit of information in the error message.

To that end I have suggested two modification, each only one line of modification:

{noformat}
jlpoole@ares /usr/local/src/xerces_2.12.2/xerces-2_12_2 $ diff src/org/apache/xerces/impl/msg/XIncludeMessages.properties ../xerces-2_12_2_work/src/org/apache/xerces/impl/msg/XIncludeMessages.properties
37c37
< XMLParseError = Error attempting to parse XML file (href=''{0}'').
---
> XMLParseError = Error attempting to parse XML file (href=''{0}'', xpointer=''{1}'').
jlpoole@ares /usr/local/src/xerces_2.12.2/xerces-2_12_2 $ diff src/org/apache/xerces/xinclude/XIncludeHandler.java ../xerces-2_12_2_work/src/org/apache/xerces/xinclude/XIncludeHandler.java
1725c1725
<                 reportFatalError("XMLParseError", new Object[] { href });
---
>                 reportFatalError("XMLParseError", new Object[] { href, xpointer });
jlpoole@ares /usr/local/src/xerces_2.12.2/xerces-2_12_2 $
{noformat}

I did some preliminary tests to see if the modifications above might break something in conditions where 1) no xpointer is present, 2) xpointer = "", and 3) xpointer is invalid, e.g. xpointer="TODO:FIXME". 






--
This message was sent by Atlassian Jira
(v8.20.10#820010)

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