Tomas Jelinek has submitted this change and it was merged.
Change subject: frontend: fix custom compatibility version field setting
......................................................................
frontend: fix custom compatibility version field setting
Fix the UI behaviour of both webadmin and userportal for setting the
VM->System tab->Custom compatibility version (CCV) drop down menu to
avoid refreshing and reseting the value after manually changing it by
the user.
Changing the CCV field value for a VM/pool/template (choosing a
different value from pool down menu), causes few VM fields
values to be reseted and refreshed back to template defaults.
this fix avoid from the CCV field itself to be reseted as well.
This change influence the template field and CCV field behaviour for the
following use cases:
New VM
New VM from Template
New pool
Edit pool
Change-Id: I4e20cdb868740241338d007686adf9f8d780bb61
Bug-Url:
https://bugzilla.redhat.com/1367405
Signed-off-by: Sharon Gratch <sgratch(a)redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingPoolModelBehavior.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewPoolModelBehavior.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmFromTemplateModelBehavior.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewVmModelBehavior.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/PoolModelBehaviorBase.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UserPortalNewVmModelBehavior.java
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
8 files changed, 65 insertions(+), 15 deletions(-)
Approvals:
Tomas Jelinek: Looks good to me, approved
Jenkins CI: Passed CI tests
Sharon Gratch: Verified
--
To view, visit
https://gerrit.ovirt.org/65809
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4e20cdb868740241338d007686adf9f8d780bb61
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sharon Gratch <sgratch(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Sharon Gratch <sgratch(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>