XalanJ2
  1. XalanJ2
  2. XALANJ-2317

Command line XSLTC 2.7.0 swallows error diagnostics

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.7
    • Fix Version/s: 2.7.1
    • Component/s: Xalan-CmdLine
    • Labels:
      None
    • Xalan info:
      PatchAvailable

      Description

      http://svn.apache.org/viewvc?view=rev&revision=338167 changed error reporting.
      Since then the command line XSLTC does not print the error diagnostic anymore.

      To fix it org.apache.xalan.xslt.Process needs to use everywhere setErrorListener(new DefaultErrorHandler(*false*)).

      1. Process.java.patch
        3 kB
        Santiago Pericas-Geertsen

        Activity

        Alfred Nathaniel created issue -
        Hide
        Brian Minchau added a comment -

        Per the JIRA triage meeting on Oct 16, 2005, Santiago P-G agreed to look at this one.

        Show
        Brian Minchau added a comment - Per the JIRA triage meeting on Oct 16, 2005, Santiago P-G agreed to look at this one.
        Brian Minchau made changes -
        Field Original Value New Value
        Assignee Santiago Pericas-Geertsen [ santiago.pericasgeertsen@sun.com ]
        Santiago Pericas-Geertsen made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Santiago Pericas-Geertsen added a comment -

        Here is the patch proposed by the reporter. Using 'false' in the constructor will ensure that messages will be send to stderr.

        Show
        Santiago Pericas-Geertsen added a comment - Here is the patch proposed by the reporter. Using 'false' in the constructor will ensure that messages will be send to stderr.
        Santiago Pericas-Geertsen made changes -
        Attachment Process.java.patch [ 12344981 ]
        Hide
        Brian Minchau added a comment -

        I have reviewed and approve the patch and committed it to the latest code.

        I tested this with a stylesheet with a bogus xsl element, <xsl:fred />
        For Xalan-J interpreted the error was reported with location information, but not for XSLTC.
        With the patch the error message was printed.

        Show
        Brian Minchau added a comment - I have reviewed and approve the patch and committed it to the latest code. I tested this with a stylesheet with a bogus xsl element, <xsl:fred /> For Xalan-J interpreted the error was reported with location information, but not for XSLTC. With the patch the error message was printed.
        Brian Minchau made changes -
        Resolution Fixed [ 1 ]
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s Latest Development Code [ 10863 ]
        Hide
        Brian Minchau added a comment -

        Would the originator of this issue please verify that this issue is fixed in the 2.7.1 release, by adding a comment to this issue, so that we can close this issue.

        A lack of response by February 1, 2008 will be taken as consent that we can close this resolved issue.

        Regards,
        Brian Minchau

        Show
        Brian Minchau added a comment - Would the originator of this issue please verify that this issue is fixed in the 2.7.1 release, by adding a comment to this issue, so that we can close this issue. A lack of response by February 1, 2008 will be taken as consent that we can close this resolved issue. Regards, Brian Minchau
        Hide
        Alfred Nathaniel added a comment -

        Yes, thanks, problem is fixed in 2.7.1.

        Show
        Alfred Nathaniel added a comment - Yes, thanks, problem is fixed in 2.7.1.
        Alfred Nathaniel made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Mark Thomas made changes -
        Workflow jira [ 12382848 ] Default workflow, editable Closed status [ 12570441 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12570441 ] jira [ 12594513 ]

          People

          • Assignee:
            Santiago Pericas-Geertsen
            Reporter:
            Alfred Nathaniel
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development