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/TDBMServiceTest.php

Spacing +5 added lines, -5 removed lines

@@ -448,7 +448,7 @@ 
     {
         $beans = $this->tdbmService->findObjects('person', null, [], 'person.id ASC', [], null, TDBMObject::class);
 
-        $results = $beans->map(function ($item) {
+        $results = $beans->map(function($item) {
             return $item->getProperty('id', 'person');
         })->toArray();
 
@@ -457,7 +457,7 @@ 
         // Same test with page
         $page = $beans->take(0, 2);
 
-        $results = $page->map(function ($item) {
+        $results = $page->map(function($item) {
             return $item->getProperty('id', 'person');
         })->toArray();
 
@@ -818,7 +818,7 @@ 
     public function testFindObjectsMethodWithoutResultIteratorClass(): void
     {
         $this->expectException(TDBMInvalidArgumentException::class);
-        $this->expectExceptionMessageMatches('/^\$resultIteratorClass should be a `' . preg_quote(ResultIterator::class, '/') . '`. `(.*)` provided\.$/');
+        $this->expectExceptionMessageMatches('/^\$resultIteratorClass should be a `'.preg_quote(ResultIterator::class, '/').'`. `(.*)` provided\.$/');
         $tdbmService = new TDBMService(new Configuration('TheCodingMachine\\TDBM\\Test\\Dao\\Bean', 'TheCodingMachine\\TDBM\\Test\\Dao', self::getConnection(), $this->getNamingStrategy(), null, null, new NullLogger()));
 
         $tdbmService->findObjects('', null, [], null, [], null, null, self::class);
@@ -827,7 +827,7 @@ 
     public function testFindObjectsFromSqlMethodWithoutResultIteratorClass(): void
     {
         $this->expectException(TDBMInvalidArgumentException::class);
-        $this->expectExceptionMessageMatches('/^\$resultIteratorClass should be a `' . preg_quote(ResultIterator::class, '/') . '`. `(.*)` provided\.$/');
+        $this->expectExceptionMessageMatches('/^\$resultIteratorClass should be a `'.preg_quote(ResultIterator::class, '/').'`. `(.*)` provided\.$/');
         $tdbmService = new TDBMService(new Configuration('TheCodingMachine\\TDBM\\Test\\Dao\\Bean', 'TheCodingMachine\\TDBM\\Test\\Dao', self::getConnection(), $this->getNamingStrategy(), null, null, new NullLogger()));
 
         $tdbmService->findObjectsFromSql('', '', null, [], null, null, null, self::class);
@@ -836,7 +836,7 @@ 
     public function testFindObjectsFromRawSqlMethodWithoutResultIteratorClass(): void
     {
         $this->expectException(TDBMInvalidArgumentException::class);
-        $this->expectExceptionMessageMatches('/^\$resultIteratorClass should be a `' . preg_quote(ResultIterator::class, '/') . '`. `(.*)` provided\.$/');
+        $this->expectExceptionMessageMatches('/^\$resultIteratorClass should be a `'.preg_quote(ResultIterator::class, '/').'`. `(.*)` provided\.$/');
         $tdbmService = new TDBMService(new Configuration('TheCodingMachine\\TDBM\\Test\\Dao\\Bean', 'TheCodingMachine\\TDBM\\Test\\Dao', self::getConnection(), $this->getNamingStrategy(), null, null, new NullLogger()));
 
         $tdbmService->findObjectsFromRawSql('', '', [], null, null, null, self::class);

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;
     }