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 <tni...@redhat.com> --- 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/56/32956/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 e1ebdae..cb3a97b 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 @@ -1236,6 +1236,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 b857c44..1c243e2 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; @@ -216,6 +217,7 @@ for (StorageDomain domain : domains) { 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/32956 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I1af7d25f10a80cf9106007fa4b1f85cd9ffb9e3a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Tal Nisan <tni...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
