[Kimchi-devel] [PATCH] [Kimchi] Storagebuttons not behaving properly

[Socorro Stoppler]

From: Socorro <socorro at linux.vnet.ibm.com>

This patch fixes the behavior of the actions - Deactivate and Undefine -
taken on the storage pools listed in the Storage tab.  Prior to the patch,
even if the menu item was disabled, the user is still able to click on it
and perform the action.  Now clicking on those disabled menu items does not
do anything which is the intended behavior.

Signed-off-by: Socorro <socorro at linux.vnet.ibm.com>
---
 ui/js/src/kimchi.storage_main.js | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/ui/js/src/kimchi.storage_main.js b/ui/js/src/kimchi.storage_main.js
index 6c99f93..ea58012 100644
--- a/ui/js/src/kimchi.storage_main.js
+++ b/ui/js/src/kimchi.storage_main.js
@@ -310,6 +310,14 @@ kimchi.storageBindClick = function() {
         $('.pool-delete').on('click', function(event) {
             event.preventDefault();
             var $pool = $(this);
+            var in_use = $pool.data('inuse');
+            if ('active' === $pool.data('stat') || in_use) {
+                $pool.parent().addClass('disabled');
+                return false;
+            } else {
+                $pool.parent().removeClass('disabled');
+            }
+
             var settings = {
                 title : i18n['KCHAPI6001M'],
                 content : i18n['KCHPOOL6001M'],
@@ -339,6 +347,15 @@ kimchi.storageBindClick = function() {
         $('.pool-deactivate').on('click', function(event) {
             event.preventDefault();
             var poolName = $(this).data('name');
+            var $poolDeactivate = $(this);
+            var in_use = $poolDeactivate.data('inuse');
+            if (in_use) {
+                $poolDeactivate.parent().addClass('disabled');
+                return false;
+            } else {
+                $poolDeactivate.parent().removeClass('disabled');
+            }
+
             var settings = {
                 title : i18n['KCHAPI6001M'],
                 content : i18n['KCHPOOL6012M'],
-- 
2.7.4