DO NOT REPLY [Bug 52776] Generator's JspFragment.invoke implementation does not reset ELContext[JspContext.class]

2012-03-09 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52776

Mark Thomas  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED

--- Comment #3 from Mark Thomas  2012-03-09 11:54:50 UTC ---
Thanks for the patch.

This has been fixed in trunk and 7.0.x and will be included in 7.0.27 onwards.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



DO NOT REPLY [Bug 52776] Generator's JspFragment.invoke implementation does not reset ELContext[JspContext.class]

2012-03-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52776

--- Comment #2 from Karl von Randow  2012-03-07 22:32:19 UTC ---
Created attachment 28437
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=28437
Patch to correct the putting/resetting of the JspContext ELContext attribute

Sorry about that, I wasn't 100% sure that the solution was the right way to do
it. Specifically that it is appropriate to remove the resetting of the
JspContext attribute from the visit(Node.InvokeAction n) method.

However the visit(Node.InvokeAction n) doesn't set the JspContext attribute, so
it probably makes sense that it doesn't reset it. The set and reset is now in
the one place in FragmentHelperClass.generatePostamble().

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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



DO NOT REPLY [Bug 52776] Generator's JspFragment.invoke implementation does not reset ELContext[JspContext.class]

2012-03-07 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=52776

--- Comment #1 from Mark Thomas  2012-03-07 15:51:15 UTC ---
If you have a working solution, care to provide a patch?

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

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