-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Open
Labels
bugSomething isn't workingSomething isn't workingpriority:minorMinor loss of function, or other problem where easy workaround is presentMinor loss of function, or other problem where easy workaround is present
Description
Robert Scholte opened MNG-7194 and commented
The PluginParameterExpressionEvaluator evaluate(String,Class)-method ends like this:
if ( value instanceof String )
{
// TODO: without #, this could just be an evaluate call...
String val = (String) value;
int exprStartDelimiter = val.indexOf( "${" );
if ( exprStartDelimiter >= 0 )
{
if ( exprStartDelimiter > 0 )
{
value = val.substring( 0, exprStartDelimiter ) + evaluate( val.substring( exprStartDelimiter ) );
}
else
{
value = evaluate( val.substring( exprStartDelimiter ) );
}
}
}Assume that pre${unknown} needs to be evaluated and unknown does not exist, the result well be "prenull"
Just ${unknown} evaluates to null, but because there's no String concat it will be a real null.
Issue Links:
- MENFORCER-363 Unexpected result from requireProperty with regex when the property is not fully resolved
("causes")
Remote Links:
- GitHub Pull Request #1570
- GitHub Pull Request #1571
- GitHub Pull Request #1573
- GitHub Pull Request #1574
0 votes, 6 watchers
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingpriority:minorMinor loss of function, or other problem where easy workaround is presentMinor loss of function, or other problem where easy workaround is present