Majkl578 / doctrine-dbal

tests/Doctrine/Tests/DBAL/Functional/Ticket/DBAL752Test.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
         $platform = $this->_conn->getDatabasePlatform()->getName();
 
-        if (!in_array($platform, array('sqlite'))) {
+        if ( ! in_array($platform, array('sqlite'))) {
             $this->markTestSkipped('Related to SQLite only');
         }
     }

tests/Doctrine/Tests/DBAL/Functional/Schema/OracleSchemaManagerTest.php

Spacing +2 added lines, -2 removed lines

@@ -14,13 +14,13 @@
     {
         parent::setUp();
 
-        if(!isset($GLOBALS['db_username'])) {
+        if ( ! isset($GLOBALS['db_username'])) {
             $this->markTestSkipped('Foo');
         }
 
         $username = $GLOBALS['db_username'];
 
-        $query = "GRANT ALL PRIVILEGES TO ".$username;
+        $query = "GRANT ALL PRIVILEGES TO " . $username;
 
         $conn = \Doctrine\Tests\TestUtil::getTempConnection();
         $conn->executeUpdate($query);

tests/Doctrine/Tests/DBAL/Functional/Schema/MySqlSchemaManagerTest.php

Spacing +11 added lines, -11 removed lines

@@ -456,17 +456,17 @@
         $this->_conn->query('DROP TABLE IF EXISTS test_column_defaults_with_create');
 
         $escapeSequences = [
-            "\\0",          // An ASCII NUL (X'00') character
-            "\\'", "''",    // Single quote
-            '\\"', '""',    // Double quote
-            '\\b',          // A backspace character
-            '\\n',          // A new-line character
-            '\\r',          // A carriage return character
-            '\\t',          // A tab character
-            '\\Z',          // ASCII 26 (Control+Z)
-            '\\\\',         // A backslash (\) character
-            '\\%',          // A percent (%) character
-            '\\_',          // An underscore (_) character
+            "\\0", // An ASCII NUL (X'00') character
+            "\\'", "''", // Single quote
+            '\\"', '""', // Double quote
+            '\\b', // A backspace character
+            '\\n', // A new-line character
+            '\\r', // A carriage return character
+            '\\t', // A tab character
+            '\\Z', // ASCII 26 (Control+Z)
+            '\\\\', // A backslash (\) character
+            '\\%', // A percent (%) character
+            '\\_', // An underscore (_) character
         ];
 
         $default = implode('+', $escapeSequences);

tests/Doctrine/Tests/DBAL/Functional/Schema/SqliteSchemaManagerTest.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
 
     public function testCreateAndDropDatabase()
     {
-        $path = dirname(__FILE__).'/test_create_and_drop_sqlite_database.sqlite';
+        $path = dirname(__FILE__) . '/test_create_and_drop_sqlite_database.sqlite';
 
         $this->_sm->createDatabase($path);
         self::assertFileExists($path);
@@ -145,7 +145,7 @@ 
         }
 
         $version = \SQLite3::version();
-        if(version_compare($version['versionString'], '3.7.16', '<')) {
+        if (version_compare($version['versionString'], '3.7.16', '<')) {
             $this->markTestSkipped('This version of sqlite doesn\'t return the order of the Primary Key.');
         }
         $this->_conn->exec(<<<EOS

tests/Doctrine/Tests/DBAL/Functional/Schema/PostgreSqlSchemaManagerTest.php

Spacing +8 added lines, -8 removed lines

@@ -20,7 +20,7 @@ 
     {
         parent::tearDown();
 
-        if (!$this->_conn) {
+        if ( ! $this->_conn) {
             return;
         }
 
@@ -219,22 +219,22 @@ 
 
     public function testListForeignKeys()
     {
-        if(!$this->_conn->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_conn->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Does not support foreign key constraints.');
         }
 
-        $fkOptions = array('SET NULL', 'SET DEFAULT', 'NO ACTION','CASCADE', 'RESTRICT');
+        $fkOptions = array('SET NULL', 'SET DEFAULT', 'NO ACTION', 'CASCADE', 'RESTRICT');
         $foreignKeys = array();
         $fkTable = $this->getTestTable('test_create_fk1');
-        for($i = 0; $i < count($fkOptions); $i++) {
+        for ($i = 0; $i < count($fkOptions); $i++) {
             $fkTable->addColumn("foreign_key_test$i", 'integer');
             $foreignKeys[] = new \Doctrine\DBAL\Schema\ForeignKeyConstraint(
-                                 array("foreign_key_test$i"), 'test_create_fk2', array('id'), "foreign_key_test_$i"."_fk", array('onDelete' => $fkOptions[$i]));
+                                 array("foreign_key_test$i"), 'test_create_fk2', array('id'), "foreign_key_test_$i" . "_fk", array('onDelete' => $fkOptions[$i]));
         }
         $this->_sm->dropAndCreateTable($fkTable);
         $this->createTestTable('test_create_fk2');
 
-        foreach($foreignKeys as $foreignKey) {
+        foreach ($foreignKeys as $foreignKey) {
             $this->_sm->createForeignKey($foreignKey, 'test_create_fk1');
         }
         $fkeys = $this->_sm->listTableForeignKeys('test_create_fk1');
@@ -244,7 +244,7 @@ 
             self::assertEquals(array('id'), array_map('strtolower', $fkeys[$i]->getForeignColumns()));
             self::assertEquals('test_create_fk2', strtolower($fkeys[0]->getForeignTableName()));
             if ($foreignKeys[$i]->getOption('onDelete') == 'NO ACTION') {
-                self::assertFalse($fkeys[$i]->hasOption('onDelete'), 'Unexpected option: '. $fkeys[$i]->getOption('onDelete'));
+                self::assertFalse($fkeys[$i]->hasOption('onDelete'), 'Unexpected option: ' . $fkeys[$i]->getOption('onDelete'));
             } else {
                 self::assertEquals($foreignKeys[$i]->getOption('onDelete'), $fkeys[$i]->getOption('onDelete'));
             }
@@ -378,7 +378,7 @@ 
      */
     public function testJsonbColumn(string $type): void
     {
-        if (!$this->_sm->getDatabasePlatform() instanceof PostgreSQL94Platform) {
+        if ( ! $this->_sm->getDatabasePlatform() instanceof PostgreSQL94Platform) {
             $this->markTestSkipped("Requires PostgresSQL 9.4+");
             return;
         }

tests/Doctrine/Tests/DBAL/Functional/Schema/SchemaManagerFunctionalTestCase.php

Spacing +66 added lines, -66 removed lines

@@ -80,7 +80,7 @@ 
      */
     public function testDropsDatabaseWithActiveConnections()
     {
-        if (! $this->_sm->getDatabasePlatform()->supportsCreateDropDatabase()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsCreateDropDatabase()) {
             $this->markTestSkipped('Cannot drop Database client side with this Driver.');
         }
 
@@ -118,7 +118,7 @@ 
     public function testDropAndCreateSequence()
     {
         if ( ! $this->_conn->getDatabasePlatform()->supportsSequences()) {
-            $this->markTestSkipped($this->_conn->getDriver()->getName().' does not support sequences.');
+            $this->markTestSkipped($this->_conn->getDriver()->getName() . ' does not support sequences.');
         }
 
         $name = 'dropcreate_sequences_test_seq';
@@ -132,7 +132,7 @@ 
     {
         $filteredList = array_filter(
             $items,
-            function (\Doctrine\DBAL\Schema\AbstractAsset $item) use ($name) : bool {
+            function(\Doctrine\DBAL\Schema\AbstractAsset $item) use ($name) : bool {
                 return $item->getShortestName($item->getNamespaceName()) === $name;
             }
         );
@@ -142,8 +142,8 @@ 
 
     public function testListSequences()
     {
-        if(!$this->_conn->getDatabasePlatform()->supportsSequences()) {
-            $this->markTestSkipped($this->_conn->getDriver()->getName().' does not support sequences.');
+        if ( ! $this->_conn->getDatabasePlatform()->supportsSequences()) {
+            $this->markTestSkipped($this->_conn->getDriver()->getName() . ' does not support sequences.');
         }
 
         $sequence = new \Doctrine\DBAL\Schema\Sequence('list_sequences_test_seq', 20, 10);
@@ -154,9 +154,9 @@ 
         self::assertInternalType('array', $sequences, 'listSequences() should return an array.');
 
         $foundSequence = null;
-        foreach($sequences as $sequence) {
+        foreach ($sequences as $sequence) {
             self::assertInstanceOf('Doctrine\DBAL\Schema\Sequence', $sequence, 'Array elements of listSequences() should be Sequence instances.');
-            if(strtolower($sequence->getName()) == 'list_sequences_test_seq') {
+            if (strtolower($sequence->getName()) == 'list_sequences_test_seq') {
                 $foundSequence = $sequence;
             }
         }
@@ -168,7 +168,7 @@ 
 
     public function testListDatabases()
     {
-        if (!$this->_sm->getDatabasePlatform()->supportsCreateDropDatabase()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsCreateDropDatabase()) {
             $this->markTestSkipped('Cannot drop Database client side with this Driver.');
         }
 
@@ -185,7 +185,7 @@ 
      */
     public function testListNamespaceNames()
     {
-        if (!$this->_sm->getDatabasePlatform()->supportsSchemas()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsSchemas()) {
             $this->markTestSkipped('Platform does not support schemas.');
         }
 
@@ -193,7 +193,7 @@ 
         $namespaces = $this->_sm->listNamespaceNames();
         $namespaces = array_map('strtolower', $namespaces);
 
-        if (!in_array('test_create_schema', $namespaces)) {
+        if ( ! in_array('test_create_schema', $namespaces)) {
             $this->_conn->executeUpdate($this->_sm->getDatabasePlatform()->getCreateSchemaSQL('test_create_schema'));
 
             $namespaces = $this->_sm->listNamespaceNames();
@@ -223,7 +223,7 @@ 
             }
         }
 
-        self::assertTrue( $foundTable , "The 'list_tables_test' table has to be found.");
+        self::assertTrue($foundTable, "The 'list_tables_test' table has to be found.");
     }
 
     public function createListTableColumns()
@@ -252,64 +252,64 @@ 
 
         self::assertArrayHasKey('id', $columns);
         self::assertEquals(0, array_search('id', $columnsKeys));
-        self::assertEquals('id',   strtolower($columns['id']->getname()));
+        self::assertEquals('id', strtolower($columns['id']->getname()));
         self::assertInstanceOf('Doctrine\DBAL\Types\IntegerType', $columns['id']->gettype());
-        self::assertEquals(false,  $columns['id']->getunsigned());
-        self::assertEquals(true,   $columns['id']->getnotnull());
-        self::assertEquals(null,   $columns['id']->getdefault());
-        self::assertInternalType('array',  $columns['id']->getPlatformOptions());
+        self::assertEquals(false, $columns['id']->getunsigned());
+        self::assertEquals(true, $columns['id']->getnotnull());
+        self::assertEquals(null, $columns['id']->getdefault());
+        self::assertInternalType('array', $columns['id']->getPlatformOptions());
 
         self::assertArrayHasKey('test', $columns);
         self::assertEquals(1, array_search('test', $columnsKeys));
         self::assertEquals('test', strtolower($columns['test']->getname()));
         self::assertInstanceOf('Doctrine\DBAL\Types\StringType', $columns['test']->gettype());
-        self::assertEquals(255,    $columns['test']->getlength());
-        self::assertEquals(false,  $columns['test']->getfixed());
-        self::assertEquals(false,  $columns['test']->getnotnull());
-        self::assertEquals('expected default',   $columns['test']->getdefault());
-        self::assertInternalType('array',  $columns['test']->getPlatformOptions());
+        self::assertEquals(255, $columns['test']->getlength());
+        self::assertEquals(false, $columns['test']->getfixed());
+        self::assertEquals(false, $columns['test']->getnotnull());
+        self::assertEquals('expected default', $columns['test']->getdefault());
+        self::assertInternalType('array', $columns['test']->getPlatformOptions());
 
-        self::assertEquals('foo',  strtolower($columns['foo']->getname()));
+        self::assertEquals('foo', strtolower($columns['foo']->getname()));
         self::assertEquals(2, array_search('foo', $columnsKeys));
         self::assertInstanceOf('Doctrine\DBAL\Types\TextType', $columns['foo']->gettype());
-        self::assertEquals(false,  $columns['foo']->getunsigned());
-        self::assertEquals(false,  $columns['foo']->getfixed());
-        self::assertEquals(true,   $columns['foo']->getnotnull());
-        self::assertEquals(null,   $columns['foo']->getdefault());
-        self::assertInternalType('array',  $columns['foo']->getPlatformOptions());
+        self::assertEquals(false, $columns['foo']->getunsigned());
+        self::assertEquals(false, $columns['foo']->getfixed());
+        self::assertEquals(true, $columns['foo']->getnotnull());
+        self::assertEquals(null, $columns['foo']->getdefault());
+        self::assertInternalType('array', $columns['foo']->getPlatformOptions());
 
-        self::assertEquals('bar',  strtolower($columns['bar']->getname()));
+        self::assertEquals('bar', strtolower($columns['bar']->getname()));
         self::assertEquals(3, array_search('bar', $columnsKeys));
         self::assertInstanceOf('Doctrine\DBAL\Types\DecimalType', $columns['bar']->gettype());
-        self::assertEquals(null,   $columns['bar']->getlength());
-        self::assertEquals(10,     $columns['bar']->getprecision());
-        self::assertEquals(4,      $columns['bar']->getscale());
-        self::assertEquals(false,  $columns['bar']->getunsigned());
-        self::assertEquals(false,  $columns['bar']->getfixed());
-        self::assertEquals(false,  $columns['bar']->getnotnull());
-        self::assertEquals(null,   $columns['bar']->getdefault());
-        self::assertInternalType('array',  $columns['bar']->getPlatformOptions());
+        self::assertEquals(null, $columns['bar']->getlength());
+        self::assertEquals(10, $columns['bar']->getprecision());
+        self::assertEquals(4, $columns['bar']->getscale());
+        self::assertEquals(false, $columns['bar']->getunsigned());
+        self::assertEquals(false, $columns['bar']->getfixed());
+        self::assertEquals(false, $columns['bar']->getnotnull());
+        self::assertEquals(null, $columns['bar']->getdefault());
+        self::assertInternalType('array', $columns['bar']->getPlatformOptions());
 
         self::assertEquals('baz1', strtolower($columns['baz1']->getname()));
         self::assertEquals(4, array_search('baz1', $columnsKeys));
         self::assertInstanceOf('Doctrine\DBAL\Types\DateTimeType', $columns['baz1']->gettype());
-        self::assertEquals(true,   $columns['baz1']->getnotnull());
-        self::assertEquals(null,   $columns['baz1']->getdefault());
-        self::assertInternalType('array',  $columns['baz1']->getPlatformOptions());
+        self::assertEquals(true, $columns['baz1']->getnotnull());
+        self::assertEquals(null, $columns['baz1']->getdefault());
+        self::assertInternalType('array', $columns['baz1']->getPlatformOptions());
 
         self::assertEquals('baz2', strtolower($columns['baz2']->getname()));
         self::assertEquals(5, array_search('baz2', $columnsKeys));
         self::assertContains($columns['baz2']->gettype()->getName(), array('time', 'date', 'datetime'));
-        self::assertEquals(true,   $columns['baz2']->getnotnull());
-        self::assertEquals(null,   $columns['baz2']->getdefault());
-        self::assertInternalType('array',  $columns['baz2']->getPlatformOptions());
+        self::assertEquals(true, $columns['baz2']->getnotnull());
+        self::assertEquals(null, $columns['baz2']->getdefault());
+        self::assertInternalType('array', $columns['baz2']->getPlatformOptions());
 
         self::assertEquals('baz3', strtolower($columns['baz3']->getname()));
         self::assertEquals(6, array_search('baz3', $columnsKeys));
         self::assertContains($columns['baz3']->gettype()->getName(), array('time', 'date', 'datetime'));
-        self::assertEquals(true,   $columns['baz3']->getnotnull());
-        self::assertEquals(null,   $columns['baz3']->getdefault());
-        self::assertInternalType('array',  $columns['baz3']->getPlatformOptions());
+        self::assertEquals(true, $columns['baz3']->getnotnull());
+        self::assertEquals(null, $columns['baz3']->getdefault());
+        self::assertInternalType('array', $columns['baz3']->getPlatformOptions());
     }
 
     /**
@@ -440,7 +440,7 @@ 
         $tableIndexes = $this->_sm->listTableIndexes('test_create_index');
         self::assertInternalType('array', $tableIndexes);
 
-        self::assertEquals('test',        strtolower($tableIndexes['test']->getName()));
+        self::assertEquals('test', strtolower($tableIndexes['test']->getName()));
         self::assertEquals(array('test'), array_map('strtolower', $tableIndexes['test']->getColumns()));
         self::assertTrue($tableIndexes['test']->isUnique());
         self::assertFalse($tableIndexes['test']->isPrimary());
@@ -448,7 +448,7 @@ 
 
     public function testCreateTableWithForeignKeys()
     {
-        if(!$this->_sm->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Platform does not support foreign keys.');
         }
 
@@ -467,16 +467,16 @@ 
 
         $fkConstraint = current($fkConstraints);
         self::assertInstanceOf('\Doctrine\DBAL\Schema\ForeignKeyConstraint', $fkConstraint);
-        self::assertEquals('test_foreign',             strtolower($fkConstraint->getForeignTableName()));
-        self::assertEquals(array('foreign_key_test'),  array_map('strtolower', $fkConstraint->getColumns()));
-        self::assertEquals(array('id'),                array_map('strtolower', $fkConstraint->getForeignColumns()));
+        self::assertEquals('test_foreign', strtolower($fkConstraint->getForeignTableName()));
+        self::assertEquals(array('foreign_key_test'), array_map('strtolower', $fkConstraint->getColumns()));
+        self::assertEquals(array('id'), array_map('strtolower', $fkConstraint->getForeignColumns()));
 
         self::assertTrue($fkTable->columnsAreIndexed($fkConstraint->getColumns()), "The columns of a foreign key constraint should always be indexed.");
     }
 
     public function testListForeignKeys()
     {
-        if(!$this->_conn->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_conn->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Does not support foreign key constraints.');
         }
 
@@ -494,11 +494,11 @@ 
         self::assertEquals(1, count($fkeys), "Table 'test_create_fk1' has to have one foreign key.");
 
         self::assertInstanceOf('Doctrine\DBAL\Schema\ForeignKeyConstraint', $fkeys[0]);
-        self::assertEquals(array('foreign_key_test'),  array_map('strtolower', $fkeys[0]->getLocalColumns()));
-        self::assertEquals(array('id'),                array_map('strtolower', $fkeys[0]->getForeignColumns()));
-        self::assertEquals('test_create_fk2',          strtolower($fkeys[0]->getForeignTableName()));
+        self::assertEquals(array('foreign_key_test'), array_map('strtolower', $fkeys[0]->getLocalColumns()));
+        self::assertEquals(array('id'), array_map('strtolower', $fkeys[0]->getForeignColumns()));
+        self::assertEquals('test_create_fk2', strtolower($fkeys[0]->getForeignTableName()));
 
-        if($fkeys[0]->hasOption('onDelete')) {
+        if ($fkeys[0]->hasOption('onDelete')) {
             self::assertEquals('CASCADE', $fkeys[0]->getOption('onDelete'));
         }
     }
@@ -518,7 +518,7 @@ 
 
     public function testAlterTableScenario()
     {
-        if(!$this->_sm->getDatabasePlatform()->supportsAlterTable()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsAlterTable()) {
             $this->markTestSkipped('Alter Table is not supported by this platform.');
         }
 
@@ -606,7 +606,7 @@ 
 
     public function testTableInNamespace()
     {
-        if (! $this->_sm->getDatabasePlatform()->supportsSchemas()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsSchemas()) {
             $this->markTestSkipped('Schema definition is not supported by this platform.');
         }
 
@@ -630,7 +630,7 @@ 
 
     public function testCreateAndListViews()
     {
-        if (!$this->_sm->getDatabasePlatform()->supportsViews()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsViews()) {
             $this->markTestSkipped('Views is not supported by this platform.');
         }
 
@@ -648,7 +648,7 @@ 
 
     public function testAutoincrementDetection()
     {
-        if (!$this->_sm->getDatabasePlatform()->supportsIdentityColumns()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsIdentityColumns()) {
             $this->markTestSkipped('This test is only supported on platforms that have autoincrement');
         }
 
@@ -669,7 +669,7 @@ 
      */
     public function testAutoincrementDetectionMulticolumns()
     {
-        if (!$this->_sm->getDatabasePlatform()->supportsIdentityColumns()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsIdentityColumns()) {
             $this->markTestSkipped('This test is only supported on platforms that have autoincrement');
         }
 
@@ -691,7 +691,7 @@ 
      */
     public function testUpdateSchemaWithForeignKeyRenaming()
     {
-        if (!$this->_sm->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('This test is only supported on platforms that have foreign keys.');
         }
 
@@ -736,7 +736,7 @@ 
      */
     public function testRenameIndexUsedInForeignKeyConstraint()
     {
-        if (! $this->_sm->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('This test is only supported on platforms that have foreign keys.');
         }
 
@@ -940,7 +940,7 @@ 
         return $table;
     }
 
-    protected function getTestTable($name, $options=array())
+    protected function getTestTable($name, $options = array())
     {
         $table = new Table($name, array(), array(), array(), false, $options);
         $table->setSchemaConfig($this->_sm->createSchemaConfig());
@@ -976,7 +976,7 @@ 
 
     public function testListForeignKeysComposite()
     {
-        if(!$this->_conn->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_conn->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('Does not support foreign key constraints.');
         }
 
@@ -995,7 +995,7 @@ 
 
         self::assertInstanceOf('Doctrine\DBAL\Schema\ForeignKeyConstraint', $fkeys[0]);
         self::assertEquals(array('id', 'foreign_key_test'), array_map('strtolower', $fkeys[0]->getLocalColumns()));
-        self::assertEquals(array('id', 'other_id'),         array_map('strtolower', $fkeys[0]->getForeignColumns()));
+        self::assertEquals(array('id', 'other_id'), array_map('strtolower', $fkeys[0]->getForeignColumns()));
     }
 
     /**
@@ -1224,7 +1224,7 @@ 
      */
     public function testDoesNotListIndexesImplicitlyCreatedByForeignKeys()
     {
-        if (! $this->_sm->getDatabasePlatform()->supportsForeignKeyConstraints()) {
+        if ( ! $this->_sm->getDatabasePlatform()->supportsForeignKeyConstraints()) {
             $this->markTestSkipped('This test is only supported on platforms that have foreign keys.');
         }
 

tests/Doctrine/Tests/DBAL/Functional/DataAccessTest.php

Spacing +33 added lines, -33 removed lines

@@ -165,7 +165,7 @@ 
         $paramInt = 1;
         $paramStr = 'foo';
 
-        $sql = "SELECT test_int, test_string FROM " . $this->_conn->quoteIdentifier($table) . " ".
+        $sql = "SELECT test_int, test_string FROM " . $this->_conn->quoteIdentifier($table) . " " .
                "WHERE test_int = " . $this->_conn->quote($paramInt) . " AND test_string = " . $this->_conn->quote($paramStr);
         $stmt = $this->_conn->prepare($sql);
         self::assertInstanceOf('Doctrine\DBAL\Statement', $stmt);
@@ -531,30 +531,30 @@ 
     {
         $p = $this->_conn->getDatabasePlatform();
         $sql = 'SELECT ';
-        $sql .= $p->getDateDiffExpression('test_datetime', $p->getCurrentTimestampSQL()) .' AS diff, ';
-        $sql .= $p->getDateAddSecondsExpression('test_datetime', 1) .' AS add_seconds, ';
-        $sql .= $p->getDateSubSecondsExpression('test_datetime', 1) .' AS sub_seconds, ';
-        $sql .= $p->getDateAddMinutesExpression('test_datetime', 5) .' AS add_minutes, ';
-        $sql .= $p->getDateSubMinutesExpression('test_datetime', 5) .' AS sub_minutes, ';
-        $sql .= $p->getDateAddHourExpression('test_datetime', 3) .' AS add_hour, ';
-        $sql .= $p->getDateSubHourExpression('test_datetime', 3) .' AS sub_hour, ';
-        $sql .= $p->getDateAddDaysExpression('test_datetime', 10) .' AS add_days, ';
-        $sql .= $p->getDateSubDaysExpression('test_datetime', 10) .' AS sub_days, ';
-        $sql .= $p->getDateAddWeeksExpression('test_datetime', 1) .' AS add_weeks, ';
-        $sql .= $p->getDateSubWeeksExpression('test_datetime', 1) .' AS sub_weeks, ';
-        $sql .= $p->getDateAddMonthExpression('test_datetime', 2) .' AS add_month, ';
-        $sql .= $p->getDateSubMonthExpression('test_datetime', 2) .' AS sub_month, ';
-        $sql .= $p->getDateAddQuartersExpression('test_datetime', 3) .' AS add_quarters, ';
-        $sql .= $p->getDateSubQuartersExpression('test_datetime', 3) .' AS sub_quarters, ';
-        $sql .= $p->getDateAddYearsExpression('test_datetime', 6) .' AS add_years, ';
-        $sql .= $p->getDateSubYearsExpression('test_datetime', 6) .' AS sub_years ';
+        $sql .= $p->getDateDiffExpression('test_datetime', $p->getCurrentTimestampSQL()) . ' AS diff, ';
+        $sql .= $p->getDateAddSecondsExpression('test_datetime', 1) . ' AS add_seconds, ';
+        $sql .= $p->getDateSubSecondsExpression('test_datetime', 1) . ' AS sub_seconds, ';
+        $sql .= $p->getDateAddMinutesExpression('test_datetime', 5) . ' AS add_minutes, ';
+        $sql .= $p->getDateSubMinutesExpression('test_datetime', 5) . ' AS sub_minutes, ';
+        $sql .= $p->getDateAddHourExpression('test_datetime', 3) . ' AS add_hour, ';
+        $sql .= $p->getDateSubHourExpression('test_datetime', 3) . ' AS sub_hour, ';
+        $sql .= $p->getDateAddDaysExpression('test_datetime', 10) . ' AS add_days, ';
+        $sql .= $p->getDateSubDaysExpression('test_datetime', 10) . ' AS sub_days, ';
+        $sql .= $p->getDateAddWeeksExpression('test_datetime', 1) . ' AS add_weeks, ';
+        $sql .= $p->getDateSubWeeksExpression('test_datetime', 1) . ' AS sub_weeks, ';
+        $sql .= $p->getDateAddMonthExpression('test_datetime', 2) . ' AS add_month, ';
+        $sql .= $p->getDateSubMonthExpression('test_datetime', 2) . ' AS sub_month, ';
+        $sql .= $p->getDateAddQuartersExpression('test_datetime', 3) . ' AS add_quarters, ';
+        $sql .= $p->getDateSubQuartersExpression('test_datetime', 3) . ' AS sub_quarters, ';
+        $sql .= $p->getDateAddYearsExpression('test_datetime', 6) . ' AS add_years, ';
+        $sql .= $p->getDateSubYearsExpression('test_datetime', 6) . ' AS sub_years ';
         $sql .= 'FROM fetch_table';
 
         $row = $this->_conn->fetchAssoc($sql);
         $row = array_change_key_case($row, CASE_LOWER);
 
         $diff = (strtotime('2010-01-01') - strtotime(date('Y-m-d'))) / 3600 / 24;
-        self::assertEquals($diff, $row['diff'], "Date difference should be approx. ".$diff." days.", 1);
+        self::assertEquals($diff, $row['diff'], "Date difference should be approx. " . $diff . " days.", 1);
         self::assertEquals('2010-01-01 10:10:11', date('Y-m-d H:i:s', strtotime($row['add_seconds'])), "Adding second should end up on 2010-01-01 10:10:11");
         self::assertEquals('2010-01-01 10:10:09', date('Y-m-d H:i:s', strtotime($row['sub_seconds'])), "Subtracting second should end up on 2010-01-01 10:10:09");
         self::assertEquals('2010-01-01 10:15:10', date('Y-m-d H:i:s', strtotime($row['add_minutes'])), "Adding minutes should end up on 2010-01-01 10:15:10");
@@ -578,15 +578,15 @@ 
         $platform = $this->_conn->getDatabasePlatform();
 
         $sql = 'SELECT ';
-        $sql .= $platform->getLocateExpression('test_string', "'oo'") .' AS locate1, ';
-        $sql .= $platform->getLocateExpression('test_string', "'foo'") .' AS locate2, ';
-        $sql .= $platform->getLocateExpression('test_string', "'bar'") .' AS locate3, ';
-        $sql .= $platform->getLocateExpression('test_string', 'test_string') .' AS locate4, ';
-        $sql .= $platform->getLocateExpression("'foo'", 'test_string') .' AS locate5, ';
-        $sql .= $platform->getLocateExpression("'barfoobaz'", 'test_string') .' AS locate6, ';
-        $sql .= $platform->getLocateExpression("'bar'", 'test_string') .' AS locate7, ';
-        $sql .= $platform->getLocateExpression('test_string', "'oo'", 2) .' AS locate8, ';
-        $sql .= $platform->getLocateExpression('test_string', "'oo'", 3) .' AS locate9 ';
+        $sql .= $platform->getLocateExpression('test_string', "'oo'") . ' AS locate1, ';
+        $sql .= $platform->getLocateExpression('test_string', "'foo'") . ' AS locate2, ';
+        $sql .= $platform->getLocateExpression('test_string', "'bar'") . ' AS locate3, ';
+        $sql .= $platform->getLocateExpression('test_string', 'test_string') . ' AS locate4, ';
+        $sql .= $platform->getLocateExpression("'foo'", 'test_string') . ' AS locate5, ';
+        $sql .= $platform->getLocateExpression("'barfoobaz'", 'test_string') . ' AS locate6, ';
+        $sql .= $platform->getLocateExpression("'bar'", 'test_string') . ' AS locate7, ';
+        $sql .= $platform->getLocateExpression('test_string', "'oo'", 2) . ' AS locate8, ';
+        $sql .= $platform->getLocateExpression('test_string', "'oo'", 3) . ' AS locate9 ';
         $sql .= 'FROM fetch_table';
 
         $row = $this->_conn->fetchAssoc($sql);
@@ -717,11 +717,11 @@ 
 
         $results = $stmt->fetchAll(
             FetchMode::CUSTOM_OBJECT,
-            __NAMESPACE__.'\\MyFetchClass'
+            __NAMESPACE__ . '\\MyFetchClass'
         );
 
         self::assertCount(1, $results);
-        self::assertInstanceOf(__NAMESPACE__.'\\MyFetchClass', $results[0]);
+        self::assertInstanceOf(__NAMESPACE__ . '\\MyFetchClass', $results[0]);
 
         self::assertEquals(1, $results[0]->test_int);
         self::assertEquals('foo', $results[0]->test_string);
@@ -760,7 +760,7 @@ 
         $results = $stmt->fetchAll();
 
         self::assertCount(1, $results);
-        self::assertInstanceOf(__NAMESPACE__.'\\MyFetchClass', $results[0]);
+        self::assertInstanceOf(__NAMESPACE__ . '\\MyFetchClass', $results[0]);
 
         self::assertEquals(1, $results[0]->test_int);
         self::assertEquals('foo', $results[0]->test_string);
@@ -785,7 +785,7 @@ 
         }
 
         self::assertCount(1, $results);
-        self::assertInstanceOf(__NAMESPACE__.'\\MyFetchClass', $results[0]);
+        self::assertInstanceOf(__NAMESPACE__ . '\\MyFetchClass', $results[0]);
 
         self::assertEquals(1, $results[0]->test_int);
         self::assertEquals('foo', $results[0]->test_string);
@@ -885,7 +885,7 @@ 
 
     private function skipOci8AndMysqli()
     {
-        if (isset($GLOBALS['db_type']) && $GLOBALS['db_type'] == "oci8")  {
+        if (isset($GLOBALS['db_type']) && $GLOBALS['db_type'] == "oci8") {
             $this->markTestSkipped("Not supported by OCI8");
         }
         if ('mysqli' == $this->_conn->getDriver()->getName()) {

tests/Doctrine/Tests/DBAL/Functional/ModifyLimitQueryTest.php

Spacing +2 added lines, -2 removed lines

@@ -16,7 +16,7 @@ 
     {
         parent::setUp();
 
-        if (!self::$tableCreated) {
+        if ( ! self::$tableCreated) {
             /* @var $sm \Doctrine\DBAL\Schema\AbstractSchemaManager */
             $table = new \Doctrine\DBAL\Schema\Table("modify_limit_table");
             $table->addColumn('test_int', 'integer');
@@ -95,7 +95,7 @@ 
         $this->_conn->insert('modify_limit_table2', array('test_int' => 2));
 
         $sql = "SELECT modify_limit_table.test_int FROM modify_limit_table " .
-               "INNER JOIN modify_limit_table2 ON modify_limit_table.test_int = modify_limit_table2.test_int ".
+               "INNER JOIN modify_limit_table2 ON modify_limit_table.test_int = modify_limit_table2.test_int " .
                "GROUP BY modify_limit_table.test_int " .
                "ORDER BY modify_limit_table.test_int ASC";
         $this->assertLimitResult(array(1, 2), $sql, 10, 0);

tests/Doctrine/Tests/DBAL/Functional/StatementTest.php

Spacing +3 added lines, -3 removed lines

@@ -265,19 +265,19 @@
     {
         return array(
             'fetch' => array(
-                function (Statement $stmt) {
+                function(Statement $stmt) {
                     return $stmt->fetch();
                 },
                 false,
             ),
             'fetch-column' => array(
-                function (Statement $stmt) {
+                function(Statement $stmt) {
                     return $stmt->fetchColumn();
                 },
                 false,
             ),
             'fetch-all' => array(
-                function (Statement $stmt) {
+                function(Statement $stmt) {
                     return $stmt->fetchAll();
                 },
                 array(),