doctrine / orm

tests/Doctrine/Tests/TestUtil.php

Spacing +2 added lines, -2 removed lines

@@ -90,7 +90,7 @@ 
     {
         $realDbParams = self::getParamsForMainConnection();
 
-        if (! self::$initialized) {
+        if ( ! self::$initialized) {
             $tmpDbParams = self::getParamsForTemporaryConnection();
 
             $realConn = DriverManager::getConnection($realDbParams);
@@ -141,7 +141,7 @@ 
 
     private static function addDbEventSubscribers(Connection $conn)
     {
-        if (! isset($GLOBALS['db_event_subscribers'])) {
+        if ( ! isset($GLOBALS['db_event_subscribers'])) {
             return;
         }
 

lib/Doctrine/ORM/Cache/DefaultQueryCache.php

Spacing +16 added lines, -16 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 = function (array $entry) use ($cm) : EntityCacheKey {
+        $generateKeys = 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 = function ($id) use ($assocMetadata) : EntityCacheKey {
+                $generateKeys = function($id) use ($assocMetadata) : EntityCacheKey {
                     return new EntityCacheKey($assocMetadata->getRootClassName(), $id);
                 };
 
@@ -210,7 +210,7 @@ 
                 //
                 // We need to unwrap those associations into proxy references,
                 // since we don't have actual data for them except for identifiers.
-                if (! ($unCachedAssociationData instanceof AssociationCacheEntry)) {
+                if ( ! ($unCachedAssociationData instanceof AssociationCacheEntry)) {
                     continue;
                 }
 
@@ -242,7 +242,7 @@ 
             throw FeatureNotImplemented::multipleRootEntities();
         }
 
-        if (! $rsm->isSelect) {
+        if ( ! $rsm->isSelect) {
             throw FeatureNotImplemented::nonSelectStatements();
         }
 
@@ -250,7 +250,7 @@ 
             throw FeatureNotImplemented::partialEntities();
         }
 
-        if (! ($key->cacheMode & Cache::MODE_PUT)) {
+        if ( ! ($key->cacheMode & Cache::MODE_PUT)) {
             return false;
         }
 
@@ -260,7 +260,7 @@ 
         $unitOfWork = $this->em->getUnitOfWork();
         $persister  = $unitOfWork->getEntityPersister($entityName);
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             throw NonCacheableEntity::fromEntity($entityName);
         }
 
@@ -272,7 +272,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;
                 }
             }
@@ -306,7 +306,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;
@@ -345,9 +345,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;
                 }
             }
@@ -367,7 +367,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/DefaultCache.php

Spacing +16 added lines, -17 removed lines

@@ -55,7 +55,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return null;
         }
 
@@ -70,7 +70,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getCollectionPersister($metadata->getProperty($association));
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return null;
         }
 
@@ -85,7 +85,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return false;
         }
 
@@ -100,7 +100,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return;
         }
 
@@ -115,7 +115,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return;
         }
 
@@ -132,7 +132,7 @@ 
         foreach ($metadatas as $metadata) {
             $persister = $this->uow->getEntityPersister($metadata->getRootClassName());
 
-            if (! ($persister instanceof CachedPersister)) {
+            if ( ! ($persister instanceof CachedPersister)) {
                 continue;
             }
 
@@ -148,7 +148,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getCollectionPersister($metadata->getProperty($association));
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return false;
         }
 
@@ -163,7 +163,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getCollectionPersister($metadata->getProperty($association));
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return;
         }
 
@@ -178,7 +178,7 @@ 
         $metadata  = $this->em->getClassMetadata($className);
         $persister = $this->uow->getCollectionPersister($metadata->getProperty($association));
 
