fixup! feat: extend Entity and adjust QBMapper to support Snowflake IDs

miaulalala · miaulalala · commit ba2f797f9ec0 · 2025-12-19T00:05:15.000+01:00
diff --git a/lib/public/AppFramework/Db/QBMapper.php b/lib/public/AppFramework/Db/QBMapper.php
@@ -116,10 +116,12 @@ public function insert(Entity $entity): Entity {
 			/** @psalm-suppress DocblockTypeContradiction */
 			$entity->setId();
 			$qb->executeStatement();
-		} else {
+		} elseif ($entity->getId() === null) {
 			$qb->executeStatement();
 			// When autoincrement is used id is always an int
 			$entity->setId($qb->getLastInsertId());
+		} else {
+			$qb->executeStatement();
 		}
 		return $entity;
 	}

Original file line number	Diff line number	Diff line change
`@@ -116,10 +116,12 @@ public function insert(Entity $entity): Entity {`
`116`	`116`	`/** @psalm-suppress DocblockTypeContradiction */`
`117`	`117`	`$entity->setId();`
`118`	`118`	`$qb->executeStatement();`
`119`		`- } else {`
	`119`	`+ } elseif ($entity->getId() === null) {`
`120`	`120`	`$qb->executeStatement();`
`121`	`121`	`// When autoincrement is used id is always an int`
`122`	`122`	`$entity->setId($qb->getLastInsertId());`
	`123`	`+ } else {`
	`124`	`+ $qb->executeStatement();`
`123`	`125`	`}`
`124`	`126`	`return $entity;`
`125`	`127`	`}`