doctrine / orm

lib/Doctrine/ORM/ORMInvalidArgumentException.php

Spacing +17 added lines, -17 removed lines

@@ -30,7 +30,7 @@ 
      */
     public static function scheduleInsertForManagedEntity($entity)
     {
-        return new self('A managed+dirty entity ' . self::objToStr($entity) . ' can not be scheduled for insertion.');
+        return new self('A managed+dirty entity '.self::objToStr($entity).' can not be scheduled for insertion.');
     }
 
     /**
@@ -40,7 +40,7 @@ 
      */
     public static function scheduleInsertForRemovedEntity($entity)
     {
-        return new self('Removed entity ' . self::objToStr($entity) . ' can not be scheduled for insertion.');
+        return new self('Removed entity '.self::objToStr($entity).' can not be scheduled for insertion.');
     }
 
     /**
@@ -50,7 +50,7 @@ 
      */
     public static function scheduleInsertTwice($entity)
     {
-        return new self('Entity ' . self::objToStr($entity) . ' can not be scheduled for insertion twice.');
+        return new self('Entity '.self::objToStr($entity).' can not be scheduled for insertion twice.');
     }
 
     /**
@@ -62,7 +62,7 @@ 
     public static function entityWithoutIdentity($className, $entity)
     {
         return new self(
-            "The given entity of type '" . $className . "' (" . self::objToStr($entity) . ') has no identity/no ' .
+            "The given entity of type '".$className."' (".self::objToStr($entity).') has no identity/no '.
             'id values set. It cannot be added to the identity map.'
         );
     }
@@ -74,7 +74,7 @@ 
      */
     public static function readOnlyRequiresManagedEntity($entity)
     {
-        return new self('Only managed entities can be marked or checked as read only. But ' . self::objToStr($entity) . ' is not');
+        return new self('Only managed entities can be marked or checked as read only. But '.self::objToStr($entity).' is not');
     }
 
     /**
@@ -85,7 +85,7 @@ 
     public static function newEntitiesFoundThroughRelationships($newEntitiesWithAssociations)
     {
         $errorMessages = array_map(
-            static function (array $newEntityWithAssociation) : string {
+            static function(array $newEntityWithAssociation) : string {
                 [$associationMetadata, $entity] = $newEntityWithAssociation;
 
                 return self::newEntityFoundThroughRelationshipMessage($associationMetadata, $entity);
@@ -154,7 +154,7 @@ 
      */
     public static function entityNotManaged($entity)
     {
-        return new self('Entity ' . self::objToStr($entity) . ' is not managed. An entity is managed if its fetched ' .
+        return new self('Entity '.self::objToStr($entity).' is not managed. An entity is managed if its fetched '.
             'from the database or registered as new through EntityManager#persist');
     }
 
@@ -166,7 +166,7 @@ 
      */
     public static function entityHasNoIdentity($entity, $operation)
     {
-        return new self('Entity has no identity, therefore ' . $operation . ' cannot be performed. ' . self::objToStr($entity));
+        return new self('Entity has no identity, therefore '.$operation.' cannot be performed. '.self::objToStr($entity));
     }
 
     /**
@@ -177,7 +177,7 @@ 
      */
     public static function entityIsRemoved($entity, $operation)
     {
-        return new self('Entity is removed, therefore ' . $operation . ' cannot be performed. ' . self::objToStr($entity));
+        return new self('Entity is removed, therefore '.$operation.' cannot be performed. '.self::objToStr($entity));
     }
 
     /**
@@ -188,7 +188,7 @@ 
      */
     public static function detachedEntityCannot($entity, $operation)
     {
-        return new self('Detached entity ' . self::objToStr($entity) . ' cannot be ' . $operation);
+        return new self('Detached entity '.self::objToStr($entity).' cannot be '.$operation);
     }
 
     /**
@@ -200,8 +200,8 @@ 
      */
     public static function invalidObject($context, $given, $parameterIndex = 1)
     {
-        return new self($context . ' expects parameter ' . $parameterIndex .
-            ' to be an entity object, ' . gettype($given) . ' given.');
+        return new self($context.' expects parameter '.$parameterIndex.
+            ' to be an entity object, '.gettype($given).' given.');
     }
 
     /**
@@ -209,7 +209,7 @@ 
      */
     public static function invalidCompositeIdentifier()
     {
-        return new self('Binding an entity with a composite primary key to a query is not supported. ' .
+        return new self('Binding an entity with a composite primary key to a query is not supported. '.
             'You should split the parameter into the explicit fields and bind them separately.');
     }
 
@@ -246,7 +246,7 @@ 
      */
     private static function objToStr($obj) : string
     {
-        return method_exists($obj, '__toString') ? (string) $obj : get_class($obj) . '@' . spl_object_id($obj);
+        return method_exists($obj, '__toString') ? (string) $obj : get_class($obj).'@'.spl_object_id($obj);
     }
 
     /**
@@ -255,15 +255,15 @@ 
     private static function newEntityFoundThroughRelationshipMessage(AssociationMetadata $association, $entity) : string
     {
         return 'A new entity was found through the relationship \''
-            . $association->getSourceEntity() . '#' . $association->getName() . '\' that was not'
-            . ' configured to cascade persist operations for entity: ' . self::objToStr($entity) . '.'
+            . $association->getSourceEntity().'#'.$association->getName().'\' that was not'
+            . ' configured to cascade persist operations for entity: '.self::objToStr($entity).'.'
             . ' To solve this issue: Either explicitly call EntityManager#persist()'
             . ' on this unknown entity or configure cascade persist'
             . ' this association in the mapping for example @ManyToOne(..,cascade={"persist"}).'
             . (method_exists($entity, '__toString')
                 ? ''
                 : ' If you cannot find out which entity causes the problem implement \''
-                . $association->getTargetEntity() . '#__toString()\' to get a clue.'
+                . $association->getTargetEntity().'#__toString()\' to get a clue.'
             );
     }
 }

lib/Doctrine/ORM/Mapping/Factory/Autoloader.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
         // remove namespace separators from remaining class name
         $fileName = str_replace('\\', '', $classNameRelativeToMetadataNamespace);
 
-        return $metadataDir . DIRECTORY_SEPARATOR . $fileName . '.php';
+        return $metadataDir.DIRECTORY_SEPARATOR.$fileName.'.php';
     }
 
     /**
@@ -63,7 +63,7 @@ 
     ) : Closure {
         $metadataNamespace = ltrim($metadataNamespace, '\\');
 
-        if (! ($notFoundCallback === null || is_callable($notFoundCallback))) {
+        if ( ! ($notFoundCallback === null || is_callable($notFoundCallback))) {
             $type = is_object($notFoundCallback) ? get_class($notFoundCallback) : gettype($notFoundCallback);
 
             throw new InvalidArgumentException(
@@ -71,7 +71,7 @@ 
             );
         }
 
-        $autoloader = static function ($className) use ($metadataDir, $metadataNamespace, $notFoundCallback) {
+        $autoloader = static function($className) use ($metadataDir, $metadataNamespace, $notFoundCallback) {
             if (strpos($className, $metadataNamespace) === 0) {
                 $file = Autoloader::resolveFile($metadataDir, $metadataNamespace, $className);
 

lib/Doctrine/ORM/Mapping/Factory/Strategy/FileWriterClassMetadataGeneratorStrategy.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 
         $this->ensureDirectoryIsReady(dirname($filePath));
 
-        $tmpFileName = $filePath . '.' . uniqid('', true);
+        $tmpFileName = $filePath.'.'.uniqid('', true);
 
         file_put_contents($tmpFileName, $sourceCode);
         @chmod($tmpFileName, 0664);
@@ -50,11 +50,11 @@ 
      */
     private function ensureDirectoryIsReady(string $directory)
     {
-        if (! is_dir($directory) && (@mkdir($directory, 0775, true) === false)) {
+        if ( ! is_dir($directory) && (@mkdir($directory, 0775, true) === false)) {
             throw new RuntimeException(sprintf('Your metadata directory "%s" must be writable', $directory));
         }
 
-        if (! is_writable($directory)) {
+        if ( ! is_writable($directory)) {
             throw new RuntimeException(sprintf('Your proxy directory "%s" must be writable', $directory));
         }
     }

lib/Doctrine/ORM/Mapping/DefaultEntityListenerResolver.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
      */
     public function register($object)
     {
-        if (! is_object($object)) {
+        if ( ! is_object($object)) {
             throw new InvalidArgumentException(sprintf('An object was expected, but got "%s".', gettype($object)));
         }
 

lib/Doctrine/ORM/Mapping/Exporter/VariableExporter.php

Spacing +5 added lines, -5 removed lines

@@ -24,30 +24,30 @@
      */
     public function export($value, int $indentationLevel = 0) : string
     {
-        if (! is_array($value)) {
+        if ( ! is_array($value)) {
             return var_export($value, true);
         }
 
         $indentation  = str_repeat(self::INDENTATION, $indentationLevel);
-        $longestKey   = array_reduce(array_keys($value), static function ($k, $v) {
+        $longestKey   = array_reduce(array_keys($value), static function($k, $v) {
             return (string) (strlen((string) $k) > strlen((string) $v) ? $k : $v);
         });
         $maxKeyLength = strlen($longestKey) + (is_numeric($longestKey) ? 0 : 2);
 
         $lines = [];
 
-        $lines[] = $indentation . '[';
+        $lines[] = $indentation.'[';
 
         foreach ($value as $entryKey => $entryValue) {
             $lines[] = sprintf(
                 '%s%s => %s,',
-                $indentation . self::INDENTATION,
+                $indentation.self::INDENTATION,
                 str_pad(var_export($entryKey, true), $maxKeyLength),
                 ltrim($this->export($entryValue, $indentationLevel + 1))
             );
         }
 
-        $lines[] = $indentation . ']';
+        $lines[] = $indentation.']';
 
         return implode(PHP_EOL, $lines);
     }

lib/Doctrine/ORM/Persisters/SqlExpressionVisitor.php

Spacing +3 added lines, -3 removed lines

@@ -74,13 +74,13 @@
 
         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/Query/Lexer.php

Spacing +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
 
             // Recognize identifiers, aliased or qualified names
             case ctype_alpha($value[0]) || $value[0] === '_' || $value[0] === '\\':
-                $name = 'Doctrine\ORM\Query\Lexer::T_' . strtoupper($value);
+                $name = 'Doctrine\ORM\Query\Lexer::T_'.strtoupper($value);
 
                 if (defined($name)) {
                     $type = constant($name);

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

Spacing +4 added lines, -4 removed lines

@@ -63,12 +63,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)
                     );
@@ -98,6 +98,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/Parser.php

Spacing +31 added lines, -31 removed lines

@@ -397,7 +397,7 @@ 
         }
 
         foreach ($this->queryComponents as $dqlAlias => $qComp) {
-            if (! isset($this->identVariableExpressions[$dqlAlias])) {
+            if ( ! isset($this->identVariableExpressions[$dqlAlias])) {
                 continue;
             }
 
@@ -461,7 +461,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,
@@ -564,7 +564,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']
@@ -574,7 +574,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']
@@ -608,7 +608,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;
@@ -616,13 +616,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);
             }
 
@@ -679,7 +679,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']
@@ -689,7 +689,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']
@@ -731,9 +731,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']
                 );
             }
@@ -749,7 +749,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 = [];
 
@@ -771,8 +771,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']);
             }
@@ -784,7 +784,7 @@ 
 
     private function processRootEntityAliasSelected()
     {
-        if (! $this->identVariableExpressions) {
+        if ( ! $this->identVariableExpressions) {
             return;
         }
 
@@ -941,7 +941,7 @@ 
 
         [$namespaceAlias, $simpleClassName] = explode(':', $this->lexer->token['value']);
 
-        return $this->em->getConfiguration()->getEntityNamespace($namespaceAlias) . '\\' . $simpleClassName;
+        return $this->em->getConfiguration()->getEntityNamespace($namespaceAlias).'\\'.$simpleClassName;
     }
 
     /**
@@ -1020,9 +1020,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.'
             );
         }
 
@@ -1036,8 +1036,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);
@@ -1067,7 +1067,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'];
             }
         }
 
@@ -1449,7 +1449,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.');
         }
 
@@ -1797,7 +1797,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;
@@ -1811,7 +1811,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;
@@ -2432,7 +2432,7 @@ 
 
         // Phase 1 AST optimization: Prevent AST\ConditionalFactor
         // if only one AST\ConditionalPrimary is defined
-        if (! $not) {
+        if ( ! $not) {
             return $conditionalPrimary;
         }
 
@@ -2451,7 +2451,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;
@@ -2619,7 +2619,7 @@ 
             $this->match(Lexer::T_OF);
         }
 
-        $collMemberExpr      = new AST\CollectionMemberExpression(
+        $collMemberExpr = new AST\CollectionMemberExpression(
             $entityExpr,
             $this->CollectionValuedPathExpression()
         );
@@ -2958,7 +2958,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');
         }
 
@@ -2988,7 +2988,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');
         }
 
@@ -3226,7 +3226,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.');
                 }
 
@@ -3237,7 +3237,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.');
                 }