Lior Vernia has uploaded a new change for review.

Change subject: webadmin: Small fix for removing profiles
......................................................................

webadmin: Small fix for removing profiles

As part of a very recent large refactoring, some regression was
introduced in the add/remove profiles widget in the edit network
dialog, that disregarded removal of profiles. This patch fixes it.

Change-Id: Iea8aa1b670dc12109da30549ab61975215e0c174
Signed-off-by: Lior Vernia <[email protected]>
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/EditNetworkModel.java
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/61/20961/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/EditNetworkModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/EditNetworkModel.java
index b3781d4..d50efac 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/EditNetworkModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/EditNetworkModel.java
@@ -107,11 +107,13 @@
                     profilesModels.add(editModel);
                     editModel.getName().setIsChangable(false);
                 }
+                originalProfileModels.clear();
+                originalProfileModels.addAll(profilesModels);
+
                 if (profilesModels.isEmpty() && !(Boolean) 
getIsVmNetwork().getEntity()) {
                     profilesModels.add(getDefaultProfile());
                 }
                 getProfiles().setItems(profilesModels);
-                originalProfileModels = profilesModels;
             }
         };
         AsyncDataProvider.getVnicProfilesByNetworkId(profilesQuery, 
getNetwork().getId());


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

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

Reply via email to