Catch the exception when a user no longer exists

Else this could result in a lot of log spam.

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
This commit is contained in:
Roeland Jago Douma 2020-01-25 20:17:50 +01:00 коммит произвёл Julius Härtl
Родитель 8b87d83cbe
Коммит 74117d916e
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4C614C6ED2CDE6DF
1 изменённых файлов: 8 добавлений и 1 удалений

Просмотреть файл

@ -343,7 +343,14 @@ class DocumentService {
* @throws NotFoundException
*/
public function getFileById($fileId, $userId = null): Node {
$files = $this->rootFolder->getUserFolder($this->userId ?? $userId)->getById($fileId);
try {
$userFolder = $this->rootFolder->getUserFolder($this->userId ?? $userId);
} catch (\OC\User\NoUserException $e) {
// It is a bit hacky to depend on internal exceptions here. But it is the best we can do for now
throw new NotFoundException();
}
$files = $userFolder->getById($fileId);
if (count($files) === 0) {
throw new NotFoundException();
}