From 03e0559afe4a8e01cd1d4833c48197a23f4258bb Mon Sep 17 00:00:00 2001 From: Ryan Fletcher Date: Thu, 12 Jul 2018 21:17:04 -0400 Subject: [PATCH] StackApiController->delete($boardId, $id) added in error handling Signed-off-by: Ryan Fletcher --- lib/Controller/StackApiController.php | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/lib/Controller/StackApiController.php b/lib/Controller/StackApiController.php index 77c08c683..f73a768e7 100644 --- a/lib/Controller/StackApiController.php +++ b/lib/Controller/StackApiController.php @@ -123,10 +123,14 @@ class StackApiController extends ApiController { * * Delete the stack specified by $id. Return the board that was deleted. */ - public function delete($boardId, $id) { - $stack = $this->service->delete($id); + public function delete($boardId, $stackId) { + $stack = $this->service->delete($stackId); - return new DataResponse($stack); + if ($stack == false || $stack == null) { + return new DataResponse("Stack Not Found", HTTP::STATUS_NOT_FOUND); + } + + return new DataResponse($stack, HTTP::STATUS_OK); } }