Kanagaraj M has uploaded a new change for review. Change subject: webadmin: Removed duplicate call to onSuccess in RunQuery ......................................................................
webadmin: Removed duplicate call to onSuccess in RunQuery In case of handleFailure=true, the onSuccess method was called twice. This leads to multiple invocation of the callback code. It is fixed now. Change-Id: Ic762d09330c5f721c65867dd6b1b9ff517842d64 Signed-off-by: Kanagaraj M <[email protected]> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java 1 file changed, 0 insertions(+), 3 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/98/10298/1 diff --git a/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java b/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java index f313380..8bb1e64 100644 --- a/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java +++ b/frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/Frontend.java @@ -215,9 +215,6 @@ // getEventsHandler().runQueryFailed(failedResult); String errorMessage = result.getExceptionString(); handleNotLoggedInEvent(errorMessage); - if (callback.isHandleFailure()) { - callback.getDel().OnSuccess(callback.getModel(), result); - } } if (callback.isHandleFailure()) { callback.getDel().OnSuccess(callback.getModel(), result); -- To view, visit http://gerrit.ovirt.org/10298 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic762d09330c5f721c65867dd6b1b9ff517842d64 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
