doctrine / orm

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;
 

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

Indentation +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
         $this->em->clear();
 
         $history = $this->em->createQuery('SELECT h FROM ' . __NAMESPACE__ . '\\DDC1250ClientHistory h WHERE h.id = ?1')
-                  ->setParameter(1, $c2->id)->getSingleResult();
+                    ->setParameter(1, $c2->id)->getSingleResult();
 
         self::assertInstanceOf(DDC1250ClientHistory::class, $history);
     }

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         $this->em->flush();
         $this->em->clear();
 
-        $history = $this->em->createQuery('SELECT h FROM ' . __NAMESPACE__ . '\\DDC1250ClientHistory h WHERE h.id = ?1')
+        $history = $this->em->createQuery('SELECT h FROM '.__NAMESPACE__.'\\DDC1250ClientHistory h WHERE h.id = ?1')
                   ->setParameter(1, $c2->id)->getSingleResult();
 
         self::assertInstanceOf(DDC1250ClientHistory::class, $history);

lib/Doctrine/ORM/Query/AST/Node.php

Indentation +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
             foreach ($props as $name => $prop) {
                 $ident += 4;
                 $str   .= str_repeat(' ', $ident) . '"' . $name . '": '
-                      . $this->dump($prop) . ',' . PHP_EOL;
+                        . $this->dump($prop) . ',' . PHP_EOL;
                 $ident -= 4;
             }
 
@@ -63,7 +63,7 @@ 
 
             foreach ($obj as $k => $v) {
                 $str .= PHP_EOL . str_repeat(' ', $ident) . '"'
-                      . $k . '" => ' . $this->dump($v) . ',';
+                        . $k . '" => ' . $this->dump($v) . ',';
                 $some = true;
             }
 

Spacing +8 added lines, -8 removed lines

@@ -54,32 +54,32 @@
         $str = '';
 
         if ($obj instanceof Node) {
-            $str  .= get_class($obj) . '(' . PHP_EOL;
+            $str  .= get_class($obj).'('.PHP_EOL;
             $props = get_object_vars($obj);
 
             foreach ($props as $name => $prop) {
                 $ident += 4;
-                $str   .= str_repeat(' ', $ident) . '"' . $name . '": '
-                      . $this->dump($prop) . ',' . PHP_EOL;
+                $str   .= str_repeat(' ', $ident).'"'.$name.'": '
+                      . $this->dump($prop).','.PHP_EOL;
                 $ident -= 4;
             }
 
-            $str .= str_repeat(' ', $ident) . ')';
+            $str .= str_repeat(' ', $ident).')';
         } elseif (is_array($obj)) {
             $ident += 4;
             $str   .= 'array(';
             $some   = false;
 
             foreach ($obj as $k => $v) {
-                $str .= PHP_EOL . str_repeat(' ', $ident) . '"'
-                      . $k . '" => ' . $this->dump($v) . ',';
+                $str .= PHP_EOL.str_repeat(' ', $ident).'"'
+                      . $k.'" => '.$this->dump($v).',';
                 $some = true;
             }
 
             $ident -= 4;
-            $str   .= ($some ? PHP_EOL . str_repeat(' ', $ident) : '') . ')';
+            $str   .= ($some ? PHP_EOL . str_repeat(' ', $ident) : '').')';
         } elseif (is_object($obj)) {
-            $str .= 'instanceof(' . get_class($obj) . ')';
+            $str .= 'instanceof('.get_class($obj).')';
         } else {
             $str .= var_export($obj, true);
         }

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

Indentation +5 added lines, -5 removed lines

