perf: Avoid re-query of already fetched lock info

Signed-off-by: Julius Härtl <[email protected]>
nextcloud · Dec 19, 2023 · 03606e5 · 03606e5
1 parent 0401fb1
commit 03606e5
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/lib/Service/LockService.php b/lib/Service/LockService.php
@@ -132,6 +132,7 @@ public function getLockForNodeIds(array $nodeIds): array {
 				$locks[$nodeId] = $this->lockCache[$nodeId];
 			} else {
 				$locksToRequest[] = $nodeId;
+				$this->lockCache[$nodeId] = false;
 			}
 		}
 		if (count($locksToRequest) === 0) {
@@ -144,10 +145,10 @@ public function getLockForNodeIds(array $nodeIds): array {
 		}
 		$newLocks = array_merge(...$newLocks);
 
+		$expiredLocks = [];
 		foreach ($newLocks as $lock) {
 			if ($lock->getETA() === 0) {
-				// TODO batch remove
-				$this->locksRequest->delete($lock);
+				$expiredLocks[] = $lock->getId();
 				$locks[$lock->getFileId()] = false;
 				$this->lockCache[$lock->getFileId()] = false;
 			} else {
@@ -156,6 +157,10 @@ public function getLockForNodeIds(array $nodeIds): array {
 			}
 		}
 
+		if (count($expiredLocks) > 0) {
+			$this->locksRequest->removeIds($expiredLocks);
+		}
+
 		return $locks;
 	}