Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: incorrect DC/Cluster handling on new pool ...................................................................... frontend: incorrect DC/Cluster handling on new pool Have one DC with two clusters. Make a template on the first cluster. Go to the new pool dialog and select the second cluster - it gets automatically reset to the first one. The problem was that the logic of selecting the DC/Cluster was implemented in the common place for new and edit dialog. For edit dialog it selected the DC/Cluster according to the selected pool. For new pool according to the selected template. While for edit pool it made sense to select the DC/Cluster according to edited pool it made no sense for new pool to select the DC/Cluster according to the selected template. Fixed by moving the logic specific for edit pool to the ExistingPoolModelBehavior from PoolModelBehaviorBase. Change-Id: I38e771d88fc300304942f750b6628862279a02ad Bug-Url: https://bugzilla.redhat.com/1002902 Signed-off-by: Tomas Jelinek <tjelinek@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/PoolModelBehaviorBase.java 3 files changed, 12 insertions(+), 12 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/18713 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38e771d88fc300304942f750b6628862279a02ad Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server