Daniel Erez has uploaded a new change for review.

Change subject: webadmin: data domain - hide import tabs when not relevant
......................................................................

webadmin: data domain - hide import tabs when not relevant

When a data domain doesn't contain unregistered entities
(VMs/Templates), 'VM/Template import' tabs should  be
hidden.

Change-Id: Ie8c9cef939e45d701224dc228f8fd964bd06b7bf
Signed-off-by: Daniel Erez <de...@redhat.com>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
1 file changed, 7 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/32/28832/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
index 863c050..a829482 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
@@ -1067,6 +1067,11 @@
             boolean isDataCenterAvailable = storage.getStorageType() != 
StorageType.GLANCE;
             boolean isGeneralAvailable = storage.getStorageType() != 
StorageType.GLANCE;
 
+            boolean isRegsiterEntityListModelSelected =
+                    getActiveDetailModel() == vmRegisterListModel || 
getActiveDetailModel() == templateRegisterListModel;
+            boolean isRegisterSubtabsAvailable = isDataStorage &&
+                    (storage.isContainsUnregisteredEntities() || 
isRegsiterEntityListModelSelected);
+
             generalModel.setIsAvailable(isGeneralAvailable);
             dcListModel.setIsAvailable(isDataCenterAvailable);
 
@@ -1075,8 +1080,8 @@
 
             vmListModel.setIsAvailable(isDataStorage);
             templateListModel.setIsAvailable(isDataStorage);
-            vmRegisterListModel.setIsAvailable(isDataStorage);
-            templateRegisterListModel.setIsAvailable(isDataStorage);
+            vmRegisterListModel.setIsAvailable(isRegisterSubtabsAvailable);
+            
templateRegisterListModel.setIsAvailable(isRegisterSubtabsAvailable);
             diskListModel.setIsAvailable(isDataStorage);
             snapshotListModel.setIsAvailable(isDataStorage);
 


-- 
To view, visit http://gerrit.ovirt.org/28832
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie8c9cef939e45d701224dc228f8fd964bd06b7bf
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <de...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to