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 <tjeli...@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 <tjeli...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjeli...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to