Make sure identifer column always has the empty value doctrine expects

flack · Jun 13, 2024 · 5134540 · 5134540
1 parent d1d0552
commit 5134540
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/api/dbobject.php b/src/api/dbobject.php
@@ -96,6 +96,9 @@ public function __set($field, $value)
                 $value = (string) $value;
             } elseif ($mapping['type'] === 'integer') {
                 $value = (int) $value;
+                if ($value === 0 && $this->cm->isIdentifier($field)) {
+                    $value = null;
+                }
             } elseif ($mapping['type'] === 'boolean') {
                 $value = (boolean) $value;
             } elseif ($mapping['type'] === 'float') {

diff --git a/test/api/mgdobjectTest.php b/test/api/mgdobjectTest.php
@@ -228,6 +228,7 @@ public function test_create()
         $this->assertEquals($topic->id + 1, $topic2->id);
 
         $topic3 = $this->make_object('midgard_topic');
+        $topic3->id = 0;
         $topic3->up = $topic->id;
         $topic3->name = __FUNCTION__ . '-3';
         $stat = $topic3->create();