Catch up latest commits

.github/workflows/continuous-integration.yml

@@ -28,7 +28,7 @@ jobs:
           tools: "cs2pr"
 
       - name: "Cache dependencies installed with composer"
-        uses: "actions/cache@v1"
+        uses: "actions/cache@v4"
         with:
           path: "~/.composer/cache"
           key: "php-${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"
@@ -64,7 +64,7 @@ jobs:
           tools: "cs2pr"
 
       - name: "Cache dependencies installed with composer"
-        uses: "actions/cache@v1"
+        uses: "actions/cache@v4"
         with:
           path: "~/.composer/cache"
           key: "php-${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"
@@ -109,7 +109,7 @@ jobs:
           tools: "cs2pr"
 
       - name: "Cache dependencies installed with composer"
-        uses: "actions/cache@v1"
+        uses: "actions/cache@v4"
         with:
           path: "~/.composer/cache"
           key: "php-${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"
@@ -161,7 +161,7 @@ jobs:
           coverage: "pcov"
 
       - name: "Cache dependencies installed with composer"
-        uses: "actions/cache@v1"
+        uses: "actions/cache@v4"
         with:
           path: "~/.composer/cache"
           key: "php-${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"
@@ -208,7 +208,7 @@ jobs:
           coverage: "pcov"
 
       - name: "Cache dependencies installed with composer"
-        uses: "actions/cache@v1"
+        uses: "actions/cache@v4"
         with:
           path: "~/.composer/cache"
           key: "php-${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"
@@ -254,7 +254,7 @@ jobs:
           coverage: "pcov"
 
       - name: "Cache dependencies installed with composer"
-        uses: "actions/cache@v1"
+        uses: "actions/cache@v4"
         with:
           path: "~/.composer/cache"
           key: "php-${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"
@@ -305,7 +305,7 @@ jobs:
           coverage: "pcov"
 
       - name: "Cache dependencies installed with composer"
-        uses: "actions/cache@v1"
+        uses: "actions/cache@v4"
         with:
           path: "~/.composer/cache"
           key: "php-${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"
@@ -352,7 +352,7 @@ jobs:
           coverage: "pcov"
 
       - name: "Cache dependencies installed with composer"
-        uses: "actions/cache@v1"
+        uses: "actions/cache@v4"
         with:
           path: "~/.composer/cache"
           key: "php-${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"
@@ -403,7 +403,7 @@ jobs:
           coverage: "pcov"
 
       - name: "Cache dependencies installed with composer"
-        uses: "actions/cache@v1"
+        uses: "actions/cache@v4"
         with:
           path: "~/.composer/cache"
           key: "php-${{ matrix.php-version }}-composer-locked-${{ hashFiles('composer.lock') }}"

src/EmptyResultIterator.php

@@ -0,0 +1,7 @@
+<?php
+
+namespace TheCodingMachine\TDBM;
+
+class EmptyResultIterator extends ResultIterator
+{
+}

src/InnerResultIterator.php

@@ -5,6 +5,7 @@
 namespace TheCodingMachine\TDBM;
 
 use Doctrine\DBAL\Driver\ResultStatement;
+use Doctrine\DBAL\ForwardCompatibility\Result;
 use Doctrine\DBAL\Platforms\AbstractPlatform;
 use Doctrine\DBAL\Platforms\MySqlPlatform;
 use Doctrine\DBAL\Statement;
@@ -135,7 +136,7 @@ public function count()
 
         if ($this->fetchStarted && $this->tdbmService->getConnection()->getDatabasePlatform() instanceof MySqlPlatform) {
             // Optimisation: we don't need a separate "count" SQL request in MySQL.
-            assert($this->statement instanceof Statement);
+            assert($this->statement instanceof Statement || $this->statement instanceof Result);
             $this->count = (int)$this->statement->rowCount();
             return $this->count;
         }

src/NativeWeakrefObjectStorage.php

@@ -94,6 +94,14 @@ public function remove(string $tableName, $id): void
         unset($this->objects[$tableName][$id]);
     }
 
