doctrine / persistence

tests/Doctrine/Tests/Common/Persistence/ObjectManagerDecoratorTest.php

Spacing +1 added lines, -1 removed lines

@@ -61,7 +61,7 @@
      */
     public function testAllMethodCallsAreDelegatedToTheWrappedInstance($method, array $parameters, $isVoidMethod)
     {
-        $returnedValue = $isVoidMethod ? null : 'INNER VALUE FROM ' . $method;
+        $returnedValue = $isVoidMethod ? null : 'INNER VALUE FROM '.$method;
         $stub          = $this->wrapped
             ->expects($this->once())
             ->method($method)

tests/Doctrine/Tests/Common/Persistence/PersistentObjectTest.php

Spacing +2 added lines, -2 removed lines

@@ -177,7 +177,7 @@ 
 
     public function getAssociationTargetClass($assocName)
     {
-        return __NAMESPACE__ . '\TestObject';
+        return __NAMESPACE__.'\TestObject';
     }
 
     public function getFieldNames()
@@ -192,7 +192,7 @@ 
 
     public function getName()
     {
-        return __NAMESPACE__ . '\TestObject';
+        return __NAMESPACE__.'\TestObject';
     }
 
     public function getReflectionClass()

Indentation +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         $this->cm = new TestObjectMetadata();
         $this->om = $this->createMock(ObjectManager::class);
         $this->om->expects($this->any())->method('getClassMetadata')
-                 ->will($this->returnValue($this->cm));
+                    ->will($this->returnValue($this->cm));
         $this->object = new TestObject();
         PersistentObject::setObjectManager($this->om);
         $this->object->injectObjectManager($this->om, $this->cm);

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

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     public function testGetAllClassNames()
     {
         $reader = new AnnotationReader();
-        $driver = new SimpleAnnotationDriver($reader, [__DIR__ . '/_files/annotation']);
+        $driver = new SimpleAnnotationDriver($reader, [__DIR__.'/_files/annotation']);
 
         $classes = $driver->getAllClassNames();
 

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

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
         $metadata = $this->createMock(ClassMetadata::class);
         $metadata->expects($this->once())->method('getFieldNames');
 
-        $driver = new PHPDriver([__DIR__ . '/_files']);
+        $driver = new PHPDriver([__DIR__.'/_files']);
         $driver->loadMetadataForClass('TestEntity', $metadata);
     }
 }

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

Spacing +8 added lines, -8 removed lines

@@ -12,7 +12,7 @@ 
 {
     public function testGetPaths()
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path]);
         self::assertSame([$path], $locator->getPaths());
@@ -31,7 +31,7 @@ 
 
     public function testUniquePaths()
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path, $path]);
         self::assertSame([$path], $locator->getPaths());
@@ -39,16 +39,16 @@ 
 
     public function testFindMappingFile()
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path], '.yml');
 
-        self::assertSame(__DIR__ . '/_files' . DIRECTORY_SEPARATOR . 'stdClass.yml', $locator->findMappingFile('stdClass'));
+        self::assertSame(__DIR__.'/_files'.DIRECTORY_SEPARATOR.'stdClass.yml', $locator->findMappingFile('stdClass'));
     }
 
     public function testFindMappingFileNotFound()
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path], '.yml');
 
@@ -59,7 +59,7 @@ 
 
     public function testGetAllClassNames()
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator       = new DefaultFileLocator([$path], '.yml');
         $allClasses    = $locator->getAllClassNames(null);
@@ -79,7 +79,7 @@ 
 
     public function testGetAllClassNamesNonMatchingFileExtension()
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path], '.xml');
         self::assertSame([], $locator->getAllClassNames('global'));
@@ -87,7 +87,7 @@ 
 
     public function testFileExists()
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path], '.yml');
 

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

Spacing +17 added lines, -17 removed lines

@@ -13,7 +13,7 @@ 
 {
     public function testGetPaths()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix]);
@@ -25,7 +25,7 @@ 
 
     public function testGetPrefixes()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix]);
@@ -42,7 +42,7 @@ 
 
     public function testFileExists()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix], '.yml');
@@ -55,7 +55,7 @@ 
 
     public function testGetAllClassNames()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator       = new SymfonyFileLocator([$path => $prefix], '.yml');
