Merge pull request #455 from GromNaN/param-types

Add parameter types

src/AbstractFixture.php

@@ -45,12 +45,11 @@ private function getReferenceRepository(): ReferenceRepository
      *
      * @see Doctrine\Common\DataFixtures\ReferenceRepository::setReference
      *
-     * @param string $name
      * @param object $object - managed object
      *
      * @return void
      */
-    public function setReference($name, $object)
+    public function setReference(string $name, object $object)
     {
         $this->getReferenceRepository()->setReference($name, $object);
     }
@@ -63,14 +62,13 @@ public function setReference($name, $object)
      *
      * @see Doctrine\Common\DataFixtures\ReferenceRepository::addReference
      *
-     * @param string $name
      * @param object $object - managed object
      *
      * @return void
      *
      * @throws BadMethodCallException - if repository already has a reference by $name.
      */
-    public function addReference($name, $object)
+    public function addReference(string $name, object $object)
     {
         $this->getReferenceRepository()->addReference($name, $object);
     }
@@ -81,15 +79,14 @@ public function addReference($name, $object)
      *
      * @see Doctrine\Common\DataFixtures\ReferenceRepository::getReference
      *
-     * @param string $name
      * @psalm-param class-string<T>|null $class
      *
      * @return object
      * @psalm-return ($class is null ? object : T)
      *
      * @template T of object
      */
