DigitaleBibliothek / Frontend / ab579da6148

Commits

Tim Donohue authored and GitHub committed ab579da6148 Merge 01 Feb 2024

Merge pull request #2790 from DSpace/backport-2778-to-dspace-7_x

[Port dspace-7_x] Dedupe patch operations by op and path

Modified

-344
+        value: value
-345
+      }));
-346
+      break;
-347
+    case JsonPatchOperationsActionTypes.NEW_JSON_PATCH_REMOVE_OPERATION:
-348
+      newBody.push(makeOperationEntry({ op: JsonPatchOperationType.remove, path: targetPath }));
-349
+      break;
-350
+    case JsonPatchOperationsActionTypes.NEW_JSON_PATCH_MOVE_OPERATION:
-351
+      newBody.push(makeOperationEntry({ op: JsonPatchOperationType.move, from: fromPath, path: targetPath }));
-352
+      break;
-353
+  }
--
+  return newBody;
-+
+  return dedupeOperationEntries(newBody);
-+
+}
-+
-+
+/**
-+
+ * Dedupe operation entries by op and path. This prevents processing unnecessary patches in a single PATCH request.
-+
+ *
-+
+ * @param body JSON patch operation object entries
-+
+ * @returns deduped JSON patch operation object entries
-+
+ */
-+
+function dedupeOperationEntries(body: JsonPatchOperationObject[]): JsonPatchOperationObject[] {
-+
+  const ops = new Map<string, any>();
-+
+  for (let i = body.length - 1; i >= 0; i--) {
-+
+    const patch = body[i].operation;
-+
+    const key = `${patch.op}-${patch.path}`;
-+
+    if (!ops.has(key)) {
-+
+      ops.set(key, patch);
-+
+    } else {
-+
+      body.splice(i, 1);
-+
+    }
-+
+  }
-+
-+
+  return body;
-376
+}
-377
-378
+function makeOperationEntry(operation) {
-379
+  return { operation: operation, timeCompleted: new Date().getTime() };
-380
+}