Do not convert falsey env vars to blank string

Only blank out undefined as that's what we've always done
This commit is contained in:
Nick O'Leary 2019-02-04 14:12:34 +00:00
parent e843f192ec
commit db3eee72b5
No known key found for this signature in database
GPG Key ID: 4F2157149161A6C9
2 changed files with 6 additions and 6 deletions

View File

@ -38,7 +38,6 @@ class Subflow extends Flow {
* @param {[type]} subflowInstance [description]
*/
constructor(parent,globalFlow,subflowDef,subflowInstance) {
console.log(subflowDef);
// console.log("CREATE SUBFLOW",subflowDef.id,subflowInstance.id);
// console.log("SubflowInstance\n"+JSON.stringify(subflowInstance," ",2));
// console.log("SubflowDef\n"+JSON.stringify(subflowDef," ",2));

View File

@ -442,22 +442,23 @@ function getSetting(node, name) {
* @memberof @node-red/util_util
*/
function evaluateEnvProperty(value, node) {
var result;
if (/^\${[^}]+}$/.test(value)) {
// ${ENV_VAR}
var name = value.substring(2,value.length-1);
var val = getSetting(node, name);
return val ? val : "";
result = getSetting(node, name);
} else if (!/\${\S+}/.test(value)) {
// ENV_VAR
var val = getSetting(node, value);
return val ? val : "";
result = getSetting(node, value);
} else {
// FOO${ENV_VAR}BAR
return value.replace(/\${([^}]+)}/g, function(match, name) {
var val = getSetting(node, name);
return (val ? val : "");
return (val === undefined)?"":val;
});
}
return (result === undefined)?"":result;
}