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(a)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(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>