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

@@ -109,7 +109,7 @@ 
     public function iterate($stmt, $resultSetMapping, array $hints = [])
     {
         @trigger_error(
-            'Method ' . __METHOD__ . '() is deprecated and will be removed in Doctrine ORM 3.0. Use getIterable() instead.',
+            'Method '.__METHOD__.'() is deprecated and will be removed in Doctrine ORM 3.0. Use getIterable() instead.',
             E_USER_DEPRECATED
         );
 
@@ -342,7 +342,7 @@ 
                         : $value;
 
                     if ($cacheKeyInfo['isIdentifier'] && $value !== null) {
-                        $id[$dqlAlias]                .= '|' . $value;
+                        $id[$dqlAlias]                .= '|'.$value;
                         $nonemptyComponents[$dqlAlias] = true;
                     }
                     break;
@@ -378,10 +378,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;
@@ -482,7 +482,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()
@@ -502,7 +502,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/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/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

@@ -57,19 +57,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

@@ -294,9 +294,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);
         }
 
@@ -428,7 +428,7 @@ 
             $user
                 ->getGroups()
                 ->matching($criteria)
-                ->map(static function (CmsGroup $group) {
+                ->map(static function(CmsGroup $group) {
                     return $group->getName();
                 })
                 ->toArray()
@@ -472,7 +472,7 @@ 
             $user
                 ->getTags()
                 ->matching($criteria)
-                ->map(static function (CmsTag $tag) {
+                ->map(static function(CmsTag $tag) {
                     return $tag->getName();
                 })
                 ->toArray()

tests/Doctrine/Tests/ORM/Functional/Ticket/DDC1400Test.php

Indentation +2 added lines, -2 removed lines

@@ -54,8 +54,8 @@
         $user1 = $this->em->getReference(DDC1400User::class, $user1->id);
 
         $this->em->createQuery('SELECT a, s FROM ' . DDC1400Article::class . ' a JOIN a.userStates s WITH s.user = :activeUser')
-                  ->setParameter('activeUser', $user1)
-                  ->getResult();
+                    ->setParameter('activeUser', $user1)
+                    ->getResult();
 
         $queryCount = $this->getCurrentQueryCount();
 

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 
         $user1 = $this->em->getReference(DDC1400User::class, $user1->id);
 
-        $this->em->createQuery('SELECT a, s FROM ' . DDC1400Article::class . ' a JOIN a.userStates s WITH s.user = :activeUser')
+        $this->em->createQuery('SELECT a, s FROM '.DDC1400Article::class.' a JOIN a.userStates s WITH s.user = :activeUser')
                   ->setParameter('activeUser', $user1)
                   ->getResult();
 

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

Indentation +6 added lines, -6 removed lines

@@ -28,11 +28,11 @@ 
     protected function configure()
     {
         $this->setName('orm:schema-tool:drop')
-             ->setDescription('Drop the complete database schema of EntityManager Storage Connection or generate the corresponding SQL output')
-             ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Instead of trying to apply generated SQLs into EntityManager Storage Connection, output them.')
-             ->addOption('force', 'f', InputOption::VALUE_NONE, "Don't ask for the deletion of the database, but force the operation to run.")
-             ->addOption('full-database', null, InputOption::VALUE_NONE, 'Instead of using the Class Metadata to detect the database table schema, drop ALL assets that the database contains.')
-             ->setHelp(<<<'EOT'
+                ->setDescription('Drop the complete database schema of EntityManager Storage Connection or generate the corresponding SQL output')
+                ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Instead of trying to apply generated SQLs into EntityManager Storage Connection, output them.')
+                ->addOption('force', 'f', InputOption::VALUE_NONE, "Don't ask for the deletion of the database, but force the operation to run.")
+                ->addOption('full-database', null, InputOption::VALUE_NONE, 'Instead of using the Class Metadata to detect the database table schema, drop ALL assets that the database contains.')
+                ->setHelp(<<<'EOT'
 Processes the schema and either drop the database schema of EntityManager Storage Connection or generate the SQL output.
 Beware that the complete database is dropped by this command, even tables that are not relevant to your metadata model.
 
@@ -42,7 +42,7 @@ 
 
     $config->setFilterSchemaAssetsExpression($regexp);
 EOT
-             );
+                );
     }
 
     /**