We're updating the issue view to help you get more done. 

AbstractCacheProvider.getOrSet(): local var unscoped when checking if null

Description

1 2 3 4 var target = get( arguments.objectKey ); if( !isNull( target ) ){ // here is the problem return target; }

The issue here is that, if Lucee/Coldfusion scope cascading is on, then when 'target' is NULL, the engine will look up the scopes for it. If you have, for instance, FORM.target, due to submitting a link building form for example (https://presidecms.atlassian.net/browse/PRESIDECMS-1634), you'll get an unexpected result out of the cache.

Should use:

 

1 2 3 if ( !isNull( local.target ) ) { return target; }

 

Status

Assignee

Luis Majano

Reporter

Dominic Watson

Labels

None

Fix versions

Priority

Major