doctrine / orm

tests/Doctrine/Tests/ORM/Functional/Ticket/DDC3644Test.php

Spacing +2 added lines, -2 removed lines

@@ -156,7 +156,7 @@ 
     {
         $this->addresses = $addresses;
 
-        $addresses->map(function ($address) {
+        $addresses->map(function($address) {
             $address->user = $this;
         });
     }
@@ -165,7 +165,7 @@ 
     {
         $this->pets = $pets;
 
-        $pets->map(function ($pet) {
+        $pets->map(function($pet) {
             $pet->owner = $this;
         });
     }

lib/Doctrine/ORM/Mapping/Builder/ToOneAssociationMetadataBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
                 );
             }
 
-            if (! $associationMetadata->isOwningSide()) {
+            if ( ! $associationMetadata->isOwningSide()) {
                 throw Mapping\MappingException::illegalInverseIdentifierAssociation(
                     $this->componentMetadata->getClassName(),
                     $this->fieldName
@@ -100,7 +100,7 @@ 
             // @todo guilhermeblanco The below block of code modifies component metadata properties, and it should be moved
             //                       to the component metadata builder instead of here.
 
-            if (! in_array($this->fieldName, $this->componentMetadata->identifier, true)) {
+            if ( ! in_array($this->fieldName, $this->componentMetadata->identifier, true)) {
                 $this->componentMetadata->identifier[] = $this->fieldName;
             }
 

lib/Doctrine/ORM/Mapping/Builder/OneToManyAssociationMetadataBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         $associationMetadata->setOwningSide(false);
         $associationMetadata->setMappedBy($this->oneToManyAnnotation->mappedBy);
 
-        if (! empty($this->oneToManyAnnotation->indexBy)) {
+        if ( ! empty($this->oneToManyAnnotation->indexBy)) {
             $associationMetadata->setIndexedBy($this->oneToManyAnnotation->indexBy);
         }
 

lib/Doctrine/ORM/Mapping/Builder/ManyToManyAssociationMetadataBuilder.php

Spacing +4 added lines, -4 removed lines

@@ -86,16 +86,16 @@ 
         $associationMetadata->setFetchMode($this->getFetchMode($this->manyToManyAnnotation->fetch));
         $associationMetadata->setOwningSide(true);
 
-        if (! empty($this->manyToManyAnnotation->mappedBy)) {
+        if ( ! empty($this->manyToManyAnnotation->mappedBy)) {
             $associationMetadata->setMappedBy($this->manyToManyAnnotation->mappedBy);
             $associationMetadata->setOwningSide(false);
         }
 
-        if (! empty($this->manyToManyAnnotation->inversedBy)) {
+        if ( ! empty($this->manyToManyAnnotation->inversedBy)) {
             $associationMetadata->setInversedBy($this->manyToManyAnnotation->inversedBy);
         }
 
-        if (! empty($this->manyToManyAnnotation->indexBy)) {
+        if ( ! empty($this->manyToManyAnnotation->indexBy)) {
             $associationMetadata->setIndexedBy($this->manyToManyAnnotation->indexBy);
         }
 
@@ -113,7 +113,7 @@ 
         $this->buildCache($associationMetadata);
 
         // Check for owning side to consider join column
-        if (! $associationMetadata->isOwningSide()) {
+        if ( ! $associationMetadata->isOwningSide()) {
             return $associationMetadata;
         }
 

lib/Doctrine/ORM/Mapping/Builder/AssociationMetadataBuilder.php

Spacing +3 added lines, -3 removed lines

@@ -90,12 +90,12 @@ 
     protected function getTargetEntity(string $targetEntity) : string
     {
         // Validate if target entity is defined
-        if (! $targetEntity) {
+        if ( ! $targetEntity) {
             throw Mapping\MappingException::missingTargetEntity($this->fieldName);
         }
 
         // Validate that target entity exists
-        if (! (class_exists($targetEntity) || interface_exists($targetEntity))) {
+        if ( ! (class_exists($targetEntity) || interface_exists($targetEntity))) {
             throw Mapping\MappingException::invalidTargetEntityClass(
                 $targetEntity,
                 $this->componentMetadata->getClassName(),
@@ -150,7 +150,7 @@ 
     {
         $fetchModeConstant = sprintf('%s::%s', Mapping\FetchMode::class, $fetchMode);
 
-        if (! defined($fetchModeConstant)) {
+        if ( ! defined($fetchModeConstant)) {
             throw Mapping\MappingException::invalidFetchMode($this->componentMetadata->getClassName(), $fetchMode);
         }
 

lib/Doctrine/ORM/Mapping/Builder/ManyToOneAssociationMetadataBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         $associationMetadata->setFetchMode($this->getFetchMode($this->manyToOneAnnotation->fetch));
         $associationMetadata->setOwningSide(true);
 
-        if (! empty($this->manyToOneAnnotation->inversedBy)) {
+        if ( ! empty($this->manyToOneAnnotation->inversedBy)) {
             $associationMetadata->setInversedBy($this->manyToOneAnnotation->inversedBy);
         }
 

lib/Doctrine/ORM/Mapping/Builder/OneToOneAssociationMetadataBuilder.php

Spacing +5 added lines, -5 removed lines

@@ -51,12 +51,12 @@ 
             $associationMetadata->setCascade(array_unique(array_merge($associationMetadata->getCascade(), ['remove'])));
         }
 
-        if (! empty($this->oneToOneAnnotation->mappedBy)) {
+        if ( ! empty($this->oneToOneAnnotation->mappedBy)) {
             $associationMetadata->setMappedBy($this->oneToOneAnnotation->mappedBy);
             $associationMetadata->setOwningSide(false);
         }
 
-        if (! empty($this->oneToOneAnnotation->inversedBy)) {
+        if ( ! empty($this->oneToOneAnnotation->inversedBy)) {
             $associationMetadata->setInversedBy($this->oneToOneAnnotation->inversedBy);
             $associationMetadata->setOwningSide(true);
         }
@@ -65,7 +65,7 @@ 
         $this->buildPrimaryKey($associationMetadata);
 
         // Check for owning side to consider join column
-        if (! $associationMetadata->isOwningSide()) {
+        if ( ! $associationMetadata->isOwningSide()) {
             return $associationMetadata;
         }
 
@@ -89,7 +89,7 @@ 
         if (count($joinColumns) === 1) {
             $joinColumn = reset($joinColumns);
 
-            if (! $associationMetadata->isPrimaryKey()) {
+            if ( ! $associationMetadata->isPrimaryKey()) {
                 $joinColumn->setUnique(true);
             }
 
@@ -98,7 +98,7 @@ 
 
         $tableMetadata = $this->componentMetadata->table;
 
-        if (! $tableMetadata) {
+        if ( ! $tableMetadata) {
             $exception = 'ClassMetadata::setTable() has to be called before defining a one to one relationship.';
 
             throw new RuntimeException($exception);

tests/Doctrine/Tests/ORM/Mapping/ClassMetadataTest.php

Spacing +8 added lines, -8 removed lines

@@ -36,7 +36,7 @@ 
 use function strtoupper;
 use function unserialize;
 
-require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
 class ClassMetadataTest extends OrmTestCase
 {
@@ -165,7 +165,7 @@ 
      */
     public function testMapAssociationInGlobalNamespace() : void
     {
-        require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+        require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
         $cm = new ClassMetadata(DoctrineGlobalArticle::class, null);
         $cm->setTable(new Mapping\TableMetadata('doctrine_global_article'));
@@ -265,7 +265,7 @@ 
      */
     public function testSetDiscriminatorMapInGlobalNamespace() : void
     {
-        require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+        require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
         $cm = new ClassMetadata('DoctrineGlobalUser', null);
         $cm->setTable(new Mapping\TableMetadata('doctrine_global_user'));
@@ -281,7 +281,7 @@ 
      */
     public function testSetSubClassesInGlobalNamespace() : void
     {
-        require_once __DIR__ . '/../../Models/Global/GlobalNamespaceModel.php';
+        require_once __DIR__.'/../../Models/Global/GlobalNamespaceModel.php';
 
         $cm = new ClassMetadata('DoctrineGlobalUser', null);
         $cm->setTable(new Mapping\TableMetadata('doctrine_global_user'));
@@ -767,7 +767,7 @@ 
     public function testEmptyFieldNameThrowsException() : void
     {
         $this->expectException(MappingException::class);
-        $this->expectExceptionMessage("The field or association mapping misses the 'fieldName' attribute in entity '" . CMS\CmsUser::class . "'.");
+        $this->expectExceptionMessage("The field or association mapping misses the 'fieldName' attribute in entity '".CMS\CmsUser::class."'.");
 
         $cm = new ClassMetadata(CMS\CmsUser::class, null);
         $cm->setTable(new Mapping\TableMetadata('cms_users'));
@@ -819,7 +819,7 @@ 
         $cm->addLifecycleCallback('postLoad', 'notfound');
 
         $this->expectException(MappingException::class);
-        $this->expectExceptionMessage("Entity '" . CMS\CmsUser::class . "' has no public method 'notfound' to be registered as lifecycle callback.");
+        $this->expectExceptionMessage("Entity '".CMS\CmsUser::class."' has no public method 'notfound' to be registered as lifecycle callback.");
 
         $cm->validateLifecycleCallbacks(new RuntimeReflectionService());
     }
@@ -841,7 +841,7 @@ 
         $cm->addProperty($association);
 
         $this->expectException(MappingException::class);
-        $this->expectExceptionMessage("The target-entity 'UnknownClass' cannot be found in '" . CMS\CmsUser::class . "#address'.");
+        $this->expectExceptionMessage("The target-entity 'UnknownClass' cannot be found in '".CMS\CmsUser::class."#address'.");
 
         $cm->validateAssociations();
     }
@@ -1089,6 +1089,6 @@ 
      */
     public function propertyToColumnName(string $propertyName, ?string $className = null) : string
     {
-        return strtolower($this->classToTableName($className)) . '_' . $propertyName;
+        return strtolower($this->classToTableName($className)).'_'.$propertyName;
     }
 }

lib/Doctrine/ORM/Mapping/Driver/DatabaseDriver.php

Spacing +7 added lines, -7 removed lines

@@ -145,8 +145,8 @@ 
     ) : Mapping\ComponentMetadata {
         $this->reverseEngineerMappingFromDatabase();
 
-        if (! isset($this->classToTableNames[$className])) {
-            throw new InvalidArgumentException('Unknown class ' . $className);
+        if ( ! isset($this->classToTableNames[$className])) {
+            throw new InvalidArgumentException('Unknown class '.$className);
         }
 
         $metadata = new Mapping\ClassMetadata($className, $parent);
@@ -175,7 +175,7 @@ 
                     }
                 }
 
-                if (! $otherFk) {
+                if ( ! $otherFk) {
                     // the definition of this many to many table does not contain
                     // enough foreign key information to continue reverse engineering.
                     continue;
@@ -258,9 +258,9 @@ 
                 $allForeignKeyColumns = array_merge($allForeignKeyColumns, $foreignKey->getLocalColumns());
             }
 
-            if (! $table->hasPrimaryKey()) {
+            if ( ! $table->hasPrimaryKey()) {
                 throw new Mapping\MappingException(
-                    'Table ' . $table->getName() . ' has no primary key. Doctrine does not ' .
+                    'Table '.$table->getName().' has no primary key. Doctrine does not '.
                     "support reverse engineering from tables that don't have a primary key."
                 );
             }
@@ -457,7 +457,7 @@ 
             }
 
             // Here we need to check if $fkColumns are the same as $primaryKeys
-            if (! array_diff($fkColumns, $primaryKeys)) {
+            if ( ! array_diff($fkColumns, $primaryKeys)) {
                 $metadata->addProperty($associationMapping);
             } else {
                 $metadata->addProperty($associationMapping);
@@ -502,7 +502,7 @@ 
      */
     private function getClassNameForTable($tableName)
     {
-        return $this->namespace . (
+        return $this->namespace.(
             $this->classNamesForTables[$tableName]
                 ?? Inflector::classify(strtolower($tableName))
         );