You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by ji...@codehaus.org on 2003/09/04 23:08:10 UTC
[jira] Updated: (JELLY-78) Null variables are passed over by scope inheritance
The following issue has been updated:
Updater: Morgan Delagrange (mailto:mdelagra@yahoo.com)
Date: Thu, 4 Sep 2003 4:07 PM
Changes:
Component changed to core / taglib.core
---------------------------------------------------------------------
For a full history of the issue, see:
http://jira.codehaus.org/secure/ViewIssue.jspa?key=JELLY-78&page=history
---------------------------------------------------------------------
View the issue:
http://jira.codehaus.org/secure/ViewIssue.jspa?key=JELLY-78
Here is an overview of the issue:
---------------------------------------------------------------------
Key: JELLY-78
Summary: Null variables are passed over by scope inheritance
Type: Bug
Status: Unassigned
Priority: Minor
Time Spent: Unknown
Remaining: Unknown
Project: jelly
Components:
core / taglib.core
Assignee:
Reporter: Scott Howlett
Created: Tue, 26 Aug 2003 2:08 PM
Updated: Thu, 4 Sep 2003 4:07 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://jira.codehaus.org/secure/Administrators.jspa
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira