doctrine / orm

tests/Doctrine/Tests/DbalTypes/NegativeToPositiveType.php

Spacing +2 added lines, -2 removed lines

@@ -38,7 +38,7 @@ 
      */
     public function convertToDatabaseValueSQL($sqlExpr, AbstractPlatform $platform)
     {
-        return 'ABS(' . $sqlExpr . ')';
+        return 'ABS('.$sqlExpr.')';
     }
 
     /**
@@ -46,6 +46,6 @@ 
      */
     public function convertToPHPValueSQL($sqlExpr, $platform)
     {
-        return '-(' . $sqlExpr . ')';
+        return '-('.$sqlExpr.')';
     }
 }

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

Indentation +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
     /**
      * @Id @Column(name="id", type="integer")
      * @GeneratedValue(strategy="AUTO")
-    */
+     */
     public $id;
 }
 

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

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
                 $this->_em->getClassMetadata(Address::class),
                 ]
             );
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
         }
     }
 

tests/Doctrine/Tests/ORM/Id/SequenceGeneratorTest.php

Spacing +2 added lines, -2 removed lines

@@ -45,13 +45,13 @@
 
         for ($i = 0; $i < 42; ++$i) {
             if ($i % 10 == 0) {
-                $this->connection->setQueryResult(new StatementArrayMock([[(int)($i / 10) * 10]]));
+                $this->connection->setQueryResult(new StatementArrayMock([[(int) ($i / 10) * 10]]));
             }
 
             $id = $this->sequenceGenerator->generate($this->entityManager, null);
 
             self::assertSame($i, $id);
-            self::assertSame((int)($i / 10) * 10 + 10, $this->sequenceGenerator->getCurrentMaxValue());
+            self::assertSame((int) ($i / 10) * 10 + 10, $this->sequenceGenerator->getCurrentMaxValue());
             self::assertSame($i + 1, $this->sequenceGenerator->getNextValue());
         }
     }

lib/Doctrine/ORM/Query/QueryException.php

Spacing +15 added lines, -15 removed lines

