Tomas Jelinek has submitted this change and it was merged.
Change subject: frontend: in some cases the display options are not taken from template
......................................................................
frontend: in some cases the display options are not taken from template
The problem was that first the options depending on the display
protocol has been copied from template to VM and then the display
type. It could happen, that you had a display protocol e.g. VLC
and than you have selected something which makes sense only for
SPICE, than this options have been ignored.
Fixed by moving the relevant options under the display protocol
setting.
Change-Id: I77a995eff3e800ba000cc3049eb1929d30ee91e8
Bug-Url:
https://bugzilla.redhat.com/952712
Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewTemplateVmModelBehavior.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
3 files changed, 10 insertions(+), 9 deletions(-)
Approvals:
Tomas Jelinek: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/16450
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I77a995eff3e800ba000cc3049eb1929d30ee91e8
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server