doctrine / orm

tests/Doctrine/Tests/OrmFunctionalTestCase.php

Spacing +20 added lines, -20 removed lines

@@ -346,7 +346,7 @@ 
         $conn = static::$sharedConn;
 
         // In case test is skipped, tearDown is called, but no setup may have run
-        if (! $conn) {
+        if ( ! $conn) {
             return;
         }
 
@@ -420,7 +420,7 @@ 
         }
 
         if (isset($this->usedModelSets['directorytree'])) {
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('file'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('file'));
             // MySQL doesn't know deferred deletions therefore only executing the second query gives errors.
             $conn->executeUpdate('DELETE FROM Directory WHERE parentDirectory_id IS NOT NULL');
             $conn->executeUpdate('DELETE FROM Directory');
@@ -513,12 +513,12 @@ 
                 )
             );
 
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-users-groups'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-group'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-phone'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-user'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-address'));
-            $conn->executeUpdate('DELETE FROM ' . $platform->quoteIdentifier('quote-city'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-users-groups'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-group'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-phone'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-user'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-address'));
+            $conn->executeUpdate('DELETE FROM '.$platform->quoteIdentifier('quote-city'));
         }
 
         if (isset($this->usedModelSets['vct_onetoone'])) {
@@ -630,7 +630,7 @@ 
         $classes = [];
 
         foreach ($classNames as $className) {
-            if (! isset(static::$entityTablesCreated[$className])) {
+            if ( ! isset(static::$entityTablesCreated[$className])) {
                 static::$entityTablesCreated[$className] = true;
                 $classes[]                               = $this->em->getClassMetadata($className);
             }
@@ -649,25 +649,25 @@ 
     {
         $this->setUpDBALTypes();
 
-        if (! isset(static::$sharedConn)) {
+        if ( ! isset(static::$sharedConn)) {
             static::$sharedConn = TestUtil::getConnection();
         }
 
         if (isset($GLOBALS['DOCTRINE_MARK_SQL_LOGS'])) {
             if (in_array(static::$sharedConn->getDatabasePlatform()->getName(), ['mysql', 'postgresql'], true)) {
-                static::$sharedConn->executeQuery('SELECT 1 /*' . static::class . '*/');
+                static::$sharedConn->executeQuery('SELECT 1 /*'.static::class.'*/');
             } elseif (static::$sharedConn->getDatabasePlatform()->getName() === 'oracle') {
-                static::$sharedConn->executeQuery('SELECT 1 /*' . static::class . '*/ FROM dual');
+                static::$sharedConn->executeQuery('SELECT 1 /*'.static::class.'*/ FROM dual');
             }
         }
 
-        if (! $this->em) {
+        if ( ! $this->em) {
             $this->em         = $this->getEntityManager();
             $this->schemaTool = new SchemaTool($this->em);
         }
 
         foreach ($this->usedModelSets as $setName => $bool) {
-            if (! isset(static::$tablesCreated[$setName])) {
+            if ( ! isset(static::$tablesCreated[$setName])) {
                 $this->setUpEntitySchema(static::$modelSets[$setName]);
 
                 static::$tablesCreated[$setName] = true;
@@ -742,8 +742,8 @@ 
 
         $config->setMetadataDriverImpl(
             $mappingDriver ?? $config->newDefaultAnnotationDriver([
-                realpath(__DIR__ . '/Models/Cache'),
-                realpath(__DIR__ . '/Models/GeoNames'),
+                realpath(__DIR__.'/Models/Cache'),
+                realpath(__DIR__.'/Models/GeoNames'),
             ])
         );
 
@@ -791,13 +791,13 @@ 
 
             foreach ($last25queries as $i => $query) {
                 $params = array_map(
-                    static function ($p) {
+                    static function($p) {
                         return is_object($p) ? get_class($p) : var_export($p, true);
                     },
                     $query['params'] ?: []
                 );
 
-                $queries .= $i . ". SQL: '" . $query['sql'] . "' Params: " . implode(', ', $params) . PHP_EOL;
+                $queries .= $i.". SQL: '".$query['sql']."' Params: ".implode(', ', $params).PHP_EOL;
             }
 
             $trace    = $e->getTrace();
@@ -810,11 +810,11 @@ 
                         break;
                     }
 
-                    $traceMsg .= $part['file'] . ':' . $part['line'] . PHP_EOL;
+                    $traceMsg .= $part['file'].':'.$part['line'].PHP_EOL;
                 }
             }
 
-            $message = '[' . get_class($e) . '] ' . $e->getMessage() . PHP_EOL . PHP_EOL . 'With queries:' . PHP_EOL . $queries . PHP_EOL . 'Trace:' . PHP_EOL . $traceMsg;
+            $message = '['.get_class($e).'] '.$e->getMessage().PHP_EOL.PHP_EOL.'With queries:'.PHP_EOL.$queries.PHP_EOL.'Trace:'.PHP_EOL.$traceMsg;
 
             throw new Exception($message, (int) $e->getCode(), $e);
         }

lib/Doctrine/ORM/Tools/Console/Command/RunDqlCommand.php

Indentation +7 added lines, -7 removed lines

@@ -33,13 +33,13 @@
     protected function configure()
     {
         $this->setName('orm:run-dql')
-             ->setDescription('Executes arbitrary DQL directly from the command line')
-             ->addArgument('dql', InputArgument::REQUIRED, 'The DQL to execute.')
-             ->addOption('hydrate', null, InputOption::VALUE_REQUIRED, 'Hydration mode of result set. Should be either: object, array, scalar or single-scalar.', 'object')
-             ->addOption('first-result', null, InputOption::VALUE_REQUIRED, 'The first result in the result set.')
-             ->addOption('max-result', null, InputOption::VALUE_REQUIRED, 'The maximum number of results in the result set.')
-             ->addOption('show-sql', null, InputOption::VALUE_NONE, 'Dump generated SQL instead of executing query')
-             ->setHelp('Executes arbitrary DQL directly from the command line.');
+                ->setDescription('Executes arbitrary DQL directly from the command line')
+                ->addArgument('dql', InputArgument::REQUIRED, 'The DQL to execute.')
+                ->addOption('hydrate', null, InputOption::VALUE_REQUIRED, 'Hydration mode of result set. Should be either: object, array, scalar or single-scalar.', 'object')
+                ->addOption('first-result', null, InputOption::VALUE_REQUIRED, 'The first result in the result set.')
+                ->addOption('max-result', null, InputOption::VALUE_REQUIRED, 'The maximum number of results in the result set.')
+                ->addOption('show-sql', null, InputOption::VALUE_NONE, 'Dump generated SQL instead of executing query')
+                ->setHelp('Executes arbitrary DQL directly from the command line.');
     }
 
     /**

Spacing +5 added lines, -5 removed lines

@@ -58,9 +58,9 @@ 
         }
 
         $hydrationModeName = $input->getOption('hydrate');
-        $hydrationMode     = 'Doctrine\ORM\Query::HYDRATE_' . strtoupper(str_replace('-', '_', $hydrationModeName));
+        $hydrationMode     = 'Doctrine\ORM\Query::HYDRATE_'.strtoupper(str_replace('-', '_', $hydrationModeName));
 
-        if (! defined($hydrationMode)) {
+        if ( ! defined($hydrationMode)) {
             throw new RuntimeException(sprintf(
                 "Hydration mode '%s' does not exist. It should be either: object. array, scalar or single-scalar.",
                 $hydrationModeName
@@ -71,7 +71,7 @@ 
         $firstResult = $input->getOption('first-result');
 
         if ($firstResult !== null) {
-            if (! is_numeric($firstResult)) {
+            if ( ! is_numeric($firstResult)) {
                 throw new LogicException("Option 'first-result' must contain an integer value");
             }
 
@@ -81,7 +81,7 @@ 
         $maxResult = $input->getOption('max-result');
 
         if ($maxResult !== null) {
-            if (! is_numeric($maxResult)) {
+            if ( ! is_numeric($maxResult)) {
                 throw new LogicException("Option 'max-result' must contain an integer value");
             }
 
@@ -95,7 +95,7 @@ 
 
         $resultSet = $query->execute([], constant($hydrationMode));
 
-        $dumper = new CliDumper(static function (string $payload) use ($output) : void {
+        $dumper = new CliDumper(static function(string $payload) use ($output) : void {
             $output->write($payload);
         });
         $dumper->dump((new VarCloner())->cloneVar($resultSet));

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/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);