I'm assuming it's valid for a CachedValue to exist but have a null value as we

properly ignore those elsewhere, so we should ignore them when traversing the cache as well.
2007-08-24 01:46:40 +00:00
parent b24786facd
commit d80cccd6bf
1 changed files with 3 additions and 2 deletions
@@ -416,8 +416,9 @@ public class PersistenceContext
        if (bin != null) {
            for (Object key : bin.enumerateKeys()) {
                CacheAdapter.CachedValue<T> element = bin.lookup((Serializable) key);
-                if (element != null) {
-                    filter.visitCacheEntry(this, cacheId, (Serializable) key, element.getValue());
+                T value;
+                if (element != null && (value = element.getValue()) != null) {
+                    filter.visitCacheEntry(this, cacheId, (Serializable) key, value);
                }
            }
        }