doctrine / orm

lib/Doctrine/ORM/PersistentObject.php

Spacing +16 added lines, -16 removed lines

@@ -80,7 +80,7 @@ 
     {
         if ($entityManager !== self::$entityManager) {
             throw new RuntimeException(
-                'Trying to use PersistentObject with different EntityManager instances. ' .
+                'Trying to use PersistentObject with different EntityManager instances. '.
                 'Was PersistentObject::setEntityManager() called?'
             );
         }
@@ -105,8 +105,8 @@ 
 
         $property = $this->cm->getProperty($field);
 
-        if (! $property) {
-            throw new BadMethodCallException("no field with name '" . $field . "' exists on '" . $this->cm->getClassName() . "'");
+        if ( ! $property) {
+            throw new BadMethodCallException("no field with name '".$field."' exists on '".$this->cm->getClassName()."'");
         }
 
         switch (true) {
@@ -118,7 +118,7 @@ 
                 $targetClassName = $property->getTargetEntity();
 
                 if ($args[0] !== null && ! ($args[0] instanceof $targetClassName)) {
-                    throw new InvalidArgumentException("Expected persistent object of type '" . $targetClassName . "'");
+                    throw new InvalidArgumentException("Expected persistent object of type '".$targetClassName."'");
                 }
 
                 $this->{$field} = $args[0];
@@ -144,8 +144,8 @@ 
 
         $property = $this->cm->getProperty($field);
 
-        if (! $property) {
-            throw new BadMethodCallException("no field with name '" . $field . "' exists on '" . $this->cm->getClassName() . "'");
+        if ( ! $property) {
+            throw new BadMethodCallException("no field with name '".$field."' exists on '".$this->cm->getClassName()."'");
         }
 
         return $this->{$field};
@@ -167,7 +167,7 @@ 
         $mappedByField    = $property->getMappedBy();
         $targetMetadata   = self::$entityManager->getClassMetadata($property->getTargetEntity());
         $targetProperty   = $targetMetadata->getProperty($mappedByField);
-        $setterMethodName = ($targetProperty instanceof ToManyAssociationMetadata ? 'add' : 'set') . $mappedByField;
+        $setterMethodName = ($targetProperty instanceof ToManyAssociationMetadata ? 'add' : 'set').$mappedByField;
 
         $targetObject->{$setterMethodName}($this);
     }
@@ -189,21 +189,21 @@ 
 
         $property = $this->cm->getProperty($field);
 
-        if (! $property) {
-            throw new BadMethodCallException("no field with name '" . $field . "' exists on '" . $this->cm->getClassName() . "'");
+        if ( ! $property) {
+            throw new BadMethodCallException("no field with name '".$field."' exists on '".$this->cm->getClassName()."'");
         }
 
-        if (! ($property instanceof ToManyAssociationMetadata)) {
-            throw new BadMethodCallException('There is no method add' . $field . '() on ' . $this->cm->getClassName());
+        if ( ! ($property instanceof ToManyAssociationMetadata)) {
+            throw new BadMethodCallException('There is no method add'.$field.'() on '.$this->cm->getClassName());
         }
 
         $targetClassName = $property->getTargetEntity();
 
-        if (! ($args[0] instanceof $targetClassName)) {
-            throw new InvalidArgumentException("Expected persistent object of type '" . $targetClassName . "'");
+        if ( ! ($args[0] instanceof $targetClassName)) {
+            throw new InvalidArgumentException("Expected persistent object of type '".$targetClassName."'");
         }
 
-        if (! ($this->{$field} instanceof Collection)) {
+        if ( ! ($this->{$field} instanceof Collection)) {
             $this->{$field} = new ArrayCollection($this->{$field} ?: []);
         }
 
@@ -225,7 +225,7 @@ 
             return;
         }
 
-        if (! self::$entityManager) {
+        if ( ! self::$entityManager) {
             throw new RuntimeException('No runtime entity manager set. Call PersistentObject#setEntityManager().');
         }
 
@@ -258,7 +258,7 @@ 
                 return $this->add($field, $args);
 
             default:
-                throw new BadMethodCallException('There is no method ' . $method . ' on ' . $this->cm->getClassName());
+                throw new BadMethodCallException('There is no method '.$method.' on '.$this->cm->getClassName());
         }
     }
 }

lib/Doctrine/ORM/OptimisticLockException.php

Spacing +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
         $expectedLockVersion = $expectedLockVersion instanceof DateTime ? $expectedLockVersion->getTimestamp() : $expectedLockVersion;
         $actualLockVersion   = $actualLockVersion instanceof DateTime ? $actualLockVersion->getTimestamp() : $actualLockVersion;
 
-        return new self('The optimistic lock failed, version ' . $expectedLockVersion . ' was expected, but is actually ' . $actualLockVersion, $entity);
+        return new self('The optimistic lock failed, version '.$expectedLockVersion.' was expected, but is actually '.$actualLockVersion, $entity);
     }
 
     /**
@@ -69,6 +69,6 @@ 
      */
     public static function notVersioned($entityName)
     {
-        return new self('Cannot obtain optimistic lock on unversioned entity ' . $entityName, null);
+        return new self('Cannot obtain optimistic lock on unversioned entity '.$entityName, null);
     }
 }

lib/Doctrine/ORM/Configuration.php

Spacing +4 added lines, -4 removed lines

@@ -161,7 +161,7 @@ 
      */
     public function newDefaultAnnotationDriver(array $paths = []) : AnnotationDriver
     {
-        AnnotationRegistry::registerFile(__DIR__ . '/Annotation/DoctrineAnnotations.php');
+        AnnotationRegistry::registerFile(__DIR__.'/Annotation/DoctrineAnnotations.php');
 
         $reader = new CachedReader(new AnnotationReader(), new ArrayCache());
 
@@ -235,7 +235,7 @@ 
     {
         $queryCacheImpl = $this->getQueryCacheImpl();
 
-        if (! $queryCacheImpl) {
+        if ( ! $queryCacheImpl) {
             throw QueryCacheNotConfigured::create();
         }
 
@@ -245,7 +245,7 @@ 
 
         $metadataCacheImpl = $this->getMetadataCacheImpl();
 
-        if (! $metadataCacheImpl) {
+        if ( ! $metadataCacheImpl) {
             throw MetadataCacheNotConfigured::create();
         }
 
@@ -453,7 +453,7 @@ 
     {
         $reflectionClass = new ReflectionClass($repositoryClassName);
 
-        if (! $reflectionClass->implementsInterface(ObjectRepository::class)) {
+        if ( ! $reflectionClass->implementsInterface(ObjectRepository::class)) {
             throw InvalidEntityRepository::fromClassName($repositoryClassName);
         }
 

lib/Doctrine/ORM/Cache/DefaultCacheFactory.php

Spacing +5 added lines, -5 removed lines

@@ -182,14 +182,14 @@ 
             : new DefaultRegion($regionName, $cacheAdapter, $lifetime);
 
         if ($cache->getUsage() === CacheUsage::READ_WRITE) {
-            if (! $this->fileLockRegionDirectory) {
+            if ( ! $this->fileLockRegionDirectory) {
                 throw new LogicException(
-                    'If you want to use a "READ_WRITE" cache an implementation of "Doctrine\ORM\Cache\ConcurrentRegion" is required, ' .
+                    'If you want to use a "READ_WRITE" cache an implementation of "Doctrine\ORM\Cache\ConcurrentRegion" is required, '.
                     'The default implementation provided by doctrine is "Doctrine\ORM\Cache\Region\FileLockRegion" if you want to use it please provide a valid directory, DefaultCacheFactory#setFileLockRegionDirectory(). '
                 );
             }
 
-            $directory = $this->fileLockRegionDirectory . DIRECTORY_SEPARATOR . $regionName;
+            $directory = $this->fileLockRegionDirectory.DIRECTORY_SEPARATOR.$regionName;
             $region    = new FileLockRegion($region, $directory, $this->regionsConfig->getLockLifetime($regionName));
         }
 
@@ -205,7 +205,7 @@ 
     {
         $cacheAdapter = clone $this->cache;
 
-        if (! $cacheAdapter instanceof CacheProvider) {
+        if ( ! $cacheAdapter instanceof CacheProvider) {
             return $cacheAdapter;
         }
 
@@ -215,7 +215,7 @@ 
             $namespace .= ':';
         }
 
-        $cacheAdapter->setNamespace($namespace . $name);
+        $cacheAdapter->setNamespace($namespace.$name);
 
         return $cacheAdapter;
     }

lib/Doctrine/ORM/Cache/Region/DefaultRegion.php

Spacing +2 added lines, -2 removed lines

@@ -101,7 +101,7 @@ 
      */
     protected function getCacheEntryKey(CacheKey $key)
     {
-        return $this->name . self::REGION_KEY_SEPARATOR . $key->hash;
+        return $this->name.self::REGION_KEY_SEPARATOR.$key->hash;
     }
 
     /**
@@ -125,7 +125,7 @@ 
      */
     public function evictAll()
     {
-        if (! $this->cache instanceof ClearableCache) {
+        if ( ! $this->cache instanceof ClearableCache) {
             throw new BadMethodCallException(sprintf(
                 'Clearing all cache entries is not supported by the supplied cache adapter of type %s',
                 get_class($this->cache)

lib/Doctrine/ORM/Cache/Region/FileLockRegion.php

Spacing +7 added lines, -7 removed lines

@@ -52,11 +52,11 @@ 
      */
     public function __construct(Region $region, $directory, $lockLifetime)
     {
-        if (! is_dir($directory) && ! @mkdir($directory, 0775, true)) {
+        if ( ! is_dir($directory) && ! @mkdir($directory, 0775, true)) {
             throw new InvalidArgumentException(sprintf('The directory "%s" does not exist and could not be created.', $directory));
         }
 
-        if (! is_writable($directory)) {
+        if ( ! is_writable($directory)) {
             throw new InvalidArgumentException(sprintf('The directory "%s" is not writable.', $directory));
         }
 
@@ -72,14 +72,14 @@ 
     {
         $filename = $this->getLockFileName($key);
 
-        if (! is_file($filename)) {
+        if ( ! is_file($filename)) {
             return false;
         }
 
         $time    = $this->getLockTime($filename);
         $content = $this->getLockContent($filename);
 
-        if (! $content || ! $time) {
+        if ( ! $content || ! $time) {
             @unlink($filename);
 
             return false;
@@ -104,7 +104,7 @@ 
      */
     private function getLockFileName(CacheKey $key)
     {
-        return $this->directory . DIRECTORY_SEPARATOR . $key->hash . '.' . self::LOCK_EXTENSION;
+        return $this->directory.DIRECTORY_SEPARATOR.$key->hash.'.'.self::LOCK_EXTENSION;
     }
 
     /**
@@ -225,7 +225,7 @@ 
         $lock     = Lock::createLockRead();
         $filename = $this->getLockFileName($key);
 
-        if (! @file_put_contents($filename, $lock->value, LOCK_EX)) {
+        if ( ! @file_put_contents($filename, $lock->value, LOCK_EX)) {
             return null;
         }
         chmod($filename, 0664);
@@ -242,7 +242,7 @@ 
             return false;
         }
 
-        if (! @unlink($this->getLockFileName($key))) {
+        if ( ! @unlink($this->getLockFileName($key))) {
             return false;
         }
 

lib/Doctrine/ORM/Cache/DefaultQueryCache.php

Spacing +15 added lines, -15 removed lines

@@ -62,17 +62,17 @@ 
      */
     public function get(QueryCacheKey $key, ResultSetMapping $rsm, array $hints = [])
     {
-        if (! ($key->cacheMode & Cache::MODE_GET)) {
+        if ( ! ($key->cacheMode & Cache::MODE_GET)) {
             return null;
         }
 
         $cacheEntry = $this->region->get($key);
 
-        if (! $cacheEntry instanceof QueryCacheEntry) {
+        if ( ! $cacheEntry instanceof QueryCacheEntry) {
             return null;
         }
 
-        if (! $this->validator->isValid($key, $cacheEntry)) {
+        if ( ! $this->validator->isValid($key, $cacheEntry)) {
             $this->region->evict($key);
 
             return null;
@@ -88,7 +88,7 @@ 
 
         $cm = $this->em->getClassMetadata($entityName);
 
-        $generateKeys = static function (array $entry) use ($cm) : EntityCacheKey {
+        $generateKeys = static function(array $entry) use ($cm) : EntityCacheKey {
             return new EntityCacheKey($cm->getRootClassName(), $entry['identifier']);
         };
 
@@ -111,7 +111,7 @@ 
                 $this->cacheLogger->entityCacheHit($regionName, $cacheKeys->identifiers[$index]);
             }
 
-            if (! $hasRelation) {
+            if ( ! $hasRelation) {
                 $result[$index] = $unitOfWork->createEntity(
                     $entityEntry->class,
                     $entityEntry->resolveAssociationEntries($this->em),
@@ -156,11 +156,11 @@ 
                     continue;
                 }
 
-                if (! isset($assoc['list']) || empty($assoc['list'])) {
+                if ( ! isset($assoc['list']) || empty($assoc['list'])) {
                     continue;
                 }
 
-                $generateKeys = static function ($id) use ($assocMetadata) : EntityCacheKey {
+                $generateKeys = static function($id) use ($assocMetadata) : EntityCacheKey {
                     return new EntityCacheKey($assocMetadata->getRootClassName(), $id);
                 };
 
@@ -239,7 +239,7 @@ 
             throw FeatureNotImplemented::multipleRootEntities();
         }
 
-        if (! $rsm->isSelect) {
+        if ( ! $rsm->isSelect) {
             throw FeatureNotImplemented::nonSelectStatements();
         }
 
@@ -247,7 +247,7 @@ 
             throw FeatureNotImplemented::partialEntities();
         }
 
-        if (! ($key->cacheMode & Cache::MODE_PUT)) {
+        if ( ! ($key->cacheMode & Cache::MODE_PUT)) {
             return false;
         }
 
@@ -257,7 +257,7 @@ 
         $unitOfWork = $this->em->getUnitOfWork();
         $persister  = $unitOfWork->getEntityPersister($entityName);
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             throw NonCacheableEntity::fromEntity($entityName);
         }
 
@@ -269,7 +269,7 @@ 
 
             if (($key->cacheMode & Cache::MODE_REFRESH) || ! $region->contains($entityKey)) {
                 // Cancel put result if entity put fail
-                if (! $persister->storeEntityCache($entity, $entityKey)) {
+                if ( ! $persister->storeEntityCache($entity, $entityKey)) {
                     return false;
                 }
             }
@@ -303,7 +303,7 @@ 
                 }
 
                 // store single nested association
-                if (! is_array($assocValue)) {
+                if ( ! is_array($assocValue)) {
                     // Cancel put result if association put fail
                     if ($this->storeAssociationCache($key, $association, $assocValue) === null) {
                         return false;
@@ -342,9 +342,9 @@ 
             $assocIdentifier = $unitOfWork->getEntityIdentifier($assocValue);
             $entityKey       = new EntityCacheKey($assocMetadata->getRootClassName(), $assocIdentifier);
 
-            if ((! $assocValue instanceof GhostObjectInterface && ($key->cacheMode & Cache::MODE_REFRESH)) || ! $assocRegion->contains($entityKey)) {
+            if (( ! $assocValue instanceof GhostObjectInterface && ($key->cacheMode & Cache::MODE_REFRESH)) || ! $assocRegion->contains($entityKey)) {
                 // Entity put fail
-                if (! $assocPersister->storeEntityCache($assocValue, $entityKey)) {
+                if ( ! $assocPersister->storeEntityCache($assocValue, $entityKey)) {
                     return null;
                 }
             }
@@ -364,7 +364,7 @@ 
 
             if (($key->cacheMode & Cache::MODE_REFRESH) || ! $assocRegion->contains($entityKey)) {
                 // Entity put fail
-                if (! $assocPersister->storeEntityCache($assocItem, $entityKey)) {
+                if ( ! $assocPersister->storeEntityCache($assocItem, $entityKey)) {
                     return null;
                 }
             }

lib/Doctrine/ORM/Cache/EntityCacheEntry.php

Spacing +2 added lines, -2 removed lines

@@ -57,8 +57,8 @@
      */
     public function resolveAssociationEntries(EntityManagerInterface $em)
     {
-        return array_map(static function ($value) use ($em) {
-            if (! ($value instanceof AssociationCacheEntry)) {
+        return array_map(static function($value) use ($em) {
+            if ( ! ($value instanceof AssociationCacheEntry)) {
                 return $value;
             }
 

tests/Doctrine/Tests/OrmPerformanceTestCase.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      */
     public function setMaxRunningTime($maxRunningTime)
     {
-        if (! (is_int($maxRunningTime) && $maxRunningTime >= 0)) {
+        if ( ! (is_int($maxRunningTime) && $maxRunningTime >= 0)) {
             throw new InvalidArgumentException();
         }