doctrine / orm

lib/Doctrine/ORM/Tools/SchemaValidator.php

Spacing +12 added lines, -12 removed lines

@@ -79,7 +79,7 @@ 
         $ce = [];
 
         foreach ($class->getDeclaredPropertiesIterator() as $fieldName => $association) {
-            if (! ($association instanceof AssociationMetadata)) {
+            if ( ! ($association instanceof AssociationMetadata)) {
                 continue;
             }
 
@@ -87,7 +87,7 @@ 
         }
 
         foreach ($class->getSubClasses() as $subClass) {
-            if (! in_array($class->getClassName(), class_parents($subClass), true)) {
+            if ( ! in_array($class->getClassName(), class_parents($subClass), true)) {
                 $message = "According to the discriminator map class, '%s' has to be a child of '%s', but these entities are not related through inheritance.";
 
                 $ce[] = sprintf($message, $subClass, $class->getClassName());
@@ -106,7 +106,7 @@ 
         $fieldName       = $association->getName();
         $targetEntity    = $association->getTargetEntity();
 
-        if (! class_exists($targetEntity) || $metadataFactory->isTransient($targetEntity)) {
+        if ( ! class_exists($targetEntity) || $metadataFactory->isTransient($targetEntity)) {
             $message = "The target entity '%s' specified on %s#%s is unknown or not an entity.";
 
             return [sprintf($message, $targetEntity, $class->getClassName(), $fieldName)];
@@ -125,7 +125,7 @@ 
 
         /** @var ClassMetadata $targetMetadata */
         $targetMetadata    = $metadataFactory->getMetadataFor($targetEntity);
-        $containsForeignId = array_filter($targetMetadata->identifier, static function ($identifier) use ($targetMetadata) {
+        $containsForeignId = array_filter($targetMetadata->identifier, static function($identifier) use ($targetMetadata) {
             $targetProperty = $targetMetadata->getProperty($identifier);
 
             return $targetProperty instanceof AssociationMetadata;
@@ -141,7 +141,7 @@ 
             /** @var AssociationMetadata $targetAssociation */
             $targetAssociation = $targetMetadata->getProperty($mappedBy);
 
-            if (! $targetAssociation) {
+            if ( ! $targetAssociation) {
                 $message = 'The association %s#%s refers to the owning side property %s#%s which does not exist.';
 
                 $ce[] = sprintf($message, $class->getClassName(), $fieldName, $targetEntity, $mappedBy);
@@ -165,7 +165,7 @@ 
             /** @var AssociationMetadata $targetAssociation */
             $targetAssociation = $targetMetadata->getProperty($inversedBy);
 
-            if (! $targetAssociation) {
+            if ( ! $targetAssociation) {
                 $message = 'The association %s#%s refers to the inverse side property %s#%s which does not exist.';
 
                 $ce[] = sprintf($message, $class->getClassName(), $fieldName, $targetEntity, $inversedBy);
@@ -209,7 +209,7 @@ 
                 $joinTable               = $association->getJoinTable();
 
                 foreach ($joinTable->getJoinColumns() as $joinColumn) {
-                    if (! in_array($joinColumn->getReferencedColumnName(), $classIdentifierColumns, true)) {
+                    if ( ! in_array($joinColumn->getReferencedColumnName(), $classIdentifierColumns, true)) {
                         $message = "The referenced column name '%s' has to be a primary key column on the target entity class '%s'.";
 
                         $ce[] = sprintf($message, $joinColumn->getReferencedColumnName(), $class->getClassName());
@@ -218,7 +218,7 @@ 
                 }
 
                 foreach ($joinTable->getInverseJoinColumns() as $inverseJoinColumn) {
-                    if (! in_array($inverseJoinColumn->getReferencedColumnName(), $targetIdentifierColumns, true)) {
+                    if ( ! in_array($inverseJoinColumn->getReferencedColumnName(), $targetIdentifierColumns, true)) {
                         $message = "The referenced column name '%s' has to be a primary key column on the target entity class '%s'.";
 
                         $ce[] = sprintf($message, $inverseJoinColumn->getReferencedColumnName(), $targetMetadata->getClassName());
@@ -228,7 +228,7 @@ 
 
                 if (count($targetIdentifierColumns) !== count($joinTable->getInverseJoinColumns())) {
                     $columnNames = array_map(
-                        static function (JoinColumnMetadata $joinColumn) {
+                        static function(JoinColumnMetadata $joinColumn) {
                             return $joinColumn->getReferencedColumnName();
                         },
                         $joinTable->getInverseJoinColumns()
@@ -243,7 +243,7 @@ 
 
                 if (count($classIdentifierColumns) !== count($joinTable->getJoinColumns())) {
                     $columnNames = array_map(
-                        static function (JoinColumnMetadata $joinColumn) {
+                        static function(JoinColumnMetadata $joinColumn) {
                             return $joinColumn->getReferencedColumnName();
                         },
                         $joinTable->getJoinColumns()
@@ -260,7 +260,7 @@ 
                 $joinColumns       = $association->getJoinColumns();
 
                 foreach ($joinColumns as $joinColumn) {
-                    if (! in_array($joinColumn->getReferencedColumnName(), $identifierColumns, true)) {
+                    if ( ! in_array($joinColumn->getReferencedColumnName(), $identifierColumns, true)) {
                         $message = "The referenced column name '%s' has to be a primary key column on the target entity class '%s'.";
 
                         $ce[] = sprintf($message, $joinColumn->getReferencedColumnName(), $targetMetadata->getClassName());
@@ -291,7 +291,7 @@ 
                     continue;
                 }
 
-                if (! ($targetProperty instanceof AssociationMetadata)) {
+                if ( ! ($targetProperty instanceof AssociationMetadata)) {
                     $message = "The association %s#%s is ordered by a property '%s' that is non-existing field on the target entity '%s'.";
 
                     $ce[] = sprintf($message, $class->getClassName(), $fieldName, $orderField, $targetMetadata->getClassName());

lib/Doctrine/ORM/UnitOfWork.php

Spacing +53 added lines, -53 removed lines

@@ -340,7 +340,7 @@ 
 
         $this->computeChangeSets();
 
-        if (! ($this->entityInsertions ||
+        if ( ! ($this->entityInsertions ||
                 $this->entityDeletions ||
                 $this->entityUpdates ||
                 $this->collectionUpdates ||
@@ -399,7 +399,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
                     }
 
@@ -482,7 +482,7 @@ 
         $oid  = spl_object_id($entity);
         $data = [];
 
-        if (! isset($this->entityChangeSets[$oid])) {
+        if ( ! isset($this->entityChangeSets[$oid])) {
             return $data;
         }
 
@@ -562,12 +562,12 @@ 
                     || ! $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())) {
                 $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;
@@ -594,7 +594,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;
                 }
 
@@ -616,7 +616,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
                         }
 
@@ -654,7 +654,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
@@ -677,7 +677,7 @@ 
 
         // Look for changes in associations of the entity
         foreach ($class->getDeclaredPropertiesIterator() as $property) {
-            if (! $property instanceof AssociationMetadata) {
+            if ( ! $property instanceof AssociationMetadata) {
                 continue;
             }
 
@@ -744,7 +744,7 @@ 
                 // Only MANAGED entities that are NOT SCHEDULED FOR INSERTION OR DELETION are processed here.
                 $oid = spl_object_id($entity);
 
-                if (! isset($this->entityInsertions[$oid]) && ! isset($this->entityDeletions[$oid]) && isset($this->entityStates[$oid])) {
+                if ( ! isset($this->entityInsertions[$oid]) && ! isset($this->entityDeletions[$oid]) && isset($this->entityStates[$oid])) {
                     $this->computeChangeSet($class, $entity);
                 }
             }
@@ -781,13 +781,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(),
@@ -798,7 +798,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;
@@ -847,7 +847,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;
         }
@@ -878,7 +878,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);
         }
 
@@ -900,7 +900,7 @@ 
                     break;
 
                 case $property instanceof FieldMetadata:
-                    if (! $property->isPrimaryKey()
+                    if ( ! $property->isPrimaryKey()
                         || ! $property->getValueGenerator()
                         || $property->getValueGenerator()->getType() !== GeneratorType::IDENTITY) {
                         $actualData[$name] = $property->getValue($entity);
@@ -914,7 +914,7 @@ 
             }
         }
 
-        if (! isset($this->originalEntityData[$oid])) {
+        if ( ! isset($this->originalEntityData[$oid])) {
             throw new RuntimeException('Cannot call recomputeSingleEntityChangeSet before computeChangeSet on an entity.');
         }
 
@@ -932,7 +932,7 @@ 
         if ($changeSet) {
             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;
             }
@@ -1002,7 +1002,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);
 
@@ -1045,7 +1045,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()) {
@@ -1092,13 +1092,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;
@@ -1106,7 +1106,7 @@ 
 
                 $weight = ! array_filter(
                     $property->getJoinColumns(),
-                    static function (JoinColumnMetadata $joinColumn) {
+                    static function(JoinColumnMetadata $joinColumn) {
                         return $joinColumn->isNullable();
                     }
                 );
@@ -1114,14 +1114,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;
@@ -1197,7 +1197,7 @@ 
     {
         $oid = spl_object_id($entity);
 
-        if (! isset($this->entityIdentifiers[$oid])) {
+        if ( ! isset($this->entityIdentifiers[$oid])) {
             throw ORMInvalidArgumentException::entityHasNoIdentity($entity, 'scheduling for update');
         }
 
@@ -1205,7 +1205,7 @@ 
             throw ORMInvalidArgumentException::entityIsRemoved($entity, 'schedule for update');
         }
 
-        if (! isset($this->entityUpdates[$oid]) && ! isset($this->entityInsertions[$oid])) {
+        if ( ! isset($this->entityUpdates[$oid]) && ! isset($this->entityInsertions[$oid])) {
             $this->entityUpdates[$oid] = $entity;
         }
     }
@@ -1280,7 +1280,7 @@ 
             return; // entity has not been persisted yet, so nothing more to do.
         }
 
-        if (! $this->isInIdentityMap($entity)) {
+        if ( ! $this->isInIdentityMap($entity)) {
             return;
         }
 
@@ -1288,7 +1288,7 @@ 
 
         unset($this->entityUpdates[$oid]);
 
-        if (! isset($this->entityDeletions[$oid])) {
+        if ( ! isset($this->entityDeletions[$oid])) {
             $this->entityDeletions[$oid] = $entity;
             $this->entityStates[$oid]    = self::STATE_REMOVED;
         }
@@ -1390,7 +1390,7 @@ 
         $persister = $this->getEntityPersister($class->getClassName());
         $id        = $persister->getIdentifier($entity);
 
-        if (! $id) {
+        if ( ! $id) {
             return self::STATE_NEW;
         }
 
@@ -1750,7 +1750,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;
             }
 
@@ -1797,7 +1797,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;
             }
 
@@ -1813,7 +1813,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,
@@ -1828,7 +1828,7 @@ 
                     break;
 
                 case $relatedEntities !== null:
-                    if (! $relatedEntities instanceof $targetEntity) {
+                    if ( ! $relatedEntities instanceof $targetEntity) {
                         throw ORMInvalidArgumentException::invalidAssociation(
                             $this->em->getClassMetadata($targetEntity),
                             $association,
@@ -1857,7 +1857,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;
             }
 
@@ -1916,7 +1916,7 @@ 
 
         switch (true) {
             case $lockMode === LockMode::OPTIMISTIC:
-                if (! $class->isVersioned()) {
+                if ( ! $class->isVersioned()) {
                     throw OptimisticLockException::notVersioned($class->getClassName());
                 }
 
@@ -1939,7 +1939,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();
                 }
 
@@ -2097,7 +2097,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;
                 }
@@ -2145,7 +2145,7 @@ 
         }
 
         foreach ($class->getDeclaredPropertiesIterator() as $field => $association) {
-            if (! ($association instanceof AssociationMetadata)) {
+            if ( ! ($association instanceof AssociationMetadata)) {
                 continue;
             }
 
@@ -2194,7 +2194,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])])) {
@@ -2244,7 +2244,7 @@ 
                 $associatedId[$targetField] = $joinColumnValue;
             }
 
-            if (! $associatedId) {
+            if ( ! $associatedId) {
                 // Foreign key is NULL
                 $association->setValue($entity, null);
                 $this->originalEntityData[$oid][$field] = null;
@@ -2253,7 +2253,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();
             }
 
@@ -2270,7 +2270,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 &&
@@ -2353,7 +2353,7 @@ 
 
     public function triggerEagerLoads()
     {
-        if (! $this->eagerLoadingEntities) {
+        if ( ! $this->eagerLoadingEntities) {
             return;
         }
 
@@ -2362,7 +2362,7 @@ 
         $this->eagerLoadingEntities = [];
 
         foreach ($eagerLoadingEntities as $entityName => $ids) {
-            if (! $ids) {
+            if ( ! $ids) {
                 continue;
             }
 
@@ -2660,7 +2660,7 @@ 
     {
         $class = $this->em->getClassMetadata(get_class($entity));
 
-        if (! $class->getProperty($propertyName)) {
+        if ( ! $class->getProperty($propertyName)) {
             return; // ignore non-persistent fields
         }
 
@@ -2669,7 +2669,7 @@ 
         // Update changeset and mark entity for synchronization
         $this->entityChangeSets[$oid][$propertyName] = [$oldValue, $newValue];
 
-        if (! isset($this->scheduledForSynchronization[$class->getRootClassName()][$oid])) {
+        if ( ! isset($this->scheduledForSynchronization[$class->getRootClassName()][$oid])) {
             $this->scheduleForSynchronization($entity);
         }
     }
@@ -2751,7 +2751,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);
     }
 
     /**
@@ -2766,7 +2766,7 @@ 
      */
     public function markReadOnly($object)
     {
-        if (! is_object($object) || ! $this->isInIdentityMap($object)) {
+        if ( ! is_object($object) || ! $this->isInIdentityMap($object)) {
             throw ORMInvalidArgumentException::readOnlyRequiresManagedEntity($object);
         }
 
@@ -2784,7 +2784,7 @@ 
      */
     public function isReadOnly($object)
     {
-        if (! is_object($object)) {
+        if ( ! is_object($object)) {
             throw ORMInvalidArgumentException::readOnlyRequiresManagedEntity($object);
         }
 
@@ -2796,7 +2796,7 @@ 
      */
     private function afterTransactionComplete()
     {
-        $this->performCallbackOnCachedPersister(static function (CachedPersister $persister) {
+        $this->performCallbackOnCachedPersister(static function(CachedPersister $persister) {
             $persister->afterTransactionComplete();
         });
     }
@@ -2806,7 +2806,7 @@ 
      */
     private function afterTransactionRolledBack()
     {
-        $this->performCallbackOnCachedPersister(static function (CachedPersister $persister) {
+        $this->performCallbackOnCachedPersister(static function(CachedPersister $persister) {
             $persister->afterTransactionRolledBack();
         });
     }
@@ -2816,7 +2816,7 @@ 
      */
     private function performCallbackOnCachedPersister(callable $callback)
     {
-        if (! $this->hasCache) {
+        if ( ! $this->hasCache) {
             return;
         }
 

lib/Doctrine/ORM/PersistentObject.php

Spacing +16 added lines, -16 removed lines

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

lib/Doctrine/ORM/OptimisticLockException.php

Spacing +2 added lines, -2 removed lines

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

lib/Doctrine/ORM/Configuration.php

Spacing +4 added lines, -4 removed lines

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

lib/Doctrine/ORM/Cache/DefaultCacheFactory.php

Spacing +5 added lines, -5 removed lines

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

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

Spacing +2 added lines, -2 removed lines

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

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

Spacing +7 added lines, -7 removed lines

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

lib/Doctrine/ORM/Cache/DefaultQueryCache.php

Spacing +15 added lines, -15 removed lines

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