This is a small code clean up with PHP CS Fixer

src/Adapter/Driver/Oci8/Connection.php

@@ -252,7 +252,7 @@ public function execute($sql)
 
     /**
      * @todo Get Last Generated Value in Connection (this might not apply)
-     * {@inheritDoc}
+     * @inheritDoc
      */
     public function getLastGeneratedValue($name = null)
     {

src/Sql/Delete.php

@@ -70,7 +70,6 @@ public function from($table): static
     }
 
     /**
-     * @param ?string $key
      * @return mixed
      */
     public function getRawState(?string $key = null)

src/Sql/Predicate/PredicateSet.php

@@ -133,8 +133,6 @@ public function addPredicates($predicates, $combination = self::OP_AND)
 
     /**
      * Return the predicates
-     *
-     * @return array
      */
     public function getPredicates(): array
     {

src/Sql/Select.php

@@ -115,7 +115,7 @@ class Select extends AbstractPreparableSql
     protected bool $prefixColumnsWithTable = true;
 
     /** @var null|string|array|TableIdentifier */
-    protected $table = null;
+    protected $table;
 
     /** @var null|string|Expression */
     protected $quantifier;
@@ -239,7 +239,7 @@ public function columns(array $columns, $prefixColumnsWithTable = true)
      * Create join clause
      *
      * @param  string|array|TableIdentifier $name
-     * @param  string|Predicate\PredicateInterface $on
+     * @param  string|PredicateInterface $on
      * @param  string|array $columns
      * @param  string $type one of the JOIN_* constants
      * @return $this Provides a fluent interface

test/integration/Adapter/Driver/Pdo/Mysql/AdapterTrait.php

@@ -16,8 +16,10 @@ trait AdapterTrait
     #[Override]
     protected function setUp(): void
     {
-        if (! is_string(getenv('TESTS_LAMINAS_DB_ADAPTER_DRIVER_MYSQL')) ||
-            strtolower((string) getenv('TESTS_LAMINAS_DB_ADAPTER_DRIVER_MYSQL')) !== 'true') {
+        if (
+            ! is_string(getenv('TESTS_LAMINAS_DB_ADAPTER_DRIVER_MYSQL')) ||
+            strtolower((string) getenv('TESTS_LAMINAS_DB_ADAPTER_DRIVER_MYSQL')) !== 'true'
+        ) {
             $this->markTestSkipped('pdo_mysql integration tests are not enabled!');
         }
 

test/integration/Adapter/Driver/Pdo/Postgresql/AdapterTrait.php

@@ -16,7 +16,10 @@ trait AdapterTrait
     #[Override]
     protected function setUp(): void
     {
-        if (! is_string(getenv('TESTS_LAMINAS_DB_ADAPTER_DRIVER_PGSQL')) || strtolower(getenv('TESTS_LAMINAS_DB_ADAPTER_DRIVER_PGSQL')) !== 'true') {
+        if (
+            ! is_string(getenv('TESTS_LAMINAS_DB_ADAPTER_DRIVER_PGSQL'))
+            || strtolower(getenv('TESTS_LAMINAS_DB_ADAPTER_DRIVER_PGSQL')) !== 'true'
+        ) {
             $this->markTestSkipped('pdo_pgsql integration tests are not enabled!');
         }
 

test/unit/Adapter/Driver/IbmDb2/IbmDb2Test.php

@@ -42,7 +42,10 @@ protected function setUp(): void
      */
     public function testRegisterConnection(): void
     {
-        $mockConnection = $this->getMockBuilder(Connection::class)->setConstructorArgs([[]])->onlyMethods(['setDriver'])->getMock();
+        $mockConnection = $this->getMockBuilder(Connection::class)
+                            ->setConstructorArgs([[]])
+                            ->onlyMethods(['setDriver'])
+                            ->getMock();
         $mockConnection->expects($this->once())->method('setDriver')->with($this->equalTo($this->ibmdb2));
         self::assertSame($this->ibmdb2, $this->ibmdb2->registerConnection($mockConnection));
     }
@@ -53,7 +56,10 @@ public function testRegisterConnection(): void
     public function testRegisterStatementPrototype(): void
     {
         $this->ibmdb2  = new IbmDb2([]);
-        $mockStatement = $this->getMockBuilder(Statement::class)->setConstructorArgs([])->onlyMethods(['setDriver'])->getMock();
+        $mockStatement = $this->getMockBuilder(Statement::class)
+                            ->setConstructorArgs([])
+                            ->onlyMethods(['setDriver'])
+                            ->getMock();
         $mockStatement->expects($this->once())->method('setDriver')->with($this->equalTo($this->ibmdb2));
         self::assertSame($this->ibmdb2, $this->ibmdb2->registerStatementPrototype($mockStatement));
     }

test/unit/Adapter/Driver/Oci8/Oci8Test.php

@@ -41,7 +41,10 @@ protected function setUp(): void
      */
     public function testRegisterConnection(): void
     {
-        $mockConnection = $this->getMockBuilder(Connection::class)->setConstructorArgs([[]])->onlyMethods(['setDriver'])->getMock();
+        $mockConnection = $this->getMockBuilder(Connection::class)
+            ->setConstructorArgs([[]])
+            ->onlyMethods(['setDriver'])
+            ->getMock();
         $mockConnection->expects($this->once())->method('setDriver')->with($this->equalTo($this->oci8));
         self::assertSame($this->oci8, $this->oci8->registerConnection($mockConnection));
     }
@@ -52,7 +55,10 @@ public function testRegisterConnection(): void
     public function testRegisterStatementPrototype(): void
     {
         $this->oci8    = new Oci8([]);
-        $mockStatement = $this->getMockBuilder(Statement::class)->setConstructorArgs([])->onlyMethods(['setDriver'])->getMock();
+        $mockStatement = $this->getMockBuilder(Statement::class)
+            ->setConstructorArgs([])
+            ->onlyMethods(['setDriver'])
+            ->getMock();
         $mockStatement->expects($this->once())->method('setDriver')->with($this->equalTo($this->oci8));
         self::assertSame($this->oci8, $this->oci8->registerStatementPrototype($mockStatement));
     }
@@ -63,7 +69,10 @@ public function testRegisterStatementPrototype(): void
     public function testRegisterResultPrototype(): void
     {
         $this->oci8    = new Oci8([]);
-        $mockStatement = $this->getMockBuilder(Result::class)->setConstructorArgs([])->onlyMethods([])->getMock();
+        $mockStatement = $this->getMockBuilder(Result::class)
+            ->setConstructorArgs([])
+            ->onlyMethods([])
+            ->getMock();
         self::assertSame($this->oci8, $this->oci8->registerResultPrototype($mockStatement));
     }
 

test/unit/Adapter/Driver/Pgsql/PgsqlTest.php

@@ -55,7 +55,10 @@ public function testCheckEnvironment(): void
      */
     public function testRegisterConnection(): void
     {
-        $mockConnection = $this->getMockBuilder(Connection::class)->setConstructorArgs([[]])->onlyMethods(['setDriver'])->getMock();
+        $mockConnection = $this->getMockBuilder(Connection::class)
+                            ->setConstructorArgs([[]])
+                            ->onlyMethods(['setDriver'])
+                            ->getMock();
         $mockConnection->expects($this->once())->method('setDriver')->with($this->equalTo($this->pgsql));
         self::assertSame($this->pgsql, $this->pgsql->registerConnection($mockConnection));
     }
@@ -66,7 +69,10 @@ public function testRegisterConnection(): void
     public function testRegisterStatementPrototype(): void
     {
         $this->pgsql   = new Pgsql([]);
-        $mockStatement = $this->getMockBuilder(Statement::class)->setConstructorArgs([])->onlyMethods(['setDriver'])->getMock();
+        $mockStatement = $this->getMockBuilder(Statement::class)
+                            ->setConstructorArgs([])
+                            ->onlyMethods(['setDriver'])
+                            ->getMock();
         $mockStatement->expects($this->once())->method('setDriver')->with($this->equalTo($this->pgsql));
         self::assertSame($this->pgsql, $this->pgsql->registerStatementPrototype($mockStatement));
     }
@@ -77,7 +83,10 @@ public function testRegisterStatementPrototype(): void
     public function testRegisterResultPrototype(): void
     {
         $this->pgsql   = new Pgsql([]);
-        $mockStatement = $this->getMockBuilder(Result::class)->setConstructorArgs([])->onlyMethods([])->getMock();
+        $mockStatement = $this->getMockBuilder(Result::class)
+                            ->setConstructorArgs([])
+                            ->onlyMethods([])
+                            ->getMock();
         self::assertSame($this->pgsql, $this->pgsql->registerResultPrototype($mockStatement));
     }
 

test/unit/Adapter/Driver/Sqlsrv/SqlsrvTest.php

@@ -42,7 +42,10 @@ protected function setUp(): void
      */
     public function testRegisterConnection(): void
     {
-        $mockConnection = $this->getMockBuilder(Connection::class)->setConstructorArgs([[]])->onlyMethods(['setDriver'])->getMock();
+        $mockConnection = $this->getMockBuilder(Connection::class)
+            ->setConstructorArgs([[]])
+            ->onlyMethods(['setDriver'])
+            ->getMock();
         $mockConnection->expects($this->once())->method('setDriver')->with($this->equalTo($this->sqlsrv));
         self::assertSame($this->sqlsrv, $this->sqlsrv->registerConnection($mockConnection));
     }
@@ -53,7 +56,10 @@ public function testRegisterConnection(): void
     public function testRegisterStatementPrototype(): void
     {
         $this->sqlsrv  = new Sqlsrv([]);
-        $mockStatement = $this->getMockBuilder(Statement::class)->setConstructorArgs([])->onlyMethods(['setDriver'])->getMock();
+        $mockStatement = $this->getMockBuilder(Statement::class)
+            ->setConstructorArgs([])
+            ->onlyMethods(['setDriver'])
+            ->getMock();
         $mockStatement->expects($this->once())->method('setDriver')->with($this->equalTo($this->sqlsrv));
         self::assertSame($this->sqlsrv, $this->sqlsrv->registerStatementPrototype($mockStatement));
     }

test/unit/Adapter/Platform/PostgresqlTest.php

@@ -120,8 +120,8 @@ public function testQuoteValueList(): void
         //    'Attempting to quote a value in Laminas\Db\Adapter\Platform\Postgresql without extension/driver'
         //    . ' support can introduce security vulnerabilities in a production environment'
         //);
-        $fooOBar = $this->platform->quoteTrustedValue("Foo O'Bar");
-        self::assertEquals($fooOBar, $this->platform->quoteValueList("Foo O'Bar"));
+        $fooBar = $this->platform->quoteTrustedValue("Foo O'Bar");
+        self::assertEquals($fooBar, $this->platform->quoteValueList("Foo O'Bar"));
     }
 
     public function testGetIdentifierSeparator(): void

test/unit/Adapter/Platform/Sql92Test.php

@@ -65,7 +65,8 @@ public function testQuoteValueRaisesNoticeWithoutPlatformSupport(): void
          */
         //$this->expectNotice();
         //$this->expectExceptionMessage(
-        //    'Attempting to quote a value without specific driver level support can introduce security vulnerabilities '
+        //    'Attempting to quote a value without specific driver level '
+        //    . ' support can introduce security vulnerabilities'
         //    . 'in a production environment.'
         //);
         $this->expectNotToPerformAssertions();
@@ -109,7 +110,8 @@ public function testQuoteValueList(): void
          */
         //$this->expectError();
         //$this->expectExceptionMessage(
-        //    'Attempting to quote a value without specific driver level support can introduce security vulnerabilities '
+        //    'Attempting to quote a value without specific driver level '
+        //    . 'support can introduce security vulnerabilities '
         //    . 'in a production environment.'
         //);
         self::assertEquals("'Foo O\\'Bar'", $this->platform->quoteValueList("Foo O'Bar"));

test/unit/Adapter/Platform/SqlServerTest.php

@@ -70,7 +70,8 @@ public function testQuoteValueRaisesNoticeWithoutPlatformSupport(): void
          */
         //$this->expectNotice();
         //$this->expectExceptionMessage(
-        //    'Attempting to quote a value in Laminas\Db\Adapter\Platform\SqlServer without extension/driver support can '
+        //    'Attempting to quote a value in Laminas\Db\Adapter\Platform\SqlServer '
+        //    . 'without extension/driver support can '
         //    . 'introduce security vulnerabilities in a production environment'
         //);
         $this->expectNotToPerformAssertions();
@@ -112,7 +113,8 @@ public function testQuoteValueList(): void
          */
         //$this->expectError();
         //$this->expectExceptionMessage(
-        //    'Attempting to quote a value in Laminas\Db\Adapter\Platform\SqlServer without extension/driver support can '
+        //    'Attempting to quote a value in Laminas\Db\Adapter\Platform\SqlServer '
+        //    . 'without extension/driver support can '
         //    . 'introduce security vulnerabilities in a production environment'
         //);
         self::assertEquals("'Foo O''Bar'", $this->platform->quoteValueList("Foo O'Bar"));

test/unit/Metadata/Source/AbstractSourceTest.php

@@ -22,7 +22,11 @@ final class AbstractSourceTest extends TestCase
     #[Override]
     protected function setUp(): void
     {
-        $this->abstractSourceMock = $this->getMockBuilder(AbstractSource::class)->setConstructorArgs([])->onlyMethods([])->disableOriginalConstructor()->getMock();
+        $this->abstractSourceMock = $this->getMockBuilder(AbstractSource::class)
+            ->setConstructorArgs([])
+            ->onlyMethods([])
+            ->disableOriginalConstructor()
+            ->getMock();
     }
 
     /**

test/unit/RowGateway/AbstractRowGatewayTest.php

@@ -174,7 +174,10 @@ public function testSaveInsertMultiKey(): void
     {
         $this->rowGateway = $this->getMockBuilder(AbstractRowGateway::class)->onlyMethods([])->getMock();
 
-        $mockSql = $this->getMockBuilder(Sql::class)->setConstructorArgs([$this->mockAdapter])->onlyMethods([])->getMock();
+        $mockSql = $this->getMockBuilder(Sql::class)
+                    ->setConstructorArgs([$this->mockAdapter])
+                    ->onlyMethods([])
+                    ->getMock();
 
         $rgPropertyValues = [
             'primaryKeyColumn' => ['one', 'two'],

test/unit/Sql/Ddl/Column/AbstractLengthColumnTest.php

@@ -17,7 +17,10 @@ final class AbstractLengthColumnTest extends TestCase
      */
     public function testSetLength(): void
     {
-        $column = $this->getMockBuilder(AbstractLengthColumn::class)->setConstructorArgs(['foo', 55])->onlyMethods([])->getMock();
+        $column = $this->getMockBuilder(AbstractLengthColumn::class)
+            ->setConstructorArgs(['foo', 55])
+            ->onlyMethods([])
+            ->getMock();
         self::assertEquals(55, $column->getLength());
         self::assertSame($column, $column->setLength(20));
         self::assertEquals(20, $column->getLength());
@@ -28,7 +31,10 @@ public function testSetLength(): void
      */
     public function testGetLength(): void
     {
-        $column = $this->getMockBuilder(AbstractLengthColumn::class)->setConstructorArgs(['foo', 55])->onlyMethods([])->getMock();
+        $column = $this->getMockBuilder(AbstractLengthColumn::class)
+            ->setConstructorArgs(['foo', 55])
+            ->onlyMethods([])
+            ->getMock();
         self::assertEquals(55, $column->getLength());
     }
 
@@ -37,7 +43,10 @@ public function testGetLength(): void
      */
     public function testGetExpressionData(): void
     {
-        $column = $this->getMockBuilder(AbstractLengthColumn::class)->setConstructorArgs(['foo', 4])->onlyMethods([])->getMock();
+        $column = $this->getMockBuilder(AbstractLengthColumn::class)
+            ->setConstructorArgs(['foo', 4])
+            ->onlyMethods([])
+            ->getMock();
 
         self::assertEquals(
             [['%s %s NOT NULL', ['foo', 'INTEGER(4)'], [$column::TYPE_IDENTIFIER, $column::TYPE_LITERAL]]],

test/unit/Sql/Ddl/Column/AbstractPrecisionColumnTest.php

@@ -19,7 +19,10 @@ final class AbstractPrecisionColumnTest extends TestCase
      */
     public function testSetDigits(): void
     {
-        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)->setConstructorArgs(['foo', 10])->onlyMethods([])->getMock();
+        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)
+            ->setConstructorArgs(['foo', 10])
+            ->onlyMethods([])
+            ->getMock();
         self::assertEquals(10, $column->getDigits());
         self::assertSame($column, $column->setDigits(12));
         self::assertEquals(12, $column->getDigits());
@@ -30,7 +33,10 @@ public function testSetDigits(): void
      */
     public function testGetDigits(): void
     {
-        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)->setConstructorArgs(['foo', 10])->onlyMethods([])->getMock();
+        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)
+            ->setConstructorArgs(['foo', 10])
+            ->onlyMethods([])
+            ->getMock();
         self::assertEquals(10, $column->getDigits());
     }
 
@@ -39,7 +45,10 @@ public function testGetDigits(): void
      */
     public function testSetDecimal(): void
     {
-        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)->setConstructorArgs(['foo', 10, 5])->onlyMethods([])->getMock();
+        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)
+            ->setConstructorArgs(['foo', 10, 5])
+            ->onlyMethods([])
+            ->getMock();
         self::assertEquals(5, $column->getDecimal());
         self::assertSame($column, $column->setDecimal(2));
         self::assertEquals(2, $column->getDecimal());
@@ -50,7 +59,10 @@ public function testSetDecimal(): void
      */
     public function testGetDecimal(): void
     {
-        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)->setConstructorArgs(['foo', 10, 5])->onlyMethods([])->getMock();
+        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)
+            ->setConstructorArgs(['foo', 10, 5])
+            ->onlyMethods([])
+            ->getMock();
         self::assertEquals(5, $column->getDecimal());
     }
 
@@ -59,7 +71,10 @@ public function testGetDecimal(): void
      */
     public function testGetExpressionData(): void
     {
-        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)->setConstructorArgs(['foo', 10, 5])->onlyMethods([])->getMock();
+        $column = $this->getMockBuilder(AbstractPrecisionColumn::class)
+            ->setConstructorArgs(['foo', 10, 5])
+            ->onlyMethods([])
+            ->getMock();
 
         self::assertEquals(
             [['%s %s NOT NULL', ['foo', 'INTEGER(10,5)'], [$column::TYPE_IDENTIFIER, $column::TYPE_LITERAL]]],

test/unit/Sql/Platform/PlatformTest.php

@@ -122,8 +122,12 @@ protected function resolveAdapter(string $platformName): Adapter
         /** @var DriverInterface|MockObject $mockDriver */
         $mockDriver = $this->getMockBuilder(DriverInterface::class)->getMock();
 
-        $mockDriver->expects($this->any())->method('formatParameterName')->willReturn('?');
-        $mockDriver->expects($this->any())->method('createStatement')->willReturnCallback(fn() => new StatementContainer());
+        $mockDriver->expects($this->any())
+            ->method('formatParameterName')
+            ->willReturn('?');
+        $mockDriver->expects($this->any())
+            ->method('createStatement')
+            ->willReturnCallback(fn() => new StatementContainer());
 
         return new Adapter($mockDriver, $platform);
     }