Merge pull request #6063 from nextcloud/backport/6054/stable34

[stable34] fix(wopi): report a locked file as 423, not 500

Author: elzody

lib/Controller/WopiController.php

@@ -665,7 +665,10 @@ public function putFile(
 				$this->wrappedFilesystemOperation($wopi, fn () => $file->putContent($content));
 			} catch (LockedException $e) {
 				$this->logger->error($e->getMessage(), ['exception' => $e]);
-				return new JSONResponse(['message' => 'File locked'], Http::STATUS_INTERNAL_SERVER_ERROR);
+				// The file is locked by another operation and we wrote nothing.
+				// Report it as such, so the client can retry rather than treat
+				// this as a server fault or as a change behind its back.
+				return new JSONResponse(['message' => 'File locked'], Http::STATUS_LOCKED);
 			}
 
 			if ($wopi->hasTemplateId()) {
@@ -808,7 +811,8 @@ public function postFile(
 			try {
 				$this->wrappedFilesystemOperation($wopi, fn () => $file->putContent($content));
 			} catch (LockedException) {
-				return new JSONResponse(['message' => 'File locked'], Http::STATUS_INTERNAL_SERVER_ERROR);
+				// As in putFile(): nothing was written, so this is not a server fault.
+				return new JSONResponse(['message' => 'File locked'], Http::STATUS_LOCKED);
 			}
 
 			// epub is exception (can be uploaded but not opened so don't try to get access token)