@@ -80,7 +80,7 @@ 
      */
     public function testInvalidCustomNamespaceSeparator()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         new SymfonyFileLocator([$path => $prefix], '.yml', null);
@@ -104,7 +104,7 @@ 
      */
     public function testGetClassNamesWithCustomNsSeparator($separator, $dir)
     {
-        $path   = __DIR__ . $dir;
+        $path   = __DIR__.$dir;
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix], '.yml', $separator);
@@ -147,30 +147,30 @@ 
      */
     public function testFindMappingFileWithCustomNsSeparator($separator, $dir, $files)
     {
-        $path   = __DIR__ . $dir;
+        $path   = __DIR__.$dir;
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix], '.yml', $separator);
 
         foreach ($files as $filePath => $className) {
-            self::assertSame(realpath($path . '/' . $filePath), realpath($locator->findMappingFile($className)));
+            self::assertSame(realpath($path.'/'.$filePath), realpath($locator->findMappingFile($className)));
         }
     }
 
 
     public function testFindMappingFile()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix], '.yml');
 
-        self::assertSame(__DIR__ . '/_files/stdClass.yml', $locator->findMappingFile('Foo\\stdClass'));
+        self::assertSame(__DIR__.'/_files/stdClass.yml', $locator->findMappingFile('Foo\\stdClass'));
     }
 
     public function testFindMappingFileNotFound()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix], '.yml');
@@ -184,13 +184,13 @@ 
     {
         // Low -> High
         $prefixes                             = [];
-        $prefixes[__DIR__ . '/_match_ns']     = 'Foo';
-        $prefixes[__DIR__ . '/_match_ns/Bar'] = 'Foo\\Bar';
+        $prefixes[__DIR__.'/_match_ns']     = 'Foo';
+        $prefixes[__DIR__.'/_match_ns/Bar'] = 'Foo\\Bar';
 
         $locator = new SymfonyFileLocator($prefixes, '.yml');
 
         self::assertSame(
-            __DIR__ . '/_match_ns/Bar/barEntity.yml',
+            __DIR__.'/_match_ns/Bar/barEntity.yml',
             $locator->findMappingFile("Foo\\Bar\\barEntity")
         );
     }
@@ -198,13 +198,13 @@ 
     public function testFindMappingFileMostSpecificNamespaceFirst()
     {
         $prefixes                             = [];
-        $prefixes[__DIR__ . '/_match_ns/Bar'] = 'Foo\\Bar';
-        $prefixes[__DIR__ . '/_match_ns']     = 'Foo';
+        $prefixes[__DIR__.'/_match_ns/Bar'] = 'Foo\\Bar';
+        $prefixes[__DIR__.'/_match_ns']     = 'Foo';
 
         $locator = new SymfonyFileLocator($prefixes, '.yml');
 
         self::assertSame(
-            __DIR__ . '/_match_ns/Bar/barEntity.yml',
+            __DIR__.'/_match_ns/Bar/barEntity.yml',
             $locator->findMappingFile("Foo\\Bar\\barEntity")
         );
     }

lib/Doctrine/Common/Persistence/PersistentObject.php

Spacing +15 added lines, -15 removed lines

@@ -76,7 +76,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?');
         }
 
@@ -96,17 +96,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()."'");
         }
     }
 
@@ -125,7 +125,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()."'");
     }
 
     /**
@@ -141,14 +141,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);
     }
 
@@ -165,15 +165,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]);
@@ -193,7 +193,7 @@ 
             return;
         }
 
-        if (! self::$objectManager) {
+        if (!self::$objectManager) {
             throw new RuntimeException('No runtime object manager set. Call PersistentObject#setObjectManager().');
         }
 
@@ -223,7 +223,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());
         }
     }
 }

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

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
     public function testGetParentClasses()
     {
         $classes = $this->reflectionService->getParentClasses(self::class);
-        self::assertTrue(count($classes) === 0, 'The test class ' . self::class . ' should have no parents according to static reflection.');
+        self::assertTrue(count($classes) === 0, 'The test class '.self::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

@@ -38,13 +38,13 @@
     public function testGetParentClasses()
     {
         $classes = $this->reflectionService->getParentClasses(self::class);
-        self::assertTrue(count($classes) >= 1, 'The test class ' . self::class . ' should have at least one parent.');
+        self::assertTrue(count($classes) >= 1, 'The test class '.self::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()