doctrine / orm

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

Spacing +4 added lines, -4 removed lines

@@ -62,12 +62,12 @@ 
      */
     public function add($arg)
     {
-        if ($arg !== null && (! $arg instanceof self || $arg->count() > 0)) {
+        if ($arg !== null && ( ! $arg instanceof self || $arg->count() > 0)) {
             // If we decide to keep Expr\Base instances, we can use this check
-            if (! is_string($arg)) {
+            if ( ! is_string($arg)) {
                 $class = get_class($arg);
 
-                if (! in_array($class, $this->allowedClasses, true)) {
+                if ( ! in_array($class, $this->allowedClasses, true)) {
                     throw new \InvalidArgumentException(
                         sprintf("Expression of type '%s' not allowed in this context.", $class)
                     );
@@ -97,6 +97,6 @@ 
             return (string) $this->parts[0];
         }
 
-        return $this->preSeparator . implode($this->separator, $this->parts) . $this->postSeparator;
+        return $this->preSeparator.implode($this->separator, $this->parts).$this->postSeparator;
     }
 }

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

Spacing +1 added lines, -1 removed lines

@@ -19,6 +19,6 @@
      */
     public static function noDispatchForNode($node)
     {
-        return new self('Double-dispatch for node ' . get_class($node) . ' is not supported.');
+        return new self('Double-dispatch for node '.get_class($node).' is not supported.');
     }
 }

lib/Doctrine/ORM/Query/AST/Functions/IdentityFunction.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
         $tableAlias       = $sqlWalker->getSQLTableAlias($tableName, $dqlAlias);
         $quotedColumnName = $platform->quoteIdentifier($joinColumn->getColumnName());
 
-        return $tableAlias . '.' . $quotedColumnName;
+        return $tableAlias.'.'.$quotedColumnName;
     }
 
     /**

lib/Doctrine/ORM/Query/AST/Functions/SizeFunction.php

Spacing +3 added lines, -3 removed lines

@@ -44,7 +44,7 @@ 
             $targetTableAlias = $sqlWalker->getSQLTableAlias($targetClass->getTableName());
             $sourceTableAlias = $sqlWalker->getSQLTableAlias($class->getTableName(), $dqlAlias);
 
-            $sql .= $targetTableName . ' ' . $targetTableAlias . ' WHERE ';
+            $sql .= $targetTableName.' '.$targetTableAlias.' WHERE ';
 
             $owningAssociation = $targetClass->getProperty($association->getMappedBy());
             $first             = true;
@@ -78,7 +78,7 @@ 
             }
 
             // join to target table
-            $sql .= $joinTableName . ' ' . $joinTableAlias . ' WHERE ';
+            $sql .= $joinTableName.' '.$joinTableAlias.' WHERE ';
 
             $joinColumns = $association->isOwningSide()
                 ? $joinTable->getJoinColumns()
@@ -104,7 +104,7 @@ 
             }
         }
 
-        return '(' . $sql . ')';
+        return '('.$sql.')';
     }
 
     /**

lib/Doctrine/ORM/Query/AST/Functions/AbsFunction.php

Spacing +2 added lines, -2 removed lines

@@ -23,9 +23,9 @@
      */
     public function getSql(SqlWalker $sqlWalker)
     {
-        return 'ABS(' . $sqlWalker->walkSimpleArithmeticExpression(
+        return 'ABS('.$sqlWalker->walkSimpleArithmeticExpression(
             $this->simpleArithmeticExpression
-        ) . ')';
+        ).')';
     }
 
     /**

lib/Doctrine/ORM/Query/ResultSetMappingBuilder.php

Spacing +5 added lines, -5 removed lines

@@ -123,7 +123,7 @@ 
         $classMetadata = $this->em->getClassMetadata($class);
         $platform      = $this->em->getConnection()->getDatabasePlatform();
 
-        if (! $this->isInheritanceSupported($classMetadata)) {
+        if ( ! $this->isInheritanceSupported($classMetadata)) {
             throw new \InvalidArgumentException(
                 'ResultSetMapping builder does not currently support your inheritance scheme.'
             );
@@ -163,7 +163,7 @@ 
                             );
                         }
 
-                        if (! $joinColumn->getType()) {
+                        if ( ! $joinColumn->getType()) {
                             $joinColumn->setType(PersisterHelper::getTypeOfColumn($referencedColumnName, $targetClass, $this->em));
                         }
 
@@ -202,7 +202,7 @@ 
     {
         switch ($mode) {
             case self::COLUMN_RENAMING_INCREMENT:
-                return $columnName . $this->sqlCounter++;
+                return $columnName.$this->sqlCounter++;
 
             case self::COLUMN_RENAMING_CUSTOM:
                 return $customRenameColumns[$columnName] ?? $columnName;
@@ -233,7 +233,7 @@ 
                 $sql .= ', ';
             }
 
-            $sql .= $tableAlias . '.';
+            $sql .= $tableAlias.'.';
 
             if (isset($this->fieldMappings[$columnName])) {
                 $class = $this->em->getClassMetadata($this->declaringClasses[$columnName]);
@@ -245,7 +245,7 @@ 
                 $sql .= $this->discriminatorColumns[$dqlAlias];
             }
 
-            $sql .= ' AS ' . $columnName;
+            $sql .= ' AS '.$columnName;
         }
 
         return $sql;

lib/Doctrine/ORM/Query/ResultSetMapping.php

Spacing +3 added lines, -3 removed lines

@@ -222,7 +222,7 @@ 
         $found = false;
 
         foreach (array_merge($this->metaMappings, $this->fieldMappings) as $columnName => $columnFieldName) {
-            if (! ($columnFieldName === $fieldName && $this->columnOwnerMap[$columnName] === $alias)) {
+            if ( ! ($columnFieldName === $fieldName && $this->columnOwnerMap[$columnName] === $alias)) {
                 continue;
             }
 
@@ -331,7 +331,7 @@ 
         // field name => class name of declaring class
         $this->declaringClasses[$columnName] = $declaringClass ?: $this->aliasMap[$alias];
 
-        if (! $this->isMixed && $this->scalarMappings) {
+        if ( ! $this->isMixed && $this->scalarMappings) {
             $this->isMixed = true;
         }
 
@@ -376,7 +376,7 @@ 
         $this->scalarMappings[$columnName] = $alias;
         $this->typeMappings[$columnName]   = $type;
 
-        if (! $this->isMixed && $this->fieldMappings) {
+        if ( ! $this->isMixed && $this->fieldMappings) {
             $this->isMixed = true;
         }
 

lib/Doctrine/ORM/Query/Parser.php

Spacing +31 added lines, -31 removed lines

@@ -395,7 +395,7 @@ 
         }
 
         foreach ($this->queryComponents as $dqlAlias => $qComp) {
-            if (! isset($this->identVariableExpressions[$dqlAlias])) {
+            if ( ! isset($this->identVariableExpressions[$dqlAlias])) {
                 continue;
             }
 
@@ -459,7 +459,7 @@ 
         $tokenStr = substr($dql, (int) $token['position'], $length);
 
         // Building informative message
-        $message = 'line 0, col ' . $tokenPos . " near '" . $tokenStr . "': Error: " . $message;
+        $message = 'line 0, col '.$tokenPos." near '".$tokenStr."': Error: ".$message;
 
         throw QueryException::semanticalError(
             $message,
@@ -562,7 +562,7 @@ 
             $identVariable = $deferredItem['expression'];
 
             // Check if IdentificationVariable exists in queryComponents
-            if (! isset($this->queryComponents[$identVariable])) {
+            if ( ! isset($this->queryComponents[$identVariable])) {
                 $this->semanticalError(
                     sprintf("'%s' is not defined.", $identVariable),
                     $deferredItem['token']
@@ -572,7 +572,7 @@ 
             $qComp = $this->queryComponents[$identVariable];
 
             // Check if queryComponent points to an AbstractSchemaName or a ResultVariable
-            if (! isset($qComp['metadata'])) {
+            if ( ! isset($qComp['metadata'])) {
                 $this->semanticalError(
                     sprintf("'%s' does not point to a Class.", $identVariable),
                     $deferredItem['token']
@@ -606,7 +606,7 @@ 
             // If the namespace is not given then assumes the first FROM entity namespace
             if (strpos($className, '\\') === false && ! class_exists($className) && strpos($fromClassName, '\\') !== false) {
                 $namespace = substr($fromClassName, 0, strrpos($fromClassName, '\\'));
-                $fqcn      = $namespace . '\\' . $className;
+                $fqcn      = $namespace.'\\'.$className;
 
                 if (class_exists($fqcn)) {
                     $expression->className = $fqcn;
@@ -614,13 +614,13 @@ 
                 }
             }
 
-            if (! class_exists($className)) {
+            if ( ! class_exists($className)) {
                 $this->semanticalError(sprintf('Class "%s" is not defined.', $className), $token);
             }
 
             $class = new \ReflectionClass($className);
 
-            if (! $class->isInstantiable()) {
+            if ( ! $class->isInstantiable()) {
                 $this->semanticalError(sprintf('Class "%s" can not be instantiated.', $className), $token);
             }
 
@@ -677,7 +677,7 @@ 
             $resultVariable = $deferredItem['expression'];
 
             // Check if ResultVariable exists in queryComponents
-            if (! isset($this->queryComponents[$resultVariable])) {
+            if ( ! isset($this->queryComponents[$resultVariable])) {
                 $this->semanticalError(
                     sprintf("'%s' is not defined.", $resultVariable),
                     $deferredItem['token']
@@ -687,7 +687,7 @@ 
             $qComp = $this->queryComponents[$resultVariable];
 
             // Check if queryComponent points to an AbstractSchemaName or a ResultVariable
-            if (! isset($qComp['resultVariable'])) {
+            if ( ! isset($qComp['resultVariable'])) {
                 $this->semanticalError(
                     sprintf("'%s' does not point to a ResultVariable.", $resultVariable),
                     $deferredItem['token']
@@ -729,9 +729,9 @@ 
             $property = $class->getProperty($field);
 
             // Check if field or association exists
-            if (! $property) {
+            if ( ! $property) {
                 $this->semanticalError(
-                    'Class ' . $class->getClassName() . ' has no field or association named ' . $field,
+                    'Class '.$class->getClassName().' has no field or association named '.$field,
                     $deferredItem['token']
                 );
             }
@@ -748,7 +748,7 @@ 
             // Validate if PathExpression is one of the expected types
             $expectedType = $pathExpression->expectedType;
 
-            if (! ($expectedType & $fieldType)) {
+            if ( ! ($expectedType & $fieldType)) {
                 // We need to recognize which was expected type(s)
                 $expectedStringTypes = [];
 
@@ -770,8 +770,8 @@ 
                 // Build the error message
                 $semanticalError  = 'Invalid PathExpression. ';
                 $semanticalError .= count($expectedStringTypes) === 1
-                    ? 'Must be a ' . $expectedStringTypes[0] . '.'
-                    : implode(' or ', $expectedStringTypes) . ' expected.';
+                    ? 'Must be a '.$expectedStringTypes[0].'.'
+                    : implode(' or ', $expectedStringTypes).' expected.';
 
                 $this->semanticalError($semanticalError, $deferredItem['token']);
             }
@@ -783,7 +783,7 @@ 
 
     private function processRootEntityAliasSelected()
     {
-        if (! $this->identVariableExpressions) {
+        if ( ! $this->identVariableExpressions) {
             return;
         }
 
@@ -940,7 +940,7 @@ 
 
         [$namespaceAlias, $simpleClassName] = explode(':', $this->lexer->token['value']);
 
-        return $this->em->getConfiguration()->getEntityNamespace($namespaceAlias) . '\\' . $simpleClassName;
+        return $this->em->getConfiguration()->getEntityNamespace($namespaceAlias).'\\'.$simpleClassName;
     }
 
     /**
@@ -1019,9 +1019,9 @@ 
     {
         $identVariable = $this->IdentificationVariable();
 
-        if (! isset($this->queryComponents[$identVariable])) {
+        if ( ! isset($this->queryComponents[$identVariable])) {
             $this->semanticalError(
-                'Identification Variable ' . $identVariable . ' used in join path expression but was not defined before.'
+                'Identification Variable '.$identVariable.' used in join path expression but was not defined before.'
             );
         }
 
@@ -1035,8 +1035,8 @@ 
         $class    = $qComp['metadata'];
         $property = $class->getProperty($field);
 
-        if (! ($property !== null && $property instanceof AssociationMetadata)) {
-            $this->semanticalError('Class ' . $class->getClassName() . ' has no association named ' . $field);
+        if ( ! ($property !== null && $property instanceof AssociationMetadata)) {
+            $this->semanticalError('Class '.$class->getClassName().' has no association named '.$field);
         }
 
         return new AST\JoinAssociationPathExpression($identVariable, $field);
@@ -1066,7 +1066,7 @@ 
             while ($this->lexer->isNextToken(Lexer::T_DOT)) {
                 $this->match(Lexer::T_DOT);
                 $this->match(Lexer::T_IDENTIFIER);
-                $field .= '.' . $this->lexer->token['value'];
+                $field .= '.'.$this->lexer->token['value'];
             }
         }
 
@@ -1450,7 +1450,7 @@ 
         // Still need to decide between IdentificationVariable or ResultVariable
         $lookaheadValue = $this->lexer->lookahead['value'];
 
-        if (! isset($this->queryComponents[$lookaheadValue])) {
+        if ( ! isset($this->queryComponents[$lookaheadValue])) {
             $this->semanticalError('Cannot group by undefined identification or result variable.');
         }
 
@@ -1798,7 +1798,7 @@ 
         while ($this->lexer->isNextToken(Lexer::T_DOT)) {
             $this->match(Lexer::T_DOT);
             $this->match(Lexer::T_IDENTIFIER);
-            $field .= '.' . $this->lexer->token['value'];
+            $field .= '.'.$this->lexer->token['value'];
         }
 
         $partialFieldSet[] = $field;
@@ -1812,7 +1812,7 @@ 
             while ($this->lexer->isNextToken(Lexer::T_DOT)) {
                 $this->match(Lexer::T_DOT);
                 $this->match(Lexer::T_IDENTIFIER);
-                $field .= '.' . $this->lexer->token['value'];
+                $field .= '.'.$this->lexer->token['value'];
             }
 
             $partialFieldSet[] = $field;
@@ -2433,7 +2433,7 @@ 
 
         // Phase 1 AST optimization: Prevent AST\ConditionalFactor
         // if only one AST\ConditionalPrimary is defined
-        if (! $not) {
+        if ( ! $not) {
             return $conditionalPrimary;
         }
 
@@ -2452,7 +2452,7 @@ 
     {
         $condPrimary = new AST\ConditionalPrimary();
 
-        if (! $this->lexer->isNextToken(Lexer::T_OPEN_PARENTHESIS)) {
+        if ( ! $this->lexer->isNextToken(Lexer::T_OPEN_PARENTHESIS)) {
             $condPrimary->simpleConditionalExpression = $this->SimpleConditionalExpression();
 
             return $condPrimary;
@@ -2620,7 +2620,7 @@ 
             $this->match(Lexer::T_OF);
         }
 
-        $collMemberExpr      = new AST\CollectionMemberExpression(
+        $collMemberExpr = new AST\CollectionMemberExpression(
             $entityExpr,
             $this->CollectionValuedPathExpression()
         );
@@ -2955,7 +2955,7 @@ 
         $lookaheadType = $this->lexer->lookahead['type'];
         $isDistinct    = false;
 
-        if (! in_array($lookaheadType, [Lexer::T_COUNT, Lexer::T_AVG, Lexer::T_MAX, Lexer::T_MIN, Lexer::T_SUM], true)) {
+        if ( ! in_array($lookaheadType, [Lexer::T_COUNT, Lexer::T_AVG, Lexer::T_MAX, Lexer::T_MIN, Lexer::T_SUM], true)) {
             $this->syntaxError('One of: MAX, MIN, AVG, SUM, COUNT');
         }
 
@@ -2985,7 +2985,7 @@ 
         $lookaheadType = $this->lexer->lookahead['type'];
         $value         = $this->lexer->lookahead['value'];
 
-        if (! in_array($lookaheadType, [Lexer::T_ALL, Lexer::T_ANY, Lexer::T_SOME], true)) {
+        if ( ! in_array($lookaheadType, [Lexer::T_ALL, Lexer::T_ANY, Lexer::T_SOME], true)) {
             $this->syntaxError('ALL, ANY or SOME');
         }
 
@@ -3223,7 +3223,7 @@ 
                 $lookaheadValue = $this->lexer->lookahead['value'];
 
                 // Validate existing component
-                if (! isset($this->queryComponents[$lookaheadValue])) {
+                if ( ! isset($this->queryComponents[$lookaheadValue])) {
                     $this->semanticalError('Cannot add having condition on undefined result variable.');
                 }
 
@@ -3234,7 +3234,7 @@ 
                 }
 
                 // Validating ResultVariable
-                if (! isset($this->queryComponents[$lookaheadValue]['resultVariable'])) {
+                if ( ! isset($this->queryComponents[$lookaheadValue]['resultVariable'])) {
                     $this->semanticalError('Cannot add having condition on a non result variable.');
                 }
 

lib/Doctrine/ORM/Query/QueryException.php

Spacing +22 added lines, -22 removed lines

@@ -32,7 +32,7 @@ 
      */
     public static function syntaxError($message, $previous = null)
     {
-        return new self('[Syntax Error] ' . $message, 0, $previous);
+        return new self('[Syntax Error] '.$message, 0, $previous);
     }
 
     /**
@@ -43,7 +43,7 @@ 
      */
     public static function semanticalError($message, $previous = null)
     {
-        return new self('[Semantical Error] ' . $message, 0, $previous);
+        return new self('[Semantical Error] '.$message, 0, $previous);
     }
 
     /**
@@ -62,7 +62,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.');
     }
 
     /**
@@ -72,7 +72,7 @@ 
      */
     public static function invalidParameterPosition($pos)
     {
-        return new self('Invalid parameter position: ' . $pos);
+        return new self('Invalid parameter position: '.$pos);
     }
 
     /**
@@ -83,7 +83,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);
     }
 
     /**
@@ -94,7 +94,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);
     }
 
     /**
@@ -104,7 +104,7 @@ 
      */
     public static function invalidParameterFormat($value)
     {
-        return new self('Invalid parameter format, ' . $value . ' given, but :<name> or ?<num> expected.');
+        return new self('Invalid parameter format, '.$value.' given, but :<name> or ?<num> expected.');
     }
 
     /**
@@ -114,7 +114,7 @@ 
      */
     public static function unknownParameter($key)
     {
-        return new self('Invalid parameter: token ' . $key . ' is not defined in the query.');
+        return new self('Invalid parameter: token '.$key.' is not defined in the query.');
     }
 
     /**
@@ -133,7 +133,7 @@ 
     public static function invalidPathExpression($pathExpr)
     {
         return new self(
-            "Invalid PathExpression '" . $pathExpr->identificationVariable . '.' . $pathExpr->field . "'."
+            "Invalid PathExpression '".$pathExpr->identificationVariable.'.'.$pathExpr->field."'."
         );
     }
 
@@ -144,7 +144,7 @@ 
      */
     public static function invalidLiteral($literal)
     {
-        return new self("Invalid literal '" . $literal . "'");
+        return new self("Invalid literal '".$literal."'");
     }
 
     /**
@@ -155,8 +155,8 @@ 
     public static function iterateWithFetchJoinCollectionNotAllowed($assoc)
     {
         return new self(
-            'Invalid query operation: Not allowed to iterate over fetch join collections ' .
-            'in class ' . $assoc['sourceEntity'] . ' association ' . $assoc['fieldName']
+            'Invalid query operation: Not allowed to iterate over fetch join collections '.
+            'in class '.$assoc['sourceEntity'].' association '.$assoc['fieldName']
         );
     }
 
@@ -166,8 +166,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.'
         );
     }
@@ -180,8 +180,8 @@ 
     public static function overwritingJoinConditionsNotYetSupported($assoc)
     {
         return new self(
-            'Unsupported query operation: It is not yet possible to overwrite the join ' .
-            'conditions in class ' . $assoc['sourceEntityName'] . ' association ' . $assoc['fieldName'] . '. ' .
+            'Unsupported query operation: It is not yet possible to overwrite the join '.
+            'conditions in class '.$assoc['sourceEntityName'].' association '.$assoc['fieldName'].'. '.
             'Use WITH to append additional join conditions to the association.'
         );
     }
@@ -192,8 +192,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.'
         );
     }
 
@@ -217,8 +217,8 @@ 
     public static function associationPathCompositeKeyNotSupported()
     {
         return new self(
-            'A single-valued association path expression to an entity with a composite primary ' .
-            'key is not supported. Explicitly name the components of the composite primary key ' .
+            'A single-valued association path expression to an entity with a composite primary '.
+            'key is not supported. Explicitly name the components of the composite primary key '.
             'in the query.'
         );
     }
@@ -231,7 +231,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.');
     }
 
@@ -243,7 +243,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."
         );
     }