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 2014/08/12 14:11:14 UTC
[1/4] git commit: Make members final when there is no need to modify
them later
Repository: wicket
Updated Branches:
refs/heads/wicket-6.x 353aa4e8b -> d0a29b0e5
Make members final when there is no need to modify them later
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/a969f599
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/a969f599
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/a969f599
Branch: refs/heads/wicket-6.x
Commit: a969f5999d9b25c867ff0dc904186cd66de6b76b
Parents: 353aa4e
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Tue Aug 12 14:08:25 2014 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Tue Aug 12 14:08:25 2014 +0200
----------------------------------------------------------------------
.../wicket/atmosphere/AtmosphereRequestHandler.java | 5 +++--
.../wicket/atmosphere/AtmosphereRequestMapper.java | 7 +++----
.../wicket/atmosphere/AtmosphereWebRequest.java | 8 ++++----
.../wicket/atmosphere/AtmosphereWebResponse.java | 2 +-
.../java/org/apache/wicket/atmosphere/EventBus.java | 15 +++++++--------
.../apache/wicket/atmosphere/EventSubscription.java | 10 +++++-----
.../java/org/apache/wicket/atmosphere/PageKey.java | 7 ++++---
7 files changed, 27 insertions(+), 27 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/a969f599/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java
index a76f440..48bd5e2e 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java
@@ -18,14 +18,15 @@ package org.apache.wicket.atmosphere;
import java.util.Collection;
-import org.apache.wicket.Application;
import org.apache.wicket.Component;
import org.apache.wicket.Page;
import org.apache.wicket.ajax.AjaxRequestTarget;
+import org.apache.wicket.behavior.Behavior;
import org.apache.wicket.protocol.http.WebApplication;
import org.apache.wicket.request.IRequestCycle;
import org.apache.wicket.request.IRequestHandler;
import org.apache.wicket.request.cycle.RequestCycle;
+import org.apache.wicket.util.lang.Args;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -63,7 +64,7 @@ public class AtmosphereRequestHandler implements IRequestHandler
this.pageKey = pageKey;
this.subscriptions = subscriptions;
this.event = event;
- this.eventSubscriptionInvoker = eventSubscriptionInvoker;
+ this.eventSubscriptionInvoker = Args.notNull(eventSubscriptionInvoker, "eventSubscriptionInvoker");
}
@Override
http://git-wip-us.apache.org/repos/asf/wicket/blob/a969f599/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestMapper.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestMapper.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestMapper.java
index 2b6b8c9..0f427e2 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestMapper.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestMapper.java
@@ -20,6 +20,7 @@ import org.apache.wicket.request.IRequestHandler;
import org.apache.wicket.request.IRequestMapper;
import org.apache.wicket.request.Request;
import org.apache.wicket.request.Url;
+import org.apache.wicket.util.lang.Args;
/**
* Internal {@link IRequestMapper} to map {@link AtmosphereWebRequest} to
@@ -29,10 +30,9 @@ import org.apache.wicket.request.Url;
*/
public class AtmosphereRequestMapper implements IRequestMapper
{
- private EventSubscriptionInvoker eventSubscriptionInvoker;
+ private final EventSubscriptionInvoker eventSubscriptionInvoker;
/**
- *
* Construct with {@link SubscribeAnnotationEventSubscriptionInvoker}
*/
public AtmosphereRequestMapper()
@@ -41,14 +41,13 @@ public class AtmosphereRequestMapper implements IRequestMapper
}
/**
- *
* Construct.
*
* @param eventSubscriptionInvoker
*/
public AtmosphereRequestMapper(EventSubscriptionInvoker eventSubscriptionInvoker)
{
- this.eventSubscriptionInvoker = eventSubscriptionInvoker;
+ this.eventSubscriptionInvoker = Args.notNull(eventSubscriptionInvoker, "eventSubscriptionInvoker");
}
@Override
http://git-wip-us.apache.org/repos/asf/wicket/blob/a969f599/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebRequest.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebRequest.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebRequest.java
index 18556de..eb8f5c1 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebRequest.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebRequest.java
@@ -43,13 +43,13 @@ import org.apache.wicket.util.upload.FileUploadException;
*/
class AtmosphereWebRequest extends ServletWebRequest
{
- private ServletWebRequest wrappedRequest;
+ private final ServletWebRequest wrappedRequest;
- private PageKey pageKey;
+ private final PageKey pageKey;
- private Collection<EventSubscription> subscriptions;
+ private final Collection<EventSubscription> subscriptions;
- private AtmosphereEvent event;
+ private final AtmosphereEvent event;
AtmosphereWebRequest(ServletWebRequest wrappedRequest, PageKey pageKey,
Collection<EventSubscription> subscriptions, AtmosphereEvent event)
http://git-wip-us.apache.org/repos/asf/wicket/blob/a969f599/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebResponse.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebResponse.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebResponse.java
index 6a86e97..de09ed8 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebResponse.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereWebResponse.java
@@ -30,7 +30,7 @@ import org.atmosphere.cpr.AtmosphereResponse;
*/
class AtmosphereWebResponse extends WebResponse
{
- private AtmosphereResponse response;
+ private final AtmosphereResponse response;
private final AppendingStringBuffer out;
private boolean redirect;
http://git-wip-us.apache.org/repos/asf/wicket/blob/a969f599/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
index 7790f83..b93849b 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
@@ -102,17 +102,17 @@ public class EventBus implements UnboundListener
return eventBus;
}
- private WebApplication application;
+ private final WebApplication application;
- private Broadcaster broadcaster;
+ private final Broadcaster broadcaster;
- private Multimap<PageKey, EventSubscription> subscriptions = HashMultimap.create();
+ private final Multimap<PageKey, EventSubscription> subscriptions = HashMultimap.create();
- private Map<String, PageKey> trackedPages = Maps.newHashMap();
+ private final Map<String, PageKey> trackedPages = Maps.newHashMap();
- private List<ResourceRegistrationListener> registrationListeners = new CopyOnWriteArrayList<ResourceRegistrationListener>();
+ private final List<ResourceRegistrationListener> registrationListeners = new CopyOnWriteArrayList<ResourceRegistrationListener>();
- private AtmosphereParameters parameters = new AtmosphereParameters();
+ private final AtmosphereParameters parameters = new AtmosphereParameters();
/**
* Creates and registers an {@code EventBus} for the given application. The first broadcaster
@@ -278,8 +278,7 @@ public class EventBus implements UnboundListener
* Unregisters all {@link EventSubscription}s for the given component, including the
* subscriptions for its behaviors.
*
- * @param page
- * @param subscription
+ * @param component
*/
public synchronized void unregister(Component component)
{
http://git-wip-us.apache.org/repos/asf/wicket/blob/a969f599/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventSubscription.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventSubscription.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventSubscription.java
index dfd8912..f7080bd 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventSubscription.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventSubscription.java
@@ -34,15 +34,15 @@ import com.google.common.base.Predicates;
*/
public class EventSubscription
{
- private String componentPath;
+ private final String componentPath;
- private Integer behaviorIndex;
+ private final Integer behaviorIndex;
- private String methodName;
+ private final String methodName;
- private Predicate<AtmosphereEvent> filter;
+ private final Predicate<AtmosphereEvent> filter;
- private Predicate<AtmosphereEvent> contextAwareFilter;
+ private final Predicate<AtmosphereEvent> contextAwareFilter;
/**
* Construct.
http://git-wip-us.apache.org/repos/asf/wicket/blob/a969f599/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/PageKey.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/PageKey.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/PageKey.java
index 61586a9..c875f8e 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/PageKey.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/PageKey.java
@@ -17,6 +17,7 @@
package org.apache.wicket.atmosphere;
import com.google.common.base.Objects;
+import org.apache.wicket.util.lang.Args;
/**
* Identifies a page by its id and the session it belongs to.
@@ -25,9 +26,9 @@ import com.google.common.base.Objects;
*/
public class PageKey
{
- private Integer pageId;
+ private final Integer pageId;
- private String sessionId;
+ private final String sessionId;
/**
* Construct.
@@ -37,7 +38,7 @@ public class PageKey
*/
public PageKey(Integer pageId, String sessionId)
{
- this.pageId = pageId;
+ this.pageId = Args.notNull(pageId, "pageId");
this.sessionId = sessionId;
}
[4/4] git commit: Use #getBroadcaster() to be able to use a custom
one for testing
Posted by mg...@apache.org.
Use #getBroadcaster() to be able to use a custom one for testing
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/d0a29b0e
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/d0a29b0e
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/d0a29b0e
Branch: refs/heads/wicket-6.x
Commit: d0a29b0e54b2cd474c01637fe278887c5351ada7
Parents: ca6e424
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Tue Aug 12 14:10:34 2014 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Tue Aug 12 14:10:34 2014 +0200
----------------------------------------------------------------------
.../src/main/java/org/apache/wicket/atmosphere/EventBus.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/d0a29b0e/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
index 7631e35..a87a9c5 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
@@ -363,7 +363,7 @@ public class EventBus implements UnboundListener
ThreadContext oldContext = ThreadContext.get(false);
try
{
- for (AtmosphereResource resource : ImmutableList.copyOf(broadcaster.getAtmosphereResources()))
+ for (AtmosphereResource resource : ImmutableList.copyOf(getBroadcaster().getAtmosphereResources()))
{
postToSingleResource(event, resource);
}
[2/4] git commit: EventBus#removeRegistrationListener() should
*remove* the listeners
Posted by mg...@apache.org.
EventBus#removeRegistrationListener() should *remove* the listeners
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/c539f680
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/c539f680
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/c539f680
Branch: refs/heads/wicket-6.x
Commit: c539f680b0be205322dfbec2cb1281e473d22c12
Parents: a969f599
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Tue Aug 12 14:09:16 2014 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Tue Aug 12 14:09:16 2014 +0200
----------------------------------------------------------------------
.../src/main/java/org/apache/wicket/atmosphere/EventBus.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/c539f680/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
index b93849b..7631e35 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
@@ -456,7 +456,7 @@ public class EventBus implements UnboundListener
*/
public void removeRegistrationListener(ResourceRegistrationListener listener)
{
- registrationListeners.add(listener);
+ registrationListeners.remove(listener);
}
private void fireRegistration(String uuid, Page page)
[3/4] git commit: Cache the got value and reuse it in a method
Posted by mg...@apache.org.
Cache the got value and reuse it in a method
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/ca6e424c
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/ca6e424c
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/ca6e424c
Branch: refs/heads/wicket-6.x
Commit: ca6e424c89d3b75216e58bf0733637aeb37c4a8c
Parents: c539f68
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Tue Aug 12 14:10:06 2014 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Tue Aug 12 14:10:06 2014 +0200
----------------------------------------------------------------------
.../wicket/atmosphere/AtmosphereRequestHandler.java | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/ca6e424c/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java
----------------------------------------------------------------------
diff --git a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java
index 48bd5e2e..8824468 100644
--- a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java
+++ b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereRequestHandler.java
@@ -70,8 +70,9 @@ public class AtmosphereRequestHandler implements IRequestHandler
@Override
public void respond(IRequestCycle requestCycle)
{
- Page page = (Page)Application.get().getMapperContext().getPageInstance(pageKey.getPageId());
- AjaxRequestTarget target = WebApplication.get().newAjaxRequestTarget(page);
+ WebApplication application = WebApplication.get();
+ Page page = (Page)application.getMapperContext().getPageInstance(pageKey.getPageId());
+ AjaxRequestTarget target = application.newAjaxRequestTarget(page);
executeHandlers(target, page);
}
@@ -85,14 +86,15 @@ public class AtmosphereRequestHandler implements IRequestHandler
Component component = page.get(componentPath);
if (component != null)
{
- if (curSubscription.getBehaviorIndex() == null)
+ Integer behaviorIndex = curSubscription.getBehaviorIndex();
+ if (behaviorIndex == null)
{
invokeMethod(target, curSubscription, component);
}
else
{
- invokeMethod(target, curSubscription,
- component.getBehaviorById(curSubscription.getBehaviorIndex()));
+ Behavior behavior = component.getBehaviorById(behaviorIndex);
+ invokeMethod(target, curSubscription, behavior);
}
}
else