[12.x] Fix dropping schema-qualified prefixed tables (#54834)

* fix dropping prefixed tables

* add tests

* fix tests

Author: hafezdivandari

src/Illuminate/Database/Schema/Grammars/MySqlGrammar.php

@@ -653,7 +653,7 @@ public function compileRenameIndex(Blueprint $blueprint, Fluent $command)
      */
     public function compileDropAllTables($tables)
     {
-        return 'drop table '.implode(', ', $this->wrapArray($tables));
+        return 'drop table '.implode(', ', $this->escapeNames($tables));
     }
 
     /**
@@ -664,7 +664,7 @@ public function compileDropAllTables($tables)
      */
     public function compileDropAllViews($views)
     {
-        return 'drop view '.implode(', ', $this->wrapArray($views));
+        return 'drop view '.implode(', ', $this->escapeNames($views));
     }
 
     /**
@@ -702,6 +702,20 @@ public function compileTableComment(Blueprint $blueprint, Fluent $command)
         );
     }
 
+    /**
+     * Quote-escape the given tables, views, or types.
+     *
+     * @param  array  $names
+     * @return array
+     */
+    public function escapeNames($names)
+    {
+        return array_map(
+            fn ($name) => (new Collection(explode('.', $name)))->map($this->wrapValue(...))->implode('.'),
+            $names
+        );
+    }
+
     /**
      * Create the column definition for a char type.
      *

src/Illuminate/Database/Schema/Grammars/PostgresGrammar.php

@@ -682,11 +682,10 @@ public function compileTableComment(Blueprint $blueprint, Fluent $command)
      */
     public function escapeNames($names)
     {
-        return array_map(static function ($name) {
-            return '"'.(new Collection(explode('.', $name)))
-                ->map(fn ($segment) => trim($segment, '\'"'))
-                ->implode('"."').'"';
-        }, $names);
+        return array_map(
+            fn ($name) => (new Collection(explode('.', $name)))->map($this->wrapValue(...))->implode('.'),
+            $names
+        );
     }
 
     /**

tests/Database/DatabaseMySqlSchemaGrammarTest.php

@@ -1490,6 +1490,22 @@ public function testDropAllViews()
         $this->assertSame('drop view `alpha`, `beta`, `gamma`', $statement);
     }
 
+    public function testDropAllTablesWithPrefixAndSchema()
+    {
+        $connection = $this->getConnection(prefix: 'prefix_');
+        $statement = $this->getGrammar($connection)->compileDropAllTables(['schema.alpha', 'schema.beta', 'schema.gamma']);
+
+        $this->assertSame('drop table `schema`.`alpha`, `schema`.`beta`, `schema`.`gamma`', $statement);
+    }
+
+    public function testDropAllViewsWithPrefixAndSchema()
+    {
+        $connection = $this->getConnection(prefix: 'prefix_');
+        $statement = $this->getGrammar($connection)->compileDropAllViews(['schema.alpha', 'schema.beta', 'schema.gamma']);
+
+        $this->assertSame('drop view `schema`.`alpha`, `schema`.`beta`, `schema`.`gamma`', $statement);
+    }
+
     public function testGrammarsAreMacroable()
     {
         // compileReplace macro.

tests/Database/DatabasePostgresSchemaGrammarTest.php

@@ -1110,6 +1110,38 @@ public function testDropAllTypesEscapesTableNames()
         $this->assertSame('drop type "alpha", "beta", "gamma" cascade', $statement);
     }
 
+    public function testDropAllTablesWithPrefixAndSchema()
+    {
+        $connection = $this->getConnection(prefix: 'prefix_');
+        $statement = $this->getGrammar($connection)->compileDropAllTables(['schema.alpha', 'schema.beta', 'schema.gamma']);
+
+        $this->assertSame('drop table "schema"."alpha", "schema"."beta", "schema"."gamma" cascade', $statement);
+    }
+
+    public function testDropAllViewsWithPrefixAndSchema()
+    {
+        $connection = $this->getConnection(prefix: 'prefix_');
+        $statement = $this->getGrammar($connection)->compileDropAllViews(['schema.alpha', 'schema.beta', 'schema.gamma']);
+
+        $this->assertSame('drop view "schema"."alpha", "schema"."beta", "schema"."gamma" cascade', $statement);
+    }
+
+    public function testDropAllTypesWithPrefixAndSchema()
+    {
+        $connection = $this->getConnection(prefix: 'prefix_');
+        $statement = $this->getGrammar($connection)->compileDropAllTypes(['schema.alpha', 'schema.beta', 'schema.gamma']);
+
+        $this->assertSame('drop type "schema"."alpha", "schema"."beta", "schema"."gamma" cascade', $statement);
+    }
+
+    public function testDropAllDomainsWithPrefixAndSchema()
+    {
+        $connection = $this->getConnection(prefix: 'prefix_');
+        $statement = $this->getGrammar($connection)->compileDropAllDomains(['schema.alpha', 'schema.beta', 'schema.gamma']);
+
+        $this->assertSame('drop domain "schema"."alpha", "schema"."beta", "schema"."gamma" cascade', $statement);
+    }
+
     public function testCompileColumns()
     {
         $connection = $this->getConnection();
@@ -1122,10 +1154,11 @@ public function testCompileColumns()
 
     protected function getConnection(
         ?PostgresGrammar $grammar = null,
-        ?PostgresBuilder $builder = null
+        ?PostgresBuilder $builder = null,
+        string $prefix = ''
     ) {
         $connection = m::mock(Connection::class)
-            ->shouldReceive('getTablePrefix')->andReturn('')
+            ->shouldReceive('getTablePrefix')->andReturn($prefix)
             ->shouldReceive('getConfig')->with('prefix_indexes')->andReturn(null)
             ->getMock();