-        if (! ($persister instanceof CachedPersister)) {
+        if ( ! ($persister instanceof CachedPersister)) {
             return;
         }
 
@@ -194,13 +194,13 @@ 
 
         foreach ($metadatas as $metadata) {
             foreach ($metadata->getDeclaredPropertiesIterator() as $association) {
-                if (! $association instanceof ToManyAssociationMetadata) {
+                if ( ! $association instanceof ToManyAssociationMetadata) {
                     continue;
                 }
 
                 $persister = $this->uow->getCollectionPersister($association);
 
-                if (! ($persister instanceof CachedPersister)) {
+                if ( ! ($persister instanceof CachedPersister)) {
                     continue;
                 }
 
@@ -228,7 +228,7 @@ 
             return;
         }
 
-        if (! isset($this->queryCaches[$regionName])) {
+        if ( ! isset($this->queryCaches[$regionName])) {
             return;
         }
 
@@ -253,11 +253,10 @@ 
     public function getQueryCache($regionName = null)
     {
         if ($regionName === null) {
-            return $this->defaultQueryCache ?:
-                $this->defaultQueryCache = $this->cacheFactory->buildQueryCache($this->em);
+            return $this->defaultQueryCache ?: $this->defaultQueryCache = $this->cacheFactory->buildQueryCache($this->em);
         }
 
-        if (! isset($this->queryCaches[$regionName])) {
+        if ( ! isset($this->queryCaches[$regionName])) {
             $this->queryCaches[$regionName] = $this->cacheFactory->buildQueryCache($this->em, $regionName);
         }
 
@@ -272,7 +271,7 @@ 
      */
     private function buildEntityCacheKey(ClassMetadata $metadata, $identifier)
     {
-        if (! is_array($identifier)) {
+        if ( ! is_array($identifier)) {
             $identifier = $this->toIdentifierArray($metadata, $identifier);
         }
 
@@ -288,7 +287,7 @@ 
      */
     private function buildCollectionCacheKey(ClassMetadata $metadata, $association, $ownerIdentifier)
     {
-        if (! is_array($ownerIdentifier)) {
+        if ( ! is_array($ownerIdentifier)) {
             $ownerIdentifier = $this->toIdentifierArray($metadata, $ownerIdentifier);
         }
 

lib/Doctrine/ORM/Cache/Persister/Collection/AbstractCollectionPersister.php

Spacing +4 added lines, -4 removed lines

@@ -143,7 +143,7 @@ 
         $targetHydrator  = $targetPersister->getEntityHydrator();
 
         // Only preserve ordering if association configured it
-        if (! ($association instanceof ToManyAssociationMetadata && $association->getIndexedBy())) {
+        if ( ! ($association instanceof ToManyAssociationMetadata && $association->getIndexedBy())) {
             // Elements may be an array or a Collection
             $elements = array_values($elements instanceof Collection ? $elements->getValues() : $elements);
         }
@@ -170,7 +170,7 @@ 
 
         $cached = $this->region->put($key, $entry);
 
-        if (! $this->cacheLogger || ! $cached) {
+        if ( ! $this->cacheLogger || ! $cached) {
             return;
         }
 
@@ -263,7 +263,7 @@ 
 
         $this->region->evict($key);
 
-        if (! $this->cacheLogger) {
+        if ( ! $this->cacheLogger) {
             return;
         }
 
@@ -283,7 +283,7 @@ 
 
         $targetRegion->evict($key);
 
-        if (! $this->cacheLogger) {
+        if ( ! $this->cacheLogger) {
             return;
         }
 

lib/Doctrine/ORM/Cache/Persister/Entity/AbstractEntityPersister.php

Spacing +4 added lines, -4 removed lines

@@ -212,7 +212,7 @@ 
             $associations = [];
 
             foreach ($this->class->getDeclaredPropertiesIterator() as $association) {
-                if (! ($association instanceof ToOneAssociationMetadata) ||
+                if ( ! ($association instanceof ToOneAssociationMetadata) ||
                     ! $association->getCache() ||
                     ($association->getFetchMode() !== FetchMode::EAGER || ! $association->isOwningSide())) {
                     continue;
@@ -261,7 +261,7 @@ 
             ? $this->persister->expandCriteriaParameters($criteria)
             : $this->persister->expandParameters($criteria);
 
-        return sha1($query . serialize($params) . serialize($orderBy) . $limit . $offset);
+        return sha1($query.serialize($params).serialize($orderBy).$limit.$offset);
     }
 
     /**
@@ -559,7 +559,7 @@ 
         $hasCache  = ($persister instanceof CachedPersister);
         $key       = null;
 
-        if (! $hasCache) {
+        if ( ! $hasCache) {
             return $this->persister->loadManyToManyCollection($association, $sourceEntity, $collection);
         }
 
@@ -598,7 +598,7 @@ 
         $persister = $uow->getCollectionPersister($association);
         $hasCache  = ($persister instanceof CachedPersister);
 
-        if (! $hasCache) {
+        if ( ! $hasCache) {
             return $this->persister->loadOneToManyCollection($association, $sourceEntity, $collection);
         }
 

lib/Doctrine/ORM/UnitOfWork.php

Spacing +61 added lines, -61 removed lines

@@ -336,7 +336,7 @@ 
 
         $this->computeChangeSets();
 
-        if (! ($this->entityInsertions ||
+        if ( ! ($this->entityInsertions ||
                 $this->entityDeletions ||
                 $this->entityUpdates ||
                 $this->collectionUpdates ||
@@ -395,7 +395,7 @@ 
             // Entity deletions come last and need to be in reverse commit order
             if ($this->entityDeletions) {
                 foreach (array_reverse($commitOrder) as $committedEntityName) {
-                    if (! $this->entityDeletions) {
+                    if ( ! $this->entityDeletions) {
                         break; // just a performance optimisation
                     }
 
@@ -478,7 +478,7 @@ 
         $oid  = spl_object_id($entity);
         $data = [];
 
-        if (! isset($this->entityChangeSets[$oid])) {
+        if ( ! isset($this->entityChangeSets[$oid])) {
             return $data;
         }
 
@@ -555,18 +555,18 @@ 
                 continue;
             }
 
-            if (! (( ! $class->isIdentifier($name)
+            if ( ! (( ! $class->isIdentifier($name)
                     || ! $class->getProperty($name) instanceof FieldMetadata
                     || ! $class->getProperty($name)->hasValueGenerator()
                     || $class->getProperty($name)->getValueGenerator()->getType() !== GeneratorType::IDENTITY
-                )) || (! $class->isVersioned() || $name === $class->versionProperty->getName())) {
+                )) || ( ! $class->isVersioned() || $name === $class->versionProperty->getName())) {
                 continue;
             }
 
             $actualData[$name] = $value;
         }
 
-        if (! isset($this->originalEntityData[$oid])) {
+        if ( ! isset($this->originalEntityData[$oid])) {
             // Entity is either NEW or MANAGED but not yet fully persisted (only has an id).
             // These result in an INSERT.
             $this->originalEntityData[$oid] = $actualData;
@@ -575,7 +575,7 @@ 
             foreach ($actualData as $propName => $actualValue) {
                 $property = $class->getProperty($propName);
 
-                if (! (($property instanceof FieldMetadata)) &&
+                if ( ! (($property instanceof FieldMetadata)) &&
                     ! (($property instanceof ToOneAssociationMetadata && $property->isOwningSide()))) {
                     continue;
                 }
@@ -595,7 +595,7 @@ 
 
             foreach ($actualData as $propName => $actualValue) {
                 // skip field, its a partially omitted one!
-                if (! (isset($originalData[$propName]) || array_key_exists($propName, $originalData))) {
+                if ( ! (isset($originalData[$propName]) || array_key_exists($propName, $originalData))) {
                     continue;
                 }
 
@@ -617,7 +617,7 @@ 
                     if ($owner === null) { // cloned
                         $actualValue->setOwner($entity, $property);
                     } elseif ($owner !== $entity) { // no clone, we have to fix
-                        if (! $actualValue->isInitialized()) {
+                        if ( ! $actualValue->isInitialized()) {
                             $actualValue->initialize(); // we have to do this otherwise the cols share state
                         }
 
@@ -655,7 +655,7 @@ 
                         // Check if original value exists
                         if ($orgValue instanceof PersistentCollection) {
                             // A PersistentCollection was de-referenced, so delete it.
-                            if (! $this->isCollectionScheduledForDeletion($orgValue)) {
+                            if ( ! $this->isCollectionScheduledForDeletion($orgValue)) {
                                 $this->scheduleCollectionDeletion($orgValue);
 
                                 $changeSet[$propName] = $orgValue; // Signal changeset, to-many associations will be ignored
@@ -678,7 +678,7 @@ 
 
         // Look for changes in associations of the entity
         foreach ($class->getDeclaredPropertiesIterator() as $property) {
-            if (! $property instanceof AssociationMetadata) {
+            if ( ! $property instanceof AssociationMetadata) {
                 continue;
             }
 
@@ -690,7 +690,7 @@ 
 
             $this->computeAssociationChanges($property, $value);
 
-            if (! ($property instanceof ManyToManyAssociationMetadata) ||
+            if ( ! ($property instanceof ManyToManyAssociationMetadata) ||
                 ! ($value instanceof PersistentCollection) ||
                 isset($this->entityChangeSets[$oid]) ||
                 ! $property->isOwningSide() ||
@@ -786,13 +786,13 @@ 
         $targetClass    = $this->em->getClassMetadata($targetEntity);
 
         foreach ($unwrappedValue as $key => $entry) {
-            if (! ($entry instanceof $targetEntity)) {
+            if ( ! ($entry instanceof $targetEntity)) {
                 throw ORMInvalidArgumentException::invalidAssociation($targetClass, $association, $entry);
             }
 
             $state = $this->getEntityState($entry, self::STATE_NEW);
 
-            if (! ($entry instanceof $targetEntity)) {
+            if ( ! ($entry instanceof $targetEntity)) {
                 throw UnexpectedAssociationValue::create(
                     $association->getSourceEntity(),
                     $association->getName(),
@@ -803,7 +803,7 @@ 
 
             switch ($state) {
                 case self::STATE_NEW:
-                    if (! in_array('persist', $association->getCascade(), true)) {
+                    if ( ! in_array('persist', $association->getCascade(), true)) {
                         $this->nonCascadedNewDetectedEntities[spl_object_id($entry)] = [$association, $entry];
 
                         break;
@@ -852,7 +852,7 @@ 
         $persister      = $this->getEntityPersister($class->getClassName());
         $generationPlan->executeImmediate($this->em, $entity);
 
-        if (! $generationPlan->containsDeferred()) {
+        if ( ! $generationPlan->containsDeferred()) {
             $id                            = $this->em->getIdentifierFlattener()->flattenIdentifier($class, $persister->getIdentifier($entity));
             $this->entityIdentifiers[$oid] = $id;
         }
@@ -883,7 +883,7 @@ 
     {
         $oid = spl_object_id($entity);
 
-        if (! isset($this->entityStates[$oid]) || $this->entityStates[$oid] !== self::STATE_MANAGED) {
+        if ( ! isset($this->entityStates[$oid]) || $this->entityStates[$oid] !== self::STATE_MANAGED) {
             throw ORMInvalidArgumentException::entityNotManaged($entity);
         }
 
@@ -905,7 +905,7 @@ 
                     break;
 
                 case ($property instanceof FieldMetadata):
-                    if (! $property->isPrimaryKey()
+                    if ( ! $property->isPrimaryKey()
                         || ! $property->getValueGenerator()
                         || $property->getValueGenerator()->getType() !== GeneratorType::IDENTITY) {
                         $actualData[$name] = $property->getValue($entity);
@@ -919,7 +919,7 @@ 
             }
         }
 
-        if (! isset($this->originalEntityData[$oid])) {
+        if ( ! isset($this->originalEntityData[$oid])) {
             throw new \RuntimeException('Cannot call recomputeSingleEntityChangeSet before computeChangeSet on an entity.');
         }
 
@@ -936,13 +936,13 @@ 
             $changeSet[$propName] = [$orgValue, $actualValue];
         }
 
-        if (! $changeSet) {
+        if ( ! $changeSet) {
             return;
         }
 
         if (isset($this->entityChangeSets[$oid])) {
             $this->entityChangeSets[$oid] = array_merge($this->entityChangeSets[$oid], $changeSet);
-        } elseif (! isset($this->entityInsertions[$oid])) {
+        } elseif ( ! isset($this->entityInsertions[$oid])) {
             $this->entityChangeSets[$oid] = $changeSet;
             $this->entityUpdates[$oid]    = $entity;
         }
@@ -1013,7 +1013,7 @@ 
                 $this->recomputeSingleEntityChangeSet($class, $entity);
             }
 
-            if (! empty($this->entityChangeSets[$oid])) {
+            if ( ! empty($this->entityChangeSets[$oid])) {
 //                echo 'Update: ';
 //                \Doctrine\Common\Util\Debug::dump($this->entityChangeSets[$oid], 3);
 
@@ -1058,7 +1058,7 @@ 
             // Entity with this $oid after deletion treated as NEW, even if the $oid
             // is obtained by a new entity because the old one went out of scope.
             //$this->entityStates[$oid] = self::STATE_NEW;
-            if (! $class->isIdentifierComposite()) {
+            if ( ! $class->isIdentifierComposite()) {
                 $property = $class->getProperty($class->getSingleIdentifierFieldName());
 
                 if ($property instanceof FieldMetadata && $property->hasValueGenerator()) {
@@ -1107,13 +1107,13 @@ 
         // Calculate dependencies for new nodes
         while ($class = array_pop($newNodes)) {
             foreach ($class->getDeclaredPropertiesIterator() as $property) {
-                if (! ($property instanceof ToOneAssociationMetadata && $property->isOwningSide())) {
+                if ( ! ($property instanceof ToOneAssociationMetadata && $property->isOwningSide())) {
                     continue;
                 }
 
                 $targetClass = $this->em->getClassMetadata($property->getTargetEntity());
 
-                if (! $calc->hasNode($targetClass->getClassName())) {
+                if ( ! $calc->hasNode($targetClass->getClassName())) {
                     $calc->addNode($targetClass->getClassName(), $targetClass);
 
                     $newNodes[] = $targetClass;
@@ -1121,7 +1121,7 @@ 
 
                 $weight = ! array_filter(
                     $property->getJoinColumns(),
-                    function (JoinColumnMetadata $joinColumn) {
+                    function(JoinColumnMetadata $joinColumn) {
                         return $joinColumn->isNullable();
                     }
                 );
@@ -1129,14 +1129,14 @@ 
                 $calc->addDependency($targetClass->getClassName(), $class->getClassName(), $weight);
 
                 // If the target class has mapped subclasses, these share the same dependency.
-                if (! $targetClass->getSubClasses()) {
+                if ( ! $targetClass->getSubClasses()) {
                     continue;
                 }
 
                 foreach ($targetClass->getSubClasses() as $subClassName) {
                     $targetSubClass = $this->em->getClassMetadata($subClassName);
 
-                    if (! $calc->hasNode($subClassName)) {
+                    if ( ! $calc->hasNode($subClassName)) {
                         $calc->addNode($targetSubClass->getClassName(), $targetSubClass);
 
                         $newNodes[] = $targetSubClass;
@@ -1184,7 +1184,7 @@ 
             $this->addToIdentityMap($entity);
         }
 
-        if (! ($entity instanceof NotifyPropertyChanged)) {
+        if ( ! ($entity instanceof NotifyPropertyChanged)) {
             return;
         }
 
@@ -1214,7 +1214,7 @@ 
     {
         $oid = spl_object_id($entity);
 
-        if (! isset($this->entityIdentifiers[$oid])) {
+        if ( ! isset($this->entityIdentifiers[$oid])) {
             throw ORMInvalidArgumentException::entityHasNoIdentity($entity, 'scheduling for update');
         }
 
@@ -1300,7 +1300,7 @@ 
             return; // entity has not been persisted yet, so nothing more to do.
         }
 
-        if (! $this->isInIdentityMap($entity)) {
+        if ( ! $this->isInIdentityMap($entity)) {
             return;
         }
 
@@ -1412,7 +1412,7 @@ 
         $persister = $this->getEntityPersister($class->getClassName());
         $id        = $persister->getIdentifier($entity);
 
-        if (! $id) {
+        if ( ! $id) {
             return self::STATE_NEW;
         }
 
@@ -1772,7 +1772,7 @@ 
         $class = $this->em->getClassMetadata(get_class($entity));
 
         foreach ($class->getDeclaredPropertiesIterator() as $association) {
-            if (! ($association instanceof AssociationMetadata && in_array('refresh', $association->getCascade(), true))) {
+            if ( ! ($association instanceof AssociationMetadata && in_array('refresh', $association->getCascade(), true))) {
                 continue;
             }
 
@@ -1819,7 +1819,7 @@ 
         }
 
         foreach ($class->getDeclaredPropertiesIterator() as $association) {
-            if (! ($association instanceof AssociationMetadata && in_array('persist', $association->getCascade(), true))) {
+            if ( ! ($association instanceof AssociationMetadata && in_array('persist', $association->getCascade(), true))) {
                 continue;
             }
 
@@ -1835,7 +1835,7 @@ 
 
                 case ($relatedEntities instanceof Collection):
                 case (is_array($relatedEntities)):
-                    if (! ($association instanceof ToManyAssociationMetadata)) {
+                    if ( ! ($association instanceof ToManyAssociationMetadata)) {
                         throw ORMInvalidArgumentException::invalidAssociation(
                             $this->em->getClassMetadata($targetEntity),
                             $association,
@@ -1850,7 +1850,7 @@ 
                     break;
 
                 case ($relatedEntities !== null):
-                    if (! $relatedEntities instanceof $targetEntity) {
+                    if ( ! $relatedEntities instanceof $targetEntity) {
                         throw ORMInvalidArgumentException::invalidAssociation(
                             $this->em->getClassMetadata($targetEntity),
                             $association,
@@ -1879,7 +1879,7 @@ 
         $class             = $this->em->getClassMetadata(get_class($entity));
 
         foreach ($class->getDeclaredPropertiesIterator() as $association) {
-            if (! ($association instanceof AssociationMetadata && in_array('remove', $association->getCascade(), true))) {
+            if ( ! ($association instanceof AssociationMetadata && in_array('remove', $association->getCascade(), true))) {
                 continue;
             }
 
@@ -1938,7 +1938,7 @@ 
 
         switch (true) {
             case $lockMode === LockMode::OPTIMISTIC:
-                if (! $class->isVersioned()) {
+                if ( ! $class->isVersioned()) {
                     throw OptimisticLockException::notVersioned($class->getClassName());
                 }
 
@@ -1961,7 +1961,7 @@ 
             case $lockMode === LockMode::NONE:
             case $lockMode === LockMode::PESSIMISTIC_READ:
             case $lockMode === LockMode::PESSIMISTIC_WRITE:
-                if (! $this->em->getConnection()->isTransactionActive()) {
+                if ( ! $this->em->getConnection()->isTransactionActive()) {
                     throw TransactionRequiredException::transactionRequired();
                 }
 
@@ -2120,7 +2120,7 @@ 
                     $entity->addPropertyChangedListener($this);
                 }
             } else {
-                if (! isset($hints[Query::HINT_REFRESH])
+                if ( ! isset($hints[Query::HINT_REFRESH])
                     || (isset($hints[Query::HINT_REFRESH_ENTITY]) && $hints[Query::HINT_REFRESH_ENTITY] !== $entity)) {
                     return $entity;
                 }
@@ -2150,7 +2150,7 @@ 
         foreach ($data as $field => $value) {
             $property = $class->getProperty($field);
 
-            if (! ($property instanceof FieldMetadata)) {
+            if ( ! ($property instanceof FieldMetadata)) {
                 continue;
             }
 
@@ -2170,7 +2170,7 @@ 
         }
 
         foreach ($class->getDeclaredPropertiesIterator() as $field => $association) {
-            if (! ($association instanceof AssociationMetadata)) {
+            if ( ! ($association instanceof AssociationMetadata)) {
                 continue;
             }
 
@@ -2219,7 +2219,7 @@ 
                 continue;
             }
 
-            if (! $association->isOwningSide()) {
+            if ( ! $association->isOwningSide()) {
                 // use the given entity association
                 if (isset($data[$field]) && is_object($data[$field]) &&
                     isset($this->entityStates[spl_object_id($data[$field])])) {
@@ -2269,7 +2269,7 @@ 
                 $associatedId[$targetField] = $joinColumnValue;
             }
 
-            if (! $associatedId) {
+            if ( ! $associatedId) {
                 // Foreign key is NULL
                 $association->setValue($entity, null);
                 $this->originalEntityData[$oid][$field] = null;
@@ -2278,7 +2278,7 @@ 
             }
 
             // @todo guilhermeblanco Can we remove the need of this somehow?
-            if (! isset($hints['fetchMode'][$class->getClassName()][$field])) {
+            if ( ! isset($hints['fetchMode'][$class->getClassName()][$field])) {
                 $hints['fetchMode'][$class->getClassName()][$field] = $association->getFetchMode();
             }
 
@@ -2295,7 +2295,7 @@ 
                     // If this is an uninitialized proxy, we are deferring eager loads,
                     // this association is marked as eager fetch, and its an uninitialized proxy (wtf!)
                     // then we can append this entity for eager loading!
-                    if (! $targetClass->isIdentifierComposite() &&
+                    if ( ! $targetClass->isIdentifierComposite() &&
                         $newValue instanceof GhostObjectInterface &&
                         isset($hints[self::HINT_DEFEREAGERLOAD]) &&
                         $hints['fetchMode'][$class->getClassName()][$field] === FetchMode::EAGER &&
@@ -2355,7 +2355,7 @@ 
             $this->originalEntityData[$oid][$field] = $newValue;
             $association->setValue($entity, $newValue);
 
-            if (! $association->getInversedBy()
+            if ( ! $association->getInversedBy()
                 || ! ($association instanceof OneToOneAssociationMetadata)
                 // @TODO refactor this
                 // we don't want to set any values in un-initialized proxies
@@ -2380,7 +2380,7 @@ 
 
     public function triggerEagerLoads()
     {
-        if (! $this->eagerLoadingEntities) {
+        if ( ! $this->eagerLoadingEntities) {
             return;
         }
 
@@ -2389,7 +2389,7 @@ 
         $this->eagerLoadingEntities = [];
 
         foreach ($eagerLoadingEntities as $entityName => $ids) {
-            if (! $ids) {
+            if ( ! $ids) {
                 continue;
             }
 
@@ -2657,7 +2657,7 @@ 
 
         $this->addToIdentityMap($entity);
 
-        if (! ($entity instanceof NotifyPropertyChanged) || $isProxy) {
+        if ( ! ($entity instanceof NotifyPropertyChanged) || $isProxy) {
             return;
         }
 
@@ -2689,7 +2689,7 @@ 
     {
         $class = $this->em->getClassMetadata(get_class($entity));
 
-        if (! $class->getProperty($propertyName)) {
+        if ( ! $class->getProperty($propertyName)) {
             return; // ignore non-persistent fields
         }
 
@@ -2768,7 +2768,7 @@ 
             return;
         }
 
-        if (! ($obj instanceof PersistentCollection)) {
+        if ( ! ($obj instanceof PersistentCollection)) {
             return;
         }
 
@@ -2784,7 +2784,7 @@ 
      */
     private static function objToStr($obj)
     {
-        return method_exists($obj, '__toString') ? (string) $obj : get_class($obj) . '@' . spl_object_id($obj);
+        return method_exists($obj, '__toString') ? (string) $obj : get_class($obj).'@'.spl_object_id($obj);
     }
 
     /**
@@ -2799,7 +2799,7 @@ 
      */
     public function markReadOnly($object)
     {
-        if (! is_object($object) || ! $this->isInIdentityMap($object)) {
+        if ( ! is_object($object) || ! $this->isInIdentityMap($object)) {
             throw ORMInvalidArgumentException::readOnlyRequiresManagedEntity($object);
         }
 
@@ -2817,7 +2817,7 @@ 
      */
     public function isReadOnly($object)
     {
-        if (! is_object($object)) {
+        if ( ! is_object($object)) {
             throw ORMInvalidArgumentException::readOnlyRequiresManagedEntity($object);
         }
 
@@ -2829,7 +2829,7 @@ 
      */
     private function afterTransactionComplete()
     {
-        $this->performCallbackOnCachedPersister(function (CachedPersister $persister) {
+        $this->performCallbackOnCachedPersister(function(CachedPersister $persister) {
             $persister->afterTransactionComplete();
         });
     }
@@ -2839,7 +2839,7 @@ 
      */
     private function afterTransactionRolledBack()
     {
-        $this->performCallbackOnCachedPersister(function (CachedPersister $persister) {
+        $this->performCallbackOnCachedPersister(function(CachedPersister $persister) {
             $persister->afterTransactionRolledBack();
         });
     }
@@ -2849,12 +2849,12 @@ 
      */
     private function performCallbackOnCachedPersister(callable $callback)
     {
-        if (! $this->hasCache) {
+        if ( ! $this->hasCache) {
             return;
         }
 
         foreach (array_merge($this->entityPersisters, $this->collectionPersisters) as $persister) {
-            if (! ($persister instanceof CachedPersister)) {
+            if ( ! ($persister instanceof CachedPersister)) {
                 continue;
             }
 
@@ -2864,7 +2864,7 @@ 
 
     private function dispatchOnFlushEvent()
     {
-        if (! $this->eventManager->hasListeners(Events::onFlush)) {
+        if ( ! $this->eventManager->hasListeners(Events::onFlush)) {
             return;
         }
 
@@ -2873,7 +2873,7 @@ 
 
     private function dispatchPostFlushEvent()
     {
-        if (! $this->eventManager->hasListeners(Events::postFlush)) {
+        if ( ! $this->eventManager->hasListeners(Events::postFlush)) {
             return;
         }
 

lib/Doctrine/ORM/Event/ListenersInvoker.php

Spacing +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
             }
         }
 
-        if (! ($invoke & self::INVOKE_MANAGER)) {
+        if ( ! ($invoke & self::INVOKE_MANAGER)) {
             return;
         }
 

lib/Doctrine/ORM/QueryBuilder.php

Spacing +16 added lines, -16 removed lines

@@ -414,7 +414,7 @@ 
     {
         $aliases = $this->getRootAliases();
 
-        if (! isset($aliases[0])) {
+        if ( ! isset($aliases[0])) {
             throw new \RuntimeException('No alias was set before invoking getRootAlias().');
         }
 
@@ -597,7 +597,7 @@ 
     public function getParameter($key)
     {
         $filteredParameters = $this->parameters->filter(
-            function (Query\Parameter $parameter) use ($key) : bool {
+            function(Query\Parameter $parameter) use ($key) : bool {
                 $parameterName = $parameter->getName();
 
                 return $key === $parameterName || (string) $key === (string) $parameterName;
@@ -673,7 +673,7 @@ 
     {
         if ($append && ($dqlPartName === 'where' || $dqlPartName === 'having')) {
             throw new \InvalidArgumentException(
-                "Using \$append = true does not have an effect with 'where' or 'having' " .
+                "Using \$append = true does not have an effect with 'where' or 'having' ".
                 'parts. See QueryBuilder#andWhere() for an example for correct usage.'
             );
         }
@@ -814,7 +814,7 @@ 
     {
         $this->type = self::DELETE;
 
-        if (! $delete) {
+        if ( ! $delete) {
             return $this;
         }
 
@@ -841,7 +841,7 @@ 
     {
         $this->type = self::UPDATE;
 
-        if (! $update) {
+        if ( ! $update) {
             return $this;
         }
 
@@ -896,7 +896,7 @@ 
     {
         $rootAliases = $this->getRootAliases();
 
-        if (! in_array($alias, $rootAliases, true)) {
+        if ( ! in_array($alias, $rootAliases, true)) {
             throw new Query\QueryException(
                 sprintf('Specified root alias %s must be set before invoking indexBy().', $alias)
             );
@@ -1057,7 +1057,7 @@ 
      */
     public function where($predicates)
     {
-        if (! (func_num_args() === 1 && $predicates instanceof Expr\Composite)) {
+        if ( ! (func_num_args() === 1 && $predicates instanceof Expr\Composite)) {
             $predicates = new Expr\Andx(func_get_args());
         }
 
@@ -1176,7 +1176,7 @@ 
      */
     public function having($having)
     {
-        if (! (func_num_args() === 1 && ($having instanceof Expr\Andx || $having instanceof Expr\Orx))) {
+        if ( ! (func_num_args() === 1 && ($having instanceof Expr\Andx || $having instanceof Expr\Orx))) {
             $having = new Expr\Andx(func_get_args());
         }
 
@@ -1274,7 +1274,7 @@ 
     public function addCriteria(Criteria $criteria)
     {
         $allAliases = $this->getAllAliases();
-        if (! isset($allAliases[0])) {
+        if ( ! isset($allAliases[0])) {
             throw new Query\QueryException('No aliases are set before invoking addCriteria().');
         }
 
@@ -1292,14 +1292,14 @@ 
             foreach ($criteria->getOrderings() as $sort => $order) {
                 $hasValidAlias = false;
                 foreach ($allAliases as $alias) {
-                    if (strpos($sort . '.', $alias . '.') === 0) {
+                    if (strpos($sort.'.', $alias.'.') === 0) {
                         $hasValidAlias = true;
                         break;
                     }
                 }
 
-                if (! $hasValidAlias) {
-                    $sort = $allAliases[0] . '.' . $sort;
+                if ( ! $hasValidAlias) {
+                    $sort = $allAliases[0].'.'.$sort;
                 }
 
                 $this->addOrderBy($sort, $order);
@@ -1373,7 +1373,7 @@ 
     private function getDQLForSelect()
     {
         $dql = 'SELECT'
-             . ($this->dqlParts['distinct']===true ? ' DISTINCT' : '')
+             . ($this->dqlParts['distinct'] === true ? ' DISTINCT' : '')
              . $this->getReducedDQLQueryPart('select', ['pre' => ' ', 'separator' => ', ']);
 
         $fromParts   = $this->getDQLPart('from');
@@ -1381,7 +1381,7 @@ 
         $fromClauses = [];
 
         // Loop through all FROM clauses
-        if (! empty($fromParts)) {
+        if ( ! empty($fromParts)) {
             $dql .= ' FROM ';
 
             foreach ($fromParts as $from) {
@@ -1389,7 +1389,7 @@ 
 
                 if ($from instanceof Expr\From && isset($joinParts[$from->getAlias()])) {
                     foreach ($joinParts[$from->getAlias()] as $join) {
-                        $fromClause .= ' ' . ((string) $join);
+                        $fromClause .= ' '.((string) $join);
                     }
                 }
 
@@ -1478,7 +1478,7 @@ 
         foreach ($this->dqlParts as $part => $elements) {
             if (is_array($this->dqlParts[$part])) {
                 foreach ($this->dqlParts[$part] as $idx => $element) {
-                    if (! is_object($element)) {
+                    if ( ! is_object($element)) {
                         continue;
                     }
 

lib/Doctrine/ORM/Query/Exec/MultiTableUpdateExecutor.php

Spacing +8 added lines, -8 removed lines

@@ -67,8 +67,8 @@ 
         // 1. Create an INSERT INTO temptable ... SELECT identifiers WHERE $AST->getWhereClause()
         $sqlWalker->setSQLTableAlias($primaryClass->getTableName(), 'i0', $updateClause->aliasIdentificationVariable);
 
-        $this->insertSql = 'INSERT INTO ' . $tempTable . ' (' . $idColumnNameList . ')'
-                . ' SELECT i0.' . implode(', i0.', array_keys($idColumns));
+        $this->insertSql = 'INSERT INTO '.$tempTable.' ('.$idColumnNameList.')'
+                . ' SELECT i0.'.implode(', i0.', array_keys($idColumns));
 
         $rangeDecl  = new AST\RangeVariableDeclaration($primaryClass->getClassName(), $updateClause->aliasIdentificationVariable);
         $fromClause = new AST\FromClause([new AST\IdentificationVariableDeclaration($rangeDecl, null, [])]);
@@ -86,7 +86,7 @@ 
         // 3. Create and store UPDATE statements
         $hierarchyClasses = array_merge(
             array_map(
-                function ($className) use ($em) {
+                function($className) use ($em) {
                     return $em->getClassMetadata($className);
                 },
                 array_reverse($primaryClass->getSubClasses())
@@ -104,14 +104,14 @@ 
                 $field    = $updateItem->pathExpression->field;
                 $property = $class->getProperty($field);
 
-                if (! $property || $class->isInheritedProperty($field)) {
+                if ( ! $property || $class->isInheritedProperty($field)) {
                     continue;
                 }
 
                 $updateSQLParts[] = $sqlWalker->walkUpdateItem($updateItem);
                 $newValue         = $updateItem->newValue;
 
-                if (! ($newValue instanceof AST\InputParameter)) {
+                if ( ! ($newValue instanceof AST\InputParameter)) {
                     continue;
                 }
 
@@ -120,7 +120,7 @@ 
                 ++$this->numParametersInUpdateClause;
             }
 
-            if (! $updateSQLParts) {
+            if ( ! $updateSQLParts) {
                 continue;
             }
 
@@ -148,8 +148,8 @@ 
             ];
         }
 
-        $this->createTempTableSql = $platform->getCreateTemporaryTableSnippetSQL() . ' ' . $tempTable . ' ('
-                . $platform->getColumnDeclarationListSQL($columnDefinitions) . ')';
+        $this->createTempTableSql = $platform->getCreateTemporaryTableSnippetSQL().' '.$tempTable.' ('
+                . $platform->getColumnDeclarationListSQL($columnDefinitions).')';
 
         $this->dropTempTableSql = $platform->getDropTemporaryTableSQL($tempTable);
     }