doctrine / orm

lib/Doctrine/ORM/Internal/Hydration/AbstractHydrator.php

Indentation +2 added lines, -2 removed lines

@@ -213,8 +213,8 @@
         $this->metadataCache = [];
 
         $this->em
-             ->getEventManager()
-             ->removeEventListener([Events::onClear], $this);
+                ->getEventManager()
+                ->removeEventListener([Events::onClear], $this);
     }
 
     /**

Spacing +6 added lines, -6 removed lines

@@ -150,7 +150,7 @@ 
     {
         $row = $this->stmt->fetch(FetchMode::ASSOCIATIVE);
 
-        if (! $row) {
+        if ( ! $row) {
             $this->cleanup();
 
             return false;
@@ -292,7 +292,7 @@ 
                         : $value;
 
                     if ($cacheKeyInfo['isIdentifier'] && $value !== null) {
-                        $id[$dqlAlias]                .= '|' . $value;
+                        $id[$dqlAlias]                .= '|'.$value;
                         $nonemptyComponents[$dqlAlias] = true;
                     }
                     break;
@@ -328,10 +328,10 @@ 
 
             // WARNING: BC break! We know this is the desired behavior to type convert values, but this
             // erroneous behavior exists since 2.0 and we're forced to keep compatibility.
-            if (! isset($cacheKeyInfo['isScalar'])) {
+            if ( ! isset($cacheKeyInfo['isScalar'])) {
                 $dqlAlias  = $cacheKeyInfo['dqlAlias'];
                 $type      = $cacheKeyInfo['type'];
-                $fieldName = $dqlAlias . '_' . $fieldName;
+                $fieldName = $dqlAlias.'_'.$fieldName;
                 $value     = $type
                     ? $type->convertToPHPValue($value, $this->platform)
                     : $value;
@@ -435,7 +435,7 @@ 
     private function getDiscriminatorValues(ClassMetadata $classMetadata) : array
     {
         $values = array_map(
-            function (string $subClass) : string {
+            function(string $subClass) : string {
                 return (string) $this->getClassMetadata($subClass)->discriminatorValue;
             },
             $classMetadata->getSubClasses()
@@ -455,7 +455,7 @@ 
      */
     protected function getClassMetadata($className)
     {
-        if (! isset($this->metadataCache[$className])) {
+        if ( ! isset($this->metadataCache[$className])) {
             $this->metadataCache[$className] = $this->em->getClassMetadata($className);
         }
 

tests/Doctrine/Tests/ORM/PersistentCollectionTest.php

Spacing +3 added lines, -3 removed lines

@@ -168,7 +168,7 @@ 
             ->expects(self::once())
             ->method('loadCollection')
             ->with($this->collection)
-            ->willReturnCallback(function (PersistentCollection $persistentCollection) use ($persistedElement) : void {
+            ->willReturnCallback(function(PersistentCollection $persistentCollection) use ($persistedElement) : void {
                 $persistentCollection->unwrap()->add($persistedElement);
             });
 
@@ -205,7 +205,7 @@ 
             ->expects(self::once())
             ->method('loadCollection')
             ->with($this->collection)
-            ->willReturnCallback(function (PersistentCollection $persistentCollection) use (
+            ->willReturnCallback(function(PersistentCollection $persistentCollection) use (
                 $persistedElement,
                 $newElementThatIsAlsoPersisted
             ) : void {
@@ -247,7 +247,7 @@ 
             ->expects(self::once())
             ->method('loadCollection')
             ->with($this->collection)
-            ->willReturnCallback(function (PersistentCollection $persistentCollection) use (
+            ->willReturnCallback(function(PersistentCollection $persistentCollection) use (
                 $persistedElement,
                 $newElementThatIsAlsoPersisted
             ) : void {

tests/Doctrine/Tests/ORM/Tools/Console/Command/InfoCommandTest.php

Indentation +8 added lines, -8 removed lines

@@ -60,13 +60,13 @@ 
         $em            = $this->createMock(EntityManagerInterface::class);
 
         $mappingDriver->method('getAllClassNames')
-                      ->willReturn([]);
+                        ->willReturn([]);
 
         $configuration->method('getMetadataDriverImpl')
-                      ->willReturn($mappingDriver);
+                        ->willReturn($mappingDriver);
 
         $em->method('getConfiguration')
-           ->willReturn($configuration);
+            ->willReturn($configuration);
 
         $application = new Application();
         $application->setHelperSet(new HelperSet(['em' => new EntityManagerHelper($em)]));
@@ -95,17 +95,17 @@ 
         $em            = $this->createMock(EntityManagerInterface::class);
 
         $mappingDriver->method('getAllClassNames')
-                      ->willReturn(['InvalidEntity']);
+                        ->willReturn(['InvalidEntity']);
 
         $configuration->method('getMetadataDriverImpl')
-                      ->willReturn($mappingDriver);
+                        ->willReturn($mappingDriver);
 
         $em->method('getConfiguration')
-           ->willReturn($configuration);
+            ->willReturn($configuration);
 
         $em->method('getClassMetadata')
-           ->with('InvalidEntity')
-           ->willThrowException(new MappingException('exception message'));
+            ->with('InvalidEntity')
+            ->willThrowException(new MappingException('exception message'));
 
         $application = new Application();
         $application->setHelperSet(new HelperSet(['em' => new EntityManagerHelper($em)]));

tests/Doctrine/Tests/ORM/Tools/Console/Command/RunDqlCommandTest.php

Spacing +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
             $this->tester->execute(
                 [
                     'command' => $this->command->getName(),
-                    'dql'     => 'SELECT e FROM ' . DateTimeModel::class . ' e',
+                    'dql'     => 'SELECT e FROM '.DateTimeModel::class.' e',
                 ]
             )
         );
@@ -80,7 +80,7 @@ 
             $this->tester->execute(
                 [
                     'command'    => $this->command->getName(),
-                    'dql'        => 'SELECT e FROM ' . DateTimeModel::class . ' e',
+                    'dql'        => 'SELECT e FROM '.DateTimeModel::class.' e',
                     '--show-sql' => 'true',
                 ]
             )

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

Indentation +8 added lines, -8 removed lines

@@ -45,14 +45,14 @@
     protected function configure()
     {
         $this->setName('orm:run-dql')
-             ->setDescription('Executes arbitrary DQL directly from the command line')
-             ->addArgument('dql', InputArgument::REQUIRED, 'The DQL to execute.')
-             ->addOption('hydrate', null, InputOption::VALUE_REQUIRED, 'Hydration mode of result set. Should be either: object, array, scalar or single-scalar.', 'object')
-             ->addOption('first-result', null, InputOption::VALUE_REQUIRED, 'The first result in the result set.')
-             ->addOption('max-result', null, InputOption::VALUE_REQUIRED, 'The maximum number of results in the result set.')
-             ->addOption('depth', null, InputOption::VALUE_REQUIRED, 'Dumping depth of Entity graph.', 7)
-             ->addOption('show-sql', null, InputOption::VALUE_NONE, 'Dump generated SQL instead of executing query')
-             ->setHelp('Executes arbitrary DQL directly from the command line.');
+                ->setDescription('Executes arbitrary DQL directly from the command line')
+                ->addArgument('dql', InputArgument::REQUIRED, 'The DQL to execute.')
+                ->addOption('hydrate', null, InputOption::VALUE_REQUIRED, 'Hydration mode of result set. Should be either: object, array, scalar or single-scalar.', 'object')
+                ->addOption('first-result', null, InputOption::VALUE_REQUIRED, 'The first result in the result set.')
+                ->addOption('max-result', null, InputOption::VALUE_REQUIRED, 'The maximum number of results in the result set.')
+                ->addOption('depth', null, InputOption::VALUE_REQUIRED, 'Dumping depth of Entity graph.', 7)
+                ->addOption('show-sql', null, InputOption::VALUE_NONE, 'Dump generated SQL instead of executing query')
+                ->setHelp('Executes arbitrary DQL directly from the command line.');
     }
 
     /**

Spacing +5 added lines, -5 removed lines

@@ -57,14 +57,14 @@ 
 
         $depth = $input->getOption('depth');
 
-        if (! is_numeric($depth)) {
+        if ( ! is_numeric($depth)) {
             throw new \LogicException("Option 'depth' must contain an integer value");
         }
 
         $hydrationModeName = $input->getOption('hydrate');
-        $hydrationMode     = 'Doctrine\ORM\Query::HYDRATE_' . strtoupper(str_replace('-', '_', $hydrationModeName));
+        $hydrationMode     = 'Doctrine\ORM\Query::HYDRATE_'.strtoupper(str_replace('-', '_', $hydrationModeName));
 
-        if (! defined($hydrationMode)) {
+        if ( ! defined($hydrationMode)) {
             throw new \RuntimeException(sprintf(
                 "Hydration mode '%s' does not exist. It should be either: object. array, scalar or single-scalar.",
                 $hydrationModeName
@@ -75,7 +75,7 @@ 
         $firstResult = $input->getOption('first-result');
 
         if ($firstResult !== null) {
-            if (! is_numeric($firstResult)) {
+            if ( ! is_numeric($firstResult)) {
                 throw new \LogicException("Option 'first-result' must contain an integer value");
             }
 
@@ -85,7 +85,7 @@ 
         $maxResult = $input->getOption('max-result');
 
         if ($maxResult !== null) {
-            if (! is_numeric($maxResult)) {
+            if ( ! is_numeric($maxResult)) {
                 throw new \LogicException("Option 'max-result' must contain an integer value");
             }
 

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

Indentation +4 added lines, -4 removed lines

@@ -45,10 +45,10 @@
     protected function configure()
     {
         $this->setName('orm:validate-schema')
-             ->setDescription('Validate the mapping files')
-             ->addOption('skip-mapping', null, InputOption::VALUE_NONE, 'Skip the mapping validation check')
-             ->addOption('skip-sync', null, InputOption::VALUE_NONE, 'Skip checking if the mapping is in sync with the database')
-             ->setHelp('Validate that the mapping files are correct and in sync with the database.');
+                ->setDescription('Validate the mapping files')
+                ->addOption('skip-mapping', null, InputOption::VALUE_NONE, 'Skip the mapping validation check')
+                ->addOption('skip-sync', null, InputOption::VALUE_NONE, 'Skip checking if the mapping is in sync with the database')
+                ->setHelp('Validate that the mapping files are correct and in sync with the database.');
     }
 
     /**

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
             $ui->text('<comment>[SKIPPED] The mapping was not checked.</comment>');
         }
 
-        if (! $skipMapping) {
+        if ( ! $skipMapping) {
             $errors = $validator->validateMapping();
 
             if ($errors) {
@@ -72,7 +72,7 @@ 
 
         if ($input->getOption('skip-sync')) {
             $ui->text('<comment>[SKIPPED] The database was not checked for synchronicity.</comment>');
-        } elseif (! $validator->schemaInSyncWithMetadata()) {
+        } elseif ( ! $validator->schemaInSyncWithMetadata()) {
             $ui->error('The database schema is not in sync with the current mapping file.');
             $exit += 2;
         } else {

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

Indentation +3 added lines, -3 removed lines

@@ -45,9 +45,9 @@
     protected function configure()
     {
         $this->setName('orm:ensure-production-settings')
-             ->setDescription('Verify that Doctrine is properly configured for a production environment')
-             ->addOption('complete', null, InputOption::VALUE_NONE, 'Flag to also inspect database connection existence.')
-             ->setHelp('Verify that Doctrine is properly configured for a production environment.');
+                ->setDescription('Verify that Doctrine is properly configured for a production environment')
+                ->addOption('complete', null, InputOption::VALUE_NONE, 'Flag to also inspect database connection existence.')
+                ->setHelp('Verify that Doctrine is properly configured for a production environment.');
     }
 
     /**

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

Indentation +5 added lines, -5 removed lines

@@ -46,11 +46,11 @@
     protected function configure()
     {
         $this->setName('orm:generate-proxies')
-             ->setAliases(['orm:generate:proxies'])
-             ->setDescription('Generates proxy classes for entity classes')
-             ->addArgument('dest-path', InputArgument::OPTIONAL, 'The path to generate your proxy classes. If none is provided, it will attempt to grab from configuration.')
-             ->addOption('filter', null, InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY, 'A string pattern used to match entities that should be processed.')
-             ->setHelp('Generates proxy classes for entity classes.');
+                ->setAliases(['orm:generate:proxies'])
+                ->setDescription('Generates proxy classes for entity classes')
+                ->addArgument('dest-path', InputArgument::OPTIONAL, 'The path to generate your proxy classes. If none is provided, it will attempt to grab from configuration.')
+                ->addOption('filter', null, InputOption::VALUE_REQUIRED | InputOption::VALUE_IS_ARRAY, 'A string pattern used to match entities that should be processed.')
+                ->setHelp('Generates proxy classes for entity classes.');
     }
 
     /**

Spacing +3 added lines, -3 removed lines

@@ -56,19 +56,19 @@
             $destPath = $em->getConfiguration()->getProxyDir();
         }
 
-        if (! is_dir($destPath)) {
+        if ( ! is_dir($destPath)) {
             mkdir($destPath, 0775, true);
         }
 
         $destPath = realpath($destPath);
 
-        if (! file_exists($destPath)) {
+        if ( ! file_exists($destPath)) {
             throw new \InvalidArgumentException(
                 sprintf("Proxies destination directory '<info>%s</info>' does not exist.", $em->getConfiguration()->getProxyDir())
             );
         }
 
-        if (! is_writable($destPath)) {
+        if ( ! is_writable($destPath)) {
             throw new \InvalidArgumentException(
                 sprintf("Proxies destination directory '<info>%s</info>' does not have write permissions.", $destPath)
             );

tests/Doctrine/Tests/ORM/Functional/ManyToManyBasicAssociationTest.php

Indentation +2 added lines, -2 removed lines

@@ -268,8 +268,8 @@
         $this->em->clear();
 
         $newUser = $this->em->createQuery('SELECT u, g FROM Doctrine\Tests\Models\CMS\CmsUser u LEFT JOIN u.groups g WHERE u.id = ?1')
-                             ->setParameter(1, $user->getId())
-                             ->getSingleResult();
+                                ->setParameter(1, $user->getId())
+                                ->getSingleResult();
         self::assertCount(0, $newUser->groups);
         self::assertInstanceOf(ManyToManyAssociationMetadata::class, $newUser->groups->getMapping());
 

Spacing +4 added lines, -4 removed lines

@@ -293,9 +293,9 @@ 
         $user->username = 'gblanco';
         $user->status   = 'developer';
 
-        for ($i=0; $i < $groupCount; ++$i) {
+        for ($i = 0; $i < $groupCount; ++$i) {
             $group       = new CmsGroup();
-            $group->name = 'Developers_' . $i;
+            $group->name = 'Developers_'.$i;
             $user->addGroup($group);
         }
 
@@ -427,7 +427,7 @@ 
             $user
                 ->getGroups()
                 ->matching($criteria)
-                ->map(function (CmsGroup $group) {
+                ->map(function(CmsGroup $group) {
                     return $group->getName();
                 })
                 ->toArray()
@@ -471,7 +471,7 @@ 
             $user
                 ->getTags()
                 ->matching($criteria)
-                ->map(function (CmsTag $tag) {
+                ->map(function(CmsTag $tag) {
                     return $tag->getName();
                 })
                 ->toArray()