doctrine / orm

lib/Doctrine/ORM/Persisters/Entity/AbstractEntityInheritancePersister.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Persisters\Entity;
 
@@ -59,6 +59,6 @@ 
             $columnType
         );
 
-        return $columnType->convertToPHPValueSQL($sql, $this->platform) . ' AS ' . $columnAlias;
+        return $columnType->convertToPHPValueSQL($sql, $this->platform).' AS '.$columnAlias;
     }
 }

Unused Use Statements -2 removed lines

@@ -4,8 +4,6 @@
 
 namespace Doctrine\ORM\Persisters\Entity;
 
-use Doctrine\ORM\Mapping\ClassMetadata;
-use Doctrine\DBAL\Types\Type;
 use Doctrine\ORM\Mapping\JoinColumnMetadata;
 
 /**

lib/Doctrine/ORM/Persisters/SqlExpressionVisitor.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Persisters;
 
@@ -82,15 +82,15 @@ 
             $expressionList[] = $this->dispatch($child);
         }
 
-        switch($expr->getType()) {
+        switch ($expr->getType()) {
             case CompositeExpression::TYPE_AND:
-                return '(' . implode(' AND ', $expressionList) . ')';
+                return '('.implode(' AND ', $expressionList).')';
 
             case CompositeExpression::TYPE_OR:
-                return '(' . implode(' OR ', $expressionList) . ')';
+                return '('.implode(' OR ', $expressionList).')';
 
             default:
-                throw new \RuntimeException("Unknown composite " . $expr->getType());
+                throw new \RuntimeException("Unknown composite ".$expr->getType());
         }
     }
 

lib/Doctrine/ORM/Utility/IdentifierFlattener.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Utility;
 
@@ -74,7 +74,7 @@ 
                 //     ? $this->unitOfWork->getEntityIdentifier($id[$field])
                 //     : $targetClassPersister->getIdentifier($id[$field])
                 // ;
-                $identifiers          = $targetClassPersister->getIdentifier($id[$field]);
+                $identifiers = $targetClassPersister->getIdentifier($id[$field]);
 
                 $associatedId = $this->flattenIdentifier($targetClassMetadata, $identifiers);
 

lib/Doctrine/ORM/PersistentCollection.php

Spacing +7 added lines, -7 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM;
 
@@ -396,7 +396,7 @@ 
      */
     public function containsKey($key)
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY &&
             $this->association->getIndexedBy()) {
@@ -413,7 +413,7 @@ 
      */
     public function contains($element)
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY) {
             $persister = $this->em->getUnitOfWork()->getCollectionPersister($this->association);
@@ -429,11 +429,11 @@ 
      */
     public function get($key)
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY &&
             $this->association->getIndexedBy()) {
-            if (!$this->typeClass->isIdentifierComposite() && $this->typeClass->isIdentifier($this->association->getIndexedBy())) {
+            if ( ! $this->typeClass->isIdentifierComposite() && $this->typeClass->isIdentifier($this->association->getIndexedBy())) {
                 return $this->em->find($this->typeClass->getClassName(), $key);
             }
 
@@ -448,7 +448,7 @@ 
      */
     public function count()
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY) {
             $persister = $this->em->getUnitOfWork()->getCollectionPersister($this->association);
@@ -602,7 +602,7 @@ 
      */
     public function slice($offset, $length = null)
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             ! $this->isDirty &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY) {

Doc Comments +2 added lines, -2 removed lines

@@ -169,7 +169,7 @@ 
      * INTERNAL:
      * Sets a keyed element in the collection during hydration.
      *
-     * @param mixed $key     The key to set.
+     * @param integer $key     The key to set.
      * @param mixed $element The element to set.
      *
      * @return void
@@ -583,7 +583,7 @@ 
      * Internal note: Tried to implement Serializable first but that did not work well
      *                with circular references. This solution seems simpler and works well.
      *
-     * @return array
+     * @return string[]
      */
     public function __sleep()
     {

lib/Doctrine/ORM/Query/QueryException.php

Spacing +14 added lines, -14 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Query;
 use Doctrine\ORM\Mapping\AssociationMetadata;
@@ -37,7 +37,7 @@ 
      */
     public static function syntaxError($message, $previous = null)
     {
-        return new self('[Syntax Error] ' . $message, 0, $previous);
+        return new self('[Syntax Error] '.$message, 0, $previous);
     }
 
     /**
@@ -48,7 +48,7 @@ 
      */
     public static function semanticalError($message, $previous = null)
     {
-        return new self('[Semantical Error] ' . $message, 0, $previous);
+        return new self('[Semantical Error] '.$message, 0, $previous);
     }
 
     /**
@@ -67,7 +67,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.');
     }
 
     /**
@@ -77,7 +77,7 @@ 
      */
     public static function invalidParameterPosition($pos)
     {
-        return new self('Invalid parameter position: ' . $pos);
+        return new self('Invalid parameter position: '.$pos);
     }
 
     /**
@@ -88,7 +88,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);
     }
 
     /**
@@ -99,7 +99,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);
     }
 
     /**
@@ -138,7 +138,7 @@ 
     public static function invalidPathExpression($pathExpr)
     {
         return new self(
-            "Invalid PathExpression '" . $pathExpr->identificationVariable . "." . $pathExpr->field . "'."
+            "Invalid PathExpression '".$pathExpr->identificationVariable.".".$pathExpr->field."'."
         );
     }
 
@@ -171,8 +171,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."
         );
     }
@@ -199,8 +199,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.'
         );
     }
 
@@ -240,7 +240,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.");
     }
 
@@ -252,7 +252,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/Printer.php

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Query;
 
@@ -50,7 +50,7 @@ 
      */
     public function startProduction($name)
     {
-        $this->println('(' . $name);
+        $this->println('('.$name);
         $this->indent++;
     }
 

lib/Doctrine/ORM/Query/FilterCollection.php

Spacing +5 added lines, -5 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Query;
 
@@ -18,7 +18,7 @@ 
     /**
      * A filter object is in CLEAN state when it has no changed parameters.
      */
-    const FILTERS_STATE_CLEAN  = 1;
+    const FILTERS_STATE_CLEAN = 1;
 
     /**
      * A filter object is in DIRTY state when it has changed parameters.
@@ -89,7 +89,7 @@ 
     public function enable($name)
     {
         if ( ! $this->has($name)) {
-            throw new \InvalidArgumentException("Filter '" . $name . "' does not exist.");
+            throw new \InvalidArgumentException("Filter '".$name."' does not exist.");
         }
 
         if ( ! $this->isEnabled($name)) {
@@ -141,7 +141,7 @@ 
     public function getFilter($name)
     {
         if ( ! $this->isEnabled($name)) {
-            throw new \InvalidArgumentException("Filter '" . $name . "' is not enabled.");
+            throw new \InvalidArgumentException("Filter '".$name."' is not enabled.");
         }
 
         return $this->enabledFilters[$name];
@@ -194,7 +194,7 @@ 
         $filterHash = '';
 
         foreach ($this->enabledFilters as $name => $filter) {
-            $filterHash .= $name . $filter;
+            $filterHash .= $name.$filter;
         }
 
         return $filterHash;

lib/Doctrine/ORM/Query/Filter/SQLFilter.php

Spacing +4 added lines, -4 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Query\Filter;
 
@@ -85,8 +85,8 @@ 
      */
     final public function getParameter($name)
     {
-        if (!isset($this->parameters[$name])) {
-            throw new \InvalidArgumentException("Parameter '" . $name . "' does not exist.");
+        if ( ! isset($this->parameters[$name])) {
+            throw new \InvalidArgumentException("Parameter '".$name."' does not exist.");
         }
 
         return $this->em->getConnection()->quote($this->parameters[$name]['value'], $this->parameters[$name]['type']);
@@ -101,7 +101,7 @@ 
      */
     final public function hasParameter($name)
     {
-        if (!isset($this->parameters[$name])) {
+        if ( ! isset($this->parameters[$name])) {
             return false;
         }
 

lib/Doctrine/ORM/Query/TreeWalkerChainIterator.php

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-declare(strict_types=1);
+declare(strict_types = 1);
 
 namespace Doctrine\ORM\Query;