[GitHub] [brooklyn-ui] algairim commented on a diff in pull request #335: brooklyn.config to take precedence before the default parameter value…

[GitBox <gi...@apache.org>]

algairim commented on code in PR #335:
URL: https://github.com/apache/brooklyn-ui/pull/335#discussion_r897616673


##########
ui-modules/utils/quick-launch/quick-launch.js:
##########
@@ -122,7 +122,7 @@ export function quickLaunchDirective() {
             if ($scope.app.config) {
                 $scope.configMap = $scope.app.config.reduce((result, config) => {
                     result[config.name] = config;
-                    const configValue = parseConfigValue(get(parsedPlan, `services[0][${BROOKLYN_CONFIG}][${config.name}]`));
+                    const configValue = parsedPlan[BROOKLYN_CONFIG] && parseConfigValue(parsedPlan[BROOKLYN_CONFIG][config.name]);

Review Comment:
   Aha! Roger that.
   I did not know how the original use-case works.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@brooklyn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org