Add return types

UPGRADE.md

@@ -6,6 +6,13 @@ awareness about deprecated code.
 - Use of our low-overhead runtime deprecation API, details:
   https://github.com/doctrine/deprecations/
 
+# Upgrade to 2.0
+
+## BC BREAK: Add return types to all the methods
+
+All return types defined in phpdoc `@return` are now defined in the method signature,
+they must be added to your code if you extend the classes or implement the interfaces.
+
 # Upgrade to 1.8
 
 Executor and Purger classes are final, they cannot be extended.

src/AbstractFixture.php

@@ -49,7 +49,7 @@ private function getReferenceRepository(): ReferenceRepository
      *
      * @return void
      */
-    public function setReference(string $name, object $object)
+    public function setReference(string $name, object $object): void
     {
         $this->getReferenceRepository()->setReference($name, $object);
     }
@@ -68,7 +68,7 @@ public function setReference(string $name, object $object)
      *
      * @throws BadMethodCallException - if repository already has a reference by $name.
      */
-    public function addReference(string $name, object $object)
+    public function addReference(string $name, object $object): void
     {
         $this->getReferenceRepository()->addReference($name, $object);
     }
@@ -86,7 +86,7 @@ public function addReference(string $name, object $object)
      *
      * @template T of object
      */
-    public function getReference(string $name, ?string $class = null)
+    public function getReference(string $name, ?string $class = null): object
     {
         if ($class === null) {
             Deprecation::trigger(
@@ -110,7 +110,7 @@ public function getReference(string $name, ?string $class = null)
      *
      * @return bool
      */
-    public function hasReference(string $name, ?string $class = null)
+    public function hasReference(string $name, ?string $class = null): bool
     {
         if ($class === null) {
             Deprecation::trigger(

src/Executor/AbstractExecutor.php

@@ -48,60 +48,50 @@ public function __construct(ObjectManager $manager)
         $this->referenceRepository = new ReferenceRepository($manager);
     }
 
-    /** @return ReferenceRepository */
-    public function getReferenceRepository()
+    public function getReferenceRepository(): ReferenceRepository
     {
         return $this->referenceRepository;
     }
 
-    public function setReferenceRepository(ReferenceRepository $referenceRepository)
+    public function setReferenceRepository(ReferenceRepository $referenceRepository): void
     {
         $this->referenceRepository = $referenceRepository;
     }
 
     /**
      * Sets the Purger instance to use for this executor instance.
-     *
-     * @return void
      */
-    public function setPurger(PurgerInterface $purger)
+    public function setPurger(PurgerInterface $purger): void
     {
         $this->purger = $purger;
     }
 
-    /** @return PurgerInterface */
-    public function getPurger()
+    public function getPurger(): PurgerInterface
     {
         return $this->purger;
     }
 
     /**
      * Set the logger callable to execute with the log() method.
-     *
-     * @return void
      */
-    public function setLogger(callable $logger)
+    public function setLogger(callable $logger): void
     {
         $this->logger = $logger;
     }
 
     /**
      * Logs a message using the logger.
-     *
-     * @return void
      */
-    public function log(string $message)
+    public function log(string $message): void
     {
         $logger = $this->logger;
         $logger($message);
     }
 
     /**
      * Load a fixture with the given persistence manager.
-     *
-     * @return void
      */
-    public function load(ObjectManager $manager, FixtureInterface $fixture)
+    public function load(ObjectManager $manager, FixtureInterface $fixture): void
     {
         if ($this->logger) {
             $prefix = '';
@@ -124,11 +114,9 @@ public function load(ObjectManager $manager, FixtureInterface $fixture)
     /**
      * Purges the database before loading.
      *
-     * @return void
-     *
      * @throws Exception if the purger is not defined.
      */
-    public function purge()
+    public function purge(): void
     {
         if ($this->purger === null) {
             throw new Exception(
@@ -149,8 +137,6 @@ public function purge()
      *
      * @param FixtureInterface[] $fixtures Array of fixtures to execute.
      * @param bool               $append   Whether to append the data fixtures or purge the database before loading.
-     *
-     * @return void
      */
-    abstract public function execute(array $fixtures, bool $append = false);
+    abstract public function execute(array $fixtures, bool $append = false): void;
 }

src/Executor/MongoDBExecutor.php

@@ -40,16 +40,14 @@ public function __construct(DocumentManager $dm, ?MongoDBPurger $purger = null)
 
     /**
      * Retrieve the DocumentManager instance this executor instance is using.
-     *
-     * @return DocumentManager
      */
-    public function getObjectManager()
+    public function getObjectManager(): DocumentManager
     {
         return $this->dm;
     }
 
     /** @inheritDoc */
-    public function setReferenceRepository(ReferenceRepository $referenceRepository)
+    public function setReferenceRepository(ReferenceRepository $referenceRepository): void
     {
         $this->dm->getEventManager()->removeEventListener(
             $this->listener->getSubscribedEvents(),
@@ -62,7 +60,7 @@ public function setReferenceRepository(ReferenceRepository $referenceRepository)
     }
 
     /** @inheritDoc */
-    public function execute(array $fixtures, bool $append = false)
+    public function execute(array $fixtures, bool $append = false): void
     {
         if ($append === false) {
             $this->purge();

src/Executor/ORMExecutor.php

@@ -16,7 +16,7 @@ class ORMExecutor extends AbstractExecutor
     use ORMExecutorCommon;
 
     /** @inheritDoc */
-    public function execute(array $fixtures, bool $append = false)
+    public function execute(array $fixtures, bool $append = false): void
     {
         $executor = $this;
         $this->em->wrapInTransaction(static function (EntityManagerInterface $em) use ($executor, $fixtures, $append) {

src/Executor/ORMExecutorCommon.php

@@ -43,16 +43,14 @@ public function __construct(EntityManagerInterface $em, ?ORMPurgerInterface $pur
 
     /**
      * Retrieve the EntityManagerInterface instance this executor instance is using.
-     *
-     * @return EntityManagerInterface
      */
-    public function getObjectManager()
+    public function getObjectManager(): EntityManagerInterface
     {
         return $this->originalManager;
     }
 
     /** @inheritDoc */
-    public function setReferenceRepository(ReferenceRepository $referenceRepository)
+    public function setReferenceRepository(ReferenceRepository $referenceRepository): void
     {
         $this->em->getEventManager()->removeEventListener(
             $this->listener->getSubscribedEvents(),

src/Executor/PHPCRExecutor.php

@@ -35,14 +35,13 @@ public function __construct(DocumentManagerInterface $dm, ?PHPCRPurger $purger =
         $this->setPurger($purger);
     }
 
-    /** @return DocumentManagerInterface */
-    public function getObjectManager()
+    public function getObjectManager(): DocumentManagerInterface
     {
         return $this->dm;
     }
 
     /** @inheritDoc */
-    public function execute(array $fixtures, bool $append = false)
+    public function execute(array $fixtures, bool $append = false): void
     {
         $that = $this;
 

src/Loader.php

@@ -73,7 +73,7 @@ class Loader
      *
      * @return array $fixtures Array of loaded fixture object instances.
      */
-    public function loadFromDirectory(string $dir)
+    public function loadFromDirectory(string $dir): array
     {
         if (! is_dir($dir)) {
             throw new InvalidArgumentException(sprintf('"%s" does not exist', $dir));
@@ -94,7 +94,7 @@ public function loadFromDirectory(string $dir)
      *
      * @return array $fixtures Array of loaded fixture object instances.
      */
-    public function loadFromFile(string $fileName)
+    public function loadFromFile(string $fileName): array
     {
         if (! is_readable($fileName)) {
             throw new InvalidArgumentException(sprintf('"%s" does not exist or is not readable', $fileName));
@@ -110,7 +110,7 @@ public function loadFromFile(string $fileName)
      *
      * @return bool
      */
-    public function hasFixture(FixtureInterface $fixture)
+    public function hasFixture(FixtureInterface $fixture): bool
     {
         return isset($this->fixtures[get_class($fixture)]);
     }
@@ -120,7 +120,7 @@ public function hasFixture(FixtureInterface $fixture)
      *
      * @return FixtureInterface
      */
-    public function getFixture(string $className)
+    public function getFixture(string $className): FixtureInterface
     {
         if (! isset($this->fixtures[$className])) {
             throw new InvalidArgumentException(sprintf(
@@ -200,7 +200,7 @@ public function getFixtures()
      *
      * @return bool
      */
-    public function isTransient(string $className)
+    public function isTransient(string $className): bool
     {
         $rc = new ReflectionClass($className);
         if ($rc->isAbstract()) {
@@ -217,7 +217,7 @@ public function isTransient(string $className)
      *
      * @return FixtureInterface
      */
-    protected function createFixture(string $class)
+    protected function createFixture(string $class): FixtureInterface
     {
         return new $class();
     }
@@ -256,7 +256,7 @@ private function orderFixturesByNumber(): void
      *
      * @return void
      */
-    private function orderFixturesByDependencies()
+    private function orderFixturesByDependencies(): void
     {
         /** @psalm-var array<class-string<DependentFixtureInterface>, int> */
         $sequenceForClasses = [];

src/OrderedFixtureInterface.php

@@ -16,5 +16,5 @@ interface OrderedFixtureInterface
      *
      * @return int
      */
-    public function getOrder();
+    public function getOrder(): int;
 }

src/ProxyReferenceRepository.php

@@ -24,7 +24,7 @@ class ProxyReferenceRepository extends ReferenceRepository
      *
      * @return string
      */
-    public function serialize()
+    public function serialize(): string
     {
         $unitOfWork       = $this->getManager()->getUnitOfWork();
         $simpleReferences = [];
@@ -49,7 +49,7 @@ public function serialize()
      *
      * @return void
      */
-    public function unserialize(string $serializedData)
+    public function unserialize(string $serializedData): void
     {
         $repositoryData = unserialize($serializedData);
 
@@ -98,7 +98,7 @@ public function unserialize(string $serializedData)
      *
      * @return bool
      */
-    public function load(string $baseCacheName)
+    public function load(string $baseCacheName): bool
     {
         $filename = $baseCacheName . '.ser';
 
@@ -124,7 +124,7 @@ public function load(string $baseCacheName)
      *
      * @return void
      */
-    public function save(string $baseCacheName)
+    public function save(string $baseCacheName): void
     {
         $serializedData = $this->serialize();
 

src/Purger/MongoDBPurger.php

@@ -27,26 +27,22 @@ public function __construct(?DocumentManager $dm = null)
 
     /**
      * Set the DocumentManager instance this purger instance should use.
-     *
-     * @return void
      */
-    public function setDocumentManager(DocumentManager $dm)
+    public function setDocumentManager(DocumentManager $dm): void
     {
         $this->dm = $dm;
     }
 
     /**
      * Retrieve the DocumentManager instance this purger instance is using.
-     *
-     * @return DocumentManager
      */
-    public function getObjectManager()
+    public function getObjectManager(): DocumentManager
     {
         return $this->dm;
     }
 
     /** @inheritDoc */
-    public function purge()
+    public function purge(): void
     {
         $metadatas = $this->dm->getMetadataFactory()->getAllMetadata();
         foreach ($metadatas as $metadata) {

src/Purger/ORMPurger.php

@@ -60,44 +60,36 @@ public function __construct(?EntityManagerInterface $em = null, array $excluded
 
     /**
      * Set the purge mode
-     *
-     * @return void
      */
-    public function setPurgeMode(int $mode)
+    public function setPurgeMode(int $mode): void
     {
         $this->purgeMode           = $mode;
         $this->cachedSqlStatements = null;
     }
 
     /**
      * Get the purge mode
-     *
-     * @return int
      */
-    public function getPurgeMode()
+    public function getPurgeMode(): int
     {
         return $this->purgeMode;
     }
 
-    /** @inheritDoc */
-    public function setEntityManager(EntityManagerInterface $em)
+    public function setEntityManager(EntityManagerInterface $em): void
     {
         $this->em                  = $em;
         $this->cachedSqlStatements = null;
     }
 
     /**
      * Retrieve the EntityManagerInterface instance this purger instance is using.
-     *
-     * @return EntityManagerInterface
      */
-    public function getObjectManager()
+    public function getObjectManager(): EntityManagerInterface
     {
         return $this->em;
     }
 
-    /** @inheritDoc */
-    public function purge()
+    public function purge(): void
     {
         $connection = $this->em->getConnection();
         array_map([$connection, 'executeStatement'], $this->getPurgeStatements());