doctrine / orm

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

Indentation +6 added lines, -6 removed lines

@@ -35,11 +35,11 @@ 
     protected function configure()
     {
         $this->setName($this->name)
-             ->setDescription('Executes (or dumps) the SQL needed to update the database schema to match the current mapping metadata')
-             ->addOption('complete', null, InputOption::VALUE_NONE, 'If defined, all assets of the database which are not relevant to the current metadata will be dropped.')
-             ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Dumps the generated SQL statements to the screen (does not execute them).')
-             ->addOption('force', 'f', InputOption::VALUE_NONE, 'Causes the generated SQL statements to be physically executed against your database.')
-             ->setHelp(<<<'EOT'
+                ->setDescription('Executes (or dumps) the SQL needed to update the database schema to match the current mapping metadata')
+                ->addOption('complete', null, InputOption::VALUE_NONE, 'If defined, all assets of the database which are not relevant to the current metadata will be dropped.')
+                ->addOption('dump-sql', null, InputOption::VALUE_NONE, 'Dumps the generated SQL statements to the screen (does not execute them).')
+                ->addOption('force', 'f', InputOption::VALUE_NONE, 'Causes the generated SQL statements to be physically executed against your database.')
+                ->setHelp(<<<'EOT'
 The <info>%command.name%</info> command generates the SQL needed to
 synchronize the database schema with the current mapping metadata of the
 default entity manager.
@@ -69,7 +69,7 @@ 
 
     $config->setFilterSchemaAssetsExpression($regexp);
 EOT
-             );
+                );
     }
 
     /**

lib/Doctrine/ORM/Tools/Console/Command/ClearCache/MetadataCommand.php

Indentation +4 added lines, -4 removed lines

@@ -30,9 +30,9 @@ 
     protected function configure()
     {
         $this->setName('orm:clear-cache:metadata')
-             ->setDescription('Clear all metadata cache of the various cache drivers')
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
-             ->setHelp(<<<'EOT'
+                ->setDescription('Clear all metadata cache of the various cache drivers')
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
+                ->setHelp(<<<'EOT'
 The <info>%command.name%</info> command is meant to clear the metadata cache of associated Entity Manager.
 It is possible to invalidate all cache entries at once - called delete -, or flushes the cache provider
 instance completely.
@@ -49,7 +49,7 @@ 
 Finally, be aware that if <info>--flush</info> option is passed, not all cache providers are able to flush entries,
 because of a limitation of its execution nature.
 EOT
-             );
+                );
     }
 
     /**

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
         $em          = $this->getHelper('em')->getEntityManager();
         $cacheDriver = $em->getConfiguration()->getMetadataCacheImpl();
 
-        if (! $cacheDriver) {
+        if ( ! $cacheDriver) {
             throw new InvalidArgumentException('No Metadata cache driver is configured on given EntityManager.');
         }
 
@@ -79,7 +79,7 @@ 
             $message = $result ? 'Successfully flushed cache entries.' : $message;
         }
 
-        if (! $result) {
+        if ( ! $result) {
             $ui->error($message);
 
             return 1;

lib/Doctrine/ORM/Tools/Console/Command/ClearCache/QueryRegionCommand.php

Indentation +6 added lines, -6 removed lines

@@ -27,11 +27,11 @@ 
     protected function configure()
     {
         $this->setName('orm:clear-cache:region:query')
-             ->setDescription('Clear a second-level cache query region')
-             ->addArgument('region-name', InputArgument::OPTIONAL, 'The query region to clear.')
-             ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all query regions will be deleted/invalidated.')
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
-             ->setHelp(<<<'EOT'
+                ->setDescription('Clear a second-level cache query region')
+                ->addArgument('region-name', InputArgument::OPTIONAL, 'The query region to clear.')
+                ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all query regions will be deleted/invalidated.')
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
+                ->setHelp(<<<'EOT'
 The <info>%command.name%</info> command is meant to clear a second-level cache query region for an associated Entity Manager.
 It is possible to delete/invalidate all query region, a specific query region or flushes the cache provider.
 
@@ -55,7 +55,7 @@ 
 Finally, be aware that if <info>--flush</info> option is passed,
 not all cache providers are able to flush entries, because of a limitation of its execution nature.
 EOT
-             );
+                );
     }
 
     /**

Spacing +2 added lines, -2 removed lines

@@ -75,7 +75,7 @@ 
             $name = Cache::DEFAULT_QUERY_REGION_NAME;
         }
 
-        if (! $cache instanceof Cache) {
+        if ( ! $cache instanceof Cache) {
             throw new InvalidArgumentException('No second-level cache is configured on the given EntityManager.');
         }
 
@@ -83,7 +83,7 @@ 
             $queryCache  = $cache->getQueryCache($name);
             $queryRegion = $queryCache->getRegion();
 
-            if (! $queryRegion instanceof DefaultRegion) {
+            if ( ! $queryRegion instanceof DefaultRegion) {
                 throw new InvalidArgumentException(sprintf(
                     'The option "--flush" expects a "Doctrine\ORM\Cache\Region\DefaultRegion", but got "%s".',
                     is_object($queryRegion) ? get_class($queryRegion) : gettype($queryRegion)

lib/Doctrine/ORM/Tools/Console/Command/ClearCache/QueryCommand.php

Indentation +4 added lines, -4 removed lines

@@ -30,9 +30,9 @@ 
     protected function configure()
     {
         $this->setName('orm:clear-cache:query')
-             ->setDescription('Clear all query cache of the various cache drivers')
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
-             ->setHelp(<<<'EOT'
+                ->setDescription('Clear all query cache of the various cache drivers')
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
+                ->setHelp(<<<'EOT'
 The <info>%command.name%</info> command is meant to clear the query cache of associated Entity Manager.
 It is possible to invalidate all cache entries at once - called delete -, or flushes the cache provider
 instance completely.
@@ -49,7 +49,7 @@ 
 Finally, be aware that if <info>--flush</info> option is passed, not all cache providers are able to flush entries,
 because of a limitation of its execution nature.
 EOT
-             );
+                );
     }
 
     /**

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
         $em          = $this->getHelper('em')->getEntityManager();
         $cacheDriver = $em->getConfiguration()->getQueryCacheImpl();
 
-        if (! $cacheDriver) {
+        if ( ! $cacheDriver) {
             throw new InvalidArgumentException('No Query cache driver is configured on given EntityManager.');
         }
 
@@ -78,7 +78,7 @@ 
             $message = $result ? 'Successfully flushed cache entries.' : $message;
         }
 
-        if (! $result) {
+        if ( ! $result) {
             $ui->error($message);
 
             return 1;

lib/Doctrine/ORM/Tools/Console/Command/ClearCache/CollectionRegionCommand.php

Indentation +8 added lines, -8 removed lines

@@ -27,13 +27,13 @@ 
     protected function configure()
     {
         $this->setName('orm:clear-cache:region:collection')
-             ->setDescription('Clear a second-level cache collection region')
-             ->addArgument('owner-class', InputArgument::OPTIONAL, 'The owner entity name.')
-             ->addArgument('association', InputArgument::OPTIONAL, 'The association collection name.')
-             ->addArgument('owner-id', InputArgument::OPTIONAL, 'The owner identifier.')
-             ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all entity regions will be deleted/invalidated.')
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
-             ->setHelp(<<<'EOT'
+                ->setDescription('Clear a second-level cache collection region')
+                ->addArgument('owner-class', InputArgument::OPTIONAL, 'The owner entity name.')
+                ->addArgument('association', InputArgument::OPTIONAL, 'The association collection name.')
+                ->addArgument('owner-id', InputArgument::OPTIONAL, 'The owner identifier.')
+                ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all entity regions will be deleted/invalidated.')
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
+                ->setHelp(<<<'EOT'
 The <info>%command.name%</info> command is meant to clear a second-level cache collection regions for an associated Entity Manager.
 It is possible to delete/invalidate all collection region, a specific collection region or flushes the cache provider.
 
@@ -57,7 +57,7 @@ 
 Finally, be aware that if <info>--flush</info> option is passed,
 not all cache providers are able to flush entries, because of a limitation of its execution nature.
 EOT
-             );
+                );
     }
 
     /**

Spacing +2 added lines, -2 removed lines

@@ -75,7 +75,7 @@ 
         $ownerId    = $input->getArgument('owner-id');
         $cache      = $em->getCache();
 
-        if (! $cache instanceof Cache) {
+        if ( ! $cache instanceof Cache) {
             throw new InvalidArgumentException('No second-level cache is configured on the given EntityManager.');
         }
 
@@ -86,7 +86,7 @@ 
         if ($input->getOption('flush')) {
             $collectionRegion = $cache->getCollectionCacheRegion($ownerClass, $assoc);
 
-            if (! $collectionRegion instanceof DefaultRegion) {
+            if ( ! $collectionRegion instanceof DefaultRegion) {
                 throw new InvalidArgumentException(sprintf(
                     'The option "--flush" expects a "Doctrine\ORM\Cache\Region\DefaultRegion", but got "%s".',
                     is_object($collectionRegion) ? get_class($collectionRegion) : gettype($collectionRegion)

lib/Doctrine/ORM/Tools/Console/Command/ClearCache/ResultCommand.php

Indentation +4 added lines, -4 removed lines

@@ -30,9 +30,9 @@ 
     protected function configure()
     {
         $this->setName('orm:clear-cache:result')
-             ->setDescription('Clear all result cache of the various cache drivers')
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
-             ->setHelp(<<<'EOT'
+                ->setDescription('Clear all result cache of the various cache drivers')
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, cache entries will be flushed instead of deleted/invalidated.')
+                ->setHelp(<<<'EOT'
 The <info>%command.name%</info> command is meant to clear the result cache of associated Entity Manager.
 It is possible to invalidate all cache entries at once - called delete -, or flushes the cache provider
 instance completely.
@@ -49,7 +49,7 @@ 
 Finally, be aware that if <info>--flush</info> option is passed, not all cache providers are able to flush entries,
 because of a limitation of its execution nature.
 EOT
-             );
+                );
     }
 
     /**

Spacing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
         $em          = $this->getHelper('em')->getEntityManager();
         $cacheDriver = $em->getConfiguration()->getResultCacheImpl();
 
-        if (! $cacheDriver) {
+        if ( ! $cacheDriver) {
             throw new InvalidArgumentException('No Result cache driver is configured on given EntityManager.');
         }
 
@@ -79,7 +79,7 @@ 
             $message = $result ? 'Successfully flushed cache entries.' : $message;
         }
 
-        if (! $result) {
+        if ( ! $result) {
             $ui->error($message);
 
             return 1;

lib/Doctrine/ORM/Tools/Console/Command/ClearCache/EntityRegionCommand.php

Indentation +7 added lines, -7 removed lines

@@ -27,12 +27,12 @@ 
     protected function configure()
     {
         $this->setName('orm:clear-cache:region:entity')
-             ->setDescription('Clear a second-level cache entity region')
-             ->addArgument('entity-class', InputArgument::OPTIONAL, 'The entity name.')
-             ->addArgument('entity-id', InputArgument::OPTIONAL, 'The entity identifier.')
-             ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all entity regions will be deleted/invalidated.')
-             ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
-             ->setHelp(<<<'EOT'
+                ->setDescription('Clear a second-level cache entity region')
+                ->addArgument('entity-class', InputArgument::OPTIONAL, 'The entity name.')
+                ->addArgument('entity-id', InputArgument::OPTIONAL, 'The entity identifier.')
+                ->addOption('all', null, InputOption::VALUE_NONE, 'If defined, all entity regions will be deleted/invalidated.')
+                ->addOption('flush', null, InputOption::VALUE_NONE, 'If defined, all cache entries will be flushed.')
+                ->setHelp(<<<'EOT'
 The <info>%command.name%</info> command is meant to clear a second-level cache entity region for an associated Entity Manager.
 It is possible to delete/invalidate all entity region, a specific entity region or flushes the cache provider.
 
@@ -56,7 +56,7 @@ 
 Finally, be aware that if <info>--flush</info> option is passed,
 not all cache providers are able to flush entries, because of a limitation of its execution nature.
 EOT
-             );
+                );
     }
 
     /**

Spacing +3 added lines, -3 removed lines

@@ -73,18 +73,18 @@
         $entityId    = $input->getArgument('entity-id');
         $cache       = $em->getCache();
 
-        if (! $cache instanceof Cache) {
+        if ( ! $cache instanceof Cache) {
             throw new InvalidArgumentException('No second-level cache is configured on the given EntityManager.');
         }
 
-        if (! $entityClass && ! $input->getOption('all')) {
+        if ( ! $entityClass && ! $input->getOption('all')) {
             throw new InvalidArgumentException('Invalid argument "--entity-class"');
         }
 
         if ($input->getOption('flush')) {
             $entityRegion = $cache->getEntityCacheRegion($entityClass);
 
-            if (! $entityRegion instanceof DefaultRegion) {
+            if ( ! $entityRegion instanceof DefaultRegion) {
                 throw new InvalidArgumentException(sprintf(
                     'The option "--flush" expects a "Doctrine\ORM\Cache\Region\DefaultRegion", but got "%s".',
                     is_object($entityRegion) ? get_class($entityRegion) : gettype($entityRegion)

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

Indentation +12 added lines, -12 removed lines

@@ -20,11 +20,11 @@ 
     public function testMixedParametersWithZeroNumber() : void
     {
         $query = $this->em->createQueryBuilder()
-                           ->select('u')
-                           ->from(CmsUser::class, 'u')
-                           ->andWhere('u.username = :username')
-                           ->andWhere('u.id = ?0')
-                           ->getQuery();
+                            ->select('u')
+                            ->from(CmsUser::class, 'u')
+                            ->andWhere('u.username = :username')
+                            ->andWhere('u.id = ?0')
+                            ->getQuery();
 
         $query->setParameter('username', 'bar');
         $query->setParameter(0, 0);
@@ -39,13 +39,13 @@ 
     public function testMixedParametersWithZeroNumberOnQueryBuilder() : void
     {
         $query = $this->em->createQueryBuilder()
-                           ->select('u')
-                           ->from(CmsUser::class, 'u')
-                           ->andWhere('u.username = :username')
-                           ->andWhere('u.id = ?0')
-                           ->setParameter('username', 'bar')
-                           ->setParameter(0, 0)
-                           ->getQuery();
+                            ->select('u')
+                            ->from(CmsUser::class, 'u')
+                            ->andWhere('u.username = :username')
+                            ->andWhere('u.id = ?0')
+                            ->setParameter('username', 'bar')
+                            ->setParameter(0, 0)
+                            ->getQuery();
 
         $query->execute();
 

lib/Doctrine/ORM/Persisters/Collection/OneToManyPersister.php

Indentation +1 added lines, -1 removed lines

@@ -247,7 +247,7 @@
 
         // 2) Build insert table records into temporary table
         $dql   = ' SELECT t0.' . implode(', t0.', $rootClass->getIdentifierFieldNames())
-               . ' FROM ' . $targetClass->getClassName() . ' t0 WHERE t0.' . $association->getMappedBy() . ' = :owner';
+                . ' FROM ' . $targetClass->getClassName() . ' t0 WHERE t0.' . $association->getMappedBy() . ' = :owner';
         $query = $this->em->createQuery($dql)->setParameter('owner', $collection->getOwner());
 
         $statement  = 'INSERT INTO ' . $tempTable . ' (' . $idColumnNameList . ') ' . $query->getSQL();

Spacing +13 added lines, -13 removed lines

@@ -33,7 +33,7 @@ 
         // the entire collection with a new would trigger this operation.
         $association = $collection->getMapping();
 
-        if (! $association->isOrphanRemoval()) {
+        if ( ! $association->isOrphanRemoval()) {
             // Handling non-orphan removal should never happen, as @OneToMany
             // can only be inverse side. For owning side one to many, it is
             // required to have a join table, which would classify as a ManyToManyPersister.
@@ -65,7 +65,7 @@ 
     {
         $association = $collection->getMapping();
 
-        if (! ($association instanceof ToManyAssociationMetadata && $association->getIndexedBy())) {
+        if ( ! ($association instanceof ToManyAssociationMetadata && $association->getIndexedBy())) {
             throw new BadMethodCallException('Selecting a collection by index is only supported on indexed collections.');
         }
 
@@ -114,7 +114,7 @@ 
     {
         $association = $collection->getMapping();
 
-        if (! ($association instanceof ToManyAssociationMetadata && $association->getIndexedBy())) {
+        if ( ! ($association instanceof ToManyAssociationMetadata && $association->getIndexedBy())) {
             throw new BadMethodCallException('Selecting a collection by index is only supported on indexed collections.');
         }
 
@@ -136,7 +136,7 @@ 
      */
     public function contains(PersistentCollection $collection, $element)
     {
-        if (! $this->isValidEntityState($element)) {
+        if ( ! $this->isValidEntityState($element)) {
             return false;
         }
 
@@ -160,12 +160,12 @@ 
     {
         $association = $collection->getMapping();
 
-        if (! $association->isOrphanRemoval()) {
+        if ( ! $association->isOrphanRemoval()) {
             // no-op: this is not the owning side, therefore no operations should be applied
             return false;
         }
 
-        if (! $this->isValidEntityState($element)) {
+        if ( ! $this->isValidEntityState($element)) {
             return false;
         }
 
@@ -203,7 +203,7 @@ 
         }
 
         $tableName = $targetClass->table->getQuotedQualifiedName($this->platform);
-        $statement = 'DELETE FROM ' . $tableName . ' WHERE ' . implode(' = ? AND ', $columns) . ' = ?';
+        $statement = 'DELETE FROM '.$tableName.' WHERE '.implode(' = ? AND ', $columns).' = ?';
 
         return $this->conn->executeUpdate($statement, $parameters);
     }
@@ -240,17 +240,17 @@ 
             ];
         }
 
-        $statement = $this->platform->getCreateTemporaryTableSnippetSQL() . ' ' . $tempTable
-            . ' (' . $this->platform->getColumnDeclarationListSQL($columnDefinitions) . ')';
+        $statement = $this->platform->getCreateTemporaryTableSnippetSQL().' '.$tempTable
+            . ' ('.$this->platform->getColumnDeclarationListSQL($columnDefinitions).')';
 
         $this->conn->executeUpdate($statement);
 
         // 2) Build insert table records into temporary table
-        $dql   = ' SELECT t0.' . implode(', t0.', $rootClass->getIdentifierFieldNames())
-               . ' FROM ' . $targetClass->getClassName() . ' t0 WHERE t0.' . $association->getMappedBy() . ' = :owner';
+        $dql   = ' SELECT t0.'.implode(', t0.', $rootClass->getIdentifierFieldNames())
+               . ' FROM '.$targetClass->getClassName().' t0 WHERE t0.'.$association->getMappedBy().' = :owner';
         $query = $this->em->createQuery($dql)->setParameter('owner', $collection->getOwner());
 
-        $statement  = 'INSERT INTO ' . $tempTable . ' (' . $idColumnNameList . ') ' . $query->getSQL();
+        $statement  = 'INSERT INTO '.$tempTable.' ('.$idColumnNameList.') '.$query->getSQL();
         $parameters = array_values($sourcePersister->getIdentifier($collection->getOwner()));
         $numDeleted = $this->conn->executeUpdate($statement, $parameters);
 
@@ -265,7 +265,7 @@ 
         // 4) Delete records on each table in the hierarchy
         $hierarchyClasses = array_merge(
             array_map(
-                function ($className) {
+                function($className) {
                     return $this->em->getClassMetadata($className);
                 },
                 array_reverse($targetClass->getSubClasses())