Repository: incubator-brooklyn
Updated Branches:
  refs/heads/master c48109a89 -> 462e5eadb


Don't display unpinned parameters

Parameters are now sorted on the server with priority == 1 if visible and null 
(received as undefined) otherwise.


Project: http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/repo
Commit: 
http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/commit/8b1d180d
Tree: http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/tree/8b1d180d
Diff: http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/diff/8b1d180d

Branch: refs/heads/master
Commit: 8b1d180d7d50b19c222d2bad58ee060f4c88c3aa
Parents: 2b23266
Author: Svetoslav Neykov <[email protected]>
Authored: Mon Nov 16 15:48:10 2015 +0200
Committer: Svetoslav Neykov <[email protected]>
Committed: Mon Nov 16 15:48:10 2015 +0200

----------------------------------------------------------------------
 .../main/webapp/assets/js/view/application-add-wizard.js | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-brooklyn/blob/8b1d180d/usage/jsgui/src/main/webapp/assets/js/view/application-add-wizard.js
----------------------------------------------------------------------
diff --git 
a/usage/jsgui/src/main/webapp/assets/js/view/application-add-wizard.js 
b/usage/jsgui/src/main/webapp/assets/js/view/application-add-wizard.js
index 77710c2..2c4f012 100644
--- a/usage/jsgui/src/main/webapp/assets/js/view/application-add-wizard.js
+++ b/usage/jsgui/src/main/webapp/assets/js/view/application-add-wizard.js
@@ -772,17 +772,12 @@ define([
                 if (catalogEntryItem!=null && catalogEntryItem.config!=null) {
                     var that = this
                     _.each(catalogEntryItem.config, function (cfg) {
-                        if (cfg.label) {
-                            configs.push( { priority: cfg.priority, html: 
_.template(RequiredConfigEntryHtml, {data:cfg}) } )
-                            // only include items with labels
+                        if (cfg.priority !== undefined) {
+                            var html = _.template(RequiredConfigEntryHtml, 
{data:cfg});
+                            that.$('.config-table').append(html)
                         }
-                        // (others might be included in future with an 
"expand" option, or priority option)
                     })
                 }
-                for (var c in configs) {
-                    that.$('.config-table').append(configs[c].html)
-                }
-                // TODO add any manual config supplied by user (in previous 
turn visiting this tab)
             }
         },
         getConfigMap:function() {

Reply via email to