Lior Vernia has uploaded a new change for review. Change subject: webadmin: Cancel button to not do anything ......................................................................
webadmin: Cancel button to not do anything Previously it acted pretty much as if OK was pressed. Change-Id: I25a872b301d6ea8a050355a4f828580eea062101 Bug-Url: https://bugzilla.redhat.com/1058017 Signed-off-by: Lior Vernia <[email protected]> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java 1 file changed, 12 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/50/23750/1 diff --git a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java index 6d02b9c..b3d361f 100644 --- a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java +++ b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/NetworkModel.java @@ -617,12 +617,20 @@ private void addQos() { NewNetworkQoSModel qosModel = new NewNetworkQoSModel(this, getSelectedDc()) { + + @Override + protected void postSaveAction(boolean succeeded) { + if (succeeded) { + List<NetworkQoS> qosItems = new ArrayList<NetworkQoS>((Collection<NetworkQoS>) getQos().getItems()); + qosItems.add(1, networkQoS); + getQos().setItems(qosItems); + getQos().setSelectedItem(networkQoS); + } + super.postSaveAction(succeeded); + } + @Override protected void cancel() { - List<NetworkQoS> qosItems = new ArrayList<NetworkQoS>((Collection<NetworkQoS>) getQos().getItems()); - qosItems.add(1, networkQoS); - getQos().setItems(qosItems); - getQos().setSelectedItem(networkQoS); sourceListModel.setConfirmWindow(null); } }; -- To view, visit http://gerrit.ovirt.org/23750 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I25a872b301d6ea8a050355a4f828580eea062101 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
