Don't use a stream response on an empty file

Streaming an empty file does boom currently.
So lets use a differnt response for that.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

1 files changed, 12 insertions, 4 deletions

diff --git a/lib/Controller/WopiController.php b/lib/Controller/WopiController.php
index a0a475bd..0fbc5ef3 100644
--- a/lib/Controller/WopiController.php
+++ b/lib/Controller/WopiController.php
@@ -353,15 +353,23 @@ class WopiController extends Controller {
 				$versionPath = '/files_versions/' . $relPath . '.v' . $version;
 				$view = new View('/' . $wopi->getOwnerUid());
 				if ($view->file_exists($versionPath)){
-					$response = new StreamResponse($view->fopen($versionPath, 'rb'));
+					$info = $view->getFileInfo($versionPath);
+					if ($info->getSize() === 0) {
+						$response = new Http\Response();
+					} else {
+						$response = new StreamResponse($view->fopen($versionPath, 'rb'));
+					}
 				}
 				else {
 					return new JSONResponse([], Http::STATUS_NOT_FOUND);
 				}
 			}
-			else
-			{
-				$response = new StreamResponse($file->fopen('rb'));
+			else {
+				if ($file->getSize() === 0) {
+					$response = new Http\Response();
+				} else {
+					$response = new StreamResponse($file->fopen('rb'));
+				}
 			}
 			$response->addHeader('Content-Disposition', 'attachment');
 			$response->addHeader('Content-Type', 'application/octet-stream');