Re: [PATCH][JASPER2] Simple patch for one of the resource error messages.

2002-11-24 Thread Mark Roth
Ryan,

That's correct - according to the JSP 2.0 specification, directives (%@ 
... %) are allowed in scriptless bodies (see JSP.1.3.10.1).

- Mark


Ryan Lubke wrote:

I think that the following error message is incorrect:

 - Scripting elements ( %@, %!, %=, % ) are disallowed here.

Since directives (%@) are not scripting elements.

I've attached a patch to correct the 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.60
diff -u -r1.60 messages.properties
--- messages.properties	16 Nov 2002 04:20:10 -	1.60
+++ messages.properties	20 Nov 2002 23:47:23 -
@@ -266,7 +266,7 @@
 jsp.error.empty.body.not.allowed=Empty body not allowed for {0}
 jsp.error.jspbody.required=Must use jsp:body to specify tag body for 
{0} if jsp:attribute is used.
 jsp.error.jspbody.emptybody.only=The {0} tag can only have 
jsp:attribute in its body.
-jsp.error.no.scriptlets=Scripting elements ( %@, %!, %=, % ) are 
disallowed here.
+jsp.error.no.scriptlets=Scripting elements ( %!, %=, % ) are 
disallowed here.
 jsp.error.internal.unexpected_node_type=Internal Error: Unexpected 
node type encountered
 jsp.error.tld.fn.invalid.signature=Invalid syntax for function 
signature in TLD.  Tag Library: {0}, Function: {1}
 jsp.error.tld.fn.duplicate.name=Duplicate function name {0} in tag 
library {1}





--
To unsubscribe, e-mail:
For additional commands, e-mail: 




--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: [PATCH][JASPER2] Simple patch for one of the resource error messages.

2002-11-22 Thread Jan Luehe
Ryan,

 I think that the following error message is incorrect:
 
  - Scripting elements ( %@, %!, %=, % ) are disallowed here.
 
 Since directives (%@) are not scripting elements.
 
 I've attached a patch to correct the messages.properties.

Thanks for your patch!

I've slightly modified it to also include the corresponding XML elements
for scripting (jsp:declaration, jsp:expression, and jsp:scriptlet).


Jan


--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]