From 9aed0ad63a20f3847e7c5bc0873e6e1d182e3b2c Mon Sep 17 00:00:00 2001
From: Joe Fiorini <joe@joefiorini.com>
Date: Wed, 11 Mar 2015 14:57:50 -0400
Subject: [PATCH] Extract list actions to directive

---
 awx/ui/static/js/shared/generator-helpers.js  |  4 +
 .../list-generator/list-generator.factory.js  | 93 ++++++++++++-------
 .../static/js/shared/list-generator/main.js   |  7 +-
 .../toolbar-button.directive.js               | 39 ++++++++
 4 files changed, 107 insertions(+), 36 deletions(-)
 create mode 100644 awx/ui/static/js/shared/list-generator/toolbar-button.directive.js

diff --git a/awx/ui/static/js/shared/generator-helpers.js b/awx/ui/static/js/shared/generator-helpers.js
index 061503300b..1c8f67c372 100644
--- a/awx/ui/static/js/shared/generator-helpers.js
+++ b/awx/ui/static/js/shared/generator-helpers.js
@@ -188,9 +188,12 @@ angular.module('GeneratorHelpers', [systemStatus.name])
 
             if (toolbar) {
                 //if this is a toolbar button, set some defaults
+                console.log('toolbar button');
                 btn.class = 'btn-xs btn-primary';
                 btn.iconSize = 'fa-lg';
                 delete btn.label;
+            } else {
+                console.log('NOT A TOOLBAR BUTTON');
             }
 
             html += "<button type=\"button\" ";
@@ -202,6 +205,7 @@ angular.module('GeneratorHelpers', [systemStatus.name])
                 html += " btn-sm";
             }
 
+            console.log('btn:', btn);
             html += (btn.awPopOver) ? " help-link-white" : "";
             html += "\" ";
             html += (btn.ngClick) ? Attr(btn, 'ngClick') : "";
diff --git a/awx/ui/static/js/shared/list-generator/list-generator.factory.js b/awx/ui/static/js/shared/list-generator/list-generator.factory.js
index f88c4d2c6a..88852cb086 100644
--- a/awx/ui/static/js/shared/list-generator/list-generator.factory.js
+++ b/awx/ui/static/js/shared/list-generator/list-generator.factory.js
@@ -71,6 +71,40 @@ export default ['$location', '$compile', '$rootScope', 'SearchWidget', 'Paginate
                     }
 
                     this.scope.list = list;
