Merge pull request #22903 from nextcloud/techdebt/16696/no-need-to-get-full-path-again

MorrisJobke · web-flow · commit d144a84df128 · 2020-09-17T11:54:47.000+02:00
Do not fetch the normalized full path again if it is already available
diff --git a/lib/private/Files/Storage/Wrapper/Encryption.php b/lib/private/Files/Storage/Wrapper/Encryption.php
@@ -263,7 +263,7 @@ public function unlink($path) {
 
 		$encryptionModule = $this->getEncryptionModule($path);
 		if ($encryptionModule) {
-			$this->keyStorage->deleteAllFileKeys($this->getFullPath($path));
+			$this->keyStorage->deleteAllFileKeys($fullPath);
 		}
 
 		return $this->storage->unlink($path);

Original file line number	Diff line number	Diff line change
`@@ -263,7 +263,7 @@ public function unlink($path) {`
`263`	`263`
`264`	`264`	`$encryptionModule = $this->getEncryptionModule($path);`
`265`	`265`	`if ($encryptionModule) {`
`266`		`- $this->keyStorage->deleteAllFileKeys($this->getFullPath($path));`
	`266`	`+ $this->keyStorage->deleteAllFileKeys($fullPath);`
`267`	`267`	`}`
`268`	`268`
`269`	`269`	`return $this->storage->unlink($path);`