From 7618aafb9045ac0cfcc33b3a1ea894b328ac2440 Mon Sep 17 00:00:00 2001 From: Rob Winch Date: Fri, 29 Jan 2016 13:23:35 -0600 Subject: [PATCH] Fix Javadoc warnings Fixes gh-347 --- spring-session/build.gradle | 1 - .../data/redis/RedisOperationsSessionRepository.java | 2 +- .../config/annotation/web/http/EnableRedisHttpSession.java | 2 +- .../session/events/AbstractSessionEvent.java | 1 + .../session/events/SessionCreatedEvent.java | 4 ++-- .../session/events/SessionDestroyedEvent.java | 4 ++-- .../session/web/http/HttpSessionManager.java | 2 +- .../session/web/http/OncePerRequestFilter.java | 7 ++++++- 8 files changed, 14 insertions(+), 9 deletions(-) diff --git a/spring-session/build.gradle b/spring-session/build.gradle index abdc990..8587b73 100644 --- a/spring-session/build.gradle +++ b/spring-session/build.gradle @@ -63,7 +63,6 @@ ext.javadocLinks = [ "http://tiles.apache.org/framework/apidocs/", "http://aopalliance.sourceforge.net/doc/", "http://www.eclipse.org/aspectj/doc/released/aspectj5rt-api/", - "http://ehcache.org/apidocs/", "http://quartz-scheduler.org/api/2.2.0/", "http://fasterxml.github.com/jackson-core/javadoc/2.3.0/", "http://fasterxml.github.com/jackson-databind/javadoc/2.3.0/", diff --git a/spring-session/src/main/java/org/springframework/session/data/redis/RedisOperationsSessionRepository.java b/spring-session/src/main/java/org/springframework/session/data/redis/RedisOperationsSessionRepository.java index c2ee9fe..2e6175c 100644 --- a/spring-session/src/main/java/org/springframework/session/data/redis/RedisOperationsSessionRepository.java +++ b/spring-session/src/main/java/org/springframework/session/data/redis/RedisOperationsSessionRepository.java @@ -552,7 +552,7 @@ public class RedisOperationsSessionRepository implements FindByPrincipalNameSess /** * Gets the prefix for the channel that SessionCreatedEvent are published to. The suffix is the session id of the session that was created. * - * @return + * @return the prefix for the channel that SessionCreatedEvent are published to */ public String getSessionCreatedChannelPrefix() { return this.keyPrefix + "event:created:"; diff --git a/spring-session/src/main/java/org/springframework/session/data/redis/config/annotation/web/http/EnableRedisHttpSession.java b/spring-session/src/main/java/org/springframework/session/data/redis/config/annotation/web/http/EnableRedisHttpSession.java index c13dbe9..72e3548 100644 --- a/spring-session/src/main/java/org/springframework/session/data/redis/config/annotation/web/http/EnableRedisHttpSession.java +++ b/spring-session/src/main/java/org/springframework/session/data/redis/config/annotation/web/http/EnableRedisHttpSession.java @@ -73,7 +73,7 @@ public @interface EnableRedisHttpSession { * same Redis instance. *

* - * @return + * @return the unique namespace for keys */ String redisNamespace() default ""; } \ No newline at end of file diff --git a/spring-session/src/main/java/org/springframework/session/events/AbstractSessionEvent.java b/spring-session/src/main/java/org/springframework/session/events/AbstractSessionEvent.java index 0fa4286..3c78d3b 100644 --- a/spring-session/src/main/java/org/springframework/session/events/AbstractSessionEvent.java +++ b/spring-session/src/main/java/org/springframework/session/events/AbstractSessionEvent.java @@ -49,6 +49,7 @@ public abstract class AbstractSessionEvent extends ApplicationEvent { * {@link SessionRepository} implementations it may not be possible to get * the original session in which case this may be null. * + * @param The type of Session * @return the expired {@link Session} or null if the data store does not support obtaining it */ @SuppressWarnings("unchecked") diff --git a/spring-session/src/main/java/org/springframework/session/events/SessionCreatedEvent.java b/spring-session/src/main/java/org/springframework/session/events/SessionCreatedEvent.java index 77fba9d..a47231d 100644 --- a/spring-session/src/main/java/org/springframework/session/events/SessionCreatedEvent.java +++ b/spring-session/src/main/java/org/springframework/session/events/SessionCreatedEvent.java @@ -35,8 +35,8 @@ public class SessionCreatedEvent extends AbstractSessionEvent { } /** - * @param source - * @param session + * @param source The Source of the SessionCreatedEvent + * @param session the Session that was created */ public SessionCreatedEvent(Object source, Session session) { super(source, session); diff --git a/spring-session/src/main/java/org/springframework/session/events/SessionDestroyedEvent.java b/spring-session/src/main/java/org/springframework/session/events/SessionDestroyedEvent.java index c70ae58..6a97f28 100644 --- a/spring-session/src/main/java/org/springframework/session/events/SessionDestroyedEvent.java +++ b/spring-session/src/main/java/org/springframework/session/events/SessionDestroyedEvent.java @@ -32,8 +32,8 @@ public class SessionDestroyedEvent extends AbstractSessionEvent { } /** - * @param source - * @param session + * @param source The Source of the SessionDestoryedEvent + * @param session the Session that was created */ public SessionDestroyedEvent(Object source, Session session) { super(source, session); diff --git a/spring-session/src/main/java/org/springframework/session/web/http/HttpSessionManager.java b/spring-session/src/main/java/org/springframework/session/web/http/HttpSessionManager.java index 069a05d..cc6a76f 100644 --- a/spring-session/src/main/java/org/springframework/session/web/http/HttpSessionManager.java +++ b/spring-session/src/main/java/org/springframework/session/web/http/HttpSessionManager.java @@ -66,7 +66,7 @@ public interface HttpSessionManager { * * * @param request the {@link HttpServletRequest} to get a new alias from - * @return + * @return Gets a new and unique Session alias. */ String getNewSessionAlias(HttpServletRequest request); } diff --git a/spring-session/src/main/java/org/springframework/session/web/http/OncePerRequestFilter.java b/spring-session/src/main/java/org/springframework/session/web/http/OncePerRequestFilter.java index 2ded8dd..871af86 100644 --- a/spring-session/src/main/java/org/springframework/session/web/http/OncePerRequestFilter.java +++ b/spring-session/src/main/java/org/springframework/session/web/http/OncePerRequestFilter.java @@ -71,12 +71,17 @@ abstract class OncePerRequestFilter implements Filter { } } - /** * Same contract as for {@code doFilter}, but guaranteed to be * just invoked once per request within a single request thread. *

Provides HttpServletRequest and HttpServletResponse arguments instead of the * default ServletRequest and ServletResponse ones. + * + * @param request the request + * @param response the response + * @param filterChain the FilterChain + * @throws ServletException thrown when a non-I/O exception has occurred + * @throws IOException thrown when an I/O exception of some sort has occurred * @see Filter#doFilter */ protected abstract void doFilterInternal(