You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2012/09/03 10:33:52 UTC

git commit: WICKET-4741 Only complete handler (no success or failure handler) is called when Ajax link produces an exception

Updated Branches:
  refs/heads/master 4f5cb035a -> cdd66a79f


WICKET-4741 Only complete handler (no success or failure handler) is called when Ajax link produces an exception

Call the success handlers before redirecting.


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/cdd66a79
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/cdd66a79
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/cdd66a79

Branch: refs/heads/master
Commit: cdd66a79f0e0bdee3ea8827bcbdb22f807518550
Parents: 4f5cb03
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Mon Sep 3 10:33:16 2012 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Mon Sep 3 10:33:16 2012 +0200

----------------------------------------------------------------------
 .../wicket/ajax/res/js/wicket-ajax-jquery.js       |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/cdd66a79/wicket-core/src/main/java/org/apache/wicket/ajax/res/js/wicket-ajax-jquery.js
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/ajax/res/js/wicket-ajax-jquery.js b/wicket-core/src/main/java/org/apache/wicket/ajax/res/js/wicket-ajax-jquery.js
index 3ba4032..e126852 100644
--- a/wicket-core/src/main/java/org/apache/wicket/ajax/res/js/wicket-ajax-jquery.js
+++ b/wicket-core/src/main/java/org/apache/wicket/ajax/res/js/wicket-ajax-jquery.js
@@ -640,6 +640,7 @@
 
 					// In case the page isn't really redirected. For example say the redirect is to an octet-stream.
 					// A file download popup will appear but the page in the browser won't change.
+					this.success(context);
 					this.done();
 
 					var rhttp  = /^http:\/\//,  // checks whether the string starts with http://