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(a)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(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: Shmuel Melamud <smelamud(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org