diff options
Diffstat (limited to 'lib/private/Archive/ZIP.php')
-rw-r--r-- | lib/private/Archive/ZIP.php | 169 |
1 files changed, 82 insertions, 87 deletions
diff --git a/lib/private/Archive/ZIP.php b/lib/private/Archive/ZIP.php index 96894bac6e9..61fd25d071d 100644 --- a/lib/private/Archive/ZIP.php +++ b/lib/private/Archive/ZIP.php @@ -1,73 +1,49 @@ <?php + /** - * @copyright Copyright (c) 2016, ownCloud, Inc. - * - * @author Arthur Schiwon <blizzz@arthur-schiwon.de> - * @author Bart Visscher <bartv@thisnet.nl> - * @author Christopher Schäpers <kondou@ts.unde.re> - * @author Christoph Wurst <christoph@winzerhof-wurst.at> - * @author Joas Schilling <coding@schilljs.com> - * @author Jörn Friedrich Dreyer <jfd@butonic.de> - * @author Morris Jobke <hey@morrisjobke.de> - * @author Robin Appelman <robin@icewind.nl> - * @author Roeland Jago Douma <roeland@famdouma.nl> - * @author Stefan Weil <sw@weilnetz.de> - * @author Thomas Müller <thomas.mueller@tmit.eu> - * - * @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 <http://www.gnu.org/licenses/> - * + * SPDX-FileCopyrightText: 2016-2024 Nextcloud GmbH and Nextcloud contributors + * SPDX-FileCopyrightText: 2016 ownCloud, Inc. + * SPDX-License-Identifier: AGPL-3.0-only */ - namespace OC\Archive; use Icewind\Streams\CallbackWrapper; -use OCP\ILogger; +use Psr\Log\LoggerInterface; class ZIP extends Archive { /** * @var \ZipArchive zip */ - private $zip = null; - private $path; + private $zip; /** - * @param string $source + * @var string */ - public function __construct($source) { + private $path; + + public function __construct(string $source) { $this->path = $source; $this->zip = new \ZipArchive(); if ($this->zip->open($source, \ZipArchive::CREATE)) { } else { - \OCP\Util::writeLog('files_archive', 'Error while opening archive '.$source, ILogger::WARN); + \OC::$server->get(LoggerInterface::class)->warning('Error while opening archive ' . $source, ['app' => 'files_archive']); } } + /** * add an empty folder to the archive * @param string $path * @return bool */ - public function addFolder($path) { + public function addFolder(string $path): bool { return $this->zip->addEmptyDir($path); } + /** * add a file to the archive - * @param string $path * @param string $source either a local file or string data - * @return bool */ - public function addFile($path, $source = '') { + public function addFile(string $path, string $source = ''): bool { if ($source and $source[0] == '/' and file_exists($source)) { $result = $this->zip->addFile($source, $path); } else { @@ -79,40 +55,39 @@ class ZIP extends Archive { } return $result; } + /** * rename a file or folder in the archive - * @param string $source - * @param string $dest - * @return boolean|null */ - public function rename($source, $dest) { + public function rename(string $source, string $dest): bool { $source = $this->stripPath($source); $dest = $this->stripPath($dest); - $this->zip->renameName($source, $dest); + return $this->zip->renameName($source, $dest); } + /** * get the uncompressed size of a file in the archive - * @param string $path - * @return int */ - public function filesize($path) { + public function filesize(string $path): false|int|float { $stat = $this->zip->statName($path); - return $stat['size']; + return $stat['size'] ?? false; } + /** * get the last modified time of a file in the archive - * @param string $path - * @return int + * @return int|false */ - public function mtime($path) { + public function mtime(string $path) { return filemtime($this->path); } + /** * get the files in a folder - * @param string $path - * @return array */ - public function getFolder($path) { + public function getFolder(string $path): array { + // FIXME: multiple calls on getFolder would traverse + // the whole file list over and over again + // maybe use a Generator or cache the list ? $files = $this->getFiles(); $folderContent = []; $pathLength = strlen($path); @@ -125,11 +100,37 @@ class ZIP extends Archive { } return $folderContent; } + + /** + * Generator that returns metadata of all files + * + * @return \Generator<array> + */ + public function getAllFilesStat() { + $fileCount = $this->zip->numFiles; + for ($i = 0;$i < $fileCount;$i++) { + yield $this->zip->statIndex($i); + } + } + + /** + * Return stat information for the given path + * + * @param string path path to get stat information on + * @return ?array stat information or null if not found + */ + public function getStat(string $path): ?array { + $stat = $this->zip->statName($path); + if (!$stat) { + return null; + } + return $stat; + } + /** * get all files in the archive - * @return array */ - public function getFiles() { + public function getFiles(): array { $fileCount = $this->zip->numFiles; $files = []; for ($i = 0;$i < $fileCount;$i++) { @@ -137,67 +138,65 @@ class ZIP extends Archive { } return $files; } + /** * get the content of a file - * @param string $path - * @return string + * @return string|false */ - public function getFile($path) { + public function getFile(string $path) { return $this->zip->getFromName($path); } + /** * extract a single file from the archive - * @param string $path - * @param string $dest - * @return boolean|null */ - public function extractFile($path, $dest) { + public function extractFile(string $path, string $dest): bool { $fp = $this->zip->getStream($path); - file_put_contents($dest, $fp); + if ($fp === false) { + return false; + } + return file_put_contents($dest, $fp) !== false; } + /** * extract the archive - * @param string $dest - * @return bool */ - public function extract($dest) { + public function extract(string $dest): bool { return $this->zip->extractTo($dest); } + /** * check if a file or folder exists in the archive - * @param string $path - * @return bool */ - public function fileExists($path) { - return ($this->zip->locateName($path) !== false) or ($this->zip->locateName($path.'/') !== false); + public function fileExists(string $path): bool { + return ($this->zip->locateName($path) !== false) or ($this->zip->locateName($path . '/') !== false); } + /** * remove a file or folder from the archive - * @param string $path - * @return bool */ - public function remove($path) { - if ($this->fileExists($path.'/')) { - return $this->zip->deleteName($path.'/'); + public function remove(string $path): bool { + if ($this->fileExists($path . '/')) { + return $this->zip->deleteName($path . '/'); } else { return $this->zip->deleteName($path); } } + /** * get a file handler - * @param string $path - * @param string $mode * @return bool|resource */ - public function getStream($path, $mode) { + public function getStream(string $path, string $mode) { if ($mode == 'r' or $mode == 'rb') { return $this->zip->getStream($path); } else { //since we can't directly get a writable stream, //make a temp copy of the file and put it back //in the archive when the stream is closed - if (strrpos($path, '.') !== false) { - $ext = substr($path, strrpos($path, '.')); + $lastPoint = strrpos($path, '.'); + if ($lastPoint !== false) { + $ext = substr($path, $lastPoint); } else { $ext = ''; } @@ -215,16 +214,12 @@ class ZIP extends Archive { /** * write back temporary files */ - public function writeBack($tmpFile, $path) { + public function writeBack(string $tmpFile, string $path): void { $this->addFile($path, $tmpFile); unlink($tmpFile); } - /** - * @param string $path - * @return string - */ - private function stripPath($path) { + private function stripPath(string $path): string { if (!$path || $path[0] == '/') { return substr($path, 1); } else { |