diff options
author | Christian Kandeler <christian.kandeler@qt.io> | 2016-11-02 15:29:44 +0100 |
---|---|---|
committer | Christian Kandeler <christian.kandeler@qt.io> | 2016-11-03 10:25:58 +0000 |
commit | 3aea73da761be9126d92c10336341b14996b42cc (patch) | |
tree | bce6b97435ddc72da13e1b1fbd768683ff3f2691 /share | |
parent | 794c88ff8570fc14502c5c716ba22bdf99c4aa9a (diff) |
ModUtils: Deal sensibly with undefined values
Change-Id: Id51424820249b05e327f8a709ab716359529131d
Reviewed-by: Jake Petroules <jake.petroules@qt.io>
Diffstat (limited to 'share')
-rw-r--r-- | share/qbs/imports/qbs/ModUtils/utils.js | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/share/qbs/imports/qbs/ModUtils/utils.js b/share/qbs/imports/qbs/ModUtils/utils.js index 2eabde78a..30302e111 100644 --- a/share/qbs/imports/qbs/ModUtils/utils.js +++ b/share/qbs/imports/qbs/ModUtils/utils.js @@ -163,9 +163,14 @@ function languagePropertyName(propertyName, fileTag) { } function modulePropertiesFromArtifacts(product, artifacts, moduleName, propertyName, langFilter) { - var result = product.moduleProperty(moduleName, languagePropertyName(propertyName, langFilter)) - for (var i in artifacts) - result = result.concat(artifacts[i].moduleProperty(moduleName, languagePropertyName(propertyName, langFilter))) + var result = product.moduleProperty( + moduleName, languagePropertyName(propertyName, langFilter)) || []; + for (var i in artifacts) { + var artifactProp = artifacts[i].moduleProperty( + moduleName, languagePropertyName(propertyName, langFilter)); + if (artifactProp) + result = result.concat(artifactProp); + } return sanitizedList(result, product, moduleName + "." + propertyName); } |