diff --git a/lib/Doctrine/Common/DataFixtures/Event/Listener/MongoDBReferenceListener.php b/lib/Doctrine/Common/DataFixtures/Event/Listener/MongoDBReferenceListener.php index ac044424..70b4a402 100644 --- a/lib/Doctrine/Common/DataFixtures/Event/Listener/MongoDBReferenceListener.php +++ b/lib/Doctrine/Common/DataFixtures/Event/Listener/MongoDBReferenceListener.php @@ -17,9 +17,6 @@ final class MongoDBReferenceListener implements EventSubscriber /** @var ReferenceRepository */ private $referenceRepository; - /** - * Initialize listener - */ public function __construct(ReferenceRepository $referenceRepository) { $this->referenceRepository = $referenceRepository; @@ -35,6 +32,8 @@ public function getSubscribedEvents() /** * Populates identities for stored references + * + * @return void */ public function postPersist(LifecycleEventArgs $args) { diff --git a/lib/Doctrine/Common/DataFixtures/Event/Listener/ORMReferenceListener.php b/lib/Doctrine/Common/DataFixtures/Event/Listener/ORMReferenceListener.php index ba43e377..9c721bfd 100644 --- a/lib/Doctrine/Common/DataFixtures/Event/Listener/ORMReferenceListener.php +++ b/lib/Doctrine/Common/DataFixtures/Event/Listener/ORMReferenceListener.php @@ -17,9 +17,6 @@ final class ORMReferenceListener implements EventSubscriber /** @var ReferenceRepository */ private $referenceRepository; - /** - * Initialize listener - */ public function __construct(ReferenceRepository $referenceRepository) { $this->referenceRepository = $referenceRepository; @@ -36,6 +33,8 @@ public function getSubscribedEvents() /** * Populates identities for stored references + * + * @return void */ public function postPersist(LifecycleEventArgs $args) { diff --git a/lib/Doctrine/Common/DataFixtures/Executor/AbstractExecutor.php b/lib/Doctrine/Common/DataFixtures/Executor/AbstractExecutor.php index f8f09f89..921d3fef 100644 --- a/lib/Doctrine/Common/DataFixtures/Executor/AbstractExecutor.php +++ b/lib/Doctrine/Common/DataFixtures/Executor/AbstractExecutor.php @@ -62,6 +62,8 @@ public function setReferenceRepository(ReferenceRepository $referenceRepository) /** * Sets the Purger instance to use for this executor instance. + * + * @return void */ public function setPurger(PurgerInterface $purger) { @@ -80,6 +82,8 @@ public function getPurger() * Set the logger callable to execute with the log() method. * * @param callable $logger + * + * @return void */ public function setLogger($logger) { @@ -90,6 +94,8 @@ public function setLogger($logger) * Logs a message using the logger. * * @param string $message + * + * @return void */ public function log($message) { @@ -99,6 +105,8 @@ public function log($message) /** * Load a fixture with the given persistence manager. + * + * @return void */ public function load(ObjectManager $manager, FixtureInterface $fixture) { @@ -123,6 +131,8 @@ 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() @@ -144,8 +154,10 @@ public function purge() /** * Executes the given array of data fixtures. * - * @param array $fixtures Array of fixtures to execute. - * @param bool $append Whether to append the data fixtures or purge the database before loading. + * @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, $append = false); } diff --git a/lib/Doctrine/Common/DataFixtures/Loader.php b/lib/Doctrine/Common/DataFixtures/Loader.php index 69c6bb0d..668eef2b 100644 --- a/lib/Doctrine/Common/DataFixtures/Loader.php +++ b/lib/Doctrine/Common/DataFixtures/Loader.php @@ -181,7 +181,7 @@ public function addFixture(FixtureInterface $fixture) /** * Returns the array of data fixtures to execute. * - * @return array $fixtures + * @psalm-return array|int, OrderedFixtureInterface> */ public function getFixtures() { diff --git a/lib/Doctrine/Common/DataFixtures/ProxyReferenceRepository.php b/lib/Doctrine/Common/DataFixtures/ProxyReferenceRepository.php index c5575bb4..8bb7b19c 100644 --- a/lib/Doctrine/Common/DataFixtures/ProxyReferenceRepository.php +++ b/lib/Doctrine/Common/DataFixtures/ProxyReferenceRepository.php @@ -62,6 +62,8 @@ public function serialize() * Unserialize reference repository * * @param string $serializedData Serialized data + * + * @return void */ public function unserialize($serializedData) { @@ -115,6 +117,8 @@ public function load($baseCacheName) * Save data fixture reference repository * * @param string $baseCacheName Base cache name + * + * @return void */ public function save($baseCacheName) { diff --git a/lib/Doctrine/Common/DataFixtures/Purger/MongoDBPurger.php b/lib/Doctrine/Common/DataFixtures/Purger/MongoDBPurger.php index 92f52fb8..b03241a5 100644 --- a/lib/Doctrine/Common/DataFixtures/Purger/MongoDBPurger.php +++ b/lib/Doctrine/Common/DataFixtures/Purger/MongoDBPurger.php @@ -18,6 +18,8 @@ class MongoDBPurger implements PurgerInterface * Construct new purger instance. * * @param DocumentManager $dm DocumentManager instance used for persistence. + * + * @return void */ public function __construct(?DocumentManager $dm = null) { @@ -26,6 +28,8 @@ public function __construct(?DocumentManager $dm = null) /** * Set the DocumentManager instance this purger instance should use. + * + * @return void */ public function setDocumentManager(DocumentManager $dm) { diff --git a/lib/Doctrine/Common/DataFixtures/Purger/ORMPurger.php b/lib/Doctrine/Common/DataFixtures/Purger/ORMPurger.php index 574a7a2e..815a5536 100644 --- a/lib/Doctrine/Common/DataFixtures/Purger/ORMPurger.php +++ b/lib/Doctrine/Common/DataFixtures/Purger/ORMPurger.php @@ -225,9 +225,9 @@ private function getCommitOrder(EntityManagerInterface $em, array $classes) } /** - * @param array $classes + * @param ClassMetadata[] $classes * - * @return array + * @return string[] */ private function getAssociationTables(array $classes, AbstractPlatform $platform) { diff --git a/lib/Doctrine/Common/DataFixtures/ReferenceRepository.php b/lib/Doctrine/Common/DataFixtures/ReferenceRepository.php index 681f7ba0..97329808 100644 --- a/lib/Doctrine/Common/DataFixtures/ReferenceRepository.php +++ b/lib/Doctrine/Common/DataFixtures/ReferenceRepository.php @@ -26,7 +26,7 @@ class ReferenceRepository * List of named references to the fixture objects * gathered during loads of fixtures * - * @var array + * @psalm-var array */ private $references = []; @@ -35,7 +35,7 @@ class ReferenceRepository * in case if reference gets unmanaged, it will * use a proxy referenced by this identity * - * @var array + * @psalm-var array */ private $identities = []; @@ -89,6 +89,8 @@ protected function getIdentifier($reference, $uow) * * @param string $name * @param object $reference + * + * @return void */ public function setReference($name, $reference) { @@ -108,6 +110,8 @@ public function setReference($name, $reference) * * @param string $name * @param mixed $identity + * + * @return void */ public function setReferenceIdentity($name, $identity) { @@ -213,7 +217,7 @@ public function hasIdentity($name) /** * Get all stored identities * - * @return array + * @psalm-return array */ public function getIdentities() { @@ -223,7 +227,7 @@ public function getIdentities() /** * Get all stored references * - * @return array + * @psalm-return array */ public function getReferences() { diff --git a/lib/Doctrine/Common/DataFixtures/Sorter/TopologicalSorter.php b/lib/Doctrine/Common/DataFixtures/Sorter/TopologicalSorter.php index e5be98c5..e1321a70 100644 --- a/lib/Doctrine/Common/DataFixtures/Sorter/TopologicalSorter.php +++ b/lib/Doctrine/Common/DataFixtures/Sorter/TopologicalSorter.php @@ -34,7 +34,7 @@ class TopologicalSorter /** * Volatile variable holding calculated nodes during sorting process. * - * @var array + * @var ClassMetadata[] */ private $sortedNodeList = []; @@ -46,8 +46,6 @@ class TopologicalSorter private $allowCyclicDependencies; /** - * Construct TopologicalSorter object - * * @param bool $allowCyclicDependencies */ public function __construct($allowCyclicDependencies = true) @@ -100,7 +98,7 @@ public function addDependency($fromHash, $toHash) * * Note: Highly performance-sensitive method. * - * @return array + * @return ClassMetadata[] * * @throws RuntimeException * @throws CircularReferenceException @@ -128,6 +126,8 @@ public function sort() * * Note: Highly performance-sensitive method. * + * @return void + * * @throws RuntimeException * @throws CircularReferenceException */ diff --git a/tests/Doctrine/Tests/Common/DataFixtures/Purger/ORMPurgerExcludeTest.php b/tests/Doctrine/Tests/Common/DataFixtures/Purger/ORMPurgerExcludeTest.php index fce00575..8f5d9128 100644 --- a/tests/Doctrine/Tests/Common/DataFixtures/Purger/ORMPurgerExcludeTest.php +++ b/tests/Doctrine/Tests/Common/DataFixtures/Purger/ORMPurgerExcludeTest.php @@ -72,7 +72,7 @@ protected function loadTestData() * Execute test purge * * @param string|null $expression - * @param array $list + * @param string[] $list */ public function executeTestPurge($expression, array $list, ?callable $filter = null): void {