[Bug 56516] VariableInfo doesn't handle scope changes correctly

2021-09-21 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=56516 --- Comment #8 from Mark Thomas --- It has been a long time comping but the move to Eclipse and Jakarta EE means I can now propose changes to the JSP spec to address this. Given the circumstances, the proposed changes are very modest and aim t

[Bug 56516] VariableInfo doesn't handle scope changes correctly

2014-05-16 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56516 --- Comment #4 from Mark Thomas --- (In reply to Konstantin Kolinko from comment #3) > Re: r1594715 > > I think this report is invalid. I'm leaning towards that conclusion. > For 2.,3. declaration of variables is illustrated in ch.JSP.

[Bug 56516] VariableInfo doesn't handle scope changes correctly

2014-05-16 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56516 --- Comment #7 from Remy Maucherat --- I also think this is invalid, although I am not the biggest expert on JSPs. The Javadoc for scopes says: /** * Scope information that scripting variable is visible only within the * star

[Bug 56516] VariableInfo doesn't handle scope changes correctly

2014-05-16 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56516 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 56516] VariableInfo doesn't handle scope changes correctly

2014-05-16 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56516 --- Comment #6 from ad...@moparisthebest.com --- I also tend to agree that declaring the variable once at the beginning *probably* isn't the right solution, however I think the way this is currently handled is still wrong. Right now the var

[Bug 56516] VariableInfo doesn't handle scope changes correctly

2014-05-15 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56516 --- Comment #2 from Mark Thomas --- This has been fixed in 8.0.x for 8.0.7 onwards. Given the history of instability around changes in the this area I plan to give folks a chance to test and review this fix in 8.0.x before I back-port it to

[Bug 56516] VariableInfo doesn't handle scope changes correctly

2014-05-15 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56516 --- Comment #3 from Konstantin Kolinko --- Re: r1594715 I think this report is invalid. In JSP specification (JSP2.3MR.pdf) there are three occurrences where an action defines a variable: 1. A action (chapter JSP.5.1) 2. A custom action

[Bug 56516] VariableInfo doesn't handle scope changes correctly

2014-05-13 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56516 --- Comment #1 from Konstantin Kolinko --- Do you expect a variable to be declared each time when VariableInfo.getDeclare() returns true? When JSP java code is generated the java scriptlets are treated as opaque. One does not analyze what c