You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by co...@jakarta.apache.org on 2004/08/27 06:46:53 UTC

[jira] Assigned: (JELLY-78) Null variables are passed over by scope inheritance

Message:

   The following issue has been re-assigned.

   Assignee: dion gillard (mailto:dion@apache.org)
---------------------------------------------------------------------
View the issue:
  http://issues.apache.org/jira/browse/JELLY-78

Here is an overview of the issue:
---------------------------------------------------------------------
        Key: JELLY-78
    Summary: Null variables are passed over by scope inheritance
       Type: Bug

     Status: Open
   Priority: Minor

    Project: jelly
 Components: 
             core / taglib.core
   Fix Fors:
             1.0-beta-4

   Assignee: dion gillard
   Reporter: Scott Howlett

    Created: Tue, 26 Aug 2003 2:08 PM
    Updated: Thu, 26 Aug 2004 9:45 PM

Description:
If I explicitly assign variable foo a null value in the current scope but it has a value in some parent scope, findVariable() will bypass the null value in favor of the non-null value up the chain.

This is because JellyContext.getVariable & findVariable just test the result of variables.get(name) against null instead of also checking with containsKey like this:

       Object value = variables.get(name);
       if ( value == null
         && !variables.containsKey(name)
         && isInherit() ) {

....


Perhaps this is an explicit design decision, but I thought it was worth mentioning.



---------------------------------------------------------------------
JIRA INFORMATION:
This message is automatically generated by JIRA.

If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa

If you want more information on JIRA, or have a bug to report see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org