Merge pull request #8614 from kenjis/fix-model-save-empty-id

fix: Model::shouldUpdate() logic

Author: kenjis

system/BaseModel.php

@@ -731,7 +731,7 @@ protected function shouldUpdate($row): bool
     {
         $id = $this->getIdValue($row);
 
-        return ! ($id === null || $id === []);
+        return ! ($id === null || $id === [] || $id === '');
     }
 
     /**

tests/system/Models/SaveModelTest.php

@@ -42,6 +42,24 @@ public function testSaveNewRecordObject(): void
         $this->seeInDatabase('job', ['name' => 'Magician']);
     }
 
+    /**
+     * @see https://github.com/codeigniter4/CodeIgniter4/issues/8613
+     */
+    public function testSaveNewRecordArrayWithEmptyStringId(): void
+    {
+        $this->createModel(JobModel::class);
+
+        $data = [
+            'id'          => '',
+            'name'        => 'Magician',
+            'description' => 'Makes peoples things dissappear.',
+        ];
+
+        $this->model->save($data);
+
+        $this->seeInDatabase('job', ['name' => 'Magician']);
+    }
+
     public function testSaveNewRecordArray(): void
     {
         $this->createModel(JobModel::class);