ahtlon/deck
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

fix: feedback

Browse Source 
Signed-off-by: Luka Trovic <luka@nextcloud.com>
This commit is contained in:
Luka Trovic
2022-03-16 17:13:29 +01:00
committed by Julius Härtl
parent bf9a51d167
commit 9f1dbd137c
3 changed files with 11 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
lib/Controller/BoardController.php
View File

@@ -27,6 +27,8 @@ use OCA\Deck\Db\Acl;
use OCA\Deck\Service\BoardService;
use OCA\Deck\Service\PermissionService;
use OCP\AppFramework\ApiController;
use OCP\AppFramework\Http;
use OCP\AppFramework\Http\DataResponse;
use OCP\IRequest;
class BoardController extends ApiController {
@@ -163,7 +165,11 @@ class BoardController extends ApiController {
* @param $newOwner
* * @return null|void
*/
public function transferOwner($boardId, $owner, $newOwner) {
return $this->boardService->transferOwnership($owner, $newOwner);
public function transferOwner(int $boardId, string $newOwner): DataResponse {
if ($this->permissionService->userIsBoardOwner($boardId, $this->userId)) {
return new DataResponse($this->boardService->transferBoardOwnership($boardId, $newOwner), HTTP::STATUS_OK);
}
return new DataResponse([], HTTP::STATUS_UNAUTHORIZED);
}
}

6
lib/Service/BoardService.php
View File

@@ -746,12 +746,6 @@ class BoardService {
$this->boardsCache = null;
}
private function getBoardOwner($boardId) {
$board = $this->boardMapper->find($boardId);
return $board->getOwner();
}
/**
* Clean a given board data from the Cache
*/