doctrine / persistence

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

Spacing +8 added lines, -8 removed lines

@@ -45,7 +45,7 @@ 
     public function testGetMetadataForAbsentClass()
     {
         $this->expectException(MappingException::class);
-        $this->cmf->getMetadataFor(__NAMESPACE__ . '\AbsentClass');
+        $this->cmf->getMetadataFor(__NAMESPACE__.'\AbsentClass');
     }
 
     public function testGetParentMetadata()
@@ -61,7 +61,7 @@ 
     {
         $metadata = $this->createMock(ClassMetadata::class);
         $cache    = new ArrayCache();
-        $cache->save(ChildEntity::class . '$CLASSMETADATA', $metadata);
+        $cache->save(ChildEntity::class.'$CLASSMETADATA', $metadata);
 
         $this->cmf->setCacheDriver($cache);
 
@@ -75,14 +75,14 @@ 
 
         $loadedMetadata = $this->cmf->getMetadataFor(ChildEntity::class);
 
-        self::assertSame($loadedMetadata, $cache->fetch(ChildEntity::class . '$CLASSMETADATA'));
+        self::assertSame($loadedMetadata, $cache->fetch(ChildEntity::class.'$CLASSMETADATA'));
     }
 
     public function testGetAliasedMetadata()
     {
         $this->cmf->getMetadataFor('prefix:ChildEntity');
 
-        self::assertTrue($this->cmf->hasMetadataFor(__NAMESPACE__ . '\ChildEntity'));
+        self::assertTrue($this->cmf->hasMetadataFor(__NAMESPACE__.'\ChildEntity'));
         self::assertTrue($this->cmf->hasMetadataFor('prefix:ChildEntity'));
     }
 
@@ -111,7 +111,7 @@ 
     {
         $classMetadata = $this->createMock(ClassMetadata::class);
 
-        $this->cmf->fallbackCallback = function () use ($classMetadata) {
+        $this->cmf->fallbackCallback = function() use ($classMetadata) {
             return $classMetadata;
         };
 
@@ -122,7 +122,7 @@ 
 
     public function testWillFailOnFallbackFailureWithNotLoadedMetadata()
     {
-        $this->cmf->fallbackCallback = function () {
+        $this->cmf->fallbackCallback = function() {
             return null;
         };
 
@@ -185,7 +185,7 @@ 
 
     protected function getFqcnFromAlias($namespaceAlias, $simpleClassName)
     {
-        return __NAMESPACE__ . '\\' . $simpleClassName;
+        return __NAMESPACE__.'\\'.$simpleClassName;
     }
 
     protected function initialize()
@@ -216,7 +216,7 @@ 
 
     protected function onNotFoundMetadata($className)
     {
-        if (! $this->fallbackCallback) {
+        if (!$this->fallbackCallback) {
             return null;
         }
 

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

Spacing +4 added lines, -4 removed lines

@@ -36,7 +36,7 @@ 
         $locator->expects($this->once())
                 ->method('findMappingFile')
                 ->with($this->equalTo('stdClass'))
-                ->will($this->returnValue(__DIR__ . '/_files/stdClass.yml'));
+                ->will($this->returnValue(__DIR__.'/_files/stdClass.yml'));
 
         $driver = new TestFileDriver($locator);
 
@@ -51,7 +51,7 @@ 
         $locator->expects($this->once())
             ->method('findMappingFile')
             ->with($this->equalTo('stdClass'))
-            ->will($this->returnValue(__DIR__ . '/_files/stdClass.yml'));
+            ->will($this->returnValue(__DIR__.'/_files/stdClass.yml'));
 
         $driver = new TestFileDriver($locator);
 
@@ -132,7 +132,7 @@ 
 
     public function testNonLocatorFallback()
     {
-        $driver = new TestFileDriver(__DIR__ . '/_files', '.yml');
+        $driver = new TestFileDriver(__DIR__.'/_files', '.yml');
         self::assertTrue($driver->isTransient('stdClass2'));
         self::assertFalse($driver->isTransient('stdClass'));
     }
@@ -141,7 +141,7 @@ 
     {
         $locator = $this->createMock(FileLocator::class);
         $locator->expects($this->any())->method('getFileExtension')->will($this->returnValue('.yml'));
-        $locator->expects($this->any())->method('getPaths')->will($this->returnValue([__DIR__ . '/_files']));
+        $locator->expects($this->any())->method('getPaths')->will($this->returnValue([__DIR__.'/_files']));
         return $locator;
     }
 }

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

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     public function testGetParentClasses()
     {
         $classes = $this->reflectionService->getParentClasses(__CLASS__);
-        self::assertTrue(count($classes) === 0, 'The test class ' . __CLASS__ . ' should have no parents according to static reflection.');
+        self::assertTrue(count($classes) === 0, 'The test class '.__CLASS__.' should have no parents according to static reflection.');
     }
 
     public function testGetReflectionClass()

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

Spacing +2 added lines, -2 removed lines

@@ -36,13 +36,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/PersistentObject.php

Spacing +15 added lines, -15 removed lines

@@ -75,7 +75,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?');
         }
 
@@ -95,17 +95,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()."'");
         }
     }
 
@@ -124,7 +124,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()."'");
     }
 
     /**
@@ -140,14 +140,14 @@ 
     {
         // add this object on the owning side as well, for obvious infinite recursion
         // reasons this is only done when called on the inverse side.
-        if (! $this->cm->isAssociationInverseSide($field)) {
+        if (!$this->cm->isAssociationInverseSide($field)) {
             return;
         }
 
         $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);
     }
 
@@ -164,15 +164,15 @@ 
      */
     private function add($field, $args)
     {
-        if (! $this->cm->hasAssociation($field) || ! $this->cm->isCollectionValuedAssociation($field)) {
-            throw new \BadMethodCallException('There is no method add' . $field . '() on ' . $this->cm->getName());
+        if (!$this->cm->hasAssociation($field) || !$this->cm->isCollectionValuedAssociation($field)) {
+            throw new \BadMethodCallException('There is no method add'.$field.'() on '.$this->cm->getName());
         }
 
         $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]);