+    /**
+     * Removes all objects from the storage.
+     */
+    public function clear(): void
+    {
+        $this->objects = array();
+    }
+
     private function cleanupDanglingWeakRefs(): void
     {
         foreach ($this->objects as $tableName => $table) {

src/ObjectStorageInterface.php

@@ -41,4 +41,9 @@ public function get(string $tableName, $id): ?DbRow;
      * @param string|int $id
      */
     public function remove(string $tableName, $id): void;
+
+    /**
+     * Removes all objects from the storage.
+     */
+    public function clear(): void;
 }

src/PageIterator.php

@@ -120,7 +120,7 @@ public static function createEmpyIterator(ResultIterator $parentResult): self
     public function getIterator()
     {
         if ($this->innerResultIterator === null) {
-            if ($this->parentResult->count() === 0) {
+            if ($this->parentResult instanceof EmptyResultIterator) {
                 $this->innerResultIterator = new EmptyInnerResultIterator();
             } elseif ($this->mode === TDBMService::MODE_CURSOR) {
                 $this->innerResultIterator = InnerResultIterator::createInnerResultIterator($this->magicSql, $this->parameters, $this->limit, $this->offset, $this->columnDescriptors, $this->objectStorage, $this->className, $this->tdbmService, $this->magicQuery, $this->logger);

src/QueryFactory/FindObjectsFromRawSqlQueryFactory.php

@@ -403,7 +403,7 @@ private function generateGroupedSqlCount(array $parsedSql): array
                 $item['delim'] = ',';
                 $innerColumns[] = $item;
             }
-            $innerColumns[count($innerColumns)-1]['delim'] = false;
+            $innerColumns[count($innerColumns) - 1]['delim'] = false;
             $parsedSql['SELECT'] = $innerColumns;
 
             $parsedSql = [

src/ResultIterator.php

@@ -177,7 +177,7 @@ public function getIterator()
 
     public function take(int $offset, int $limit): PageIterator
     {
-        if ($this->totalCount === 0) {
+        if ($this instanceof EmptyResultIterator) {
             return PageIterator::createEmpyIterator($this);
         }
         return PageIterator::createResultIterator($this, $this->queryFactory->getMagicSql(), $this->parameters, $limit, $offset, $this->queryFactory->getColumnDescriptors(), $this->objectStorage, $this->className, $this->tdbmService, $this->magicQuery, $this->mode, $this->logger);

src/TDBMService.php

@@ -1559,4 +1559,14 @@ public function setLogLevel(string $level): void
     {
         $this->logger = new LevelFilter($this->rootLogger, $level);
     }
+
+    /**
+     * Clear TDBM's bean cache
+     *
+     * This can be used in long-running processes to cleanup everything.
+     */
+    public function clearBeanCache(): void
+    {
+        $this->objectStorage->clear();
+    }
 }

src/Utils/BeanDescriptor.php

@@ -1333,7 +1333,7 @@ private function generateFindByDaoCodeForIndex(Index $index, string $beanNamespa
             $parameter = new ParameterGenerator(ltrim($element->getSafeVariableName(), '$'));
             if (!$first && !($element->isCompulsory() && $index->isUnique())) {
                 $parameterType = '?';
-            //$functionParameter = '?';
+                //$functionParameter = '?';
             } else {
                 $parameterType = '';
                 //$functionParameter = '';
@@ -1765,7 +1765,7 @@ private function generateGetForeignKeys(array $fks): MethodGenerator
      * @param string $indent
      * @return string
      */
-    private function psr2VarExport($var, string $indent=''): string
+    private function psr2VarExport($var, string $indent = ''): string
     {
         if (is_array($var)) {
             $indexed = array_keys($var) === range(0, count($var) - 1);

src/Utils/DbalUtils.php

@@ -34,6 +34,8 @@ public static function generateTypes(array $parameters): array
                 $types[$key] = Connection::PARAM_INT_ARRAY;
             } elseif (is_int($value)) {
                 $types[$key] = ParameterType::INTEGER;
+            } elseif (is_bool($value)) {
+                $types[$key] = ParameterType::BOOLEAN;
             }
         }
 

tests/AbstractTDBMObjectTest.php

@@ -44,7 +44,7 @@ public function testEmptyResultIterator()
 
     public function testEmptyPageIterator()
     {
-        $a = ResultIterator::createEmpyIterator();
+        $a = EmptyResultIterator::createEmpyIterator();
         $b = $a->take(0, 10);
         foreach ($b as $empty) {
             throw new \LogicException("Not supposed to iterate on an empty page iterator.");

tests/Dao/TestPersonDao.php

@@ -1,4 +1,5 @@
 <?php
+
 /*
  * This file has been automatically generated by TDBM.
  * You can edit this file as it will not be overwritten.

tests/NativeWeakrefObjectStorageTest.php

@@ -30,7 +30,7 @@ public function testDanglingPointers(): void
         $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));

tests/Performance/ManyToOneBench.php

@@ -73,18 +73,18 @@ private static function initSchema(Connection $connection): void
             $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,
             ]);
         }
     }

tests/TDBMAbstractServiceTest.php

@@ -211,9 +211,9 @@ private static function initSchema(Connection $connection): void
             ->column('manager_id')->references('contact')->null();
 
         $db->table('users')
-            ->addAnnotation('AddTrait', ['name'=>TestUserTrait::class], false)
-            ->addAnnotation('AddTrait', ['name'=>TestOtherUserTrait::class, 'modifiers'=>['\\'.TestOtherUserTrait::class.'::method1 insteadof \\'.TestUserTrait::class, '\\'.TestUserTrait::class.'::method1 as method1renamed']], false)
-            ->addAnnotation('AddTraitOnDao', ['name'=>TestUserDaoTrait::class], false)
+            ->addAnnotation('AddTrait', ['name' => TestUserTrait::class], false)
+            ->addAnnotation('AddTrait', ['name' => TestOtherUserTrait::class, 'modifiers' => ['\\'.TestOtherUserTrait::class.'::method1 insteadof \\'.TestUserTrait::class, '\\'.TestUserTrait::class.'::method1 as method1renamed']], false)
+            ->addAnnotation('AddTraitOnDao', ['name' => TestUserDaoTrait::class], false)
             ->implementsInterface(TestUserInterface::class)
             ->implementsInterfaceOnDao(TestUserDaoInterface::class)
             ->extends('contact')
@@ -721,7 +721,7 @@ private static function initSchema(Connection $connection): void
 
         self::insert($connection, 'tracks', [
             'album_id' => 1,
-            'title' =>'Pigs on the Wing 1',
+            'title' => 'Pigs on the Wing 1',
             // Note: Oracle does not have a TIME column type
             'duration' => $timeType->convertToDatabaseValue(new DateTimeImmutable('1970-01-01 00:01:25'), $connection->getDatabasePlatform()),
         ]);

tests/Utils/Annotation/AnnotationParserTest.php

@@ -19,7 +19,7 @@ public function testParse(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $column = new Column('foo', Type::getType(Type::STRING), ['comment'=>'@UUID']);
+        $column = new Column('foo', Type::getType(Type::STRING), ['comment' => '@UUID']);
         $table = new Table('bar');
         $annotations = $parser->getColumnAnnotations($column, $table);
 
@@ -40,7 +40,7 @@ public function testParseMultiLine(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $column = new Column('foo', Type::getType(Type::STRING), ['comment'=>"\n@UUID"]);
+        $column = new Column('foo', Type::getType(Type::STRING), ['comment' => "\n@UUID"]);
         $table = new Table('bar');
         $annotations = $parser->getColumnAnnotations($column, $table);
 
@@ -54,7 +54,7 @@ public function testParseMultiAnnotations(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $column = new Column('foo', Type::getType(Type::STRING), ['comment'=>"\n@UUID\n@Autoincrement"]);
+        $column = new Column('foo', Type::getType(Type::STRING), ['comment' => "\n@UUID\n@Autoincrement"]);
         $table = new Table('bar');
         $annotations = $parser->getColumnAnnotations($column, $table);
 
@@ -68,7 +68,7 @@ public function testException(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $table = new Table('bar', [], [], [], 0, ['comment'=>"@UUID\n@UUID"]);
+        $table = new Table('bar', [], [], [], 0, ['comment' => "@UUID\n@UUID"]);
         $annotations = $parser->getTableAnnotations($table);
 
         $this->expectException(TDBMException::class);
@@ -81,7 +81,7 @@ public function testParseParameters(): void
             'UUID' => UUID::class,
             'Autoincrement' => Autoincrement::class
         ]);
-        $table = new Table('bar', [], [], [], 0, ['comment'=>'@UUID("v4")']);
+        $table = new Table('bar', [], [], [], 0, ['comment' => '@UUID("v4")']);
         $annotations = $parser->getTableAnnotations($table);
 
         $annotation = $annotations->findAnnotation(UUID::class);
@@ -94,7 +94,7 @@ public function testParseOldUUID(): void
             'UUID' => UUID::class,
         ]);
         // First generation UUID did not use the Doctrine syntax.
-        $table = new Table('bar', [], [], [], 0, ['comment'=>'@UUID v4']);
+        $table = new Table('bar', [], [], [], 0, ['comment' => '@UUID v4']);
         $annotations = $parser->getTableAnnotations($table);
 
         $annotation = $annotations->findAnnotation(UUID::class);

tests/Utils/DbalUtilsTest.php

@@ -17,6 +17,6 @@ public function testGenerateTypes(): void
             'key4' => 1,
         ];
 
-        $this->assertSame(['key2'=>Connection::PARAM_INT_ARRAY, 'key3'=>Connection::PARAM_STR_ARRAY, 'key4'=>ParameterType::INTEGER], DbalUtils::generateTypes($params));
+        $this->assertSame(['key2' => Connection::PARAM_INT_ARRAY, 'key3' => Connection::PARAM_STR_ARRAY, 'key4' => ParameterType::INTEGER], DbalUtils::generateTypes($params));
     }
 }

tests/Utils/DefaultNamingStrategyTest.php

@@ -131,7 +131,7 @@ public function testExceptions(): void
 
     public function testBeanAnnotation(): void
     {
-        $table = new Table('chevaux', [], [], [], 0, ['comment'=>'@Bean(name="Cheval")']);
+        $table = new Table('chevaux', [], [], [], 0, ['comment' => '@Bean(name="Cheval")']);
         $strategy = $this->getDefaultNamingStrategyWithStubTables([$table]);
         $this->assertSame('ChevalDao', $strategy->getDaoClassName('chevaux'));
     }