+                    this.scope.mode = options.mode;
+
+                    this.scope.performAction = function(action) {
+                        console.log('performing:', action);
+                        this.scope.$eval(action);
+                    }.bind(this);
+
+                    this.scope.shouldHideAction = function(options) {
+                        return _.tap(this.scope.$eval(options.ngHide), function(value) { console.log('shouldHide:', value, this.scope.selected_group_id, options); }.bind(this));
+                    }.bind(this);
+                    this.scope.canShowAction = function(action) {
+                        var base = $location.path().replace(/^\//, '').split('/')[0];
+                        var inActionMode = options.mode === action.mode || action.mode === 'all';
+                        var onScreenForAction =
+                            (!options.basePaths) ||
+                                (options.basePaths.indexOf(base) > -1);
+                        var scopeShow = action.ngShow ? this.scope.$eval(action.ngShow) : true;
+
+                        if (this.scope.shouldHideAction(action)) {
+                            return false;
+                        } else if (!scopeShow) {
+                            console.log('scopeShow false', action.ngShow);
+                            return false;
+                        } else {
+                            return _.tap(inActionMode && onScreenForAction, function(value) {
+                                console.log('mode ', value, action);
+                            });
+                        }
+
+                        // return _.tap(scopeShow || , function(value) {
+                        //     console.log('canShow:', value, options.mode, action.mode, scopeShow);
+                        // });
+                    }.bind(this);
+
 
                     $compile(element)(this.scope);
 
@@ -203,7 +237,6 @@ export default ['$location', '$compile', '$rootScope', 'SearchWidget', 'Paginate
 
                         if (options.mode !== 'lookup') {
                             //actions
-                            base = $location.path().replace(/^\//, '').split('/')[0];
                             html += "<div class=\"";
                             if (options.searchSize && !options.listSize) {
                                 // User supplied searchSize, calc the remaining
@@ -221,42 +254,34 @@ export default ['$location', '$compile', '$rootScope', 'SearchWidget', 'Paginate
                             }
                             html += "\">\n";
 
-                            html += "<div class=\"list-actions\">\n";
+            html += "<div class=\"list-actions\">\n";
+            html += '<span ng-repeat="(name, options) in list.actions">';
+            html += '<toolbar-button name="name" options="options"  on-selected="performAction(action)" ng-show="canShowAction(options)"></toolbar-button>';
+            html += '</span>';
 
-                            // Add toolbar buttons or 'actions'
-                            for (action in list.actions) {
-                                if (list.actions[action].mode === 'all' || list.actions[action].mode === options.mode) {
-                                    if ((list.actions[action].basePaths === undefined) ||
-                                        (list.actions[action].basePaths && list.actions[action].basePaths.indexOf(base) > -1)) {
-                                        html += this.button({
-                                            btn: list.actions[action],
-                                            action: action,
-                                            toolbar: true
-                                        });
-                                    }
-                                }
-                            }
+            //select instructions
+            if (options.mode === 'select' && list.selectInstructions) {
+                btn = {
+                    awPopOver: list.selectInstructions,
+                    dataPlacement: 'left',
+                    dataContainer: 'body',
+                    'class': 'btn-xs btn-help',
+                    awToolTip: 'Click for help',
+                    dataTitle: 'Help',
+                    iconSize: 'fa-lg'
+                };
+                //html += this.button(btn, 'select');
+                html += this.button({
+                    btn: btn,
+                    action: 'help',
+                    toolbar: true
+                });
+            }
 
-                            //select instructions
-                            if (options.mode === 'select' && list.selectInstructions) {
-                                btn = {
-                                    awPopOver: list.selectInstructions,
-                                    dataPlacement: 'left',
-                                    dataContainer: 'body',
-                                    'class': 'btn-xs btn-help',
-                                    awToolTip: 'Click for help',
-                                    dataTitle: 'Help',
-                                    iconSize: 'fa-lg'
-                                };
-                                //html += this.button(btn, 'select');
-                                html += this.button({
-                                    btn: btn,
-                                    action: 'help',
-                                    toolbar: true
-                                });
-                            }
 
-                            html += "</div><!-- list-acitons -->\n";
+
+            html += "</div><!-- list-acitons -->\n";
+
                             html += "</div><!-- list-actions-column -->\n";
                         } else {
                             //lookup
diff --git a/awx/ui/static/js/shared/list-generator/main.js b/awx/ui/static/js/shared/list-generator/main.js
index dba9519d25..1af299d520 100644
--- a/awx/ui/static/js/shared/list-generator/main.js
+++ b/awx/ui/static/js/shared/list-generator/main.js
@@ -1,5 +1,8 @@
 import generateList from './list-generator.factory';
+import toolbarButton from './toolbar-button.directive';
+import generatorHelpers from 'tower/shared/generator-helpers';
 
 export default
-    angular.module('listGenerator', [])
-        .factory('generateList', generateList);
+    angular.module('listGenerator', [generatorHelpers.name])
+        .factory('generateList', generateList)
+        .directive('toolbarButton', toolbarButton);
diff --git a/awx/ui/static/js/shared/list-generator/toolbar-button.directive.js b/awx/ui/static/js/shared/list-generator/toolbar-button.directive.js
new file mode 100644
index 0000000000..5b0e0288db
--- /dev/null
+++ b/awx/ui/static/js/shared/list-generator/toolbar-button.directive.js
@@ -0,0 +1,39 @@
+export default ['$compile', 'Button', function($compile, Button) {
+    return {
+        restrict: 'E',
+        scope: {
+            name: '=',
+            options: '=',
+            onSelected: '&'
+        },
+        link: function(scope, element, attrs) {
+            var html = '';
+
+            // Save the ngClick property from
+            // legacy list actions
+            scope.action = scope.options.ngClick;
+
+
+            var btnOptions = _.clone(scope.options);
+            btnOptions.ngClick = "onSelected({ action: action })";
+
+            // These should be taken care of by
+            // using ng-show & ng-hide on this
+            // directive
+            delete btnOptions.ngHide;
+            delete btnOptions.ngShow;
+
+            // console.log('options:', scope.options);
+
+            html += Button({
+                btn: btnOptions,
+                action: scope.name,
+                toolbar: true
+            });
+
+            element.html(html);
+
+            $compile(element.contents())(scope);
+        }
+    };
+}];