Tomas Jelinek has uploaded a new change for review.

Change subject: userportal,webadmin: Changing cluster of existing VM changes OS 
type
......................................................................

userportal,webadmin: Changing cluster of existing VM changes OS type

Fixed by selecting the already selected OS type if it is still applicable under
the context of the newly selected cluster.

Change-Id: Ifaf27934e200e69beeb0912db22f26525b85e9b1
Bug-Url: https://bugzilla.redhat.com/1080056
Signed-off-by: Tomas Jelinek <[email protected]>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/25/26525/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
index 12eb0f8..9cb3297 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
@@ -1134,7 +1134,12 @@
 
         if (cluster != null) {
             vmOsValues = AsyncDataProvider.getOsIds(cluster.getArchitecture());
+            Integer selectedOsId = getModel().getOSType().getSelectedItem();
             getModel().getOSType().setItems(vmOsValues);
+            if (selectedOsId != null && vmOsValues.contains(selectedOsId)) {
+                getModel().getOSType().setSelectedItem(selectedOsId);
+            }
+
             postOsItemChanged();
         }
 


-- 
To view, visit http://gerrit.ovirt.org/26525
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifaf27934e200e69beeb0912db22f26525b85e9b1
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to