doctrine / persistence

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

Spacing +4 added lines, -4 removed lines

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

lib/Doctrine/Persistence/Mapping/MappingException.php

Spacing +3 added lines, -3 removed lines

@@ -29,7 +29,7 @@ 
 
     public static function pathRequired() : self
     {
-        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.');
     }
 
@@ -37,11 +37,11 @@ 
         ?string $path = null
     ) : self {
         if ($path !== null) {
-            $path = '[' . $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/Persistence/Mapping/Driver/StaticPHPDriver.php

Spacing +3 added lines, -3 removed lines

@@ -82,7 +82,7 @@ 
         $includedFiles = [];
 
         foreach ($this->paths as $path) {
-            if (! is_dir($path)) {
+            if (!is_dir($path)) {
                 throw MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
             }
 
@@ -109,7 +109,7 @@ 
 
             $sourceFile = $rc->getFileName();
 
-            if (! in_array($sourceFile, $includedFiles, true) || $this->isTransient($className)) {
+            if (!in_array($sourceFile, $includedFiles, true) || $this->isTransient($className)) {
                 continue;
             }
 
@@ -126,6 +126,6 @@ 
      */
     public function isTransient(string $className) : bool
     {
-        return ! method_exists($className, 'loadMetadata');
+        return !method_exists($className, 'loadMetadata');
     }
 }

lib/Doctrine/Persistence/Mapping/Driver/SymfonyFileLocator.php

Spacing +14 added lines, -14 removed lines

@@ -129,12 +129,12 @@ 
      */
     public function fileExists(string $className) : bool
     {
-        $defaultFileName = str_replace('\\', $this->nsSeparator, $className) . $this->fileExtension;
+        $defaultFileName = str_replace('\\', $this->nsSeparator, $className).$this->fileExtension;
 
         foreach ($this->paths as $path) {
-            if (! isset($this->prefixes[$path])) {
+            if (!isset($this->prefixes[$path])) {
                 // global namespace class
-                if (is_file($path . DIRECTORY_SEPARATOR . $defaultFileName)) {
+                if (is_file($path.DIRECTORY_SEPARATOR.$defaultFileName)) {
                     return true;
                 }
 
@@ -143,11 +143,11 @@ 
 
             $prefix = $this->prefixes[$path];
 
-            if (strpos($className, $prefix . '\\') !== 0) {
+            if (strpos($className, $prefix.'\\') !== 0) {
                 continue;
             }
 
-            $filename = $path . '/' . strtr(substr($className, strlen($prefix) + 1), '\\', $this->nsSeparator) . $this->fileExtension;
+            $filename = $path.'/'.strtr(substr($className, strlen($prefix) + 1), '\\', $this->nsSeparator).$this->fileExtension;
 
             if (is_file($filename)) {
                 return true;
@@ -169,7 +169,7 @@ 
         $classes = [];
 
         foreach ($this->paths as $path) {
-            if (! is_dir($path)) {
+            if (!is_dir($path)) {
                 throw MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
             }
 
@@ -194,7 +194,7 @@ 
                         '\\'
                     );
 
-                    $classes[] = $this->prefixes[$path] . str_replace(DIRECTORY_SEPARATOR, '\\', $nsSuffix) . '\\' . str_replace($this->nsSeparator, '\\', $fileName);
+                    $classes[] = $this->prefixes[$path].str_replace(DIRECTORY_SEPARATOR, '\\', $nsSuffix).'\\'.str_replace($this->nsSeparator, '\\', $fileName);
                 } else {
                     /** @var string $className */
                     $className = str_replace($this->nsSeparator, '\\', $fileName);
@@ -212,11 +212,11 @@ 
      */
     public function findMappingFile(string $className) : string
     {
-        $defaultFileName = str_replace('\\', $this->nsSeparator, $className) . $this->fileExtension;
+        $defaultFileName = str_replace('\\', $this->nsSeparator, $className).$this->fileExtension;
         foreach ($this->paths as $path) {
-            if (! isset($this->prefixes[$path])) {
-                if (is_file($path . DIRECTORY_SEPARATOR . $defaultFileName)) {
-                    return $path . DIRECTORY_SEPARATOR . $defaultFileName;
+            if (!isset($this->prefixes[$path])) {
+                if (is_file($path.DIRECTORY_SEPARATOR.$defaultFileName)) {
+                    return $path.DIRECTORY_SEPARATOR.$defaultFileName;
                 }
 
                 continue;
@@ -224,11 +224,11 @@ 
 
             $prefix = $this->prefixes[$path];
 
-            if (strpos($className, $prefix . '\\') !== 0) {
+            if (strpos($className, $prefix.'\\') !== 0) {
                 continue;
             }
 
-            $filename = $path . '/' . strtr(substr($className, strlen($prefix) + 1), '\\', $this->nsSeparator) . $this->fileExtension;
+            $filename = $path.'/'.strtr(substr($className, strlen($prefix) + 1), '\\', $this->nsSeparator).$this->fileExtension;
             if (is_file($filename)) {
                 return $filename;
             }
@@ -239,7 +239,7 @@ 
 
         throw MappingException::mappingFileNotFound(
             $className,
-            substr($className, $pos + 1) . $this->fileExtension
+            substr($className, $pos + 1).$this->fileExtension
         );
     }
 

lib/Doctrine/Persistence/Mapping/Driver/FileDriver.php

Spacing +5 added lines, -5 removed lines

@@ -82,10 +82,10 @@ 
 
         $result = $this->loadMappingFile($this->locator->findMappingFile($className));
 
-        if (! isset($result[$className])) {
+        if (!isset($result[$className])) {
             throw MappingException::invalidMappingFile(
                 $className,
-                str_replace('\\', '.', $className) . $this->locator->getFileExtension()
+                str_replace('\\', '.', $className).$this->locator->getFileExtension()
             );
         }
 
@@ -107,7 +107,7 @@ 
             return false;
         }
 
-        return ! $this->locator->fileExists($className);
+        return !$this->locator->fileExists($className);
     }
 
     /**
@@ -165,8 +165,8 @@ 
         }
 
         foreach ($this->locator->getPaths() as $path) {
-            $file = $path . '/' . $this->globalBasename . $this->locator->getFileExtension();
-            if (! is_file($file)) {
+            $file = $path.'/'.$this->globalBasename.$this->locator->getFileExtension();
+            if (!is_file($file)) {
                 continue;
             }
 

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

Spacing +4 added lines, -4 removed lines

@@ -89,7 +89,7 @@ 
             return;
         }
 
-        if (! is_array($paths)) {
+        if (!is_array($paths)) {
             $paths = [$paths];
         }
 
@@ -199,7 +199,7 @@ 
         $includedFiles = [];
 
         foreach ($this->paths as $path) {
-            if (! is_dir($path)) {
+            if (!is_dir($path)) {
                 throw MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
             }
 
@@ -208,7 +208,7 @@ 
                     new RecursiveDirectoryIterator($path, FilesystemIterator::SKIP_DOTS),
                     RecursiveIteratorIterator::LEAVES_ONLY
                 ),
-                '/^.+' . preg_quote($this->fileExtension) . '$/i',
+                '/^.+'.preg_quote($this->fileExtension).'$/i',
                 RecursiveRegexIterator::GET_MATCH
             );
 
@@ -244,7 +244,7 @@ 
 
             $sourceFile = $rc->getFileName();
 
-            if (! in_array($sourceFile, $includedFiles, true) || $this->isTransient($className)) {
+            if (!in_array($sourceFile, $includedFiles, true) || $this->isTransient($className)) {
                 continue;
             }
 

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

Spacing +6 added lines, -6 removed lines

@@ -94,12 +94,12 @@ 
      */
     public function findMappingFile(string $className) : string
     {
-        $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;
             }
         }
 
@@ -118,7 +118,7 @@ 
         $classes = [];
 
         foreach ($this->paths as $path) {
-            if (! is_dir($path)) {
+            if (!is_dir($path)) {
                 throw MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
             }
 
@@ -151,11 +151,11 @@ 
      */
     public function fileExists(string $className) : bool
     {
-        $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)) {
+            if (is_file($path.DIRECTORY_SEPARATOR.$fileName)) {
                 return true;
             }
         }

lib/Doctrine/Persistence/AbstractManagerRegistry.php

Spacing +5 added lines, -5 removed lines

@@ -91,7 +91,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)
             );
@@ -148,7 +148,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)
             );
@@ -169,7 +169,7 @@ 
         if (strpos($class, ':') !== false) {
             [$namespaceAlias, $simpleClassName] = explode(':', $class, 2);
 
-            $class = $this->getAliasNamespace($namespaceAlias) . '\\' . $simpleClassName;
+            $class = $this->getAliasNamespace($namespaceAlias).'\\'.$simpleClassName;
         }
 
         $proxyClass = new ReflectionClass($class);
@@ -188,7 +188,7 @@ 
             /** @var ObjectManager $manager */
             $manager = $this->getService($id);
 
-            if (! $manager->getMetadataFactory()->isTransient($class)) {
+            if (!$manager->getMetadataFactory()->isTransient($class)) {
                 return $manager;
             }
         }
@@ -247,7 +247,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));
         }
 

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

Spacing +8 added lines, -8 removed lines

@@ -14,7 +14,7 @@ 
 {
     public function testGetPaths() : void
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path]);
         self::assertSame([$path], $locator->getPaths());
@@ -33,7 +33,7 @@ 
 
     public function testUniquePaths() : void
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path, $path]);
         self::assertSame([$path], $locator->getPaths());
@@ -41,16 +41,16 @@ 
 
     public function testFindMappingFile() : void
     {
-        $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() : void
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path], '.yml');
 
@@ -61,7 +61,7 @@ 
 
     public function testGetAllClassNames() : void
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator       = new DefaultFileLocator([$path], '.yml');
         $allClasses    = $locator->getAllClassNames('');
@@ -81,7 +81,7 @@ 
 
     public function testGetAllClassNamesNonMatchingFileExtension() : void
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path], '.xml');
         self::assertSame([], $locator->getAllClassNames('global'));
@@ -89,7 +89,7 @@ 
 
     public function testFileExists() : void
     {
-        $path = __DIR__ . '/_files';
+        $path = __DIR__.'/_files';
 
         $locator = new DefaultFileLocator([$path], '.yml');