@@ -51,7 +51,7 @@ 
      */
     public static function syntaxError($message, $previous = null)
     {
-        return new self('[Syntax Error] ' . $message, 0, $previous);
+        return new self('[Syntax Error] '.$message, 0, $previous);
     }
 
     /**
@@ -62,7 +62,7 @@ 
      */
     public static function semanticalError($message, $previous = null)
     {
-        return new self('[Semantical Error] ' . $message, 0, $previous);
+        return new self('[Semantical Error] '.$message, 0, $previous);
     }
 
     /**
@@ -81,7 +81,7 @@ 
      */
     public static function invalidParameterType($expected, $received)
     {
-        return new self('Invalid parameter type, ' . $received . ' given, but ' . $expected . ' expected.');
+        return new self('Invalid parameter type, '.$received.' given, but '.$expected.' expected.');
     }
 
     /**
@@ -91,7 +91,7 @@ 
      */
     public static function invalidParameterPosition($pos)
     {
-        return new self('Invalid parameter position: ' . $pos);
+        return new self('Invalid parameter position: '.$pos);
     }
 
     /**
@@ -102,7 +102,7 @@ 
      */
     public static function tooManyParameters($expected, $received)
     {
-        return new self('Too many parameters: the query defines ' . $expected . ' parameters and you bound ' . $received);
+        return new self('Too many parameters: the query defines '.$expected.' parameters and you bound '.$received);
     }
 
     /**
@@ -113,7 +113,7 @@ 
      */
     public static function tooFewParameters($expected, $received)
     {
-        return new self('Too few parameters: the query defines ' . $expected . ' parameters but you only bound ' . $received);
+        return new self('Too few parameters: the query defines '.$expected.' parameters but you only bound '.$received);
     }
 
     /**
@@ -152,7 +152,7 @@ 
     public static function invalidPathExpression($pathExpr)
     {
         return new self(
-            "Invalid PathExpression '" . $pathExpr->identificationVariable . "." . $pathExpr->field . "'."
+            "Invalid PathExpression '".$pathExpr->identificationVariable.".".$pathExpr->field."'."
         );
     }
 
@@ -185,8 +185,8 @@ 
     public static function partialObjectsAreDangerous()
     {
         return new self(
-            "Loading partial objects is dangerous. Fetch full objects or consider " .
-            "using a different fetch mode. If you really want partial objects, " .
+            "Loading partial objects is dangerous. Fetch full objects or consider ".
+            "using a different fetch mode. If you really want partial objects, ".
             "set the doctrine.forcePartialLoad query hint to TRUE."
         );
     }
@@ -213,8 +213,8 @@ 
     public static function associationPathInverseSideNotSupported(PathExpression $pathExpr)
     {
         return new self(
-            'A single-valued association path expression to an inverse side is not supported in DQL queries. ' .
-            'Instead of "' . $pathExpr->identificationVariable . '.' . $pathExpr->field . '" use an explicit join.'
+            'A single-valued association path expression to an inverse side is not supported in DQL queries. '.
+            'Instead of "'.$pathExpr->identificationVariable.'.'.$pathExpr->field.'" use an explicit join.'
         );
     }
 
@@ -226,8 +226,8 @@ 
     public static function iterateWithFetchJoinNotAllowed($assoc)
     {
         return new self(
-            "Iterate with fetch join in class " . $assoc['sourceEntity'] .
-            " using association " . $assoc['fieldName'] . " not allowed."
+            "Iterate with fetch join in class ".$assoc['sourceEntity'].
+            " using association ".$assoc['fieldName']." not allowed."
         );
     }
 
@@ -251,7 +251,7 @@ 
      */
     public static function instanceOfUnrelatedClass($className, $rootClass)
     {
-        return new self("Cannot check if a child of '" . $rootClass . "' is instanceof '" . $className . "', " .
+        return new self("Cannot check if a child of '".$rootClass."' is instanceof '".$className."', ".
             "inheritance hierarchy does not exists between these two classes.");
     }
 
@@ -263,7 +263,7 @@ 
     public static function invalidQueryComponent($dqlAlias)
     {
         return new self(
-            "Invalid query component given for DQL alias '" . $dqlAlias . "', ".
+            "Invalid query component given for DQL alias '".$dqlAlias."', ".
             "requires 'metadata', 'parent', 'relation', 'map', 'nestingLevel' and 'token' keys."
         );
     }

lib/Doctrine/ORM/Query/Expr/OrderBy.php

Spacing +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
     public function add($sort, $order = null)
     {
         $order = ! $order ? 'ASC' : $order;
-        $this->parts[] = $sort . ' '. $order;
+        $this->parts[] = $sort.' '.$order;
     }
 
     /**
@@ -102,6 +102,6 @@ 
      */
     public function __toString()
     {
-        return $this->preSeparator . implode($this->separator, $this->parts) . $this->postSeparator;
+        return $this->preSeparator.implode($this->separator, $this->parts).$this->postSeparator;
     }
 }

lib/Doctrine/ORM/Tools/Pagination/LimitSubqueryOutputWalker.php

Spacing +11 added lines, -11 removed lines

@@ -208,8 +208,8 @@ 
         $sqlIdentifier = $this->getSQLIdentifier($AST);
 
         if ($hasOrderBy) {
-            $orderGroupBy = ' GROUP BY ' . implode(', ', $sqlIdentifier);
-            $sqlIdentifier[] = 'MIN(' . $this->walkResultVariable('dctrn_rownum') . ') AS dctrn_minrownum';
+            $orderGroupBy = ' GROUP BY '.implode(', ', $sqlIdentifier);
+            $sqlIdentifier[] = 'MIN('.$this->walkResultVariable('dctrn_rownum').') AS dctrn_minrownum';
         }
 
         // Build the counter query
@@ -220,7 +220,7 @@ 
         );
 
         if ($hasOrderBy) {
-            $sql .= $orderGroupBy . $outerOrderBy;
+            $sql .= $orderGroupBy.$outerOrderBy;
         }
 
         // Apply the limit and offset.
