[feature] Add location information (source, line-number, and column-number) to XSLT errors

Author: adamretter

exist-core/src/main/java/org/exist/http/servlets/XSLTServlet.java

@@ -89,7 +89,7 @@ public class XSLTServlet extends HttpServlet {
         new XSLTErrorsListener<ServletException>(true, false) {
 
             @Override
-            protected void raiseError(String error, Exception ex) throws ServletException {
+            protected void raiseError(final String error, final TransformerException ex) throws ServletException {
                 throw new ServletException(error, ex);
             }
         };

exist-core/src/main/java/org/exist/xquery/functions/transform/Transform.java

@@ -188,7 +188,7 @@ public Sequence eval(Sequence[] args, Sequence contextSequence) throws XPathExce
         final XSLTErrorsListener<XPathException> errorListener =
             new XSLTErrorsListener<XPathException>(stopOnError, stopOnWarn) {
                 @Override
-                protected void raiseError(String error, Exception ex) throws XPathException {
+                protected void raiseError(final String error, final TransformerException ex) throws XPathException {
                     throw new XPathException(Transform.this, error, ex);
                 }
             };

exist-core/src/main/java/org/exist/xslt/XSLTErrorsListener.java

@@ -42,14 +42,14 @@ private enum ErrorType {
   private final boolean stopOnWarn;
 
   @Nullable private ErrorType errorType;
-  @Nullable private Exception exception;
+  @Nullable private TransformerException exception;
 
   public XSLTErrorsListener(final boolean stopOnError, final boolean stopOnWarn) {
     this.stopOnError = stopOnError;
     this.stopOnWarn = stopOnWarn;
   }
 
-  protected abstract void raiseError(String error, Exception ex) throws E;
+  protected abstract void raiseError(String error, TransformerException ex) throws E;
 
   public void checkForErrors() throws E {
     if (errorType == null) {
@@ -59,16 +59,16 @@ public void checkForErrors() throws E {
     switch (errorType) {
       case WARNING:
         if (stopOnWarn) {
-          raiseError("XSL transform reported warning: " + exception.getMessage(), exception);
+          raiseError("XSL transform reported warning: " + exception.getMessageAndLocation(), exception);
         }
         break;
       case ERROR:
         if (stopOnError) {
-          raiseError("XSL transform reported error: " + exception.getMessage(), exception);
+          raiseError("XSL transform reported error: " + exception.getMessageAndLocation(), exception);
         }
         break;
       case FATAL:
-        raiseError("XSL transform reported error: " + exception.getMessage(), exception);
+        raiseError("XSL transform reported error: " + exception.getMessageAndLocation(), exception);
     }
   }