Updated Branches:
  refs/heads/wicket-1.5.x ed9eb8de5 -> 93e9186fa

WICKET-4311 fixes modal window reopen bug when used with page creator


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

Branch: refs/heads/wicket-1.5.x
Commit: 93e9186fa7efe3e486714f103e7444eeaeb4a6a6
Parents: ed9eb8d
Author: Igor Vaynberg <[email protected]>
Authored: Sun Jan 8 22:16:23 2012 -0800
Committer: Igor Vaynberg <[email protected]>
Committed: Sun Jan 8 22:16:23 2012 -0800

----------------------------------------------------------------------
 .../ajax/markup/html/modal/ModalWindow.java        |   11 +++++++++++
 1 files changed, 11 insertions(+), 0 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/93e9186f/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java
----------------------------------------------------------------------
diff --git 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java
 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java
index 8132893..daf7a91 100644
--- 
a/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java
+++ 
b/wicket-extensions/src/main/java/org/apache/wicket/extensions/ajax/markup/html/modal/ModalWindow.java
@@ -253,6 +253,17 @@ public class ModalWindow extends Panel
 
                add(newCloseButtonBehavior());
                add(new WindowClosedBehavior());
+
+               // install a default callback that will force
+               // WindowClosedBehavior to be executed
+               setWindowClosedCallback(new WindowClosedCallback()
+               {
+                       public void onClose(AjaxRequestTarget target)
+                       {
+                               // noop
+                       }
+               });
+
        }
 
        @Override

Reply via email to