You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2013/01/23 20:40:40 UTC

svn commit: r1437649 - in /tomcat/trunk/java/org/apache/jasper: compiler/DefaultErrorHandler.java resources/LocalStrings.properties resources/LocalStrings_es.properties

Author: markt
Date: Wed Jan 23 19:40:40 2013
New Revision: 1437649

URL: http://svn.apache.org/viewvc?rev=1437649&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=54466
Improve error message by including the name of the file when the java file generated from a tag file cannot be compiled.
Based on a patch by Sheldon Shao.

Modified:
    tomcat/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java
    tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties
    tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties

Modified: tomcat/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java?rev=1437649&r1=1437648&r2=1437649&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java (original)
+++ tomcat/trunk/java/org/apache/jasper/compiler/DefaultErrorHandler.java Wed Jan 23 19:40:40 2013
@@ -87,7 +87,8 @@ class DefaultErrorHandler implements Err
                 buf.append(details[i].getJspExtract());
             } else {
                 args = new Object[] {
-                        Integer.valueOf(details[i].getJavaLineNumber()) };
+                        Integer.valueOf(details[i].getJavaLineNumber()),
+                        details[i].getJavaFileName() };
                 buf.append(Constants.NEWLINE);
                 buf.append(Constants.NEWLINE);
                 buf.append(Localizer.getMessage("jsp.error.java.line.number",

Modified: tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties?rev=1437649&r1=1437648&r2=1437649&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties (original)
+++ tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties Wed Jan 23 19:40:40 2013
@@ -236,7 +236,7 @@ jsp.error.invalid.tagdir=Tag file direct
 jsp.error.needAlternateJavaEncoding=Default java encoding {0} is invalid on your java platform. An alternate can be specified via the 'javaEncoding' parameter of JspServlet.
 #Error when compiling, used for jsp line number error messages
 jsp.error.single.line.number=An error occurred at line: {0} in the jsp file: {1}
-jsp.error.java.line.number=An error occurred at line: {0} in the generated java file
+jsp.error.java.line.number=An error occurred at line: [{0}] in the generated java file: [{1}]
 jsp.error.location=line: {0}, column: {1}
 jsp.error.corresponding.servlet=Generated servlet error:\n
 jsp.error.jspbody.required=Must use jsp:body to specify tag body for {0} if jsp:attribute is used.

Modified: tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties?rev=1437649&r1=1437648&r2=1437649&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties (original)
+++ tomcat/trunk/java/org/apache/jasper/resources/LocalStrings_es.properties Wed Jan 23 19:40:40 2013
@@ -234,7 +234,7 @@ jsp.error.invalid.tagdir = El directorio
 jsp.error.needAlternateJavaEncoding = La codificaci\u00F3n java por defecto {0} es incorrecta en tu plataforma java. Se puede especificar una alternativa v\u00EDa par\u00E1metro 'javaEncoding' de JspServlet.
 #Error when compiling, used for jsp line number error messages
 jsp.error.single.line.number = Ha tenido lugar un error en la l\u00EDnea\: {0} en el archivo jsp\: {1}
-jsp.error.java.line.number = Ha tenido lugar un error en la l\u00EDnea\: {0} en el fichero java generado
+jsp.error.java.line.number = Ha tenido lugar un error en la l\u00EDnea\: [{0}] en el fichero java generado: [{1}]
 jsp.error.location = l\u00EDnea\: {0}, columna\: {1}
 jsp.error.corresponding.servlet = Error de servlet generado\:\n
 jsp.error.jspbody.required = Se debe de usar jsp\:body para especificar cuerpo tag para {0} si se usa jsp\:attribute.



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