Change in ovirt-engine[master]: frontend: in some cases the display options are not taken fr...

[tjelinek at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server