-    public function getReference($name, ?string $class = null)
+    public function getReference(string $name, ?string $class = null)
     {
         if ($class === null) {
             Deprecation::trigger(
@@ -109,12 +106,11 @@ public function getReference($name, ?string $class = null)
      *
      * @see Doctrine\Common\DataFixtures\ReferenceRepository::hasReference
      *
-     * @param string $name
-     * @psalm-param class-string $class
+     * @psalm-param class-string|null $class
      *
      * @return bool
      */
-    public function hasReference($name, ?string $class = null)
+    public function hasReference(string $name, ?string $class = null)
     {
         if ($class === null) {
             Deprecation::trigger(

src/Executor/AbstractExecutor.php

@@ -76,23 +76,19 @@ public function getPurger()
     /**
      * Set the logger callable to execute with the log() method.
      *
-     * @param callable $logger
-     *
      * @return void
      */
-    public function setLogger($logger)
+    public function setLogger(callable $logger)
     {
         $this->logger = $logger;
     }
 
     /**
      * Logs a message using the logger.
      *
-     * @param string $message
-     *
      * @return void
      */
-    public function log($message)
+    public function log(string $message)
     {
         $logger = $this->logger;
         $logger($message);

src/Executor/MongoDBExecutor.php

@@ -60,7 +60,7 @@ public function setReferenceRepository(ReferenceRepository $referenceRepository)
     }
 
     /** @inheritDoc */
-    public function execute(array $fixtures, $append = false)
+    public function execute(array $fixtures, bool $append = false)
     {
         if ($append === false) {
             $this->purge();

src/Executor/MultipleTransactionORMExecutor.php

@@ -11,7 +11,7 @@ final class MultipleTransactionORMExecutor extends AbstractExecutor
     use ORMExecutorCommon;
 
     /** @inheritDoc */
-    public function execute(array $fixtures, $append = false): void
+    public function execute(array $fixtures, bool $append = false): void
     {
         $executor = $this;
         if ($append === false) {

src/Executor/ORMExecutor.php

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

src/Executor/PHPCRExecutor.php

@@ -40,7 +40,7 @@ public function getObjectManager()
     }
 
     /** @inheritDoc */
-    public function execute(array $fixtures, $append = false)
+    public function execute(array $fixtures, bool $append = false)
     {
         $that = $this;
 

src/Loader.php

@@ -73,7 +73,7 @@ class Loader
      *
      * @return array $fixtures Array of loaded fixture object instances.
      */
-    public function loadFromDirectory($dir)
+    public function loadFromDirectory(string $dir)
     {
         if (! is_dir($dir)) {
             throw new InvalidArgumentException(sprintf('"%s" does not exist', $dir));
@@ -94,7 +94,7 @@ public function loadFromDirectory($dir)
      *
      * @return array $fixtures Array of loaded fixture object instances.
      */
-    public function loadFromFile($fileName)
+    public function loadFromFile(string $fileName)
     {
         if (! is_readable($fileName)) {
             throw new InvalidArgumentException(sprintf('"%s" does not exist or is not readable', $fileName));
@@ -108,23 +108,19 @@ public function loadFromFile($fileName)
     /**
      * Has fixture?
      *
-     * @param FixtureInterface $fixture
-     *
      * @return bool
      */
-    public function hasFixture($fixture)
+    public function hasFixture(FixtureInterface $fixture)
     {
         return isset($this->fixtures[get_class($fixture)]);
     }
 
     /**
      * Get a specific fixture instance
      *
-     * @param string $className
-     *
      * @return FixtureInterface
      */
-    public function getFixture($className)
+    public function getFixture(string $className)
     {
         if (! isset($this->fixtures[$className])) {
             throw new InvalidArgumentException(sprintf(
@@ -204,7 +200,7 @@ public function getFixtures()
      *
      * @return bool
      */
-    public function isTransient($className)
+    public function isTransient(string $className)
     {
         $rc = new ReflectionClass($className);
         if ($rc->isAbstract()) {
@@ -219,11 +215,9 @@ public function isTransient($className)
     /**
      * Creates the fixture object from the class.
      *
-     * @param string $class
-     *
      * @return FixtureInterface
      */
-    protected function createFixture($class)
+    protected function createFixture(string $class)
     {
         return new $class();
     }

src/ProxyReferenceRepository.php

@@ -49,7 +49,7 @@ public function serialize()
      *
      * @return void
      */
-    public function unserialize($serializedData)
+    public function unserialize(string $serializedData)
     {
         $repositoryData = unserialize($serializedData);
 
@@ -98,7 +98,7 @@ public function unserialize($serializedData)
      *
      * @return bool
      */
-    public function load($baseCacheName)
+    public function load(string $baseCacheName)
     {
         $filename = $baseCacheName . '.ser';
 
@@ -124,7 +124,7 @@ public function load($baseCacheName)
      *
      * @return void
      */
-    public function save($baseCacheName)
+    public function save(string $baseCacheName)
     {
         $serializedData = $this->serialize();
 

src/Purger/ORMPurger.php

@@ -59,11 +59,9 @@ public function __construct(?EntityManagerInterface $em = null, array $excluded
     /**
      * Set the purge mode
      *
-     * @param int $mode
-     *
      * @return void
      */
-    public function setPurgeMode($mode)
+    public function setPurgeMode(int $mode)
     {
         $this->purgeMode           = $mode;
         $this->cachedSqlStatements = null;

src/ReferenceRepository.php

@@ -75,7 +75,7 @@ public function __construct(ObjectManager $manager)
      *
      * @return array
      */
-    protected function getIdentifier($reference, $uow)
+    protected function getIdentifier(object $reference, object $uow)
     {
         // In case Reference is not yet managed in UnitOfWork
         if (! $this->hasIdentifier($reference)) {
@@ -103,12 +103,9 @@ protected function getIdentifier($reference, $uow)
      * and referenced to $reference. If $name
      * already is set, it overrides it
      *
-     * @param string $name
-     * @param object $reference
-     *
      * @return void
      */
-    public function setReference($name, $reference)
+    public function setReference(string $name, object $reference)
     {
         $class = $this->getRealClass(get_class($reference));
 
@@ -134,13 +131,12 @@ public function setReference($name, $reference)
     /**
      * Store the identifier of a reference
      *
-     * @param string            $name
      * @param mixed             $identity
      * @param class-string|null $class
      *
      * @return void
      */
-    public function setReferenceIdentity($name, $identity, ?string $class = null)
+    public function setReferenceIdentity(string $name, $identity, ?string $class = null)
     {
         if ($class === null) {
             Deprecation::trigger(
@@ -166,14 +162,13 @@ public function setReferenceIdentity($name, $identity, ?string $class = null)
      * the record is inserted, be sure tu use this method
      * after $object is flushed
      *
-     * @param string $name
      * @param object $object - managed object
      *
      * @return void
      *
      * @throws BadMethodCallException - if repository already has a reference by $name.
      */
-    public function addReference($name, $object)
+    public function addReference(string $name, object $object)
     {
         // For BC, to be removed in next major.
         if (isset($this->references[$name])) {
@@ -199,7 +194,6 @@ public function addReference($name, $object)
      * Loads an object using stored reference
      * named by $name
      *
-     * @param string $name
      * @psalm-param class-string<T>|null $class
      *
      * @return object
@@ -209,7 +203,7 @@ public function addReference($name, $object)
      *
      * @template T of object
      */
-    public function getReference($name, ?string $class = null)
+    public function getReference(string $name, ?string $class = null)
     {
         if ($class === null) {
             Deprecation::trigger(
@@ -256,12 +250,11 @@ public function getReference($name, ?string $class = null)
      * Check if an object is stored using reference
      * named by $name
      *
-     * @param string $name
      * @psalm-param class-string $class
      *
      * @return bool
      */
-    public function hasReference($name, ?string $class = null)
+    public function hasReference(string $name, ?string $class = null)
     {
         if ($class === null) {
             Deprecation::trigger(
@@ -281,11 +274,9 @@ public function hasReference($name, ?string $class = null)
      * Searches for reference names in the
      * list of stored references
      *
-     * @param object $reference
-     *
      * @return array<string>
      */
-    public function getReferenceNames($reference)
+    public function getReferenceNames(object $reference)
     {
         $class = $this->getRealClass(get_class($reference));
         if (! isset($this->referencesByClass[$class])) {
@@ -298,12 +289,11 @@ public function getReferenceNames($reference)
     /**
      * Checks if reference has identity stored
      *
-     * @param string            $name
      * @param class-string|null $class
      *
      * @return bool
      */
-    public function hasIdentity($name, ?string $class = null)
+    public function hasIdentity(string $name, ?string $class = null)
     {
         if ($class === null) {
             Deprecation::trigger(
@@ -380,19 +370,17 @@ public function getManager()
      *
      * @return string
      */
-    protected function getRealClass($className)
+    protected function getRealClass(string $className)
     {
         return $this->manager->getClassMetadata($className)->getName();
     }
 
     /**
      * Checks if object has identifier already in unit of work.
      *
-     * @param object $reference
-     *
      * @return bool
      */
-    private function hasIdentifier($reference)
+    private function hasIdentifier(object $reference)
     {
         // in case if reference is set after flush, store its identity
         $uow = $this->manager->getUnitOfWork();