refactor(Streamer): inject IDateTimeZone as constructor arg

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

Author: susnux

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

@@ -20,6 +20,7 @@
 use OCP\Files\IFilenameValidator;
 use OCP\Files\Mount\IMountManager;
 use OCP\IConfig;
+use OCP\IDateTimeZone;
 use OCP\IDBConnection;
 use OCP\IL10N;
 use OCP\IPreview;
@@ -79,6 +80,7 @@ public function createServer(string $baseUri,
 			$objectTree,
 			$this->logger,
 			$this->eventDispatcher,
+			\OCP\Server::get(IDateTimeZone::class),
 		));
 
 		// Some WebDAV clients do require Class 2 WebDAV support (locking), since

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

@@ -15,6 +15,7 @@
 use OCP\Files\File as NcFile;
 use OCP\Files\Folder as NcFolder;
 use OCP\Files\Node as NcNode;
+use OCP\IDateTimeZone;
 use Psr\Log\LoggerInterface;
 use Sabre\DAV\Server;
 use Sabre\DAV\ServerPlugin;
@@ -41,6 +42,7 @@ public function __construct(
 		private Tree $tree,
 		private LoggerInterface $logger,
 		private IEventDispatcher $eventDispatcher,
+		private IDateTimeZone $timezoneFactory,
 	) {
 	}
 
@@ -163,7 +165,7 @@ public function handleDownload(Request $request, Response $response): ?bool {
 			// Full folder is loaded to rename the archive to the folder name
 			$archiveName = $folder->getName();
 		}
-		$streamer = new Streamer($tarRequest, -1, count($content));
+		$streamer = new Streamer($tarRequest, -1, count($content), $this->timezoneFactory);
 		$streamer->sendHeaders($archiveName);
 		// For full folder downloads we also add the folder itself to the archive
 		if (empty($files)) {

apps/dav/lib/Server.php

@@ -75,6 +75,7 @@
 use OCP\IAppConfig;
 use OCP\ICacheFactory;
 use OCP\IConfig;
+use OCP\IDateTimeZone;
 use OCP\IPreview;
 use OCP\IRequest;
 use OCP\IUserSession;
@@ -230,6 +231,7 @@ public function __construct(
 			$this->server->tree,
 			$logger,
 			$eventDispatcher,
+			\OCP\Server::get(IDateTimeZone::class),
 		));
 		$this->server->addPlugin(\OCP\Server::get(PaginatePlugin::class));
 

lib/private/Streamer.php

@@ -41,7 +41,12 @@ public static function isUserAgentPreferTar(IRequest $request): bool {
 	 * @param int $numberOfFiles The number of files (and directories) that will
 	 *                           be included in the streamed file
 	 */
-	public function __construct(IRequest|bool $preferTar, int|float $size, int $numberOfFiles) {
+	public function __construct(
+		IRequest|bool $preferTar,
+		int|float $size,
+		int $numberOfFiles,
+		private IDateTimeZone $timezoneFactory,
+	) {
 		if ($preferTar instanceof IRequest) {
 			$preferTar = self::isUserAgentPreferTar($preferTar);
 		}
@@ -197,7 +202,7 @@ private function fixTimestamp(int $timestamp): int {
 		if ($this->streamerInstance instanceof ZipStreamer) {
 			// Zip does not support any timezone information
 			// while tar is interpreted as Unix time the Zip time is interpreted as local time of the user...
-			$zone = \OCP\Server::get(IDateTimeZone::class)->getTimeZone($timestamp);
+			$zone = $this->timezoneFactory->getTimeZone($timestamp);
 			$timestamp += $zone->getOffset(new \DateTimeImmutable('@' . (string)$timestamp));
 		}
 		return $timestamp;

lib/public/AppFramework/Http/ZipResponse.php

@@ -9,6 +9,7 @@
 
 use OC\Streamer;
 use OCP\AppFramework\Http;
+use OCP\IDateTimeZone;
 use OCP\IRequest;
 
 /**
@@ -65,7 +66,7 @@ public function callback(IOutput $output) {
 			$size += $resource['size'];
 		}
 
-		$zip = new Streamer($this->request, $size, $files);
+		$zip = new Streamer($this->request, $size, $files, \OCP\Server::get(IDateTimeZone::class));
 		$zip->sendHeaders($this->name);
 
 		foreach ($this->resources as $resource) {