WICKET-5676 Double click causes internal error using 
AjaxFallbackDefaultDataTable

Use the default channel name for Ajax navigation links


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

Branch: refs/heads/WICKET-5677
Commit: 7ff5623b9af4e4b1ddfd3dee8d5bfde775de09c0
Parents: 64c8567
Author: Martin Tzvetanov Grigorov <mgrigo...@apache.org>
Authored: Mon Aug 18 12:26:01 2014 +0300
Committer: Martin Tzvetanov Grigorov <mgrigo...@apache.org>
Committed: Mon Aug 18 12:26:01 2014 +0300

----------------------------------------------------------------------
 .../main/java/org/apache/wicket/ajax/AjaxChannel.java   | 12 +++++++++++-
 .../paging/AjaxPagingNavigationIncrementLink.java       |  4 +---
 .../navigation/paging/AjaxPagingNavigationLink.java     |  4 +---
 3 files changed, 13 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/7ff5623b/wicket-core/src/main/java/org/apache/wicket/ajax/AjaxChannel.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/ajax/AjaxChannel.java 
b/wicket-core/src/main/java/org/apache/wicket/ajax/AjaxChannel.java
index e60a5c3..45b050c 100644
--- a/wicket-core/src/main/java/org/apache/wicket/ajax/AjaxChannel.java
+++ b/wicket-core/src/main/java/org/apache/wicket/ajax/AjaxChannel.java
@@ -58,9 +58,19 @@ public class AjaxChannel implements IClusterable
        }
 
        /**
+        * The name of the default channel
+        */
+       public static final String DEFAULT_NAME = "0";
+
+       /**
+        * The type of the default channel
+        */
+       public static final Type DEFAULT_TYPE = Type.QUEUE;
+
+       /**
         * The default channel for all Ajax calls
         */
-       public static final AjaxChannel DEFAULT = new AjaxChannel("0", 
Type.QUEUE);
+       public static final AjaxChannel DEFAULT = new AjaxChannel(DEFAULT_NAME, 
DEFAULT_TYPE);
 
        private final String name;
 

http://git-wip-us.apache.org/repos/asf/wicket/blob/7ff5623b/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationIncrementLink.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationIncrementLink.java
 
b/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationIncrementLink.java
index 6f47714..0dcd652 100644
--- 
a/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationIncrementLink.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationIncrementLink.java
@@ -51,8 +51,6 @@ public class AjaxPagingNavigationIncrementLink extends 
PagingNavigationIncrement
 {
        private static final long serialVersionUID = 1L;
 
-       public static final String PAGING_CHANNEL_NAME = "ajax-paging";
-
        /**
         * Constructor.
         * 
@@ -94,7 +92,7 @@ public class AjaxPagingNavigationIncrementLink extends 
PagingNavigationIncrement
                        protected void 
updateAjaxAttributes(AjaxRequestAttributes attributes)
                        {
                                super.updateAjaxAttributes(attributes);
-                               attributes.setChannel(new 
AjaxChannel(PAGING_CHANNEL_NAME, AjaxChannel.Type.ACTIVE));
+                               attributes.setChannel(new 
AjaxChannel(AjaxChannel.DEFAULT_NAME, AjaxChannel.Type.ACTIVE));
                                
AjaxPagingNavigationIncrementLink.this.updateAjaxAttributes(attributes);
                        }
                };

http://git-wip-us.apache.org/repos/asf/wicket/blob/7ff5623b/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationLink.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationLink.java
 
b/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationLink.java
index c8bcb47..05e2f85 100644
--- 
a/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationLink.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/ajax/markup/html/navigation/paging/AjaxPagingNavigationLink.java
@@ -16,8 +16,6 @@
  */
 package org.apache.wicket.ajax.markup.html.navigation.paging;
 
-import static 
org.apache.wicket.ajax.markup.html.navigation.paging.AjaxPagingNavigationIncrementLink.PAGING_CHANNEL_NAME;
-
 import org.apache.wicket.ajax.AjaxChannel;
 import org.apache.wicket.ajax.AjaxRequestTarget;
 import org.apache.wicket.ajax.attributes.AjaxRequestAttributes;
@@ -78,7 +76,7 @@ public class AjaxPagingNavigationLink extends 
PagingNavigationLink<Void> impleme
                        protected void 
updateAjaxAttributes(AjaxRequestAttributes attributes)
                        {
                                super.updateAjaxAttributes(attributes);
-                               attributes.setChannel(new 
AjaxChannel(PAGING_CHANNEL_NAME, AjaxChannel.Type.ACTIVE));
+                               attributes.setChannel(new 
AjaxChannel(AjaxChannel.DEFAULT_NAME, AjaxChannel.Type.ACTIVE));
                                
AjaxPagingNavigationLink.this.updateAjaxAttributes(attributes);
                        }
                };

Reply via email to