Tal Nisan has uploaded a new change for review. Change subject: webadmin: Data center list of a storage domain should be sorted by DC name ......................................................................
webadmin: Data center list of a storage domain should be sorted by DC name Change-Id: I1af7d25f10a80cf9106007fa4b1f85cd9ffb9e3a Related-To: https://bugzilla.redhat.com/1117228 Signed-off-by: Tal Nisan <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java 2 files changed, 12 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/32/32932/1 diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java index d2337eb..a4549dc 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java @@ -1237,6 +1237,16 @@ } } + public final static class StorageDomainByPoolNameComparator implements Comparator<StorageDomain>, Serializable { + private static final long serialVersionUID = 990203400356561666L; + private LexoNumericComparator lexoNumeric = new LexoNumericComparator(); + + @Override + public int compare(StorageDomain domain1, StorageDomain domain2) { + return lexoNumeric.compare(domain1.getStoragePoolName(), domain2.getStoragePoolName()); + } + } + public final static class VDSGroupComparator implements Comparator<VDSGroup>, Serializable { private static final long serialVersionUID = 990203400356561587L; private LexoNumericComparator lexoNumeric = new LexoNumericComparator(); diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java index 0251852..1e9c349 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java @@ -1,6 +1,7 @@ package org.ovirt.engine.ui.uicommonweb.models.storage; import java.util.ArrayList; +import java.util.Collections; import java.util.List; import org.ovirt.engine.core.common.VdcActionUtils; @@ -220,6 +221,7 @@ : Guid.Empty.toString(); domain.setId(domain.getStoragePoolId()); } + Collections.sort(domains, new Linq.StorageDomainByPoolNameComparator()); searchableListModel.setItems(domains); setIsEmpty(((List) searchableListModel.getItems()).size() == 0); } -- To view, visit http://gerrit.ovirt.org/32932 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I1af7d25f10a80cf9106007fa4b1f85cd9ffb9e3a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
