Tal Nisan has uploaded a new change for review.

Change subject: webadmin: Added storage type column to disks table
......................................................................

webadmin: Added storage type column to disks table

Change-Id: I7c9566ee36967f587a2fee31c6db7aa40dcc2955
Signed-off-by: Tal Nisan <tni...@redhat.com>
Relates-To: https://bugzilla.redhat.com/1038053
---
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/disks/DisksViewColumns.java
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/BaseVmDiskListModelTable.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/storage/SubTabStorageDiskView.java
4 files changed, 22 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/03/23503/1

diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java
index b1bb781..eed3eb6 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/CommonApplicationConstants.java
@@ -825,6 +825,9 @@
     @DefaultStringValue("Storage Domain")
     String storageDomainDisk();
 
+    @DefaultStringValue("Storage Type")
+    String storageTypeDisk();
+
     @DefaultStringValue("Storage Domain(s)")
     String storageDomainsDisk();
 
diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/disks/DisksViewColumns.java
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/disks/DisksViewColumns.java
index 457af5e..6d35bbd 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/disks/DisksViewColumns.java
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/disks/DisksViewColumns.java
@@ -8,6 +8,7 @@
 import org.ovirt.engine.core.common.businessentities.DiskInterface;
 import org.ovirt.engine.core.common.businessentities.ImageStatus;
 import org.ovirt.engine.core.common.businessentities.LunDisk;
+import org.ovirt.engine.core.common.businessentities.StorageType;
 import org.ovirt.engine.core.common.businessentities.VolumeType;
 import org.ovirt.engine.core.common.utils.SizeConverter;
 import org.ovirt.engine.ui.common.CommonApplicationConstants;
@@ -154,6 +155,18 @@
 
     public static final StorageDomainsColumn storageDomainsColumn = new 
StorageDomainsColumn();
 
+    public static final TextColumnWithTooltip<Disk> storageTypeColumn = new 
EnumColumn<Disk, StorageType>() {
+        @Override
+        protected StorageType getRawValue(Disk object) {
+            if (object.getDiskStorageType() != DiskStorageType.IMAGE) {
+                return null;
+            }
+            DiskImage disk = (DiskImage) object;
+
+            return disk.getStorageTypes().isEmpty() ? null : 
disk.getStorageTypes().get(0);
+        }
+    };
+
     public static final DiskSizeColumn<Disk> sizeColumn = new 
DiskSizeColumn<Disk>() {
         @Override
         protected Long getRawValue(Disk object) {
diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/BaseVmDiskListModelTable.java
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/BaseVmDiskListModelTable.java
index 3dab2f0..c90eb8a 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/BaseVmDiskListModelTable.java
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/vm/BaseVmDiskListModelTable.java
@@ -109,6 +109,9 @@
                 DisksViewColumns.storageDomainsColumn, 
constants.storageDomainDisk(), images, "125px"); //$NON-NLS-1$
 
         getTable().ensureColumnPresent(
+                DisksViewColumns.storageTypeColumn, 
constants.storageTypeDisk(), images, "100px"); //$NON-NLS-1$
+
+        getTable().ensureColumnPresent(
                 DisksViewColumns.dateCreatedColumn, 
constants.creationDateDisk(), images, "120px"); //$NON-NLS-1$
 
         getTable().ensureColumnPresent(
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/storage/SubTabStorageDiskView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/storage/SubTabStorageDiskView.java
index 9714785..66ee7aa 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/storage/SubTabStorageDiskView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/storage/SubTabStorageDiskView.java
@@ -61,6 +61,9 @@
                 DisksViewColumns.storageDomainsColumn, 
constants.storageDomainDisk(), true, "170px"); //$NON-NLS-1$
 
         getTable().ensureColumnPresent(
+                DisksViewColumns.storageTypeColumn, 
constants.storageDomainDisk(), true, "100px"); //$NON-NLS-1$
+
+        getTable().ensureColumnPresent(
                 DisksViewColumns.dateCreatedColumn, 
constants.creationDateDisk(), true, "150px"); //$NON-NLS-1$
 
         getTable().ensureColumnPresent(


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

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

Reply via email to