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 2009/07/05 15:25:21 UTC

svn commit: r791243 - in /tomcat/trunk/java/org/apache/jasper: compiler/TagLibraryInfoImpl.java resources/LocalStrings.properties

Author: markt
Date: Sun Jul  5 13:25:21 2009
New Revision: 791243

URL: http://svn.apache.org/viewvc?rev=791243&view=rev
Log:
Improve error message to include TDL uri when there is a problem with a tld so user knows which TLD to look at

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

Modified: tomcat/trunk/java/org/apache/jasper/compiler/TagLibraryInfoImpl.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/jasper/compiler/TagLibraryInfoImpl.java?rev=791243&r1=791242&r2=791243&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/compiler/TagLibraryInfoImpl.java (original)
+++ tomcat/trunk/java/org/apache/jasper/compiler/TagLibraryInfoImpl.java Sun Jul  5 13:25:21 2009
@@ -278,11 +278,11 @@
 
         if (tlibversion == null) {
             err.jspError("jsp.error.tld.mandatory.element.missing",
-                    "tlib-version");
+                    "tlib-version", uri);
         }
         if (jspversion == null) {
             err.jspError("jsp.error.tld.mandatory.element.missing",
-                    "jsp-version");
+                    "jsp-version", uri);
         }
 
         this.tags = new TagInfo[tagVector.size()];

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=791243&r1=791242&r2=791243&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties (original)
+++ tomcat/trunk/java/org/apache/jasper/resources/LocalStrings.properties Sun Jul  5 13:25:21 2009
@@ -335,7 +335,7 @@
 jsp.error.tld.fn.duplicate.name=Duplicate function name {0} in tag library {1}
 jsp.error.tld.fn.invalid.signature.commaexpected=Invalid syntax for function signature in TLD.  Comma ',' expected.  Tag Library: {0}, Function: {1}.
 jsp.error.tld.fn.invalid.signature.parenexpected=Invalid syntax for function signature in TLD.  Parenthesis '(' expected.  Tag Library: {0}, Function: {1}.
-jsp.error.tld.mandatory.element.missing=Mandatory TLD element missing or empty: {0}
+jsp.error.tld.mandatory.element.missing=Mandatory TLD element {0} missing or empty in TLD {1} 
 jsp.error.dynamic.attributes.not.implemented=The {0} tag declares that it accepts dynamic attributes but does not implement the required interface
 jsp.error.nomatching.fragment=Cannot find an attribute directive (with name={0} and fragment=true) prior to the fragment directive.
 jsp.error.attribute.noequal=equal symbol expected



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