Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: [setupNetworks] Unsync net params can be changed (#848354) ...................................................................... webadmin: [setupNetworks] Unsync net params can be changed (#848354) https://bugzilla.redhat.com/848354 The bug was fixed in a previous commit, but the fix worked just in case the "is to sync" was checked an unchecked without closing the dialog. If the edit network dialog is colsed and then opened again and the "is to sync" is changed back to false. The original "before sync" values should be reverted- this didn't work in the prevoius commit. Change-Id: Ia00641b123f109d443d98e2c99e727099c2a9d16 Signed-off-by: Alona Kaplan <alkaplan@redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostInterfaceModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostManagementNetworkModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostSetupNetworksModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/NetworkParameters.java 4 files changed, 30 insertions(+), 25 deletions(-) Approvals: Alona Kaplan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7684 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia00641b123f109d443d98e2c99e727099c2a9d16 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alona Kaplan <alkaplan@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik@redhat.com>