diff options
author | Carl Schwan <carl@carlschwan.eu> | 2022-06-21 17:10:48 +0300 |
---|---|---|
committer | Carl Schwan <carl@carlschwan.eu> | 2022-06-21 21:23:32 +0300 |
commit | 9b6aa4d59bb8c6a5d8472416350ae62714e52ec1 (patch) | |
tree | a693b9e4d95a96b0e0321eb08e5c5045007dc2c0 | |
parent | dbc2c2325ea194b4588515e5fad988b7ab9bcf3f (diff) |
Make it easier to debug issue #32304more-debug-lazyuserfolder
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
-rw-r--r-- | lib/private/Files/Node/LazyFolder.php | 7 | ||||
-rw-r--r-- | lib/private/Files/Node/LazyUserFolder.php | 19 |
2 files changed, 19 insertions, 7 deletions
diff --git a/lib/private/Files/Node/LazyFolder.php b/lib/private/Files/Node/LazyFolder.php index 1bae0f52e59..33b2164c5ae 100644 --- a/lib/private/Files/Node/LazyFolder.php +++ b/lib/private/Files/Node/LazyFolder.php @@ -27,6 +27,7 @@ declare(strict_types=1); namespace OC\Files\Node; use OC\Files\Utils\PathHelper; +use OCP\Files\Folder; use OCP\Constants; /** @@ -37,8 +38,8 @@ use OCP\Constants; * * @package OC\Files\Node */ -class LazyFolder implements \OCP\Files\Folder { - /** @var \Closure */ +class LazyFolder implements Folder { + /** @var \Closure(): Folder */ private $folderClosure; /** @var LazyFolder | null */ @@ -49,7 +50,7 @@ class LazyFolder implements \OCP\Files\Folder { /** * LazyFolder constructor. * - * @param \Closure $folderClosure + * @param \Closure(): Folder $folderClosure */ public function __construct(\Closure $folderClosure, array $data = []) { $this->folderClosure = $folderClosure; diff --git a/lib/private/Files/Node/LazyUserFolder.php b/lib/private/Files/Node/LazyUserFolder.php index c85a356ddd3..8e5ac179629 100644 --- a/lib/private/Files/Node/LazyUserFolder.php +++ b/lib/private/Files/Node/LazyUserFolder.php @@ -27,7 +27,10 @@ use OCP\Files\FileInfo; use OCP\Constants; use OCP\Files\IRootFolder; use OCP\Files\NotFoundException; +use OCP\Files\Folder; +use OCP\Files\File; use OCP\IUser; +use Psr\Log\LoggerInterface; class LazyUserFolder extends LazyFolder { private IRootFolder $root; @@ -38,14 +41,22 @@ class LazyUserFolder extends LazyFolder { $this->root = $rootFolder; $this->user = $user; $this->path = '/' . $user->getUID() . '/files'; - parent::__construct(function () use ($user) { + parent::__construct(function () use ($user): Folder { try { - return $this->root->get('/' . $user->getUID() . '/files'); + $node = $this->root->get($this->path); + if ($node instanceof File) { + $e = new \RuntimeException(); + \OCP\Server::get(LoggerInterface::class)->error('User root storage is not a folder: ' . $this->path, [ + 'exception' => $e, + ]); + throw $e; + } + return $node; } catch (NotFoundException $e) { if (!$this->root->nodeExists('/' . $user->getUID())) { $this->root->newFolder('/' . $user->getUID()); } - return $this->root->newFolder('/' . $user->getUID() . '/files'); + return $this->root->newFolder($this->path); } }, [ 'path' => $this->path, @@ -56,7 +67,7 @@ class LazyUserFolder extends LazyFolder { } public function get($path) { - return $this->root->get('/' . $this->user->getUID() . '/files/' . ltrim($path, '/')); + return $this->root->get('/' . $this->user->getUID() . '/files' . ltrim($path, '/')); } /** |