add postgres table escaping and regression test

laravel · Mar 31, 2017 · f55331d · f55331d
1 parent 12dad33
commit f55331d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/Illuminate/Database/Schema/Grammars/PostgresGrammar.php b/src/Illuminate/Database/Schema/Grammars/PostgresGrammar.php
@@ -159,7 +159,7 @@ public function compileDropIfExists(Blueprint $blueprint, Fluent $command)
      */
     public function compileDropAllTables($tables)
     {
-        return 'drop table '.implode(',', $tables).' cascade';
+        return 'drop table "'.implode('","', $tables).'" cascade';
     }
 
     /**

diff --git a/tests/Database/DatabasePostgresSchemaGrammarTest.php b/tests/Database/DatabasePostgresSchemaGrammarTest.php
@@ -546,6 +546,13 @@ public function testAddingMacAddress()
         $this->assertEquals('alter table "users" add column "foo" macaddr not null', $statements[0]);
     }
 
+    public function testDropAllTablesEscapesTableNames()
+    {
+        $statement = $this->getGrammar()->compileDropAllTables(['alpha', 'beta', 'gamma']);
+
+        $this->assertEquals('drop table "alpha","beta","gamma" cascade', $statement);
+    }
+
     protected function getConnection()
     {
         return m::mock('Illuminate\Database\Connection');