Majkl578 / doctrine-dbal

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

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
             $sm->createTable($table);
 
 
-        } catch(\Exception $e) {
+        } catch (\Exception $e) {
         }
 
         $this->_conn->executeUpdate('DELETE FROM master_slave_table');

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

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