kinman 2003/01/13 15:50:48 Modified: jasper2/src/share/org/apache/jasper/compiler Parser.java jasper2/src/share/org/apache/jasper/resources messages.properties messages_es.properties messages_fr.properties messages_ja.properties Log: - Fix 15851, 16027, and 1600: output appropiate message when a jsp:attribute or jsp:body is nested in another jsp:attribute or jsp:body. Revision Changes Path 1.56 +14 -9 jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/compiler/Parser.java Index: Parser.java =================================================================== RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/compiler/Parser.java,v retrieving revision 1.55 retrieving revision 1.56 diff -u -r1.55 -r1.56 --- Parser.java 11 Jan 2003 00:51:36 -0000 1.55 +++ Parser.java 13 Jan 2003 23:50:48 -0000 1.56 @@ -1568,10 +1568,6 @@ err.jspError( reader.mark(), "jsp.error.not.in.template", "Expression language" ); } else if (reader.matches("<jsp:")) { - if ((parent instanceof Node.NamedAttribute) && - reader.matches("attribute")) { - err.jspError(reader.mark(), "jsp.error.nested.jspattribute"); - } err.jspError( reader.mark(), "jsp.error.not.in.template", "Standard actions" ); } else if (parseCustomTag(parent)) { @@ -1651,6 +1647,16 @@ return; } + // Check for nested jsp:body or jsp:attribute + if (tag.equals("jsp:body") || tag.equals("jsp:attribute")) { + if (reader.matches("<jsp:attribute")) { + err.jspError(reader.mark(), "jsp.error.nested.jspattribute"); + } + else if (reader.matches("<jsp:body")) { + err.jspError(reader.mark(), "jsp.error.nested.jspbody"); + } + } + if( bodyType.equalsIgnoreCase( TagInfo.BODY_CONTENT_JSP ) ) { parseElements( parent ); } @@ -1696,8 +1702,7 @@ reader.skipSpaces(); } parseBody(namedAttributeNode, "jsp:attribute", - getAttributeBodyType(parent, - attrs.getValue("name"))); + TagInfo.BODY_CONTENT_JSP); if (namedAttributeNode.isTrim()) { Node.Nodes subElems = namedAttributeNode.getBody(); if (subElems != null) { 1.79 +2 -1 jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/resources/messages.properties Index: messages.properties =================================================================== RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/resources/messages.properties,v retrieving revision 1.78 retrieving revision 1.79 diff -u -r1.78 -r1.79 --- messages.properties 11 Jan 2003 00:11:06 -0000 1.78 +++ messages.properties 13 Jan 2003 23:50:48 -0000 1.79 @@ -351,3 +351,4 @@ jsp.error.plugin.wrongRootElement=Name of root element in {0} different from {1} jsp.error.attribute.invalidPrefix=The attribute prefix {0} does not correspond to any imported tag library jsp.error.nested.jspattribute=a jsp:attribute standard action cannot be nested within another jsp:attribute standard action +jsp.error.nested.jspbody=a jsp:body standard action cannot be nested within another jsp:body or jsp:attribute standard action 1.28 +2 -1 jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/resources/messages_es.properties Index: messages_es.properties =================================================================== RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/resources/messages_es.properties,v retrieving revision 1.27 retrieving revision 1.28 diff -u -r1.27 -r1.28 --- messages_es.properties 11 Jan 2003 00:11:06 -0000 1.27 +++ messages_es.properties 13 Jan 2003 23:50:48 -0000 1.28 @@ -259,3 +259,4 @@ jsp.error.xml.invalidHighSurrogate= jsp.error.attributes.not.allowed= jsp.error.nested.jspattribute= +jsp.error.nested.jspbody= 1.11 +2 -1 jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/resources/messages_fr.properties Index: messages_fr.properties =================================================================== RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/resources/messages_fr.properties,v retrieving revision 1.10 retrieving revision 1.11 diff -u -r1.10 -r1.11 --- messages_fr.properties 11 Jan 2003 00:11:06 -0000 1.10 +++ messages_fr.properties 13 Jan 2003 23:50:48 -0000 1.11 @@ -298,3 +298,4 @@ jsp.error.useBean.noSession=Il est illégal pour useBean d''utiliser une portée de session (session scope) quand la page JSP indique (via la directive de page) qu''elle ne participe pas aux sessions jsp.error.attributes.not.allowed = {0} ne doit avoir aucun attribut jsp.error.nested.jspattribute= +jsp.error.nested.jspbody= 1.28 +2 -1 jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/resources/messages_ja.properties Index: messages_ja.properties =================================================================== RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/resources/messages_ja.properties,v retrieving revision 1.27 retrieving revision 1.28 diff -u -r1.27 -r1.28 --- messages_ja.properties 11 Jan 2003 00:11:06 -0000 1.27 +++ messages_ja.properties 13 Jan 2003 23:50:48 -0000 1.28 @@ -290,3 +290,4 @@ jsp.error.xml.invalidHighSurrogate= jsp.error.attributes.not.allowed= jsp.error.nested.jspattribute= +jsp.error.nested.jspbody=
-- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>