From 768c291cd10a6e4e36a702b3429e288a3a807038 Mon Sep 17 00:00:00 2001 From: Darien Hager Date: Fri, 10 Apr 2015 11:15:30 -0700 Subject: [PATCH] Stumbled across a bug where signatures didn't match, but also the current persister-type didn't support getCacheRegion(). Unsure of exact mechanism, but clearly the constructor doesn't take the second argument anyway, may be old code. --- lib/Doctrine/ORM/Cache/DefaultCacheFactory.php | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/lib/Doctrine/ORM/Cache/DefaultCacheFactory.php b/lib/Doctrine/ORM/Cache/DefaultCacheFactory.php index 69d2fd1c1..82843d359 100644 --- a/lib/Doctrine/ORM/Cache/DefaultCacheFactory.php +++ b/lib/Doctrine/ORM/Cache/DefaultCacheFactory.php @@ -180,13 +180,7 @@ class DefaultCacheFactory implements CacheFactory */ public function buildCollectionHydrator(EntityManagerInterface $em, array $mapping) { - /* @var $targetPersister \Doctrine\ORM\Cache\Persister\CachedPersister */ - $targetPersister = $em->getUnitOfWork()->getEntityPersister($mapping['targetEntity']); - - return new DefaultCollectionHydrator( - $em, - $targetPersister->getCacheRegion() - ); + return new DefaultCollectionHydrator($em); } /**