@@ -192,7 +192,7 @@ 
             return;
         }
 
-        if (! self::$objectManager) {
+        if (!self::$objectManager) {
             throw new \RuntimeException('No runtime object manager set. Call PersistentObject#setObjectManager().');
         }
 
@@ -222,7 +222,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

@@ -89,7 +89,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));
         }
 
@@ -144,7 +144,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));
         }
 
@@ -159,7 +159,7 @@ 
         // 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);
@@ -167,7 +167,7 @@ 
         if ($proxyClass->implementsInterface($this->proxyInterfaceName)) {
             $parentClass = $proxyClass->getParentClass();
 
-            if (! $parentClass) {
+            if (!$parentClass) {
                 return null;
             }
 
@@ -177,7 +177,7 @@ 
         foreach ($this->managers as $id) {
             $manager = $this->getService($id);
 
-            if (! $manager->getMetadataFactory()->isTransient($class)) {
+            if (!$manager->getMetadataFactory()->isTransient($class)) {
                 return $manager;
             }
         }
@@ -221,7 +221,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 +4 added lines, -4 removed lines

@@ -29,7 +29,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.');
     }
 
@@ -40,12 +40,12 @@ 
      */
     public static function fileMappingDriversRequireConfiguredDirectoryPath($path = null)
     {
-        if (! empty($path)) {
-            $path = '[' . $path . ']';
+        if (!empty($path)) {
+            $path = '['.$path.']';
         }
 
         return new self(sprintf(
-            'File mapping drivers must have a valid directory path, ' .
+            'File mapping drivers must have a valid directory path, '.
             'however the given path %s seems to be incorrect!',
             $path
         ));

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

Spacing +6 added lines, -6 removed lines

@@ -78,7 +78,7 @@ 
      */
     public function getAllMetadata()
     {
-        if (! $this->initialized) {
+        if (!$this->initialized) {
             $this->initialize();
         }
 
@@ -176,7 +176,7 @@ 
 
         try {
             if ($this->cacheDriver) {
-                $cached = $this->cacheDriver->fetch($realClassName . $this->cacheSalt);
+                $cached = $this->cacheDriver->fetch($realClassName.$this->cacheSalt);
                 if ($cached instanceof ClassMetadata) {
                     $this->loadedMetadata[$realClassName] = $cached;
 
@@ -184,7 +184,7 @@ 
                 } else {
                     foreach ($this->loadMetadata($realClassName) as $loadedClassName) {
                         $this->cacheDriver->save(
-                            $loadedClassName . $this->cacheSalt,
+                            $loadedClassName.$this->cacheSalt,
                             $this->loadedMetadata[$loadedClassName],
                             null
                         );
@@ -196,7 +196,7 @@ 
         } catch (MappingException $loadingException) {
             $fallbackMetadataResponse = $this->onNotFoundMetadata($realClassName);
 
-            if (! $fallbackMetadataResponse) {
+            if (!$fallbackMetadataResponse) {
                 throw $loadingException;
             }
 
@@ -277,7 +277,7 @@ 
      */
     protected function loadMetadata($name)
     {
-        if (! $this->initialized) {
+        if (!$this->initialized) {
             $this->initialize();
         }
 
@@ -364,7 +364,7 @@ 
      */
     public function isTransient($class)
     {
-        if (! $this->initialized) {
+        if (!$this->initialized) {
             $this->initialize();
         }