Fix invalid Javadoc references.

See #2175

Author: mp911de

src/main/java/org/springframework/data/redis/cache/BatchStrategies.java

@@ -44,7 +44,7 @@ private BatchStrategies() {
 	 * A {@link BatchStrategy} using a single {@code KEYS} and {@code DEL} command to remove all matching keys.
 	 * {@code KEYS} scans the entire keyspace of the Redis database and can block the Redis worker thread for a long time
 	 * on large keyspaces.
-	 * <p/>
+	 * <p>
 	 * {@code KEYS} is supported for standalone and clustered (sharded) Redis operation modes.
 	 *
 	 * @return batching strategy using {@code KEYS}.
@@ -56,7 +56,7 @@ public static BatchStrategy keys() {
 	/**
 	 * A {@link BatchStrategy} using a {@code SCAN} cursors and potentially multiple {@code DEL} commands to remove all
 	 * matching keys. This strategy allows a configurable batch size to optimize for scan batching.
-	 * <p/>
+	 * <p>
 	 * Note that using the {@code SCAN} strategy might be not supported on all drivers and Redis operation modes.
 	 *
 	 * @return batching strategy using {@code SCAN}.

src/main/java/org/springframework/data/redis/cache/BatchStrategy.java

@@ -19,9 +19,9 @@
 
 /**
  * A {@link BatchStrategy} to be used with {@link RedisCacheWriter}.
- * <p/>
+ * <p>
  * Mainly used to clear the cache.
- * <p/>
+ * <p>
  * Predefined strategies using the {@link BatchStrategies#keys() KEYS} or {@link BatchStrategies#scan(int) SCAN}
  * commands can be found in {@link BatchStrategies}.
  *

src/main/java/org/springframework/data/redis/cache/RedisCache.java

@@ -39,7 +39,7 @@
 
 /**
  * {@link org.springframework.cache.Cache} implementation using for Redis as underlying store.
- * <p/>
+ * <p>
  * Use {@link RedisCacheManager} to create {@link RedisCache} instances.
  *
  * @author Christoph Strobl

src/main/java/org/springframework/data/redis/cache/RedisCacheConfiguration.java

@@ -362,6 +362,7 @@ public void configureKeyConverters(Consumer<ConverterRegistry> registryConsumer)
 	 * <ul>
 	 * <li>{@link String} to {@link byte byte[]} using UTF-8 encoding.</li>
 	 * <li>{@link SimpleKey} to {@link String}</li>
+	 * </ul>
 	 *
 	 * @param registry must not be {@literal null}.
 	 */

src/main/java/org/springframework/data/redis/cache/RedisCacheManager.java

@@ -32,10 +32,10 @@
 
 /**
  * {@link org.springframework.cache.CacheManager} backed by a {@link RedisCache Redis} cache.
- * <p />
+ * <p>
  * This cache manager creates caches by default upon first write. Empty caches are not visible on Redis due to how Redis
  * represents empty data structures.
- * <p />
+ * <p>
  * Caches requiring a different {@link RedisCacheConfiguration} than the default configuration can be specified via
  * {@link RedisCacheManagerBuilder#withInitialCacheConfigurations(Map)}.
  *
@@ -170,7 +170,7 @@ public RedisCacheManager(RedisCacheWriter cacheWriter, RedisCacheConfiguration d
 	 * <dt>locking</dt>
 	 * <dd>disabled</dd>
 	 * <dt>batch strategy</dt>
-	 * <dd>{@link BatchStrategy#keys() KEYS}</dd>
+	 * <dd>{@link BatchStrategies#keys()}</dd>
 	 * <dt>cache configuration</dt>
 	 * <dd>{@link RedisCacheConfiguration#defaultCacheConfig()}</dd>
 	 * <dt>initial caches</dt>
@@ -423,7 +423,7 @@ public RedisCacheManagerBuilder withCacheConfiguration(String cacheName,
 
 		/**
 		 * Disable in-flight {@link org.springframework.cache.Cache} creation for unconfigured caches.
-		 * <p />
+		 * <p>
 		 * {@link RedisCacheManager#getMissingCache(String)} returns {@literal null} for any unconfigured
 		 * {@link org.springframework.cache.Cache} instead of a new {@link RedisCache} instance. This allows eg.
 		 * {@link org.springframework.cache.support.CompositeCacheManager} to chime in.

src/main/java/org/springframework/data/redis/connection/BitFieldSubCommands.java

@@ -113,7 +113,7 @@ protected BitFieldSubCommands set(BitFieldSet set) {
 	}
 
 	/**
-	 * Obtain a new {@link BitFieldIncrByBuilder} for creating and adding a {@link BitFieldIncrby} sub command.
+	 * Obtain a new {@link BitFieldIncrByBuilder} for creating and adding a {@link BitFieldIncrBy} sub command.
 	 *
 	 * @param type must not be {@literal null}.
 	 * @return
@@ -125,7 +125,7 @@ public BitFieldIncrByBuilder incr(BitFieldType type) {
 	/**
 	 * Create new {@link BitFieldSubCommands} adding given {@link BitFieldIncrBy} to the sub commands.
 	 *
-	 * @param get must not be {@literal null}.
+	 * @param incrBy must not be {@literal null}.
 	 * @return
 	 */
 	protected BitFieldSubCommands incr(BitFieldIncrBy incrBy) {

src/main/java/org/springframework/data/redis/connection/ReactiveClusterServerCommands.java

@@ -56,7 +56,7 @@ public interface ReactiveClusterServerCommands extends ReactiveServerCommands {
 	 * Get time unix timestamp of last successful {@link #bgSave()} operation in seconds.
 	 *
 	 * @param node must not be {@literal null}.
-	 * @return @return {@link Mono} wrapping unix timestamp.
+	 * @return {@link Mono} wrapping unix timestamp.
 	 * @throws IllegalArgumentException when {@code node} is {@literal null}.
 	 * @see RedisServerCommands#lastSave()
 	 */
@@ -108,7 +108,6 @@ public interface ReactiveClusterServerCommands extends ReactiveServerCommands {
 	 * <li>cpu utilization</li>
 	 * <li>replication</li>
 	 * </ul>
-	 * <p>
 	 *
 	 * @param node must not be {@literal null}.
 	 * @return {@link Mono} wrapping server information.

src/main/java/org/springframework/data/redis/connection/ReactiveGeoCommands.java

@@ -17,8 +17,6 @@
 
 import static org.springframework.data.redis.connection.RedisGeoCommands.*;
 
-import org.springframework.data.redis.domain.geo.GeoReference;
-import org.springframework.data.redis.domain.geo.GeoShape;
 import reactor.core.publisher.Flux;
 import reactor.core.publisher.Mono;
 
@@ -43,6 +41,8 @@
 import org.springframework.data.redis.connection.ReactiveRedisConnection.MultiValueResponse;
 import org.springframework.data.redis.connection.ReactiveRedisConnection.NumericResponse;
 import org.springframework.data.redis.connection.RedisGeoCommands.GeoRadiusCommandArgs.Flag;
+import org.springframework.data.redis.domain.geo.GeoReference;
+import org.springframework.data.redis.domain.geo.GeoShape;
 import org.springframework.lang.Nullable;
 import org.springframework.util.Assert;
 
@@ -1279,7 +1279,7 @@ public static GeoSearchCommand within(GeoShape shape) {
 		/**
 		 * Sets the geoset {@literal key}. Constructs a new command instance with all previously configured properties.
 		 *
-		 * @param member must not be {@literal null}.
+		 * @param reference must not be {@literal null}.
 		 * @return a new {@link GeoSearchCommand} with {@literal key} applied.
 		 */
 		public GeoSearchCommand at(GeoReference<ByteBuffer> reference) {
@@ -1292,7 +1292,7 @@ public GeoSearchCommand at(GeoReference<ByteBuffer> reference) {
 		/**
 		 * Sets the geoset {@literal key}. Constructs a new command instance with all previously configured properties.
 		 *
-		 * @param member must not be {@literal null}.
+		 * @param key must not be {@literal null}.
 		 * @return a new {@link GeoSearchCommand} with {@literal key} applied.
 		 */
 		public GeoSearchCommand in(ByteBuffer key) {
@@ -1370,7 +1370,7 @@ public static GeoSearchStoreCommand within(GeoShape shape) {
 		/**
 		 * Sets the geoset {@literal key}. Constructs a new command instance with all previously configured properties.
 		 *
-		 * @param member must not be {@literal null}.
+		 * @param reference must not be {@literal null}.
 		 * @return a new {@link GeoSearchStoreCommand} with {@literal key} applied.
 		 */
 		public GeoSearchStoreCommand at(GeoReference<ByteBuffer> reference) {
@@ -1383,7 +1383,7 @@ public GeoSearchStoreCommand at(GeoReference<ByteBuffer> reference) {
 		/**
 		 * Sets the geoset {@literal key}. Constructs a new command instance with all previously configured properties.
 		 *
-		 * @param member must not be {@literal null}.
+		 * @param key must not be {@literal null}.
 		 * @return a new {@link GeoSearchStoreCommand} with {@literal key} applied.
 		 */
 		public GeoSearchStoreCommand in(ByteBuffer key) {
@@ -1396,7 +1396,7 @@ public GeoSearchStoreCommand in(ByteBuffer key) {
 		/**
 		 * Sets the geoset {@literal destKey}. Constructs a new command instance with all previously configured properties.
 		 *
-		 * @param member must not be {@literal null}.
+		 * @param destKey must not be {@literal null}.
 		 * @return a new {@link GeoSearchStoreCommand} with {@literal destKey} applied.
 		 */
 		public GeoSearchStoreCommand storeAt(ByteBuffer destKey) {

src/main/java/org/springframework/data/redis/connection/ReactiveKeyCommands.java

@@ -93,7 +93,7 @@ public CopyCommand to(ByteBuffer targetKey) {
 		/**
 		 * Applies {@code replace}. Constructs a new command instance with all previously configured properties.
 		 *
-		 * @param key must not be {@literal null}.
+		 * @param replace
 		 * @return a new {@link CopyCommand} with {@literal replace} applied.
 		 */
 		public CopyCommand replace(boolean replace) {

src/main/java/org/springframework/data/redis/connection/ReactiveListCommands.java

@@ -830,7 +830,7 @@ default Mono<ByteBuffer> bLMove(ByteBuffer sourceKey, ByteBuffer destinationKey,
 	 * Atomically returns and removes the first/last element (head/tail depending on the {@code from} argument) of the
 	 * list stored at {@code sourceKey}, and pushes the element at the first/last element (head/tail depending on the
 	 * {@code to} argument) of the list stored at {@code destinationKey}.
-	 * <p/>
+	 * <p>
 	 * <b>Blocks connection</b> until element available or {@code timeout} reached.
 	 *
 	 * @param commands must not be {@literal null}.