anmolbabu has uploaded a new change for review. Change subject: engine : In the bricks sub-tab for a server, sort the bricks ......................................................................
engine : In the bricks sub-tab for a server, sort the bricks In the bricks sub-tab for a server, sort the bricks Change-Id: Ifeecb7bb6d020c21167801573bb733aace7e7d9f Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1024649 Signed-off-by: Anmol Babu <[email protected]> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterBrickEntity.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostBricksListModel.java 2 files changed, 10 insertions(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/81/20781/1 diff --git a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterBrickEntity.java b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterBrickEntity.java index 771aefd..7cf85f7 100644 --- a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterBrickEntity.java +++ b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterBrickEntity.java @@ -1,5 +1,8 @@ package org.ovirt.engine.core.common.businessentities.gluster; +import java.io.Serializable; +import java.util.Comparator; + import javax.validation.constraints.NotNull; import org.ovirt.engine.core.common.asynctasks.gluster.GlusterAsyncTask; @@ -20,7 +23,7 @@ * @see GlusterVolumeEntity * @see GlusterBrickStatus */ -public class GlusterBrickEntity extends IVdcQueryable implements BusinessEntityWithStatus<Guid, GlusterStatus>, GlusterTaskSupport { +public class GlusterBrickEntity extends IVdcQueryable implements BusinessEntityWithStatus<Guid, GlusterStatus>, GlusterTaskSupport, Comparator<GlusterBrickEntity>, Serializable{ private static final long serialVersionUID = 7119439284741452278L; @NotNull(message = "VALIDATION.GLUSTER.BRICK.ID.NOT_NULL", groups = { RemoveBrick.class }) @@ -206,4 +209,8 @@ this.volumeName = volumeName; } + @Override + public int compare(GlusterBrickEntity arg0, GlusterBrickEntity arg1) { + return arg0.getVolumeName().compareTo(arg1.getVolumeName()); + } } diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostBricksListModel.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostBricksListModel.java index 458f3fe..2e6ba86 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostBricksListModel.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostBricksListModel.java @@ -1,5 +1,6 @@ package org.ovirt.engine.ui.uicommonweb.models.hosts; +import java.util.Collections; import java.util.List; import org.ovirt.engine.core.common.businessentities.VDS; @@ -70,6 +71,7 @@ @Override public void onSuccess(Object model, Object returnValue) { List<GlusterBrickEntity> glusterBricks = (List<GlusterBrickEntity>) returnValue; + Collections.sort(glusterBricks, new GlusterBrickEntity()); setItems(glusterBricks); } }), getEntity().getId()); -- To view, visit http://gerrit.ovirt.org/20781 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ifeecb7bb6d020c21167801573bb733aace7e7d9f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: anmolbabu <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
