From a9fa220e6824a76bbb8d4b65a9b80f25383cdee4 Mon Sep 17 00:00:00 2001 From: Jakob Sack Date: Sun, 2 Sep 2018 15:48:13 +0200 Subject: Add fix response implements #7589 --- lib/public/AppFramework/Http/ZipResponse.php | 75 ++++++++++++++++++++++++++++ 1 file changed, 75 insertions(+) create mode 100644 lib/public/AppFramework/Http/ZipResponse.php (limited to 'lib/public') diff --git a/lib/public/AppFramework/Http/ZipResponse.php b/lib/public/AppFramework/Http/ZipResponse.php new file mode 100644 index 00000000000..27ff30d4019 --- /dev/null +++ b/lib/public/AppFramework/Http/ZipResponse.php @@ -0,0 +1,75 @@ + + * @author Roeland Jago Douma + * + * @license AGPL-3.0 + * + * This code is free software: you can redistribute it and/or modify + * it under the terms of the GNU Affero General Public License, version 3, + * as published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Affero General Public License for more details. + * + * You should have received a copy of the GNU Affero General Public License, version 3, + * along with this program. If not, see + * + */ + +/** + * Public library to send several files in one zip archive. + */ + +namespace OCP\AppFramework\Http; + +use OCP\IRequest; +use OC\Streamer; + +class ZipResponse extends Response implements ICallbackResponse { + /** @var resource[] Files to be added to the zip response */ + private $resources; + /** @var string Filename that the zip file should have */ + private $name; + private $request; + + public function __construct(string $name = 'output', IRequest $request) { + $this->name = $name; + $this->request = $request; + } + + public function addResource($r, string $internalName, int $size, int $time = -1) { + if (!\is_resource($r)) { + return; + } + + $this->resources[] = [ + 'resource' => $r, + 'internalName' => $internalName, + 'size' => $size, + 'time' => $time, + ]; + } + + public function callback(IOutput $output) { + $size = 0; + $files = count($this->resources); + + foreach ($this->resources as $resource) { + $size = $size + $resource['size']; + } + + $zip = new Streamer($this->request, $size, $files); + $zip->sendHeaders($this->name); + + foreach ($this->resources as $resource) { + $zip->addFileFromStream($resource['resource'], $resource['internalName'], $resource['size'], $resource['time']); + } + + $zip->finalize(); + } +} -- cgit v1.2.3 From a891f42a5de0b0a2a67d03525d81c028fdb3caf1 Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Tue, 2 Oct 2018 08:16:28 +0200 Subject: fixup! Add fix response Signed-off-by: Roeland Jago Douma --- lib/public/AppFramework/Http/ZipResponse.php | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'lib/public') diff --git a/lib/public/AppFramework/Http/ZipResponse.php b/lib/public/AppFramework/Http/ZipResponse.php index 27ff30d4019..c70482a0a9b 100644 --- a/lib/public/AppFramework/Http/ZipResponse.php +++ b/lib/public/AppFramework/Http/ZipResponse.php @@ -1,6 +1,8 @@ * * @author Jakob Sack * @author Roeland Jago Douma @@ -37,7 +39,7 @@ class ZipResponse extends Response implements ICallbackResponse { private $name; private $request; - public function __construct(string $name = 'output', IRequest $request) { + public function __construct(IRequest $request, string $name = 'output') { $this->name = $name; $this->request = $request; } @@ -60,7 +62,7 @@ class ZipResponse extends Response implements ICallbackResponse { $files = count($this->resources); foreach ($this->resources as $resource) { - $size = $size + $resource['size']; + $size += $resource['size']; } $zip = new Streamer($this->request, $size, $files); -- cgit v1.2.3 From 7d9052d4b9d4ac690c32d7f9e79da2a06b0df055 Mon Sep 17 00:00:00 2001 From: Roeland Jago Douma Date: Tue, 2 Oct 2018 08:17:27 +0200 Subject: fixup! Add fix response Signed-off-by: Roeland Jago Douma --- lib/public/AppFramework/Http/ZipResponse.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/public') diff --git a/lib/public/AppFramework/Http/ZipResponse.php b/lib/public/AppFramework/Http/ZipResponse.php index c70482a0a9b..712af17d2d7 100644 --- a/lib/public/AppFramework/Http/ZipResponse.php +++ b/lib/public/AppFramework/Http/ZipResponse.php @@ -46,7 +46,7 @@ class ZipResponse extends Response implements ICallbackResponse { public function addResource($r, string $internalName, int $size, int $time = -1) { if (!\is_resource($r)) { - return; + throw new \InvalidArgumentException('No resource provided'); } $this->resources[] = [ -- cgit v1.2.3 From bcbffdb644a125e436d40ba83fdd45dba8362181 Mon Sep 17 00:00:00 2001 From: Morris Jobke Date: Tue, 2 Oct 2018 22:35:31 +0200 Subject: Add PHPDoc Signed-off-by: Morris Jobke --- lib/public/AppFramework/Http/ZipResponse.php | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) (limited to 'lib/public') diff --git a/lib/public/AppFramework/Http/ZipResponse.php b/lib/public/AppFramework/Http/ZipResponse.php index 712af17d2d7..630efb38c7d 100644 --- a/lib/public/AppFramework/Http/ZipResponse.php +++ b/lib/public/AppFramework/Http/ZipResponse.php @@ -23,15 +23,16 @@ declare(strict_types=1); * */ -/** - * Public library to send several files in one zip archive. - */ - namespace OCP\AppFramework\Http; use OCP\IRequest; use OC\Streamer; +/** + * Public library to send several files in one zip archive. + * + * @since 15.0.0 + */ class ZipResponse extends Response implements ICallbackResponse { /** @var resource[] Files to be added to the zip response */ private $resources; @@ -39,11 +40,17 @@ class ZipResponse extends Response implements ICallbackResponse { private $name; private $request; + /** + * @since 15.0.0 + */ public function __construct(IRequest $request, string $name = 'output') { $this->name = $name; $this->request = $request; } + /** + * @since 15.0.0 + */ public function addResource($r, string $internalName, int $size, int $time = -1) { if (!\is_resource($r)) { throw new \InvalidArgumentException('No resource provided'); @@ -57,6 +64,9 @@ class ZipResponse extends Response implements ICallbackResponse { ]; } + /** + * @since 15.0.0 + */ public function callback(IOutput $output) { $size = 0; $files = count($this->resources); -- cgit v1.2.3