luehe 2002/08/19 14:16:47
Modified: jasper2/src/share/org/apache/jasper/compiler Tag:
tomcat_4_branch ScriptingVariabler.java
Log:
Fixed bug which caused VariableInfos to be ignored when determining which scripting
vars need to be declared
Revision Changes Path
No revision
No revision
1.1.2.2 +20 -20
jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/compiler/ScriptingVariabler.java
Index: ScriptingVariabler.java
===================================================================
RCS file:
/home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/compiler/ScriptingVariabler.java,v
retrieving revision 1.1.2.1
retrieving revision 1.1.2.2
diff -u -r1.1.2.1 -r1.1.2.2
--- ScriptingVariabler.java 17 Aug 2002 00:14:23 -0000 1.1.2.1
+++ ScriptingVariabler.java 19 Aug 2002 21:16:46 -0000 1.1.2.2
@@ -130,18 +130,14 @@
else
ownRange = parent.getNumCount();
- if (tagVarInfos != null) {
- for (int i=0; i<tagVarInfos.length; i++) {
- if (tagVarInfos[i].getScope() != scope
- || !tagVarInfos[i].getDeclare()) {
+ if (varInfos != null) {
+ for (int i=0; i<varInfos.length; i++) {
+ if (varInfos[i].getScope() != scope
+ || !varInfos[i].getDeclare()) {
continue;
}
- String varName = tagVarInfos[i].getNameGiven();
- if (varName == null) {
- varName = n.getTagData().getAttributeString(
- tagVarInfos[i].getNameFromAttribute());
- }
-
+ String varName = varInfos[i].getVarName();
+
if (scope == VariableInfo.AT_BEGIN
|| scope == VariableInfo.AT_END
|| (scope == VariableInfo.NESTED
@@ -151,22 +147,26 @@
if (currentRange == null
|| ownRange.compareTo(currentRange) > 0) {
scriptVars.put(varName, ownRange);
- vec.add(tagVarInfos[i]);
+ vec.add(varInfos[i]);
}
} else {
// scope equals NESTED AND node implements BodyTag
if (n.getCustomNestingLevel() == 0) {
- vec.add(tagVarInfos[i]);
+ vec.add(varInfos[i]);
}
}
}
} else {
- for (int i=0; i<varInfos.length; i++) {
- if (varInfos[i].getScope() != scope
- || !varInfos[i].getDeclare()) {
+ for (int i=0; i<tagVarInfos.length; i++) {
+ if (tagVarInfos[i].getScope() != scope
+ || !tagVarInfos[i].getDeclare()) {
continue;
}
- String varName = varInfos[i].getVarName();
+ String varName = tagVarInfos[i].getNameGiven();
+ if (varName == null) {
+ varName = n.getTagData().getAttributeString(
+ tagVarInfos[i].getNameFromAttribute());
+ }
if (scope == VariableInfo.AT_BEGIN
|| scope == VariableInfo.AT_END
@@ -177,7 +177,7 @@
if (currentRange == null
|| ownRange.compareTo(currentRange) > 0) {
scriptVars.put(varName, ownRange);
- vec.add(varInfos[i]);
+ vec.add(tagVarInfos[i]);
}
} else {
// scope equals NESTED AND node implements BodyTag
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
