Tomas Jelinek has uploaded a new change for review.
Change subject: frontend: old DC used if DC is selected in tree
......................................................................
frontend: old DC used if DC is selected in tree
The problem was in UnitVmModel which used the StoragePool entity from the
system tree and not the one just updated from backend.
Fixed by using the correct one.
Change-Id: I29bd052f7ed6dacd014e0c4348584881ee15ef02
Signed-off-by: Tomas Jelinek <tjeli...@redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/57/17957/1
diff --git
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
index 328acb2..9e18960 100644
---
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
+++
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java
@@ -1919,6 +1919,10 @@
Guid selectedCluster) {
StoragePool dataCenter = getDataCenterAccordingSystemTree(model,
dataCenters);
+
+ // the dataCenters are the entities just downloaded from server while
the dataCenter can be a cached one from the system tree
+ dataCenter = dataCenter == null ? null :
findDataCenterById(dataCenters, dataCenter.getId());
+
List<VDSGroup> possibleClusters = getClusterAccordingSystemTree(model,
clusters);
if (dataCenter == null || possibleClusters == null) {
getDataCenterWithClustersList().setIsChangable(false);
--
To view, visit http://gerrit.ovirt.org/17957
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I29bd052f7ed6dacd014e0c4348584881ee15ef02
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjeli...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