@@ -323,7 +323,7 @@ 
         foreach ($orderByPathExpressions as $pathExpression) {
             $idVar = $pathExpression->identificationVariable;
             $field = $pathExpression->field;
-            if (!isset($selects[$idVar])) {
+            if ( ! isset($selects[$idVar])) {
                 $selects[$idVar] = [];
             }
             $selects[$idVar][$field] = true;
@@ -334,7 +334,7 @@ 
         foreach ($AST->selectClause->selectExpressions as $selectExpression) {
             if ($selectExpression instanceof SelectExpression) {
                 $idVar = $selectExpression->expression;
-                if (!is_string($idVar)) {
+                if ( ! is_string($idVar)) {
                     continue;
                 }
                 $field = $selectExpression->fieldIdentificationVariable;
@@ -367,10 +367,10 @@ 
         array $sqlIdentifier,
         string $innerSql,
         string $sql,
-        ?OrderByClause $orderByClause
+        ? OrderByClause $orderByClause
     ) : string {
         // If the sql statement has an order by clause, we need to wrap it in a new select distinct statement
-        if (! $orderByClause) {
+        if ( ! $orderByClause) {
             return $sql;
         }
 
@@ -412,7 +412,7 @@ 
             $orderByItems[] = $orderByItemString;
             $identifier     = \substr($orderByItemString, 0, \strrpos($orderByItemString, ' '));
 
-            if (! \in_array($identifier, $identifiers, true)) {
+            if ( ! \in_array($identifier, $identifiers, true)) {
                 $identifiers[] = $identifier;
             }
         }
@@ -444,7 +444,7 @@ 
             $class = $metadataList[$dqlAliasForFieldAlias];
 
             // If the field is from a joined child table, we won't be ordering on it.
-            if (! isset($class->fieldMappings[$fieldName])) {
+            if ( ! isset($class->fieldMappings[$fieldName])) {
                 continue;
             }
 
@@ -465,7 +465,7 @@ 
                 // for the joined parent table.
                 $otherClassMetadata = $this->em->getClassMetadata($fieldMapping['declared']);
 
-                if (! $otherClassMetadata->isMappedSuperclass) {
+                if ( ! $otherClassMetadata->isMappedSuperclass) {
                     $sqlTableAliasForFieldAlias = $this->getSQLTableAlias($otherClassMetadata->getTableName(), $dqlAliasForFieldAlias);
                 }
             }
@@ -582,7 +582,7 @@ 
      */
     public function walkPathExpression($pathExpr)
     {
-        if (!$this->inSubSelect && !$this->platformSupportsRowNumber() && !in_array($pathExpr, $this->orderByPathExpressions)) {
+        if ( ! $this->inSubSelect && ! $this->platformSupportsRowNumber() && ! in_array($pathExpr, $this->orderByPathExpressions)) {
             $this->orderByPathExpressions[] = $pathExpr;
         }
 

tests/Doctrine/Performance/Mock/NonProxyLoadingUnitOfWork.php

Unused Use Statements -1 removed lines

@@ -2,7 +2,6 @@
 
 namespace Doctrine\Performance\Mock;
 
-use Doctrine\ORM\Query;
 use Doctrine\ORM\UnitOfWork;
 use Doctrine\Tests\ORM\Performance\PersisterMock;
 

tests/Doctrine/Performance/Hydration/SimpleInsertPerformanceBench.php

Spacing +3 added lines, -3 removed lines

@@ -40,8 +40,8 @@ 
         for ($i = 1; $i <= 10000; ++$i) {
             $user           = new CMS\CmsUser;
             $user->status   = 'user';
-            $user->username = 'user' . $i;
-            $user->name     = 'Mr.Smith-' . $i;
+            $user->username = 'user'.$i;
+            $user->name     = 'Mr.Smith-'.$i;
 
             $this->users[$i] = $user;
         }
@@ -52,7 +52,7 @@ 
         foreach ($this->users as $key => $user) {
             $this->entityManager->persist($user);
 
-            if (! ($key % 20)) {
+            if ( ! ($key % 20)) {
                 $this->entityManager->flush();
                 $this->entityManager->clear();
             }