From e86f81152f6be723a1d8d7b486264c8ced8bc1ac Mon Sep 17 00:00:00 2001
From: Brian Federle <brian.federle@citrix.com>
Date: Fri, 22 Mar 2013 10:28:09 -0700
Subject: [PATCH] List view UI: fix blank actions column

Fixes issue where blank actions column will show if there
are header actions, but no standard actions
---
 ui/scripts/ui/widgets/listView.js | 32 +++++++++++++++++++++++++++++--
 1 file changed, 30 insertions(+), 2 deletions(-)

diff --git a/ui/scripts/ui/widgets/listView.js b/ui/scripts/ui/widgets/listView.js
index 76ce526f4d6..bb3a4249b6c 100644
--- a/ui/scripts/ui/widgets/listView.js
+++ b/ui/scripts/ui/widgets/listView.js
@@ -648,7 +648,21 @@
     }
 
     // Actions column
-    if (actions && renderActionCol(actions)) {
+    var actionsArray = $.map(actions, function(v, k) {
+      if (k == 'add') {
+        v.isAdd = true;
+      }
+
+      return v;
+    });
+    var headerActionsArray = $.grep(
+      actionsArray,
+      function(action) {
+        return action.isHeader || action.isAdd;
+      }
+    );
+
+    if (actions && renderActionCol(actions) && actionsArray.length != headerActionsArray.length) {
       $thead.find('tr').append(
         $('<th></th>')
           .html(_l('label.actions'))
@@ -1014,7 +1028,21 @@
       $tr.data('jsonObj', dataItem);
       $tr.data('list-view-action-filter', options.actionFilter);
 
-      if (actions && renderActionCol(actions)) {
+      var actionsArray = $.map(actions, function(v, k) {
+        if (k == 'add') {
+          v.isAdd = true;
+        }
+
+        return v;
+      });
+      var headerActionsArray = $.grep(
+        actionsArray,
+        function(action) {
+          return action.isHeader || action.isAdd;
+        }
+      );
+
+      if (actions && renderActionCol(actions) && actionsArray.length != headerActionsArray.length) {
         var allowedActions = $.map(actions, function(value, key) {
           return key;
         });