feat(cache): use the insertIgnoreConflict connexion method so that conflicts are properly handled

In the FileCache, use insertIgnoreConflict to avoid conflict error messages on PostgreSQL. Use the
new optional callback argument to hintShardKey even though insertIgnoreConflict doesn't support that

Closes #19494

Signed-off-by: Thomas Citharel <tcit@tcit.fr>

Author: tcitworld

lib/private/Files/Cache/Cache.php

@@ -394,38 +394,9 @@ public function update($id, array $data) {
 		}
 
 		if (count($extensionValues)) {
-			try {
-				$query = $this->getQueryBuilder();
-				$query->insert('filecache_extended');
-				$query->hintShardKey('storage', $this->getNumericStorageId());
-
-				$query->setValue('fileid', $query->createNamedParameter($id, IQueryBuilder::PARAM_INT));
-				foreach ($extensionValues as $column => $value) {
-					$query->setValue($column, $query->createNamedParameter($value));
-				}
-
-				$query->executeStatement();
-			} catch (Exception $e) {
-				if ($e->getReason() !== Exception::REASON_UNIQUE_CONSTRAINT_VIOLATION) {
-					throw $e;
-				}
-				$query = $this->getQueryBuilder();
-				$query->update('filecache_extended')
-					->whereFileId($id)
-					->hintShardKey('storage', $this->getNumericStorageId())
-					->andWhere($query->expr()->orX(...array_map(function ($key, $value) use ($query) {
-						return $query->expr()->orX(
-							$query->expr()->neq($key, $query->createNamedParameter($value)),
-							$query->expr()->isNull($key)
-						);
-					}, array_keys($extensionValues), array_values($extensionValues))));
-
-				foreach ($extensionValues as $key => $value) {
-					$query->set($key, $query->createNamedParameter($value));
-				}
-
-				$query->executeStatement();
-			}
+			$this->connection->insertIgnoreConflict('filecache_extended', array_merge(['fileid' => $id], $extensionValues), function (IQueryBuilder $queryCallback) {
+				$queryCallback->hintShardKey('storage', $this->getNumericStorageId());
+			});
 		}
 
 		$path = $this->getPathById($id);