doctrine / persistence

tests/Doctrine/Tests/Common/Persistence/Mapping/RuntimeReflectionServiceTest.php

Spacing +2 added lines, -2 removed lines

@@ -35,13 +35,13 @@
     public function testGetParentClasses()
     {
         $classes = $this->reflectionService->getParentClasses(__CLASS__);
-        self::assertTrue(count($classes) >= 1, "The test class " . __CLASS__ . " should have at least one parent.");
+        self::assertTrue(count($classes) >= 1, "The test class ".__CLASS__." should have at least one parent.");
     }
 
     public function testGetParentClassesForAbsentClass()
     {
         $this->expectException(MappingException::class);
-        $this->reflectionService->getParentClasses(__NAMESPACE__ . '\AbsentClass');
+        $this->reflectionService->getParentClasses(__NAMESPACE__.'\AbsentClass');
     }
 
     public function testGetReflectionClass()

lib/Doctrine/Common/Persistence/Event/PreUpdateEventArgs.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
      */
     private function assertValidField($field)
     {
-        if ( ! isset($this->entityChangeSet[$field])) {
+        if (!isset($this->entityChangeSet[$field])) {
             throw new \InvalidArgumentException(sprintf(
                 'Field "%s" is not a valid field of the entity "%s" in PreUpdateEventArgs.',
                 $field,

lib/Doctrine/Common/Persistence/PersistentObject.php

Spacing +13 added lines, -13 removed lines

@@ -81,7 +81,7 @@ 
     public function injectObjectManager(ObjectManager $objectManager, ClassMetadata $classMetadata)
     {
         if ($objectManager !== self::$objectManager) {
-            throw new \RuntimeException("Trying to use PersistentObject with different ObjectManager instances. " .
+            throw new \RuntimeException("Trying to use PersistentObject with different ObjectManager instances. ".
                 "Was PersistentObject::setObjectManager() called?");
         }
 
@@ -101,17 +101,17 @@ 
      */
     private function set($field, $args)
     {
-        if ($this->cm->hasField($field) && ! $this->cm->isIdentifier($field)) {
+        if ($this->cm->hasField($field) && !$this->cm->isIdentifier($field)) {
             $this->$field = $args[0];
         } elseif ($this->cm->hasAssociation($field) && $this->cm->isSingleValuedAssociation($field)) {
             $targetClass = $this->cm->getAssociationTargetClass($field);
-            if ( ! ($args[0] instanceof $targetClass) && $args[0] !== null) {
-                throw new \InvalidArgumentException("Expected persistent object of type '" . $targetClass . "'");
+            if (!($args[0] instanceof $targetClass) && $args[0] !== null) {
+                throw new \InvalidArgumentException("Expected persistent object of type '".$targetClass."'");
             }
             $this->$field = $args[0];
             $this->completeOwningSide($field, $targetClass, $args[0]);
         } else {
-            throw new \BadMethodCallException("no field with name '" . $field . "' exists on '" . $this->cm->getName() . "'");
+            throw new \BadMethodCallException("no field with name '".$field."' exists on '".$this->cm->getName()."'");
         }
     }
 
@@ -130,7 +130,7 @@ 
             return $this->$field;
         }
 
-        throw new \BadMethodCallException("no field with name '" . $field . "' exists on '" . $this->cm->getName() . "'");
+        throw new \BadMethodCallException("no field with name '".$field."' exists on '".$this->cm->getName()."'");
     }
 
     /**
@@ -150,7 +150,7 @@ 
             $mappedByField  = $this->cm->getAssociationMappedByTargetField($field);
             $targetMetadata = self::$objectManager->getClassMetadata($targetClass);
 
-            $setter = ($targetMetadata->isCollectionValuedAssociation($mappedByField) ? "add" : "set") . $mappedByField;
+            $setter = ($targetMetadata->isCollectionValuedAssociation($mappedByField) ? "add" : "set").$mappedByField;
             $targetObject->$setter($this);
         }
     }
@@ -170,16 +170,16 @@ 
     {
         if ($this->cm->hasAssociation($field) && $this->cm->isCollectionValuedAssociation($field)) {
             $targetClass = $this->cm->getAssociationTargetClass($field);
-            if ( ! ($args[0] instanceof $targetClass)) {
-                throw new \InvalidArgumentException("Expected persistent object of type '" . $targetClass . "'");
+            if (!($args[0] instanceof $targetClass)) {
+                throw new \InvalidArgumentException("Expected persistent object of type '".$targetClass."'");
             }
-            if ( ! ($this->$field instanceof Collection)) {
+            if (!($this->$field instanceof Collection)) {
                 $this->$field = new ArrayCollection($this->$field ?: []);
             }
             $this->$field->add($args[0]);
             $this->completeOwningSide($field, $targetClass, $args[0]);
         } else {
-            throw new \BadMethodCallException("There is no method add" . $field . "() on " . $this->cm->getName());
+            throw new \BadMethodCallException("There is no method add".$field."() on ".$this->cm->getName());
         }
     }
 
@@ -196,7 +196,7 @@ 
             return;
         }
 
-        if ( ! self::$objectManager) {
+        if (!self::$objectManager) {
             throw new \RuntimeException("No runtime object manager set. Call PersistentObject#setObjectManager().");
         }
 
@@ -226,7 +226,7 @@ 
         } elseif ($command == "add") {
             $this->add($field, $args);
         } else {
-            throw new \BadMethodCallException("There is no method " . $method . " on " . $this->cm->getName());
+            throw new \BadMethodCallException("There is no method ".$method." on ".$this->cm->getName());
         }
     }
 }

lib/Doctrine/Common/Persistence/AbstractManagerRegistry.php

Spacing +6 added lines, -6 removed lines

@@ -103,7 +103,7 @@ 
             $name = $this->defaultConnection;
         }
 
-        if ( ! isset($this->connections[$name])) {
+        if (!isset($this->connections[$name])) {
             throw new \InvalidArgumentException(sprintf('Doctrine %s Connection named "%s" does not exist.', $this->name, $name));
         }
 
@@ -158,7 +158,7 @@ 
             $name = $this->defaultManager;
         }
 
-        if ( ! isset($this->managers[$name])) {
+        if (!isset($this->managers[$name])) {
             throw new \InvalidArgumentException(sprintf('Doctrine %s Manager named "%s" does not exist.', $this->name, $name));
         }
 
@@ -173,13 +173,13 @@ 
         // Check for namespace alias
         if (strpos($class, ':') !== false) {
             list($namespaceAlias, $simpleClassName) = explode(':', $class, 2);
-            $class                                  = $this->getAliasNamespace($namespaceAlias) . '\\' . $simpleClassName;
+            $class                                  = $this->getAliasNamespace($namespaceAlias).'\\'.$simpleClassName;
         }
 
         $proxyClass = new \ReflectionClass($class);
 
         if ($proxyClass->implementsInterface($this->proxyInterfaceName)) {
-            if ( ! $parentClass = $proxyClass->getParentClass()) {
+            if (!$parentClass = $proxyClass->getParentClass()) {
                 return null;
             }
 
@@ -189,7 +189,7 @@ 
         foreach ($this->managers as $id) {
             $manager = $this->getService($id);
 
-            if ( ! $manager->getMetadataFactory()->isTransient($class)) {
+            if (!$manager->getMetadataFactory()->isTransient($class)) {
                 return $manager;
             }
         }
@@ -233,7 +233,7 @@ 
             $name = $this->defaultManager;
         }
 
-        if ( ! isset($this->managers[$name])) {
+        if (!isset($this->managers[$name])) {
             throw new \InvalidArgumentException(sprintf('Doctrine %s Manager named "%s" does not exist.', $this->name, $name));
         }
 

lib/Doctrine/Common/Persistence/Mapping/RuntimeReflectionService.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function getParentClasses($class)
     {
-        if ( ! class_exists($class)) {
+        if (!class_exists($class)) {
             throw MappingException::nonExistingClass($class);
         }
 

lib/Doctrine/Common/Persistence/Mapping/MappingException.php

Spacing +7 added lines, -7 removed lines

@@ -16,8 +16,8 @@ 
      */
     public static function classNotFoundInNamespaces($className, $namespaces)
     {
-        return new self("The class '" . $className . "' was not found in the " .
-            "chain configured namespaces " . implode(", ", $namespaces));
+        return new self("The class '".$className."' was not found in the ".
+            "chain configured namespaces ".implode(", ", $namespaces));
     }
 
     /**
@@ -25,7 +25,7 @@ 
      */
     public static function pathRequired()
     {
-        return new self("Specifying the paths to your entities is required " .
+        return new self("Specifying the paths to your entities is required ".
             "in the AnnotationDriver to retrieve all class names.");
     }
 
@@ -36,13 +36,13 @@ 
      */
     public static function fileMappingDriversRequireConfiguredDirectoryPath($path = null)
     {
-        if ( ! empty($path)) {
-            $path = '[' . $path . ']';
+        if (!empty($path)) {
+            $path = '['.$path.']';
         }
 
         return new self(
-            'File mapping drivers must have a valid directory path, ' .
-            'however the given path ' . $path . ' seems to be incorrect!'
+            'File mapping drivers must have a valid directory path, '.
+            'however the given path '.$path.' seems to be incorrect!'
         );
     }
 

lib/Doctrine/Common/Persistence/Mapping/Driver/DefaultFileLocator.php

Spacing +6 added lines, -6 removed lines

@@ -90,12 +90,12 @@ 
      */
     public function findMappingFile($className)
     {
-        $fileName = str_replace('\\', '.', $className) . $this->fileExtension;
+        $fileName = str_replace('\\', '.', $className).$this->fileExtension;
 
         // Check whether file exists
         foreach ($this->paths as $path) {
-            if (is_file($path . DIRECTORY_SEPARATOR . $fileName)) {
-                return $path . DIRECTORY_SEPARATOR . $fileName;
+            if (is_file($path.DIRECTORY_SEPARATOR.$fileName)) {
+                return $path.DIRECTORY_SEPARATOR.$fileName;
             }
         }
 
@@ -111,7 +111,7 @@ 
 
         if ($this->paths) {
             foreach ($this->paths as $path) {
-                if ( ! is_dir($path)) {
+                if (!is_dir($path)) {
                     throw MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
                 }
 
@@ -141,11 +141,11 @@ 
      */
     public function fileExists($className)
     {
-        $fileName = str_replace('\\', '.', $className) . $this->fileExtension;
+        $fileName = str_replace('\\', '.', $className).$this->fileExtension;
 
         // Check whether file exists
         foreach ((array) $this->paths as $path) {
-            if (is_file($path . DIRECTORY_SEPARATOR . $fileName)) {
+            if (is_file($path.DIRECTORY_SEPARATOR.$fileName)) {
                 return true;
             }
         }

lib/Doctrine/Common/Persistence/Mapping/Driver/AnnotationDriver.php

Spacing +5 added lines, -5 removed lines

@@ -178,7 +178,7 @@ 
             return $this->classNames;
         }
 
-        if ( ! $this->paths) {
+        if (!$this->paths) {
             throw MappingException::pathRequired();
         }
 
@@ -186,7 +186,7 @@ 
         $includedFiles = [];
 
         foreach ($this->paths as $path) {
-            if ( ! is_dir($path)) {
+            if (!is_dir($path)) {
                 throw MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
             }
 
@@ -195,14 +195,14 @@ 
                     new \RecursiveDirectoryIterator($path, \FilesystemIterator::SKIP_DOTS),
                     \RecursiveIteratorIterator::LEAVES_ONLY
                 ),
-                '/^.+' . preg_quote($this->fileExtension) . '$/i',
+                '/^.+'.preg_quote($this->fileExtension).'$/i',
                 \RecursiveRegexIterator::GET_MATCH
             );
 
             foreach ($iterator as $file) {
                 $sourceFile = $file[0];
 
-                if ( ! preg_match('(^phar:)i', $sourceFile)) {
+                if (!preg_match('(^phar:)i', $sourceFile)) {
                     $sourceFile = realpath($sourceFile);
                 }
 
@@ -226,7 +226,7 @@ 
         foreach ($declared as $className) {
             $rc         = new \ReflectionClass($className);
             $sourceFile = $rc->getFileName();
-            if (in_array($sourceFile, $includedFiles) && ! $this->isTransient($className)) {
+            if (in_array($sourceFile, $includedFiles) && !$this->isTransient($className)) {
                 $classes[] = $className;
             }
         }

lib/Doctrine/Common/Persistence/Mapping/Driver/MappingDriverChain.php

Spacing +1 added lines, -1 removed lines

@@ -106,7 +106,7 @@
         foreach ($this->drivers as $namespace => $driver) {
             $oid = spl_object_hash($driver);
 
-            if ( ! isset($driverClasses[$oid])) {
+            if (!isset($driverClasses[$oid])) {
                 $driverClasses[$oid] = $driver->getAllClassNames();
             }