@@ -21,13 +21,13 @@ 
     protected function configure()
     {
         $this->setName('orm:info')
-             ->setDescription('Show basic information about all mapped entities')
-             ->setHelp(<<<'EOT'
+                ->setDescription('Show basic information about all mapped entities')
+                ->setHelp(<<<'EOT'
 The <info>%command.name%</info> shows basic information about which
 entities exist and possibly if their mapping information contains errors or
 not.
 EOT
-             );
+                );
     }
 
     /**
@@ -41,8 +41,8 @@ 
         $entityManager = $this->getHelper('em')->getEntityManager();
 
         $entityClassNames = $entityManager->getConfiguration()
-                                          ->getMetadataDriverImpl()
-                                          ->getAllClassNames();
+                                            ->getMetadataDriverImpl()
+                                            ->getAllClassNames();
 
         if (! $entityClassNames) {
             $ui->caution(

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
                                           ->getMetadataDriverImpl()
                                           ->getAllClassNames();
 
-        if (! $entityClassNames) {
+        if ( ! $entityClassNames) {
             $ui->caution(
                 [
                     'You do not have any mapped Doctrine ORM entities according to the current configuration.',

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

Indentation +2 added lines, -2 removed lines

@@ -26,8 +26,8 @@
     {
         $qb = $this->em->createQueryBuilder();
         $qb->select('p', 'r')
-           ->from(__NAMESPACE__ . '\DDC698Privilege', 'p')
-           ->leftJoin('p.roles', 'r');
+            ->from(__NAMESPACE__ . '\DDC698Privilege', 'p')
+            ->leftJoin('p.roles', 'r');
 
         self::assertSQLEquals(
             'SELECT t0."privilegeID" AS c0, t0."name" AS c1, t1."roleID" AS c2, t1."name" AS c3, t1."shortName" AS c4 FROM "Privileges" t0 LEFT JOIN "RolePrivileges" t2 ON t0."privilegeID" = t2."privilegeID" LEFT JOIN "Roles" t1 ON t1."roleID" = t2."roleID"',

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     {
         $qb = $this->em->createQueryBuilder();
         $qb->select('p', 'r')
-           ->from(__NAMESPACE__ . '\DDC698Privilege', 'p')
+           ->from(__NAMESPACE__.'\DDC698Privilege', 'p')
            ->leftJoin('p.roles', 'r');
 
         self::assertSQLEquals(

tests/Doctrine/Tests/ORM/Query/QueryTest.php

Indentation +13 added lines, -13 removed lines

@@ -88,18 +88,18 @@ 
     {
         $q  = $this->em->createQuery('select a from Doctrine\Tests\Models\CMS\CmsArticle a');
         $q2 = $q->expireQueryCache(true)
-          ->setQueryCacheLifetime(3600)
-          ->setQueryCacheDriver(null)
-          ->expireResultCache(true)
-          ->setHint('foo', 'bar')
-          ->setHint('bar', 'baz')
-          ->setParameter(1, 'bar')
-          ->setParameters(new ArrayCollection([new Parameter(2, 'baz')]))
-          ->setResultCacheDriver(null)
-          ->setResultCacheId('foo')
-          ->setDQL('foo')
-          ->setFirstResult(10)
-          ->setMaxResults(10);
+            ->setQueryCacheLifetime(3600)
+            ->setQueryCacheDriver(null)
+            ->expireResultCache(true)
+            ->setHint('foo', 'bar')
+            ->setHint('bar', 'baz')
+            ->setParameter(1, 'bar')
+            ->setParameters(new ArrayCollection([new Parameter(2, 'baz')]))
+            ->setResultCacheDriver(null)
+            ->setResultCacheId('foo')
+            ->setDQL('foo')
+            ->setFirstResult(10)
+            ->setMaxResults(10);
 
         self::assertSame($q2, $q);
     }
@@ -256,7 +256,7 @@ 
         $this->em->getConfiguration()->setResultCacheImpl(new ArrayCache());
 
         $query = $this->em->createQuery('SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u')
-                          ->useResultCache(true);
+                            ->useResultCache(true);
 
         /** @var DriverConnectionMock $driverConnectionMock */
         $driverConnectionMock = $this->em->getConnection()

Spacing +5 added lines, -5 removed lines

@@ -314,7 +314,7 @@ 
      */
     public function testGetParameterTypeJuggling() : void
     {
-        $query = $this->em->createQuery('select u from ' . CmsUser::class . ' u where u.id = ?0');
+        $query = $this->em->createQuery('select u from '.CmsUser::class.' u where u.id = ?0');
 
         $query->setParameter(0, 0);
 
@@ -328,7 +328,7 @@ 
      */
     public function testSetParameterWithNameZeroIsNotOverridden() : void
     {
-        $query = $this->em->createQuery('select u from ' . CmsUser::class . ' u where u.id != ?0 and u.username = :name');
+        $query = $this->em->createQuery('select u from '.CmsUser::class.' u where u.id != ?0 and u.username = :name');
 
         $query->setParameter(0, 0);
         $query->setParameter('name', 'Doctrine');
@@ -343,7 +343,7 @@ 
      */
     public function testSetParameterWithNameZeroDoesNotOverrideAnotherParameter() : void
     {
-        $query = $this->em->createQuery('select u from ' . CmsUser::class . ' u where u.id != ?0 and u.username = :name');
+        $query = $this->em->createQuery('select u from '.CmsUser::class.' u where u.id != ?0 and u.username = :name');
 
         $query->setParameter('name', 'Doctrine');
         $query->setParameter(0, 0);
@@ -358,7 +358,7 @@ 
      */
     public function testSetParameterWithTypeJugglingWorks() : void
     {
-        $query = $this->em->createQuery('select u from ' . CmsUser::class . ' u where u.id != ?0 and u.username = :name');
+        $query = $this->em->createQuery('select u from '.CmsUser::class.' u where u.id != ?0 and u.username = :name');
 
         $query->setParameter('0', 1);
         $query->setParameter('name', 'Doctrine');
@@ -378,7 +378,7 @@ 
     {
         $this->em->getConfiguration()->setResultCacheImpl(new ArrayCache());
 
-        $query = $this->em->createQuery('SELECT u FROM ' . CmsUser::class . ' u');
+        $query = $this->em->createQuery('SELECT u FROM '.CmsUser::class.' u');
         $query->useResultCache(true);
         $query->setResultCacheProfile();
 

tests/Doctrine/Tests/ORM/Cache/Persister/Collection/AbstractCollectionPersisterTest.php

Indentation +2 added lines, -2 removed lines

@@ -86,8 +86,8 @@
         $this->em                  = $this->getTestEntityManager();
         $this->region              = $this->createRegion();
         $this->collectionPersister = $this->getMockBuilder(CollectionPersister::class)
-                                           ->setMethods($this->collectionPersisterMockMethods)
-                                           ->getMock();
+                                            ->setMethods($this->collectionPersisterMockMethods)
+                                            ->getMock();
     }
 
     /**