doctrine / orm

lib/Doctrine/ORM/Tools/Setup.php

Spacing +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
     {
         $cache = self::createCacheInstance($isDevMode, $cache);
 
-        if (! $cache instanceof CacheProvider) {
+        if ( ! $cache instanceof CacheProvider) {
             return $cache;
         }
 
@@ -96,7 +96,7 @@ 
             $namespace .= ':';
         }
 
-        $cache->setNamespace($namespace . 'dc2_' . md5($proxyDir) . '_'); // to avoid collisions
+        $cache->setNamespace($namespace.'dc2_'.md5($proxyDir).'_'); // to avoid collisions
 
         return $cache;
     }

tests/Doctrine/Tests/ORM/Tools/SetupTest.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
 
     public function tearDown() : void
     {
-        if (! $this->originalIncludePath) {
+        if ( ! $this->originalIncludePath) {
             return;
         }
 
@@ -74,7 +74,7 @@ 
         $config = Setup::createConfiguration(false, __DIR__);
         $cache  = $config->getMetadataCacheImpl();
 
-        self::assertSame('dc2_' . md5(__DIR__) . '_', $cache->getNamespace());
+        self::assertSame('dc2_'.md5(__DIR__).'_', $cache->getNamespace());
     }
 
     /**
@@ -85,7 +85,7 @@ 
         $config = Setup::createConfiguration(false, __DIR__, new ArrayCache());
         $cache  = $config->getMetadataCacheImpl();
 
-        self::assertSame('dc2_' . md5(__DIR__) . '_', $cache->getNamespace());
+        self::assertSame('dc2_'.md5(__DIR__).'_', $cache->getNamespace());
     }
 
     /**
@@ -100,7 +100,7 @@ 
         $cache  = $config->getMetadataCacheImpl();
 
         self::assertSame($originalCache, $cache);
-        self::assertSame('foo:dc2_' . md5(__DIR__) . '_', $cache->getNamespace());
+        self::assertSame('foo:dc2_'.md5(__DIR__).'_', $cache->getNamespace());
     }
 
     /**

tests/Doctrine/Performance/Query/QueryBoundParameterProcessingBench.php

Spacing +2 added lines, -2 removed lines

@@ -57,8 +57,8 @@
         $this->parsedQueryWithDeclaredParameterType = $entityManager->createQuery($dql);
 
         foreach (range(1, 10) as $index) {
-            $this->parsedQueryWithInferredParameterType->setParameter('parameter' . $index, new DateTime());
-            $this->parsedQueryWithDeclaredParameterType->setParameter('parameter' . $index, new DateTime(), DateTimeType::DATETIME);
+            $this->parsedQueryWithInferredParameterType->setParameter('parameter'.$index, new DateTime());
+            $this->parsedQueryWithDeclaredParameterType->setParameter('parameter'.$index, new DateTime(), DateTimeType::DATETIME);
         }
 
         // Force parsing upfront - we don't benchmark that bit in this scenario

tests/Doctrine/Tests/Mocks/DriverMock.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      */
     public function getDatabasePlatform()
     {
-        if (! $this->platformMock) {
+        if ( ! $this->platformMock) {
             $this->platformMock = new DatabasePlatformMock();
         }
 

tests/Doctrine/Tests/Mocks/ConnectionMock.php

Spacing +1 added lines, -1 removed lines

@@ -116,7 +116,7 @@
     public function quote($input, $type = null)
     {
         if (is_string($input)) {
-            return "'" . $input . "'";
+            return "'".$input."'";
         }
 
         return $input;

tests/Doctrine/Tests/ORM/Functional/Locking/LockAgentWorker.php

Spacing +9 added lines, -9 removed lines

@@ -36,7 +36,7 @@ 
 
         while ($worker->work()) {
             if ($worker->returnCode() !== GEARMAN_SUCCESS) {
-                echo 'return_code: ' . $worker->returnCode() . "\n";
+                echo 'return_code: '.$worker->returnCode()."\n";
                 break;
             }
         }
@@ -61,14 +61,14 @@ 
 
     public function findWithLock($job)
     {
-        return $this->process($job, static function ($fixture, $em) {
+        return $this->process($job, static function($fixture, $em) {
             $entity = $em->find($fixture['entityName'], $fixture['entityId'], $fixture['lockMode']);
         });
     }
 
     public function dqlWithLock($job)
     {
-        return $this->process($job, static function ($fixture, $em) {
+        return $this->process($job, static function($fixture, $em) {
             /** @var Doctrine\ORM\Query $query */
             $query = $em->createQuery($fixture['dql']);
 
@@ -80,7 +80,7 @@ 
 
     public function lock($job)
     {
-        return $this->process($job, static function ($fixture, $em) {
+        return $this->process($job, static function($fixture, $em) {
             $entity = $em->find($fixture['entityName'], $fixture['entityId']);
 
             $em->lock($entity, $fixture['lockMode']);
@@ -89,18 +89,18 @@ 
 
     protected function processWorkload($job)
     {
-        echo 'Received job: ' . $job->handle() . ' for function ' . $job->functionName() . "\n";
+        echo 'Received job: '.$job->handle().' for function '.$job->functionName()."\n";
 
         $workload = $job->workload();
         $workload = unserialize($workload);
 
-        if (! isset($workload['conn']) || ! is_array($workload['conn'])) {
+        if ( ! isset($workload['conn']) || ! is_array($workload['conn'])) {
             throw new InvalidArgumentException('Missing Database parameters');
         }
 
         $this->em = $this->createEntityManager($workload['conn']);
 
-        if (! isset($workload['fixture'])) {
+        if ( ! isset($workload['fixture'])) {
             throw new InvalidArgumentException('Missing Fixture parameters');
         }
 
@@ -110,11 +110,11 @@ 
     protected function createEntityManager($conn)
     {
         $config = new Configuration();
-        $config->setProxyDir(__DIR__ . '/../../../Proxies');
+        $config->setProxyDir(__DIR__.'/../../../Proxies');
         $config->setProxyNamespace('MyProject\Proxies');
         $config->setAutoGenerateProxyClasses(true);
 
-        $annotDriver = $config->newDefaultAnnotationDriver([__DIR__ . '/../../../Models/']);
+        $annotDriver = $config->newDefaultAnnotationDriver([__DIR__.'/../../../Models/']);
         $config->setMetadataDriverImpl($annotDriver);
 
         $cache = new ArrayCache();

lib/Doctrine/ORM/PersistentObject.php

Spacing +16 added lines, -16 removed lines

@@ -79,7 +79,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?'
             );
         }
@@ -104,8 +104,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) {
@@ -117,7 +117,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];
@@ -143,8 +143,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};
@@ -166,7 +166,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);
     }
@@ -188,21 +188,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} ?: []);
         }
 
@@ -224,7 +224,7 @@ 
             return;
         }
 
-        if (! self::$entityManager) {
+        if ( ! self::$entityManager) {
             throw new RuntimeException('No runtime entity manager set. Call PersistentObject#setEntityManager().');
         }
 
@@ -254,7 +254,7 @@ 
             case 'add':
                 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/PersistentCollection.php

Spacing +10 added lines, -10 removed lines

@@ -331,7 +331,7 @@ 
         //       association (table). Without initializing the collection.
         $removed = parent::remove($key);
 
-        if (! $removed) {
+        if ( ! $removed) {
             return $removed;
         }
 
@@ -352,7 +352,7 @@ 
      */
     public function removeElement($element)
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY) {
             if ($this->collection->contains($element)) {
@@ -366,7 +366,7 @@ 
 
         $removed = parent::removeElement($element);
 
-        if (! $removed) {
+        if ( ! $removed) {
             return $removed;
         }
 
@@ -387,7 +387,7 @@ 
      */
     public function containsKey($key)
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY &&
             $this->association->getIndexedBy()) {
@@ -404,7 +404,7 @@ 
      */
     public function contains($element)
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY) {
             $persister = $this->em->getUnitOfWork()->getCollectionPersister($this->association);
@@ -420,11 +420,11 @@ 
      */
     public function get($key)
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY &&
             $this->association->getIndexedBy()) {
-            if (! $this->typeClass->isIdentifierComposite() && $this->typeClass->isIdentifier($this->association->getIndexedBy())) {
+            if ( ! $this->typeClass->isIdentifierComposite() && $this->typeClass->isIdentifier($this->association->getIndexedBy())) {
                 return $this->em->find($this->typeClass->getClassName(), $key);
             }
 
@@ -439,7 +439,7 @@ 
      */
     public function count()
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY) {
             $persister = $this->em->getUnitOfWork()->getCollectionPersister($this->association);
@@ -503,7 +503,7 @@ 
      */
     public function offsetSet($offset, $value)
     {
-        if (! isset($offset)) {
+        if ( ! isset($offset)) {
             $this->add($value);
 
             return;
@@ -598,7 +598,7 @@ 
      */
     public function slice($offset, $length = null)
     {
-        if (! $this->initialized &&
+        if ( ! $this->initialized &&
             ! $this->isDirty &&
             $this->association !== null &&
             $this->association->getFetchMode() === FetchMode::EXTRA_LAZY) {

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

Spacing +6 added lines, -6 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);