fix(dav): preserves the old hook-accessibility contract while still improving consistency

Signed-off-by: Josh <josh.t.richards@gmail.com>

Author: joshtrichards

apps/dav/lib/Connector/Sabre/File.php

@@ -345,8 +345,9 @@ public function put($data) {
 
 	private function finalizeUpload(IStorage $storage, string $internalPath, bool $exists, ?View $view): void {
 		// Since we skipped the view for the final publish step, finalize the file
-		// state explicitly here: update cache/bookkeeping, persist metadata, emit
-		// post-write hooks, and only then downgrade the lock.
+		// state explicitly here: update cache/bookkeeping, persist metadata, then
+		// downgrade to a shared lock before emitting post-write hooks so listeners
+		// can still access the file.
 		$storage->getUpdater()->update($internalPath);
 
 		$fileInfoUpdate = [
@@ -381,16 +382,17 @@ private function finalizeUpload(IStorage $storage, string $internalPath, bool $e
 		$this->fileView->putFileInfo($this->path, $fileInfoUpdate);
 		$this->refreshInfo();
 
-		if ($view) {
-			$this->emitPostHooks($exists);
-		}
-
-		// Keep the exclusive lock until all bookkeeping and metadata updates are complete.
+		// Downgrade to shared lock before post hooks so legacy hook consumers can
+		// still access the file during post_write.
 		try {
 			$this->changeLock(ILockingProvider::LOCK_SHARED);
 		} catch (LockedException $e) {
 			throw new FileLocked($e->getMessage(), $e->getCode(), $e);
 		}
+
+		if ($view) {
+			$this->emitPostHooks($exists);
+		}
 	}
 
 	private function getPartFileBasePath($path) {