Majkl578 / doctrine-dbal

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

Spacing +5 added lines, -5 removed lines

@@ -22,7 +22,7 @@ 
             $table->setPrimaryKey(array('id'));
 
             $this->_conn->getSchemaManager()->createTable($table);
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
 
         }
         $this->_conn->executeUpdate('DELETE FROM write_table');
@@ -161,7 +161,7 @@ 
         $sequence = new \Doctrine\DBAL\Schema\Sequence('write_table_id_seq');
         try {
             $this->_conn->getSchemaManager()->createSequence($sequence);
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
         }
 
         $sequences = $this->_conn->getSchemaManager()->listSequences();
@@ -184,7 +184,7 @@ 
             $this->markTestSkipped("Test only works consistently on platforms that support sequences and don't support identity columns.");
         }
 
-        self::assertFalse($this->_conn->lastInsertId( null ));
+        self::assertFalse($this->_conn->lastInsertId(null));
 
     }
 
@@ -256,7 +256,7 @@ 
     {
         $platform = $this->_conn->getDatabasePlatform();
 
-        if ( ! ($platform->supportsIdentityColumns() || $platform->usesSequenceEmulatedIdentityColumns()) ) {
+        if ( ! ($platform->supportsIdentityColumns() || $platform->usesSequenceEmulatedIdentityColumns())) {
             $this->markTestSkipped(
                 'Test only works on platforms with identity columns or sequence emulated identity columns.'
             );
@@ -268,7 +268,7 @@ 
 
         try {
             $this->_conn->getSchemaManager()->dropTable($table->getQuotedName($platform));
-        } catch(\Exception $e) { }
+        } catch (\Exception $e) { }
 
         foreach ($platform->getCreateTableSQL($table) as $sql) {
             $this->_conn->exec($sql);

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

Spacing +3 added lines, -3 removed lines

@@ -19,7 +19,7 @@ 
     {
         parent::setUp();
 
-        if ( !($this->_conn->getDriver() instanceof ExceptionConverterDriver)) {
+        if ( ! ($this->_conn->getDriver() instanceof ExceptionConverterDriver)) {
             $this->markTestSkipped('Driver does not support special exception handling.');
         }
     }
@@ -164,7 +164,7 @@ 
     {
         $platform = $this->_conn->getDatabasePlatform();
 
-        if (!$platform->supportsForeignKeyConstraints()) {
+        if ( ! $platform->supportsForeignKeyConstraints()) {
             $this->markTestSkipped("Only fails on platforms with foreign key constraints.");
         }
 
@@ -286,7 +286,7 @@ 
             $this->markTestSkipped("Only fails this way on sqlite");
         }
 
-        $filename = sprintf('%s/%s', sys_get_temp_dir(), 'doctrine_failed_connection_'.$mode.'.db');
+        $filename = sprintf('%s/%s', sys_get_temp_dir(), 'doctrine_failed_connection_' . $mode . '.db');
 
         if (file_exists($filename)) {
             chmod($filename, 0200); // make the file writable again, so it can be removed on Windows

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

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
         parent::setUp();
 
         $platform = $this->_conn->getDatabasePlatform()->getName();
-        if (!in_array($platform, array('mysql', 'sqlite'))) {
+        if ( ! in_array($platform, array('mysql', 'sqlite'))) {
             $this->markTestSkipped('Currently restricted to MySQL and SQLite.');
         }
     }

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

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
 
         $platform = $this->_conn->getDatabasePlatform()->getName();
 
-        if (!in_array($platform, array('postgresql'))) {
+        if ( ! in_array($platform, array('postgresql'))) {
             $this->markTestSkipped('Currently restricted to PostgreSQL');
         }
 

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

Indentation +1 added lines, -1 removed lines

@@ -159,7 +159,7 @@
 
         $tableIndexes = $this->_sm->listTableIndexes('non_default_pk_order');
 
-         self::assertCount(1, $tableIndexes);
+            self::assertCount(1, $tableIndexes);
 
         self::assertArrayHasKey('primary', $tableIndexes, 'listTableIndexes() has to return a "primary" array key.');
         self::assertEquals(array('other_id', 'id'), array_map('strtolower', $tableIndexes['primary']->getColumns()));

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

Indentation +1 added lines, -1 removed lines

@@ -229,7 +229,7 @@
         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');

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