fix: Catch unique constraint violation when creating new documents

Signed-off-by: Julius Härtl <jus@bitgrid.net>
nextcloud · Jun 13, 2023 · cfd0aa6 · cfd0aa6
1 parent b9ca139
commit cfd0aa6
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/lib/Service/DocumentService.php b/lib/Service/DocumentService.php
@@ -154,8 +154,17 @@ public function createDocument(File $file): Document {
 		$document->setLastSavedVersionTime($file->getFileInfo()->getMtime());
 		$document->setLastSavedVersionEtag($file->getEtag());
 		$document->setBaseVersionEtag($file->getEtag());
-		$document = $this->documentMapper->insert($document);
-		$this->cache->set('document-version-'.$document->getId(), 0);
+		try {
+			$document = $this->documentMapper->insert($document);
+			$this->cache->set('document-version-'.$document->getId(), 0);
+		} catch (Exception $e) {
+			if ($e->getReason() === Exception::REASON_UNIQUE_CONSTRAINT_VIOLATION) {
+				// Document might have been created in the meantime
+				return $this->documentMapper->find($file->getFileInfo()->getId());
+			}
+
+			throw $e;
+		}
 		return $document;
 	}