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-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org