doctrine / orm

tests/Doctrine/Tests/ORM/Query/QueryTest.php

Spacing +8 added lines, -8 removed lines

@@ -168,7 +168,7 @@ 
             9 => "St Julien"
         ];
 
-        $query  = $this->_em
+        $query = $this->_em
                 ->createQuery("SELECT a FROM Doctrine\Tests\Models\CMS\CmsAddress a WHERE a.city IN (:cities)")
                 ->setParameter('cities', $cities);
 
@@ -184,7 +184,7 @@ 
      */
     public function testProcessParameterValueClassMetadata()
     {
-        $query  = $this->_em->createQuery("SELECT a FROM Doctrine\Tests\Models\CMS\CmsAddress a WHERE a.city IN (:cities)");
+        $query = $this->_em->createQuery("SELECT a FROM Doctrine\Tests\Models\CMS\CmsAddress a WHERE a.city IN (:cities)");
         $this->assertEquals(
             CmsAddress::class,
             $query->processParameterValue($this->_em->getClassMetadata(CmsAddress::class))
@@ -332,7 +332,7 @@ 
      */
     public function testGetParameterTypeJuggling() : void
     {
-        $query = $this->_em->createQuery('select u from ' . CmsUser::class . ' u where u.id = ?0');
+        $query = $this->_em->createQuery('select u from '.CmsUser::class.' u where u.id = ?0');
 
         $query->setParameter(0, 0);
 
@@ -346,7 +346,7 @@ 
      */
     public function testSetParameterWithNameZeroIsNotOverridden() : void
     {
-        $query = $this->_em->createQuery('select u from ' . CmsUser::class . ' u where u.id != ?0 and u.username = :name');
+        $query = $this->_em->createQuery('select u from '.CmsUser::class.' u where u.id != ?0 and u.username = :name');
 
         $query->setParameter(0, 0);
         $query->setParameter('name', 'Doctrine');
@@ -361,7 +361,7 @@ 
      */
     public function testSetParameterWithNameZeroDoesNotOverrideAnotherParameter() : void
     {
-        $query = $this->_em->createQuery('select u from ' . CmsUser::class . ' u where u.id != ?0 and u.username = :name');
+        $query = $this->_em->createQuery('select u from '.CmsUser::class.' u where u.id != ?0 and u.username = :name');
 
         $query->setParameter('name', 'Doctrine');
         $query->setParameter(0, 0);
@@ -376,7 +376,7 @@ 
      */
     public function testSetParameterWithTypeJugglingWorks() : void
     {
-        $query = $this->_em->createQuery('select u from ' . CmsUser::class . ' u where u.id != ?0 and u.username = :name');
+        $query = $this->_em->createQuery('select u from '.CmsUser::class.' u where u.id != ?0 and u.username = :name');
 
         $query->setParameter('0', 1);
         $query->setParameter('name', 'Doctrine');
@@ -396,7 +396,7 @@ 
     {
         $this->_em->getConfiguration()->setResultCacheImpl(new ArrayCache());
 
-        $query = $this->_em->createQuery('SELECT u FROM ' . CmsUser::class . ' u');
+        $query = $this->_em->createQuery('SELECT u FROM '.CmsUser::class.' u');
         $query->useResultCache(true);
         $query->setResultCacheProfile();
 
@@ -414,7 +414,7 @@ 
             ->expects(self::never())
             ->method('getSingleIdentifierValue');
 
-        $query = $this->_em->createQuery('SELECT d FROM ' . DateTimeModel::class . ' d WHERE d.datetime = :value');
+        $query = $this->_em->createQuery('SELECT d FROM '.DateTimeModel::class.' d WHERE d.datetime = :value');
 
         $query->setParameter('value', new DateTime(), Type::DATETIME);
 

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/Performance/EntityManagerFactory.php

Spacing +7 added lines, -7 removed lines

@@ -19,12 +19,12 @@ 
     {
         $config = new Configuration();
 
-        $config->setProxyDir(__DIR__ . '/../Tests/Proxies');
+        $config->setProxyDir(__DIR__.'/../Tests/Proxies');
         $config->setProxyNamespace('Doctrine\Tests\Proxies');
         $config->setAutoGenerateProxyClasses(ProxyFactory::AUTOGENERATE_EVAL);
         $config->setMetadataDriverImpl($config->newDefaultAnnotationDriver([
-            realpath(__DIR__ . '/Models/Cache'),
-            realpath(__DIR__ . '/Models/GeoNames'),
+            realpath(__DIR__.'/Models/Cache'),
+            realpath(__DIR__.'/Models/GeoNames'),
         ], true));
 
         $entityManager = EntityManager::create(
@@ -45,13 +45,13 @@ 
     {
         $config = new Configuration();
 
-        $config->setProxyDir(__DIR__ . '/../Tests/Proxies');
+        $config->setProxyDir(__DIR__.'/../Tests/Proxies');
         $config->setProxyNamespace('Doctrine\Tests\Proxies');
         $config->setAutoGenerateProxyClasses(ProxyFactory::AUTOGENERATE_EVAL);
         $config->setMetadataDriverImpl($config->newDefaultAnnotationDriver([
-            realpath(__DIR__ . '/Models/Cache'),
-            realpath(__DIR__ . '/Models/Generic'),
-            realpath(__DIR__ . '/Models/GeoNames'),
+            realpath(__DIR__.'/Models/Cache'),
+            realpath(__DIR__.'/Models/Generic'),
+            realpath(__DIR__.'/Models/GeoNames'),
         ], true));
 
         // A connection that doesn't really do anything

lib/Doctrine/ORM/Query/QueryExpressionVisitor.php

Spacing +11 added lines, -11 removed lines

@@ -114,7 +114,7 @@ 
             $expressionList[] = $this->dispatch($child);
         }
 
-        switch($expr->getType()) {
+        switch ($expr->getType()) {
             case CompositeExpression::TYPE_AND:
                 return new Expr\Andx($expressionList);
 
@@ -122,7 +122,7 @@ 
                 return new Expr\Orx($expressionList);
 
             default:
-                throw new \RuntimeException("Unknown composite " . $expr->getType());
+                throw new \RuntimeException("Unknown composite ".$expr->getType());
         }
     }
 
@@ -136,10 +136,10 @@ 
             throw new QueryException('No aliases are set before invoking walkComparison().');
         }
 
-        $field = $this->queryAliases[0] . '.' . $comparison->getField();
+        $field = $this->queryAliases[0].'.'.$comparison->getField();
 
-        foreach($this->queryAliases as $alias) {
-            if(strpos($comparison->getField() . '.', $alias . '.') === 0) {
+        foreach ($this->queryAliases as $alias) {
+            if (strpos($comparison->getField().'.', $alias.'.') === 0) {
                 $field = $comparison->getField();
                 break;
             }
@@ -149,13 +149,13 @@ 
 
         foreach ($this->parameters as $parameter) {
             if ($parameter->getName() === $parameterName) {
-                $parameterName .= '_' . count($this->parameters);
+                $parameterName .= '_'.count($this->parameters);
                 break;
             }
         }
 
         $parameter = new Parameter($parameterName, $this->walkValue($comparison->getValue()));
-        $placeholder = ':' . $parameterName;
+        $placeholder = ':'.$parameterName;
 
         switch ($comparison->getOperator()) {
             case Comparison::IN:
@@ -182,19 +182,19 @@ 
 
                 return $this->expr->neq($field, $placeholder);
             case Comparison::CONTAINS:
-                $parameter->setValue('%' . $parameter->getValue() . '%', $parameter->getType());
+                $parameter->setValue('%'.$parameter->getValue().'%', $parameter->getType());
                 $this->parameters[] = $parameter;
 
                 return $this->expr->like($field, $placeholder);
             case Comparison::MEMBER_OF:
                 return $this->expr->isMemberOf($comparison->getField(), $comparison->getValue()->getValue());
             case Comparison::STARTS_WITH:
-                $parameter->setValue($parameter->getValue() . '%', $parameter->getType());
+                $parameter->setValue($parameter->getValue().'%', $parameter->getType());
                 $this->parameters[] = $parameter;
 
                 return $this->expr->like($field, $placeholder);
             case Comparison::ENDS_WITH:
-                $parameter->setValue('%' . $parameter->getValue(), $parameter->getType());
+                $parameter->setValue('%'.$parameter->getValue(), $parameter->getType());
                 $this->parameters[] = $parameter;
 
                 return $this->expr->like($field, $placeholder);
@@ -210,7 +210,7 @@ 
                     );
                 }
 
-                throw new \RuntimeException("Unknown comparison operator: " . $comparison->getOperator());
+                throw new \RuntimeException("Unknown comparison operator: ".$comparison->getOperator());
         }
     }
 

lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php

Spacing +13 added lines, -13 removed lines

@@ -105,7 +105,7 @@ 
      */
     protected function onNotFoundMetadata($className)
     {
-        if (! $this->evm->hasListeners(Events::onClassMetadataNotFound)) {
+        if ( ! $this->evm->hasListeners(Events::onClassMetadataNotFound)) {
             return;
         }
 
@@ -162,7 +162,7 @@ 
             $this->completeIdGeneratorMapping($class);
         }
 
-        if (!$class->isMappedSuperclass) {
+        if ( ! $class->isMappedSuperclass) {
             foreach ($class->embeddedClasses as $property => $embeddableClass) {
 
                 if (isset($embeddableClass['inherited'])) {
@@ -187,7 +187,7 @@ 
 
                 $identifier = $embeddableMetadata->getIdentifier();
 
-                if (! empty($identifier)) {
+                if ( ! empty($identifier)) {
                     $this->inheritIdGeneratorMapping($class, $embeddableMetadata);
                 }
 
@@ -257,7 +257,7 @@ 
      */
     protected function validateRuntimeMetadata($class, $parent)
     {
-        if ( ! $class->reflClass ) {
+        if ( ! $class->reflClass) {
             // only validate if there is a reflection class instance
             return;
         }
@@ -267,7 +267,7 @@ 
         $class->validateLifecycleCallbacks($this->getReflectionService());
 
         // verify inheritance
-        if ( ! $class->isMappedSuperclass && !$class->isInheritanceTypeNone()) {
+        if ( ! $class->isMappedSuperclass && ! $class->isInheritanceTypeNone()) {
             if ( ! $parent) {
                 if (count($class->discriminatorMap) == 0) {
                     throw MappingException::missingDiscriminatorMap($class->name);
@@ -428,7 +428,7 @@ 
     {
         foreach ($parentClass->associationMappings as $field => $mapping) {
             if ($parentClass->isMappedSuperclass) {
-                if ($mapping['type'] & ClassMetadata::TO_MANY && !$mapping['isOwningSide']) {
+                if ($mapping['type'] & ClassMetadata::TO_MANY && ! $mapping['isOwningSide']) {
                     throw MappingException::illegalToManyAssociationOnMappedSuperclass($parentClass->name, $field);
                 }
                 $mapping['sourceEntity'] = $subClass->name;
@@ -477,11 +477,11 @@ 
 
             $parentClass->mapEmbedded(
                 [
-                    'fieldName' => $prefix . '.' . $property,
+                    'fieldName' => $prefix.'.'.$property,
                     'class' => $embeddableMetadata->name,
                     'columnPrefix' => $embeddableClass['columnPrefix'],
                     'declaredField' => $embeddableClass['declaredField']
-                            ? $prefix . '.' . $embeddableClass['declaredField']
+                            ? $prefix.'.'.$embeddableClass['declaredField']
                             : $prefix,
                     'originalField' => $embeddableClass['originalField'] ?: $property,
                 ]
@@ -499,7 +499,7 @@ 
      */
     private function addInheritedIndexes(ClassMetadata $subClass, ClassMetadata $parentClass)
     {
-        if (! $parentClass->isMappedSuperclass) {
+        if ( ! $parentClass->isMappedSuperclass) {
             return;
         }
 
@@ -707,14 +707,14 @@ 
                     throw new ORMException("Can't instantiate custom generator : no custom generator definition");
                 }
                 if ( ! class_exists($definition['class'])) {
-                    throw new ORMException("Can't instantiate custom generator : " .
+                    throw new ORMException("Can't instantiate custom generator : ".
                         $definition['class']);
                 }
                 $class->setIdGenerator(new $definition['class']);
                 break;
 
             default:
-                throw new ORMException("Unknown generator type: " . $class->generatorType);
+                throw new ORMException("Unknown generator type: ".$class->generatorType);
         }
     }
 
@@ -764,7 +764,7 @@ 
      */
     protected function getFqcnFromAlias($namespaceAlias, $simpleClassName)
     {
-        return $this->em->getConfiguration()->getEntityNamespace($namespaceAlias) . '\\' . $simpleClassName;
+        return $this->em->getConfiguration()->getEntityNamespace($namespaceAlias).'\\'.$simpleClassName;
     }
 
     /**
@@ -788,7 +788,7 @@ 
      */
     private function getTargetPlatform()
     {
-        if (!$this->targetPlatform) {
+        if ( ! $this->targetPlatform) {
             $this->targetPlatform = $this->em->getConnection()->getDatabasePlatform();
         }
 

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

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
         $returnableItems = [];
 
         foreach ($keysToRetrieve as $index => $key) {
-            if (! $items[$key] instanceof CacheEntry) {
+            if ( ! $items[$key] instanceof CacheEntry) {
                 return null;
             }
 

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

Spacing +4 added lines, -4 removed lines

@@ -96,7 +96,7 @@ 
     {
         $entry = $this->cache->fetch($this->getCacheEntryKey($key));
 
-        if (! $entry instanceof CacheEntry) {
+        if ( ! $entry instanceof CacheEntry) {
             return null;
         }
 
@@ -114,7 +114,7 @@ 
             $entryKey   = $this->getCacheEntryKey($key);
             $entryValue = $this->cache->fetch($entryKey);
 
-            if (! $entryValue instanceof CacheEntry) {
+            if ( ! $entryValue instanceof CacheEntry) {
                 return null;
             }
 
@@ -130,7 +130,7 @@ 
      */
     protected function getCacheEntryKey(CacheKey $key)
     {
-        return $this->name . self::REGION_KEY_SEPARATOR . $key->hash;
+        return $this->name.self::REGION_KEY_SEPARATOR.$key->hash;
     }
 
     /**
@@ -154,7 +154,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)

tests/Doctrine/Tests/ORM/Cache/DefaultRegionTest.php

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
     public function corruptedDataDoesNotLeakIntoApplicationWhenGettingSingleEntry() : void
     {
         $key1 = new CacheKeyMock('key.1');
-        $this->cache->save($this->region->getName() . '_' . $key1->hash, 'a-very-invalid-value');
+        $this->cache->save($this->region->getName().'_'.$key1->hash, 'a-very-invalid-value');
 
         self::assertTrue($this->region->contains($key1));
         self::assertNull($this->region->get($key1));
@@ -116,7 +116,7 @@ 
     public function corruptedDataDoesNotLeakIntoApplicationWhenGettingMultipleEntries() : void
     {
         $key1 = new CacheKeyMock('key.1');
-        $this->cache->save($this->region->getName() . '_' . $key1->hash, 'a-very-invalid-value');
+        $this->cache->save($this->region->getName().'_'.$key1->hash, 'a-very-invalid-value');
 
         self::assertTrue($this->region->contains($key1));
         self::assertNull($this->region->getMultiple(new CollectionCacheEntry([$key1])));

tests/Doctrine/Tests/ORM/Cache/MultiGetRegionTest.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     public function corruptedDataDoesNotLeakIntoApplication() : void
     {
         $key1 = new CacheKeyMock('key.1');
-        $this->cache->save($this->region->getName() . '_' . $key1->hash, 'a-very-invalid-value');
+        $this->cache->save($this->region->getName().'_'.$key1->hash, 'a-very-invalid-value');
 
         self::assertTrue($this->region->contains($key1));
         self::assertNull($this->region->getMultiple(new CollectionCacheEntry([$key1])));