thecodingmachine / tdbm

tests/Performance/ManyToOneBench.php

Spacing +6 added lines, -6 removed lines

@@ -73,18 +73,18 @@ 
             $connection->exec($sqlStmt);
         }
 
-        for ($i = 1; $i<200; $i++) {
+        for ($i = 1; $i < 200; $i++) {
             TDBMAbstractServiceTest::insert($connection, 'countries', [
                 'id' => $i,
                 'label' => 'Country '.$i,
             ]);
         }
 
-        for ($i = 1; $i<1000; $i++) {
+        for ($i = 1; $i < 1000; $i++) {
             TDBMAbstractServiceTest::insert($connection, 'users', [
                 'id' => $i,
                 'name' => 'User '.$i,
-                'country_id' => ($i%199) +1,
+                'country_id' => ($i % 199) + 1,
             ]);
         }
     }
@@ -96,8 +96,8 @@ 
         $schemaLockFileDumper = new SchemaLockFileDumper($connection, new ArrayCache(), Configuration::getDefaultLockFilePath());
         $tdbmSchemaAnalyzer = new TDBMSchemaAnalyzer($connection, new ArrayCache(), $schemaAnalyzer, $schemaLockFileDumper);
         $tdbmDaoGenerator = new TDBMDaoGenerator(self::createConfiguration(), $tdbmSchemaAnalyzer);
-        $rootPath = __DIR__ . '/../';
-        self::recursiveDelete(__DIR__. '/../../src/Test/Dao/');
+        $rootPath = __DIR__.'/../';
+        self::recursiveDelete(__DIR__.'/../../src/Test/Dao/');
 
         $tdbmDaoGenerator->generateAllDaosAndBeans();
     }
