diff options
author | Paul Tirk <paultirk@paultirk.com> | 2021-03-24 19:01:32 +0100 |
---|---|---|
committer | Sean Molenaar <SMillerDev@users.noreply.github.com> | 2021-04-08 10:23:11 +0200 |
commit | b45426937eca4324cafa27ee34c1ae8219cc8802 (patch) | |
tree | 7444538e77a71ee3ba3602649421ac632b52b331 /lib/Controller | |
parent | d4ace06ff54b8e3232e7d5730f7f159292924963 (diff) |
return error if no folder name is provided
Signed-off-by: Paul Tirk <paultirk@paultirk.com>
Diffstat (limited to 'lib/Controller')
-rw-r--r-- | lib/Controller/FolderApiV2Controller.php | 22 |
1 files changed, 10 insertions, 12 deletions
diff --git a/lib/Controller/FolderApiV2Controller.php b/lib/Controller/FolderApiV2Controller.php index a1fdcd44c..e849f1cbc 100644 --- a/lib/Controller/FolderApiV2Controller.php +++ b/lib/Controller/FolderApiV2Controller.php @@ -49,19 +49,17 @@ class FolderApiV2Controller extends ApiController */ public function create($name) { - try { - $this->folderService->purgeDeleted($this->getUserId(), false); - $responseData = $this->serializeEntityV2( - $this->folderService->create($this->getUserId(), $name) - ); - return $this->responseV2([ - 'folder' => $responseData - ]); - } catch (ServiceValidationException $ex) { - return $this->errorResponseV2($ex, Http::STATUS_BAD_REQUEST); - } catch (ServiceConflictException $ex) { - return $this->errorResponseV2($ex, Http::STATUS_CONFLICT); + if (empty($name)) { + return $this->errorResponseV2('folder name is empty', 1, Http::STATUS_BAD_REQUEST); } + + $this->folderService->purgeDeleted($this->getUserId(), false); + $responseData = $this->serializeEntityV2( + $this->folderService->create($this->getUserId(), $name) + ); + return $this->responseV2([ + 'folder' => $responseData + ]); } /** |