doctrine / persistence

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

Spacing +6 added lines, -6 removed lines

@@ -90,12 +90,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;
             }
         }
 
@@ -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(string $className) : bool
     {
-        $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/Persistence/Mapping/Driver/AnnotationDriver.php

Spacing +6 added lines, -6 removed lines

@@ -80,7 +80,7 @@ 
     {
         $this->reader = $reader;
 
-        if (! $paths) {
+        if (!$paths) {
             return;
         }
 
@@ -184,7 +184,7 @@ 
             return $this->classNames;
         }
 
-        if (! $this->paths) {
+        if (!$this->paths) {
             throw MappingException::pathRequired();
         }
 
@@ -192,7 +192,7 @@ 
         $includedFiles = [];
 
         foreach ($this->paths as $path) {
-            if (! is_dir($path)) {
+            if (!is_dir($path)) {
                 throw MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
             }
 
@@ -201,14 +201,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);
                 }
 
@@ -235,7 +235,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)) {
                 continue;
             }
 

lib/Doctrine/Persistence/Mapping/MappingException.php

Spacing +4 added lines, -4 removed lines

@@ -26,18 +26,18 @@
 
     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.');
     }
 
     public static function fileMappingDriversRequireConfiguredDirectoryPath(?string $path = null) : self
     {
-        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/Persistence/Event/PreUpdateEventArgs.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
      */
     private function assertValidField(string $field) : void
     {
-        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,

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

Spacing +6 added lines, -6 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 = static function () use ($classMetadata) {
+        $this->cmf->fallbackCallback = static function() use ($classMetadata) {
             return $classMetadata;
         };
 
@@ -122,7 +122,7 @@ 
 
     public function testWillFailOnFallbackFailureWithNotLoadedMetadata()
     {
-        $this->cmf->fallbackCallback = static function () {
+        $this->cmf->fallbackCallback = static function() {
             return null;
         };
 

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

Spacing +17 added lines, -17 removed lines

@@ -14,7 +14,7 @@ 
 {
     public function testGetPaths()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix]);
@@ -26,7 +26,7 @@ 
 
     public function testGetPrefixes()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix]);
@@ -43,7 +43,7 @@ 
 
     public function testFileExists()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix], '.yml');
@@ -56,7 +56,7 @@ 
 
     public function testGetAllClassNames()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $locator       = new SymfonyFileLocator([$path => $prefix], '.yml');
@@ -77,7 +77,7 @@ 
 
     public function testInvalidCustomNamespaceSeparator()
     {
-        $path   = __DIR__ . '/_files';
+        $path   = __DIR__.'/_files';
         $prefix = 'Foo';
 
         $this->expectException(InvalidArgumentException::class);
@@ -104,7 +104,7 @@ 
      */
     public function testGetClassNamesWithCustomNsSeparator($separator, $dir)
     {
-        $path   = __DIR__ . $dir;
+        $path   = __DIR__.$dir;
         $prefix = 'Foo';
 
         $locator = new SymfonyFileLocator([$path => $prefix], '.yml', $separator);
@@ -149,29 +149,29 @@ 
      */
     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');
@@ -185,13 +185,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")
         );
     }
@@ -199,13 +199,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")
         );
     }

tests/Doctrine/Tests/Persistence/ManagerRegistryTest.php

Spacing +2 added lines, -2 removed lines

@@ -166,7 +166,7 @@ 
 
     private function getManagerFactory()
     {
-        return function (string $name) {
+        return function(string $name) {
             $mock = $this->createMock(ObjectManager::class);
 
             $driver   = $this->createMock(MappingDriver::class);
@@ -212,7 +212,7 @@ 
 
     protected function getService(string $name) : object
     {
-        if (! isset($this->services[$name])) {
+        if (!isset($this->services[$name])) {
             $this->services[$name] = call_user_func($this->managerFactory, $name);
         }
 

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

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
 
     protected function getFqcnFromAlias(string $namespaceAlias, string $simpleClassName) : string
     {
-        return __NAMESPACE__ . '\\' . $simpleClassName;
+        return __NAMESPACE__.'\\'.$simpleClassName;
     }
 
     protected function initialize() : void
@@ -67,7 +67,7 @@ 
 
     protected function onNotFoundMetadata(string $className) : ?ClassMetadata
     {
-        if (! $this->fallbackCallback) {
+        if (!$this->fallbackCallback) {
             return null;
         }
 

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

Spacing +14 added lines, -14 removed lines

@@ -127,12 +127,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;
                 }
 
@@ -141,11 +141,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;
@@ -164,7 +164,7 @@ 
 
         if ($this->paths) {
             foreach ((array) $this->paths as $path) {
-                if (! is_dir($path)) {
+                if (!is_dir($path)) {
                     throw MappingException::fileMappingDriversRequireConfiguredDirectoryPath($path);
                 }
 
@@ -189,7 +189,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);
@@ -208,11 +208,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;
@@ -220,11 +220,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;
             }
@@ -232,7 +232,7 @@ 
 
         throw MappingException::mappingFileNotFound(
             $className,
-            substr($className, strrpos($className, '\\') + 1) . $this->fileExtension
+            substr($className, strrpos($className, '\\') + 1).$this->fileExtension
         );
     }