DATAREDIS-364 - Open-up API in RedisCacheManager for easier extensibility.

We now allow users to build custom CacheManagers based on RedisCacheManager more easily by making a broader API accessible to sub-classes.
In that sense we also allowing RedisCache to be used in custom implementations by making it public.

Original pull request: #120.

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

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2014 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -37,7 +37,7 @@ import org.springframework.util.Assert;
  * @author Thomas Darimont
  */
 @SuppressWarnings("unchecked")
-class RedisCache implements Cache {
+public class RedisCache implements Cache {
 
 	private static final int PAGE_SIZE = 128;
 	private final String name;
@@ -56,7 +56,8 @@ class RedisCache implements Cache {
 	 * @param template
 	 * @param expiration
 	 */
-	RedisCache(String name, byte[] prefix, RedisTemplate<? extends Object, ? extends Object> template, long expiration) {
+	public RedisCache(String name, byte[] prefix, RedisTemplate<? extends Object, ? extends Object> template,
+			long expiration) {
 
 		Assert.hasText(name, "non-empty cache name is required");
 		this.name = name;

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

@@ -1,5 +1,5 @@
 /*
- * Copyright 2011-2014 the original author or authors.
+ * Copyright 2011-2015 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -176,7 +176,7 @@ public class RedisCacheManager extends AbstractTransactionSupportingCacheManager
 	 * @param caches must not be {@literal null}
 	 * @return
 	 */
-	private Collection<? extends Cache> addConfiguredCachesIfNecessary(Collection<? extends Cache> caches) {
+	protected Collection<? extends Cache> addConfiguredCachesIfNecessary(Collection<? extends Cache> caches) {
 
 		Assert.notNull(caches, "Caches must not be null!");
 
@@ -202,18 +202,18 @@ public class RedisCacheManager extends AbstractTransactionSupportingCacheManager
 		return result;
 	}
 
-	private Cache createAndAddCache(String cacheName) {
+	protected Cache createAndAddCache(String cacheName) {
 		addCache(createCache(cacheName));
 		return super.getCache(cacheName);
 	}
 
 	@SuppressWarnings("unchecked")
-	private RedisCache createCache(String cacheName) {
+	protected RedisCache createCache(String cacheName) {
 		long expiration = computeExpiration(cacheName);
 		return new RedisCache(cacheName, (usePrefix ? cachePrefix.prefix(cacheName) : null), template, expiration);
 	}
 
-	private long computeExpiration(String name) {
+	protected long computeExpiration(String name) {
 		Long expiration = null;
 		if (expires != null) {
 			expiration = expires.get(name);
@@ -221,9 +221,9 @@ public class RedisCacheManager extends AbstractTransactionSupportingCacheManager
 		return (expiration != null ? expiration.longValue() : defaultExpiration);
 	}
 
-	private List<RedisCache> loadAndInitRemoteCaches() {
+	protected List<Cache> loadAndInitRemoteCaches() {
 
-		List<RedisCache> caches = new ArrayList<RedisCache>();
+		List<Cache> caches = new ArrayList<Cache>();
 
 		try {
 			Set<String> cacheNames = loadRemoteCacheKeys();
@@ -244,7 +244,7 @@ public class RedisCacheManager extends AbstractTransactionSupportingCacheManager
 	}
 
 	@SuppressWarnings("unchecked")
-	private Set<String> loadRemoteCacheKeys() {
+	protected Set<String> loadRemoteCacheKeys() {
 		return (Set<String>) template.execute(new RedisCallback<Set<String>>() {
 
 			@Override
@@ -264,4 +264,17 @@ public class RedisCacheManager extends AbstractTransactionSupportingCacheManager
 			}
 		});
 	}
+
+	@SuppressWarnings("rawtypes")
+	protected RedisTemplate getTemplate() {
+		return template;
+	}
+
+	protected RedisCachePrefix getCachePrefix() {
+		return cachePrefix;
+	}
+
+	protected boolean isUsePrefix() {
+		return usePrefix;
+	}
 }