@@ -113,7 +113,7 @@ 
         if (is_file($str)) {
             return @unlink($str);
         } elseif (is_dir($str)) {
-            $scan = glob(rtrim($str, '/') . '/*');
+            $scan = glob(rtrim($str, '/').'/*');
             foreach ($scan as $index => $path) {
                 self::recursiveDelete($path);
             }

tests/WeakrefObjectStorageTest.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
         $objectStorage = new WeakrefObjectStorage();
         $dbRow = $this->createMock(DbRow::class);
 
-        for ($i=0; $i<10001; $i++) {
+        for ($i = 0; $i < 10001; $i++) {
             $objectStorage->set('foo', $i, clone $dbRow);
         }
         $this->assertNull($objectStorage->get('foo', 42));

tests/AbstractTDBMObjectTest.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
         $this->assertEquals(null, $a->first());
         $this->assertEquals(null, isset($a[0])); //an empty resultIterator must implement arrayAccess
         $this->assertEquals([], $a->toArray());
-        foreach ($a->map(function ($foo) {
+        foreach ($a->map(function($foo) {
         }) as $empty) {
             throw new \LogicException("Not supposed to iterate on an empty iterator.");
         }
@@ -51,7 +51,7 @@ 
         $this->assertEquals(0, $b->count());
         $this->assertEquals([], $b->toArray());
         $this->assertEquals(0, $b->totalCount());
-        $c = $b->map(function ($foo) {
+        $c = $b->map(function($foo) {
         });
         foreach ($c as $empty) {
             throw new \LogicException("Not supposed to iterate on an empty iterator.");

tests/Utils/DbalUtilsTest.php

Spacing +2 added lines, -2 removed lines

@@ -12,8 +12,8 @@
     {
         $params = [
             'key1' => 'foo',
-            'key2' => [1,2,3],
-            'key3' => [1,2,'baz'],
+            'key2' => [1, 2, 3],
+            'key3' => [1, 2, 'baz'],
             'key4' => 1,
         ];
 

tests/MapIteratorTest.php

Spacing +4 added lines, -4 removed lines

@@ -38,7 +38,7 @@ 
             {
                 return new ArrayIterator($this);
             }
-        }, function ($item) {
+        }, function($item) {
             return $item;
         });
 
@@ -48,7 +48,7 @@ 
     public function testConstructorException1(): void
     {
         $this->expectException('TheCodingMachine\TDBM\TDBMException');
-        $mapIterator = new MapIterator(new \DateTime(), function ($item) {
+        $mapIterator = new MapIterator(new \DateTime(), function($item) {
             return $item;
         });
     }
@@ -56,7 +56,7 @@ 
     public function testConstructorException2(): void
     {
         $this->expectException('TheCodingMachine\TDBM\TDBMException');
-        $mapIterator = new MapIterator(array(1, 2, 3), function () {
+        $mapIterator = new MapIterator(array(1, 2, 3), function() {
             return $item;
         });
     }
@@ -64,7 +64,7 @@ 
     public function testJsonSerialize(): void
     {
         $value = array(1, 2, 3);
-        $mapIterator = new MapIterator($value, function ($item) {
+        $mapIterator = new MapIterator($value, function($item) {
             return $item;
         });
 

tests/AlterableResultIteratorTest.php

Spacing +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
 
         $alterableResultIterator = new AlterableResultIterator($iterator);
 
-        $map = $alterableResultIterator->map(function ($item) {
+        $map = $alterableResultIterator->map(function($item) {
             return $item->foo;
         });
 

tests/NativeWeakrefObjectStorageTest.php

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
         $objectStorage = new NativeWeakrefObjectStorage();
         $dbRow = $this->createMock(DbRow::class);
 
-        for ($i=0; $i<10001; $i++) {
+        for ($i = 0; $i < 10001; $i++) {
             $objectStorage->set('foo', $i, clone $dbRow);
         }
         $this->assertNull($objectStorage->get('foo', 42));

src/Schema/ForeignKey.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     public function getCacheKey(): string
     {
         if ($this->cacheKey === null) {
-            $this->cacheKey = 'from__' . implode(',', $this->getUnquotedLocalColumns()) . '__to__table__' . $this->getForeignTableName() . '__columns__' . implode(',', $this->getUnquotedForeignColumns());
+            $this->cacheKey = 'from__'.implode(',', $this->getUnquotedLocalColumns()).'__to__table__'.$this->getForeignTableName().'__columns__'.implode(',', $this->getUnquotedForeignColumns());
         }
         return $this->cacheKey;
     }

src/Utils/PivotTableMethodsDescriptor.php

Spacing +9 added lines, -9 removed lines

@@ -201,7 +201,7 @@ 
 
         $getter = new MethodGenerator($this->getName());
         $getterDocBlock = new DocBlockGenerator(sprintf('Returns the list of %s associated to this bean via the %s pivot table.', $remoteBeanName, $this->pivotTable->getName()));
-        $getterDocBlock->setTag(new ReturnTag([ $fqcnRemoteBeanName.'[]' ]));
+        $getterDocBlock->setTag(new ReturnTag([$fqcnRemoteBeanName.'[]']));
         $getterDocBlock->setWordWrap(false);
         $getter->setDocBlock($getterDocBlock);
         $getter->setReturnType('array');
@@ -210,7 +210,7 @@ 
 
         $adder = new MethodGenerator('add'.$singularName);
         $adderDocBlock = new DocBlockGenerator(sprintf('Adds a relationship with %s associated to this bean via the %s pivot table.', $remoteBeanName, $this->pivotTable->getName()));
-        $adderDocBlock->setTag(new ParamTag($variableName, [ $fqcnRemoteBeanName ]));
+        $adderDocBlock->setTag(new ParamTag($variableName, [$fqcnRemoteBeanName]));
         $adderDocBlock->setWordWrap(false);
         $adder->setDocBlock($adderDocBlock);
         $adder->setReturnType('void');
@@ -219,7 +219,7 @@ 
 
         $remover = new MethodGenerator('remove'.$singularName);
         $removerDocBlock = new DocBlockGenerator(sprintf('Deletes the relationship with %s associated to this bean via the %s pivot table.', $remoteBeanName, $this->pivotTable->getName()));
-        $removerDocBlock->setTag(new ParamTag($variableName, [ $fqcnRemoteBeanName ]));
+        $removerDocBlock->setTag(new ParamTag($variableName, [$fqcnRemoteBeanName]));
         $removerDocBlock->setWordWrap(false);
         $remover->setDocBlock($removerDocBlock);
         $remover->setReturnType('void');
@@ -228,8 +228,8 @@ 
 
         $has = new MethodGenerator('has'.$singularName);
         $hasDocBlock = new DocBlockGenerator(sprintf('Returns whether this bean is associated with %s via the %s pivot table.', $remoteBeanName, $this->pivotTable->getName()));
-        $hasDocBlock->setTag(new ParamTag($variableName, [ $fqcnRemoteBeanName ]));
-        $hasDocBlock->setTag(new ReturnTag([ 'bool' ]));
+        $hasDocBlock->setTag(new ParamTag($variableName, [$fqcnRemoteBeanName]));
+        $hasDocBlock->setTag(new ReturnTag(['bool']));
         $hasDocBlock->setWordWrap(false);
         $has->setDocBlock($hasDocBlock);
         $has->setReturnType('bool');
@@ -239,15 +239,15 @@ 
         $setter = new MethodGenerator('set'.$pluralName);
         $setterDocBlock = new DocBlockGenerator(sprintf('Sets all relationships with %s associated to this bean via the %s pivot table.
 Exiting relationships will be removed and replaced by the provided relationships.', $remoteBeanName, $this->pivotTable->getName()));
-        $setterDocBlock->setTag(new ParamTag($pluralVariableName, [ $fqcnRemoteBeanName.'[]' ]));
-        $setterDocBlock->setTag(new ReturnTag([ 'void' ]));
+        $setterDocBlock->setTag(new ParamTag($pluralVariableName, [$fqcnRemoteBeanName.'[]']));
+        $setterDocBlock->setTag(new ReturnTag(['void']));
         $setterDocBlock->setWordWrap(false);
         $setter->setDocBlock($setterDocBlock);
         $setter->setReturnType('void');
         $setter->setParameter(new ParameterGenerator($pluralVariableName, 'array'));
         $setter->setBody(sprintf('$this->setRelationships(%s, $%s);', $pathKey, $pluralVariableName));
 
-        return [ $getter, $adder, $remover, $has, $setter ];
+        return [$getter, $adder, $remover, $has, $setter];
     }
 
     /**
@@ -276,7 +276,7 @@ 
         /** @var Annotation\JsonFormat|null $jsonFormat */
         $jsonFormat = $this->findRemoteAnnotation(Annotation\JsonFormat::class);
         if ($jsonFormat !== null) {
-            $method = $jsonFormat->method ?? 'get' . ucfirst($jsonFormat->property);
+            $method = $jsonFormat->method ?? 'get'.ucfirst($jsonFormat->property);
             $format = "$method()";
         } else {
             $stopRecursion = $this->findRemoteAnnotation(Annotation\JsonRecursive::class) ? '' : 'true';