doctrine / orm

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())

lib/Doctrine/ORM/QueryBuilder.php

Indentation +15 added lines, -15 removed lines

@@ -1344,9 +1344,9 @@ 
     private function getDQLForDelete()
     {
         return 'DELETE'
-              . $this->getReducedDQLQueryPart('from', ['pre' => ' ', 'separator' => ', '])
-              . $this->getReducedDQLQueryPart('where', ['pre' => ' WHERE '])
-              . $this->getReducedDQLQueryPart('orderBy', ['pre' => ' ORDER BY ', 'separator' => ', ']);
+                . $this->getReducedDQLQueryPart('from', ['pre' => ' ', 'separator' => ', '])
+                . $this->getReducedDQLQueryPart('where', ['pre' => ' WHERE '])
+                . $this->getReducedDQLQueryPart('orderBy', ['pre' => ' ORDER BY ', 'separator' => ', ']);
     }
 
     /**
@@ -1355,10 +1355,10 @@ 
     private function getDQLForUpdate()
     {
         return 'UPDATE'
-              . $this->getReducedDQLQueryPart('from', ['pre' => ' ', 'separator' => ', '])
-              . $this->getReducedDQLQueryPart('set', ['pre' => ' SET ', 'separator' => ', '])
-              . $this->getReducedDQLQueryPart('where', ['pre' => ' WHERE '])
-              . $this->getReducedDQLQueryPart('orderBy', ['pre' => ' ORDER BY ', 'separator' => ', ']);
+                . $this->getReducedDQLQueryPart('from', ['pre' => ' ', 'separator' => ', '])
+                . $this->getReducedDQLQueryPart('set', ['pre' => ' SET ', 'separator' => ', '])
+                . $this->getReducedDQLQueryPart('where', ['pre' => ' WHERE '])
+                . $this->getReducedDQLQueryPart('orderBy', ['pre' => ' ORDER BY ', 'separator' => ', ']);
     }
 
     /**
@@ -1367,8 +1367,8 @@ 
     private function getDQLForSelect()
     {
         $dql = 'SELECT'
-             . ($this->dqlParts['distinct']===true ? ' DISTINCT' : '')
-             . $this->getReducedDQLQueryPart('select', ['pre' => ' ', 'separator' => ', ']);
+                . ($this->dqlParts['distinct']===true ? ' DISTINCT' : '')
+                . $this->getReducedDQLQueryPart('select', ['pre' => ' ', 'separator' => ', ']);
 
         $fromParts   = $this->getDQLPart('from');
         $joinParts   = $this->getDQLPart('join');
@@ -1392,10 +1392,10 @@ 
         }
 
         $dql .= implode(', ', $fromClauses)
-              . $this->getReducedDQLQueryPart('where', ['pre' => ' WHERE '])
-              . $this->getReducedDQLQueryPart('groupBy', ['pre' => ' GROUP BY ', 'separator' => ', '])
-              . $this->getReducedDQLQueryPart('having', ['pre' => ' HAVING '])
-              . $this->getReducedDQLQueryPart('orderBy', ['pre' => ' ORDER BY ', 'separator' => ', ']);
+                . $this->getReducedDQLQueryPart('where', ['pre' => ' WHERE '])
+                . $this->getReducedDQLQueryPart('groupBy', ['pre' => ' GROUP BY ', 'separator' => ', '])
+                . $this->getReducedDQLQueryPart('having', ['pre' => ' HAVING '])
+                . $this->getReducedDQLQueryPart('orderBy', ['pre' => ' ORDER BY ', 'separator' => ', ']);
 
         return $dql;
     }
@@ -1415,8 +1415,8 @@ 
         }
 
         return ($options['pre'] ?? '')
-             . (is_array($queryPart) ? implode($options['separator'], $queryPart) : $queryPart)
-             . ($options['post'] ?? '');
+                . (is_array($queryPart) ? implode($options['separator'], $queryPart) : $queryPart)
+                . ($options['post'] ?? '');
     }
 
     /**

Spacing +15 added lines, -15 removed lines

@@ -417,7 +417,7 @@ 
     {
         $aliases = $this->getRootAliases();
 
-        if (! isset($aliases[0])) {
+        if ( ! isset($aliases[0])) {
             throw new RuntimeException('No alias was set before invoking getRootAlias().');
         }
 
@@ -600,7 +600,7 @@ 
     public function getParameter($key)
     {
         $filteredParameters = $this->parameters->filter(
-            static function (Query\Parameter $parameter) use ($key) : bool {
+            static function(Query\Parameter $parameter) use ($key) : bool {
                 $parameterName = $parameter->getName();
 
                 return $key === $parameterName || (string) $key === (string) $parameterName;
@@ -676,7 +676,7 @@ 
     {
         if ($append && ($dqlPartName === 'where' || $dqlPartName === 'having')) {
             throw new InvalidArgumentException(
-                "Using \$append = true does not have an effect with 'where' or 'having' " .
+                "Using \$append = true does not have an effect with 'where' or 'having' ".
                 'parts. See QueryBuilder#andWhere() for an example for correct usage.'
             );
         }
@@ -817,7 +817,7 @@ 
     {
         $this->type = self::DELETE;
 
-        if (! $delete) {
+        if ( ! $delete) {
             return $this;
         }
 
@@ -844,7 +844,7 @@ 
     {
         $this->type = self::UPDATE;
 
-        if (! $update) {
+        if ( ! $update) {
             return $this;
         }
 
@@ -899,7 +899,7 @@ 
     {
         $rootAliases = $this->getRootAliases();
 
-        if (! in_array($alias, $rootAliases, true)) {
+        if ( ! in_array($alias, $rootAliases, true)) {
             throw new Query\QueryException(
                 sprintf('Specified root alias %s must be set before invoking indexBy().', $alias)
             );
@@ -1060,7 +1060,7 @@ 
      */
     public function where($predicates)
     {
-        if (! (func_num_args() === 1 && $predicates instanceof Expr\Composite)) {
+        if ( ! (func_num_args() === 1 && $predicates instanceof Expr\Composite)) {
             $predicates = new Expr\Andx(func_get_args());
         }
 
@@ -1179,7 +1179,7 @@ 
      */
     public function having($having)
     {
-        if (! (func_num_args() === 1 && ($having instanceof Expr\Andx || $having instanceof Expr\Orx))) {
+        if ( ! (func_num_args() === 1 && ($having instanceof Expr\Andx || $having instanceof Expr\Orx))) {
             $having = new Expr\Andx(func_get_args());
         }
 
@@ -1277,7 +1277,7 @@ 
     public function addCriteria(Criteria $criteria)
     {
         $allAliases = $this->getAllAliases();
-        if (! isset($allAliases[0])) {
+        if ( ! isset($allAliases[0])) {
             throw new Query\QueryException('No aliases are set before invoking addCriteria().');
         }
 
@@ -1295,14 +1295,14 @@ 
             foreach ($criteria->getOrderings() as $sort => $order) {
                 $hasValidAlias = false;
                 foreach ($allAliases as $alias) {
-                    if (strpos($sort . '.', $alias . '.') === 0) {
+                    if (strpos($sort.'.', $alias.'.') === 0) {
                         $hasValidAlias = true;
                         break;
                     }
                 }
 
-                if (! $hasValidAlias) {
-                    $sort = $allAliases[0] . '.' . $sort;
+                if ( ! $hasValidAlias) {
+                    $sort = $allAliases[0].'.'.$sort;
                 }
 
                 $this->addOrderBy($sort, $order);
@@ -1376,7 +1376,7 @@ 
     private function getDQLForSelect()
     {
         $dql = 'SELECT'
-             . ($this->dqlParts['distinct']===true ? ' DISTINCT' : '')
+             . ($this->dqlParts['distinct'] === true ? ' DISTINCT' : '')
              . $this->getReducedDQLQueryPart('select', ['pre' => ' ', 'separator' => ', ']);
 
         $fromParts   = $this->getDQLPart('from');
@@ -1384,7 +1384,7 @@ 
         $fromClauses = [];
 
         // Loop through all FROM clauses
-        if (! empty($fromParts)) {
+        if ( ! empty($fromParts)) {
             $dql .= ' FROM ';
 
             foreach ($fromParts as $from) {
@@ -1392,7 +1392,7 @@ 
 
                 if ($from instanceof Expr\From && isset($joinParts[$from->getAlias()])) {
                     foreach ($joinParts[$from->getAlias()] as $join) {
-                        $fromClause .= ' ' . ((string) $join);
+                        $fromClause .= ' '.((string) $join);
                     }
                 }
 

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

Indentation +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $this->em->flush();
 
         $asset->getAttributes()
-              ->removeElement($attribute1);
+                ->removeElement($attribute1);
 
         $idToBeRemoved = $attribute1->id;