[patch] Patch to remove variable from VariableMapper if page scope

taglibs-49526.patch (text/plain), 1.13 KB, created by Jeremy Boynes on 2010-06-29 23:48:43 UTC

(hide)

Description: Patch to remove variable from VariableMapper if page scope

Filename:

MIME Type:

Creator: Jeremy Boynes

Created: 2010-06-29 23:48:43 UTC

Size: 1.13 KB

patch

obsolete

>Index: src/main/java/org/apache/taglibs/standard/tag/common/core/SetSupport.java >=================================================================== >--- src/main/java/org/apache/taglibs/standard/tag/common/core/SetSupport.java (revision 959167) >+++ src/main/java/org/apache/taglibs/standard/tag/common/core/SetSupport.java (working copy) >@@ -128,12 +128,10 @@ > //set variable in var Mapper > vm.setVariable(var, (ValueExpression)result); > } else { >- /* >- //else if not valueExpression - make sure to remove it from the Var mapper >- //if the scope is page, should I remove this? >- if (vm.resolveVariable(var)!=null) { >+ // make sure to remove it from the VariableMapper if we will be setting into page scope >+ if (scope == PageContext.PAGE_SCOPE && vm.resolveVariable(var) != null) { > vm.setVariable(var, null); >- }*/ >+ } > pageContext.setAttribute(var, result, scope); > } > } else { View Attachment As Diff View Attachment As Raw

This is ASF Bugzilla: the Apache Software Foundation bug system. In case of problems with the functioning of ASF Bugzilla, please contact bugzilla-admin@apache.org. Please Note: this e-mail address is only for reporting problems with ASF Bugzilla. Mail about any other subject will be silently ignored.

• • Home
  • | New
  • | Browse
  • | Search
  • |
    [?]
  • | Reports
  • | Help
  • | New Account
  • | Log In
    Remember [x]
  • | Forgot Password
    Login: [x]