Typehint EntityManagerInterface only

UPGRADE.md

@@ -1,5 +1,9 @@
 # Upgrade to 3.0
 
+## BC Break: ``EntityManagerInterface`` is now used instead of ``EntityManager`` in typehints
+
+`Sequencing\Generator#generate()` now takes ``EntityManagerInterface`` as its first argument instead of ``EntityManager``. If you have any custom generators, please update your code accordingly.
+
 ## BC Break: Removed `EntityManager#flush($entity)` and `EntityManager#flush($entities)`
 
 If your code relies on single entity flushing optimisations via

lib/Doctrine/ORM/AbstractQuery.php

@@ -264,7 +264,7 @@ abstract public function getSQL();
     /**
      * Retrieves the associated EntityManager of this Query instance.
      *
-     * @return \Doctrine\ORM\EntityManager
+     * @return \Doctrine\ORM\EntityManagerInterface
      */
     public function getEntityManager()
     {

lib/Doctrine/ORM/Cache/DefaultEntityHydrator.php

@@ -24,7 +24,7 @@
 class DefaultEntityHydrator implements EntityHydrator
 {
     /**
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;
 

lib/Doctrine/ORM/EntityRepository.php

@@ -31,7 +31,7 @@ class EntityRepository implements ObjectRepository, Selectable
     protected $entityName;
 
     /**
-     * @var EntityManager
+     * @var EntityManagerInterface
      */
     protected $em;
 
@@ -43,8 +43,8 @@ class EntityRepository implements ObjectRepository, Selectable
     /**
      * Initializes a new <tt>EntityRepository</tt>.
      *
-     * @param EntityManager         $em    The EntityManager to use.
-     * @param Mapping\ClassMetadata $class The class descriptor.
+     * @param EntityManagerInterface $em    The EntityManager to use.
+     * @param Mapping\ClassMetadata  $class The class descriptor.
      */
     public function __construct(EntityManagerInterface $em, Mapping\ClassMetadata $class)
     {
@@ -249,7 +249,7 @@ public function getClassName()
     }
 
     /**
-     * @return EntityManager
+     * @return EntityManagerInterface
      */
     protected function getEntityManager()
     {

lib/Doctrine/ORM/Event/LifecycleEventArgs.php

@@ -30,7 +30,7 @@ public function getEntity()
     /**
      * Retrieves associated EntityManager.
      *
-     * @return \Doctrine\ORM\EntityManager
+     * @return \Doctrine\ORM\EntityManagerInterface
      */
     public function getEntityManager()
     {

lib/Doctrine/ORM/Event/OnClearEventArgs.php

@@ -35,7 +35,7 @@ public function __construct(EntityManagerInterface $em)
     /**
      * Retrieves associated EntityManager.
      *
-     * @return \Doctrine\ORM\EntityManager
+     * @return \Doctrine\ORM\EntityManagerInterface
      */
     public function getEntityManager()
     {

lib/Doctrine/ORM/Event/OnFlushEventArgs.php

@@ -36,7 +36,7 @@ public function __construct(EntityManagerInterface $em)
     /**
      * Retrieve associated EntityManager.
      *
-     * @return \Doctrine\ORM\EntityManager
+     * @return \Doctrine\ORM\EntityManagerInterface
      */
     public function getEntityManager()
     {

lib/Doctrine/ORM/Event/PostFlushEventArgs.php

@@ -17,7 +17,7 @@
 class PostFlushEventArgs extends EventArgs
 {
     /**
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;
 
@@ -34,7 +34,7 @@ public function __construct(EntityManagerInterface $em)
     /**
      * Retrieves associated EntityManager.
      *
-     * @return \Doctrine\ORM\EntityManager
+     * @return \Doctrine\ORM\EntityManagerInterface
      */
     public function getEntityManager()
     {

lib/Doctrine/ORM/Event/PreFlushEventArgs.php

@@ -19,7 +19,7 @@
 class PreFlushEventArgs extends EventArgs
 {
     /**
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;
 
@@ -34,7 +34,7 @@ public function __construct(EntityManagerInterface $em)
     }
 
     /**
-     * @return \Doctrine\ORM\EntityManager
+     * @return \Doctrine\ORM\EntityManagerInterface
      */
     public function getEntityManager()
     {

lib/Doctrine/ORM/Query/FilterCollection.php

@@ -35,7 +35,7 @@ class FilterCollection
     /**
      * The EntityManager that "owns" this FilterCollection instance.
      *
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;
 

lib/Doctrine/ORM/Query/Parser.php

@@ -122,7 +122,7 @@ class Parser
     /**
      * The EntityManager.
      *
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;
 
@@ -227,7 +227,7 @@ public function getParserResult()
     /**
      * Gets the EntityManager used by the parser.
      *
-     * @return \Doctrine\ORM\EntityManager
+     * @return \Doctrine\ORM\EntityManagerInterface
      */
     public function getEntityManager()
     {

lib/Doctrine/ORM/Query/SqlWalker.php

@@ -83,7 +83,7 @@ class SqlWalker implements TreeWalker
     private $parserResult;
 
     /**
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;
 
@@ -196,7 +196,7 @@ public function getConnection()
     /**
      * Gets the EntityManager used by the walker.
      *
-     * @return \Doctrine\ORM\EntityManager
+     * @return \Doctrine\ORM\EntityManagerInterface
      */
     public function getEntityManager()
     {

lib/Doctrine/ORM/QueryBuilder.php

@@ -262,7 +262,7 @@ public function getType()
     /**
      * Gets the associated EntityManager for this query builder.
      *
-     * @return EntityManager
+     * @return EntityManagerInterface
      */
     public function getEntityManager()
     {

lib/Doctrine/ORM/Sequencing/BigIntegerIdentityGenerator.php

@@ -4,7 +4,7 @@
 
 namespace Doctrine\ORM\Sequencing;
 
-use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\EntityManagerInterface;
 
 /**
  * Id generator that obtains IDs from special "identity" columns. These are columns
@@ -35,7 +35,7 @@ public function __construct($sequenceName = null)
     /**
      * {@inheritdoc}
      */
-    public function generate(EntityManager $em, $entity)
+    public function generate(EntityManagerInterface $em, $entity)
     {
         return (string) $em->getConnection()->lastInsertId($this->sequenceName);
     }

lib/Doctrine/ORM/Sequencing/Generator.php

@@ -5,19 +5,19 @@
 
 namespace Doctrine\ORM\Sequencing;
 
-use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\EntityManagerInterface;
 
 interface Generator
 {
     /**
      * Generates an identifier for an entity.
      *
-     * @param EntityManager $em
-     * @param object        $entity
+     * @param EntityManagerInterface $em
+     * @param object                 $entity
      *
      * @return \Generator
      */
-    public function generate(EntityManager $em, $entity);
+    public function generate(EntityManagerInterface $em, $entity);
 
     /**
      * Gets whether this generator is a post-insert generator which means that

lib/Doctrine/ORM/Sequencing/IdentityGenerator.php

@@ -4,7 +4,7 @@
 
 namespace Doctrine\ORM\Sequencing;
 
-use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\EntityManagerInterface;
 
 /**
  * Id generator that obtains IDs from special "identity" columns. These are columns
@@ -35,7 +35,7 @@ public function __construct($sequenceName = null)
     /**
      * {@inheritDoc}
      */
-    public function generate(EntityManager $em, $entity)
+    public function generate(EntityManagerInterface $em, $entity)
     {
         return (int) $em->getConnection()->lastInsertId($this->sequenceName);
     }

lib/Doctrine/ORM/Sequencing/SequenceGenerator.php

@@ -4,7 +4,7 @@
 
 namespace Doctrine\ORM\Sequencing;
 
-use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\EntityManagerInterface;
 use Serializable;
 
 /**
@@ -54,7 +54,7 @@ public function __construct($sequenceName, $allocationSize)
     /**
      * {@inheritdoc}
      */
-    public function generate(EntityManager $em, $entity)
+    public function generate(EntityManagerInterface $em, $entity)
     {
         if ($this->maxValue === null || $this->nextValue == $this->maxValue) {
             // Allocate new values

lib/Doctrine/ORM/Sequencing/TableGenerator.php

@@ -4,7 +4,7 @@
 
 namespace Doctrine\ORM\Sequencing;
 
-use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\EntityManagerInterface;
 
 /**
  * Id generator that uses a single-row database table and a hi/lo algorithm.
@@ -57,7 +57,7 @@ public function __construct($tableName, $sequenceName = 'default', $allocationSi
     /**
      * {@inheritdoc}
      */
-    public function generate(EntityManager $em, $entity)
+    public function generate(EntityManagerInterface $em, $entity)
     {
         if ($this->maxValue === null || $this->nextValue === $this->maxValue) {
             // Allocate new values

lib/Doctrine/ORM/Sequencing/UuidGenerator.php

@@ -4,7 +4,7 @@
 
 namespace Doctrine\ORM\Sequencing;
 
-use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\EntityManagerInterface;
 
 /**
  * Represents an ID generator that uses the database UUID expression
@@ -17,7 +17,7 @@ class UuidGenerator implements Generator
     /**
      * {@inheritdoc}
      */
-    public function generate(EntityManager $em, $entity)
+    public function generate(EntityManagerInterface $em, $entity)
     {
         $conn = $em->getConnection();
         $sql = 'SELECT ' . $conn->getDatabasePlatform()->getGuidExpression();

lib/Doctrine/ORM/Tools/Console/Command/InfoCommand.php

@@ -39,7 +39,7 @@ protected function configure()
      */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
-        /* @var $entityManager \Doctrine\ORM\EntityManager */
+        /* @var $entityManager \Doctrine\ORM\EntityManagerInterface */
         $entityManager = $this->getHelper('em')->getEntityManager();
 
         $entityClassNames = $entityManager->getConfiguration()

lib/Doctrine/ORM/Tools/Console/Command/SchemaTool/AbstractCommand.php

@@ -38,7 +38,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
     {
         $emHelper = $this->getHelper('em');
 
-        /* @var $em \Doctrine\ORM\EntityManager */
+        /* @var $em \Doctrine\ORM\EntityManagerInterface */
         $em = $emHelper->getEntityManager();
 
         $metadatas = $em->getMetadataFactory()->getAllMetadata();

lib/Doctrine/ORM/Tools/Pagination/LimitSubqueryOutputWalker.php

@@ -59,7 +59,7 @@ class LimitSubqueryOutputWalker extends SqlWalker
     private $maxResults;
 
     /**
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;
 

tests/Doctrine/Tests/EventListener/CacheMetadataListener.php

@@ -5,7 +5,7 @@
 namespace Doctrine\Tests\EventListener;
 
 use Doctrine\Common\Persistence\Event\LoadClassMetadataEventArgs;
-use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\ORM\Mapping\CacheMetadata;
 use Doctrine\ORM\Mapping\CacheUsage;
 use Doctrine\ORM\Mapping\ClassMetadata;
@@ -57,10 +57,10 @@ private function recordVisit(ClassMetadata $metadata)
     }
 
     /**
-     * @param ClassMetadata $metadata
-     * @param EntityManager $em
+     * @param ClassMetadata          $metadata
+     * @param EntityManagerInterface $em
      */
-    protected function enableCaching(ClassMetadata $metadata, EntityManager $em)
+    protected function enableCaching(ClassMetadata $metadata, EntityManagerInterface $em)
     {
         if ($this->isVisited($metadata)) {
             return; // Already handled in the past

tests/Doctrine/Tests/Mocks/SequenceMock.php

@@ -4,7 +4,7 @@
 
 namespace Doctrine\Tests\Mocks;
 
-use Doctrine\ORM\EntityManager;
+use Doctrine\ORM\EntityManagerInterface;
 use Doctrine\ORM\Sequencing\SequenceGenerator;
 
 /**
@@ -20,7 +20,7 @@ class SequenceMock extends SequenceGenerator
     /**
      * {@inheritdoc}
      */
-    public function generate(EntityManager $em, $entity)
+    public function generate(EntityManagerInterface $em, $entity)
     {
         return $this->sequenceNumber++;
     }

tests/Doctrine/Tests/ORM/Cache/DefaultCacheFactoryTest.php

@@ -41,7 +41,7 @@ class DefaultCacheFactoryTest extends OrmTestCase
     private $factory;
 
     /**
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;
 

tests/Doctrine/Tests/ORM/Cache/DefaultEntityHydratorTest.php

@@ -25,7 +25,7 @@ class DefaultEntityHydratorTest extends OrmTestCase
     private $structure;
 
     /**
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;
 

tests/Doctrine/Tests/ORM/Cache/DefaultQueryCacheTest.php

@@ -36,7 +36,7 @@ class DefaultQueryCacheTest extends OrmTestCase
     private $queryCache;
 
     /**
-     * @var \Doctrine\ORM\EntityManager
+     * @var \Doctrine\ORM\EntityManagerInterface
      */
     private $em;