Change in ovirt-engine[master]: webadmin: Correctly restore cluster architecture selection

ofrenkel at redhat.com ofrenkel at redhat.com
Thu Jun 25 14:43:39 UTC 2015


Omer Frenkel has submitted this change and it was merged.

Change subject: webadmin: Correctly restore cluster architecture selection
......................................................................


webadmin: Correctly restore cluster architecture selection

In ClusterModel, when list of CPU architectures is populated, save the
current selection before that and restore it afterwards.

Also, implemented correct behaviour when current selection is null and
current entity is also null, which caused exception.

Change-Id: I0a838f35cf037783f14fc96f5b66f0aa3674d89b
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1223742
Signed-off-by: Shmuel Melamud <smelamud at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
1 file changed, 8 insertions(+), 6 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Martin Betak: Looks good to me, but someone else must approve
  Shmuel Leib Melamud: Verified



-- 
To view, visit https://gerrit.ovirt.org/41853
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0a838f35cf037783f14fc96f5b66f0aa3674d89b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Shmuel Melamud <smelamud at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list