fix: invalid INSERT query when builder uses table alias

Author: kenjis

system/Database/BaseBuilder.php

@@ -1863,7 +1863,7 @@ public function getCompiledInsert(bool $reset = true)
         }
 
         $sql = $this->_insert(
-            $this->db->protectIdentifiers($this->QBFrom[0], true, null, false),
+            $this->db->protectIdentifiers($this->removeAlias($this->QBFrom[0]), true, null, false),
             array_keys($this->QBSet),
             array_values($this->QBSet)
         );
@@ -1895,7 +1895,7 @@ public function insert($set = null, ?bool $escape = null)
         }
 
         $sql = $this->_insert(
-            $this->db->protectIdentifiers($this->QBFrom[0], true, $escape, false),
+            $this->db->protectIdentifiers($this->removeAlias($this->QBFrom[0]), true, $escape, false),
             array_keys($this->QBSet),
             array_values($this->QBSet)
         );
@@ -1914,6 +1914,19 @@ public function insert($set = null, ?bool $escape = null)
         return false;
     }
 
+    protected function removeAlias(string $from): string
+    {
+        if (strpos($from, ' ') !== false) {
+            // if the alias is written with the AS keyword, remove it
+            $from = preg_replace('/\s+AS\s+/i', ' ', $from);
+
+            $parts = explode(' ', $from);
+            $from  = $parts[0];
+        }
+
+        return $from;
+    }
+
     /**
      * This method is used by both insert() and getCompiledInsert() to
      * validate that the there data is actually being set and that table

tests/system/Database/Builder/InsertTest.php

@@ -85,6 +85,35 @@ public function testInsertObject()
         $this->assertSame($expectedBinds, $builder->getBinds());
     }
 
+    /**
+     * @see https://github.com/codeigniter4/CodeIgniter4/issues/5365
+     */
+    public function testInsertWithTableAlias()
+    {
+        $builder = $this->db->table('jobs as j');
+
+        $insertData = [
+            'id'   => 1,
+            'name' => 'Grocery Sales',
+        ];
+        $builder->testMode()->insert($insertData, true);
+
+        $expectedSQL   = 'INSERT INTO "jobs" ("id", "name") VALUES (1, \'Grocery Sales\')';
+        $expectedBinds = [
+            'id' => [
+                1,
+                true,
+            ],
+            'name' => [
+                'Grocery Sales',
+                true,
+            ],
+        ];
+
+        $this->assertSame($expectedSQL, str_replace("\n", ' ', $builder->getCompiledInsert()));
+        $this->assertSame($expectedBinds, $builder->getBinds());
+    }
+
     public function testThrowsExceptionOnNoValuesSet()
     {
         $builder = $this->db->table('jobs');