nextcloud / deck

lib/Service/CardService.php

Spacing +9 added lines, -9 removed lines

@@ -52,7 +52,7 @@ 
 
 	public function create($title, $stackId, $type, $order, $owner) {
 		$this->permissionService->checkPermission($this->stackMapper, $stackId, Acl::PERMISSION_EDIT);
-		if($this->boardService->isArchived($this->stackMapper, $stackId)) {
+		if ($this->boardService->isArchived($this->stackMapper, $stackId)) {
 			throw new StatusException('Operation not allowed. This board is archived.');
 		}
 		$card = new Card();
@@ -67,7 +67,7 @@ 
 
 	public function delete($id) {
 		$this->permissionService->checkPermission($this->cardMapper, $id, Acl::PERMISSION_EDIT);
-		if($this->boardService->isArchived($this->cardMapper, $id)) {
+		if ($this->boardService->isArchived($this->cardMapper, $id)) {
 			throw new StatusException('Operation not allowed. This board is archived.');
 		}
 		return $this->cardMapper->delete($this->cardMapper->find($id));
@@ -75,7 +75,7 @@ 
 
 	public function update($id, $title, $stackId, $type, $order, $description, $owner, $duedate) {
 		$this->permissionService->checkPermission($this->cardMapper, $id, Acl::PERMISSION_EDIT);
-		if($this->boardService->isArchived($this->cardMapper, $id)) {
+		if ($this->boardService->isArchived($this->cardMapper, $id)) {
 			throw new StatusException('Operation not allowed. This board is archived.');
 		}
 		$card = $this->cardMapper->find($id);
@@ -94,7 +94,7 @@ 
 
 	public function rename($id, $title) {
 		$this->permissionService->checkPermission($this->cardMapper, $id, Acl::PERMISSION_EDIT);
-		if($this->boardService->isArchived($this->cardMapper, $id)) {
+		if ($this->boardService->isArchived($this->cardMapper, $id)) {
 			throw new StatusException('Operation not allowed. This board is archived.');
 		}
 		$card = $this->cardMapper->find($id);
@@ -107,7 +107,7 @@ 
 
 	public function reorder($id, $stackId, $order) {
 		$this->permissionService->checkPermission($this->cardMapper, $id, Acl::PERMISSION_EDIT);
-		if($this->boardService->isArchived($this->cardMapper, $id)) {
+		if ($this->boardService->isArchived($this->cardMapper, $id)) {
 			throw new StatusException('Operation not allowed. This board is archived.');
 		}
 		$cards = $this->cardMapper->findAll($stackId);
@@ -138,7 +138,7 @@ 
 
 	public function archive($id) {
 		$this->permissionService->checkPermission($this->cardMapper, $id, Acl::PERMISSION_EDIT);
-		if($this->boardService->isArchived($this->cardMapper, $id)) {
+		if ($this->boardService->isArchived($this->cardMapper, $id)) {
 			throw new StatusException('Operation not allowed. This board is archived.');
 		}
 		$card = $this->cardMapper->find($id);
@@ -148,7 +148,7 @@ 
 
 	public function unarchive($id) {
 		$this->permissionService->checkPermission($this->cardMapper, $id, Acl::PERMISSION_EDIT);
-		if($this->boardService->isArchived($this->cardMapper, $id)) {
+		if ($this->boardService->isArchived($this->cardMapper, $id)) {
 			throw new StatusException('Operation not allowed. This board is archived.');
 		}
 		$card = $this->cardMapper->find($id);
@@ -158,7 +158,7 @@ 
 
 	public function assignLabel($cardId, $labelId) {
 		$this->permissionService->checkPermission($this->cardMapper, $cardId, Acl::PERMISSION_EDIT);
-		if($this->boardService->isArchived($this->cardMapper, $cardId)) {
+		if ($this->boardService->isArchived($this->cardMapper, $cardId)) {
 			throw new StatusException('Operation not allowed. This board is archived.');
 		}
 		$card = $this->cardMapper->find($cardId);
@@ -170,7 +170,7 @@ 
 
 	public function removeLabel($cardId, $labelId) {
 		$this->permissionService->checkPermission($this->cardMapper, $cardId, Acl::PERMISSION_EDIT);
-		if($this->boardService->isArchived($this->cardMapper, $cardId)) {
+		if ($this->boardService->isArchived($this->cardMapper, $cardId)) {
 			throw new StatusException('Operation not allowed. This board is archived.');
 		}
 		$card = $this->cardMapper->find($cardId);

templates/main.php

Spacing +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
 Util::addScript('deck', 'vendor/markdown-it-link-target/dist/markdown-it-link-target.min');
 Util::addScript('deck', 'vendor/jquery-timepicker/jquery.ui.timepicker');
 
-if(!\OC::$server->getConfig()->getSystemValue('debug', false)) {
+if (!\OC::$server->getConfig()->getSystemValue('debug', false)) {
 	Util::addScript('deck', 'public/app');
 } else {
 	// Load seperate JS files when debug mode is enabled
@@ -49,9 +49,9 @@ 
 		'filters' => ['boardFilterAcl', 'cardFilter', 'cardSearchFilter', 'iconWhiteFilter', 'lightenColorFilter', 'orderObjectBy', 'dateFilters', 'textColorFilter'],
 		'service' => ['ApiService', 'BoardService', 'CardService', 'LabelService', 'StackService', 'StatusService'],
 	];
-	foreach($js as $folder=>$files) {
+	foreach ($js as $folder=>$files) {
 		foreach ($files as $file) {
-			Util::addScript('deck', $folder.'/'.$file);
+			Util::addScript('deck', $folder . '/' . $file);
 		}
 	}
 }

lib/Controller/BoardApiController.php

Indentation +102 added lines, -102 removed lines

@@ -38,109 +38,109 @@
  */
 class BoardApiController extends ApiController {
 
-    private $service;
-    private $userInfo;
-
-    /**
-     * @param string $appName
-     * @param IRequest $request
-     * @param IUserManager $userManager
-     * @param IGroupManager $groupManager
-     * @param BoardService $service
-     * @param $userId
-     */
-    public function __construct($appName, IRequest $request, IUserManager $userManager, IGroupManager $groupManager, BoardService $service, $userId) {
-        parent::__construct($appName, $request);
-        $this->service = $service;
-        $this->userId = $userId;
-        $this->userManager = $userManager;
-        $this->groupManager = $groupManager;
-    }
-
-    /**
-     * @NoAdminRequired
-     * @CORS
-     * @NoCSRFRequired
-     *
-     * Return all of the boards that the current user has access to.
-     */
-    public function index() {
-        $boards = $this->service->findAll($this->getUserInfo());
-
-        return new DataResponse($boards);
-    }
-
-    /**
-     * @NoAdminRequired
-     * @CORS
-     * @NoCSRFRequired
-     *
-     * @params $id
-     *
-     * Return the board specified by $id.
-     */
-    public function get($id) {
-        $board = $this->service->find($id);
-
-        // FIXME: this should probably 404 if the board has been deleted
-
-        return new DataResponse($board);
-    }
-
-    /**
-     * @NoAdminRequired
-     * @CORS
-     * @NoCSRFRequired
-     *
-     * @params $title
-     * @params $color
-     *
-     * Create a board with the specified title and color.
-     */
-    public function create($title, $color) {
-        $board = $this->service->create($title, $this->userId, $color);
-
-        return new DataResponse($board);
-    }
-
-    /**
-     * @NoAdminRequired
-     * @CORS
-     * @NoCSRFRequired
-     *
-     * @params $id
-     *
-     * Delete the board specified by $id.  Return the board that was deleted.
-     */
-    public function delete($id) {
-        $board = $this->service->delete($id);
-
-        return new DataResponse($board);
-    }
-
-    /**
-     * @NoAdminRequired
-     * @CORS
-     * @NoCSRFRequired
-     *
-     * @params $id
-     *
-     * Undo the deletion of the board specified by $id.
-     */
-    public function undoDelete($id) {
-        $board = $this->service->find($id);
-        $this->service->deleteUndo($id);
-
-        return new DataResponse($board);
-    }
-
-    // this is taken from BoardController, but it's not ideal
-    private function getUserInfo() {
-        $groups = $this->groupManager->getUserGroupIds(
+	private $service;
+	private $userInfo;
+
+	/**
+	 * @param string $appName
+	 * @param IRequest $request
+	 * @param IUserManager $userManager
+	 * @param IGroupManager $groupManager
+	 * @param BoardService $service
+	 * @param $userId
+	 */
+	public function __construct($appName, IRequest $request, IUserManager $userManager, IGroupManager $groupManager, BoardService $service, $userId) {
+		parent::__construct($appName, $request);
+		$this->service = $service;
+		$this->userId = $userId;
+		$this->userManager = $userManager;
+		$this->groupManager = $groupManager;
+	}
+
+	/**
+	 * @NoAdminRequired
+	 * @CORS
+	 * @NoCSRFRequired
+	 *
+	 * Return all of the boards that the current user has access to.
+	 */
+	public function index() {
+		$boards = $this->service->findAll($this->getUserInfo());
+
+		return new DataResponse($boards);
+	}
+
+	/**
+	 * @NoAdminRequired
+	 * @CORS
+	 * @NoCSRFRequired
+	 *
+	 * @params $id
+	 *
+	 * Return the board specified by $id.
+	 */
+	public function get($id) {
+		$board = $this->service->find($id);
+
+		// FIXME: this should probably 404 if the board has been deleted
+
+		return new DataResponse($board);
+	}
+
+	/**
+	 * @NoAdminRequired
+	 * @CORS
+	 * @NoCSRFRequired
+	 *
+	 * @params $title
+	 * @params $color
+	 *
+	 * Create a board with the specified title and color.
+	 */
+	public function create($title, $color) {
+		$board = $this->service->create($title, $this->userId, $color);
+
+		return new DataResponse($board);
+	}
+
+	/**
+	 * @NoAdminRequired
+	 * @CORS
+	 * @NoCSRFRequired
+	 *
+	 * @params $id
+	 *
+	 * Delete the board specified by $id.  Return the board that was deleted.
+	 */
+	public function delete($id) {
+		$board = $this->service->delete($id);
+
+		return new DataResponse($board);
+	}
+
+	/**
+	 * @NoAdminRequired
+	 * @CORS
+	 * @NoCSRFRequired
+	 *
+	 * @params $id
+	 *
+	 * Undo the deletion of the board specified by $id.
+	 */
+	public function undoDelete($id) {
+		$board = $this->service->find($id);
+		$this->service->deleteUndo($id);
+
+		return new DataResponse($board);
+	}
+
+	// this is taken from BoardController, but it's not ideal
+	private function getUserInfo() {
+		$groups = $this->groupManager->getUserGroupIds(
 			$this->userManager->get($this->userId)
 		);
-        return ['user' => $this->userId,
-                'groups' => $groups];
-    }
+		return ['user' => $this->userId,
+				'groups' => $groups];
+	}
 
 }

lib/Controller/StackApiController.php

Indentation +50 added lines, -50 removed lines

@@ -36,62 +36,62 @@
  */
 class StackApiController extends ApiController {
 
-    private $service;
-    private $userInfo;
+	private $service;
+	private $userInfo;
 
-    /**
-     * @param string $appName
-     * @param IRequest $request
-     * @param StackService $service
-     */
-    public function __construct($appName, IRequest $request, StackService $service) {
-        parent::__construct($appName, $request);
-        $this->service = $service;
-    }
+	/**
+	 * @param string $appName
+	 * @param IRequest $request
+	 * @param StackService $service
+	 */
+	public function __construct($appName, IRequest $request, StackService $service) {
+		parent::__construct($appName, $request);
+		$this->service = $service;
+	}
 
-    /**
-     * @NoAdminRequired
-     * @CORS
-     * @NoCSRFRequired
-     *
-     * Return all of the stacks in the specified board.
-     */
-    public function index($boardId) {
-        $stacks = $this->service->findAll($boardId);
+	/**
+	 * @NoAdminRequired
+	 * @CORS
+	 * @NoCSRFRequired
+	 *
+	 * Return all of the stacks in the specified board.
+	 */
+	public function index($boardId) {
+		$stacks = $this->service->findAll($boardId);
 
-        return new DataResponse($stacks);
-    }
+		return new DataResponse($stacks);
+	}
 
-    /**
-     * @NoAdminRequired
-     * @CORS
-     * @NoCSRFRequired
-     *
-     * @params $title
-     * @params $order
-     *
-     * Create a stack with the specified title and order.
-     */
-    public function create($boardId, $title, $order) {
-        // this throws a StatusException that needs to be caught and handled
-        $stack = $this->service->create($title, $boardId, $order);
+	/**
+	 * @NoAdminRequired
+	 * @CORS
+	 * @NoCSRFRequired
+	 *
+	 * @params $title
+	 * @params $order
+	 *
+	 * Create a stack with the specified title and order.
+	 */
+	public function create($boardId, $title, $order) {
+		// this throws a StatusException that needs to be caught and handled
+		$stack = $this->service->create($title, $boardId, $order);
 
-        return new DataResponse($stack);
-    }
+		return new DataResponse($stack);
+	}
 
-    /**
-     * @NoAdminRequired
-     * @CORS
-     * @NoCSRFRequired
-     *
-     * @params $id
-     *
-     * Delete the stack specified by $id.  Return the board that was deleted.
-     */
-    public function delete($boardId, $id) {
-        $stack = $this->service->delete($id);
+	/**
+	 * @NoAdminRequired
+	 * @CORS
+	 * @NoCSRFRequired
+	 *
+	 * @params $id
+	 *
+	 * Delete the stack specified by $id.  Return the board that was deleted.
+	 */
+	public function delete($boardId, $id) {
+		$stack = $this->service->delete($id);
 
-        return new DataResponse($stack);
-    }
+		return new DataResponse($stack);
+	}
 
 }

appinfo/routes.php

Indentation +11 added lines, -11 removed lines

@@ -62,18 +62,18 @@
 		['name' => 'label#update', 'url' => '/labels/{labelId}', 'verb' => 'PUT'],
 		['name' => 'label#delete', 'url' => '/labels/{labelId}', 'verb' => 'DELETE'],
 
-        // api
-        ['name' => 'board_api#index', 'url' => '/api/v1.0/boards', 'verb' => 'GET'],
-        ['name' => 'board_api#get', 'url' => '/api/v1.0/board/{id}', 'verb' => 'GET'],
-        ['name' => 'board_api#create', 'url' => '/api/v1.0/board', 'verb' => 'POST'],
-        ['name' => 'board_api#delete', 'url' => '/api/v1.0/board/{id}', 'verb' => 'DELETE'],
-        ['name' => 'board_api#undo_delete', 'url' => '/api/v1.0/board/{id}/undo_delete', 'verb' => 'POST'],
+		// api
+		['name' => 'board_api#index', 'url' => '/api/v1.0/boards', 'verb' => 'GET'],
+		['name' => 'board_api#get', 'url' => '/api/v1.0/board/{id}', 'verb' => 'GET'],
+		['name' => 'board_api#create', 'url' => '/api/v1.0/board', 'verb' => 'POST'],
+		['name' => 'board_api#delete', 'url' => '/api/v1.0/board/{id}', 'verb' => 'DELETE'],
+		['name' => 'board_api#undo_delete', 'url' => '/api/v1.0/board/{id}/undo_delete', 'verb' => 'POST'],
 
-        ['name' => 'stack_api#index', 'url' => '/api/v1.0/board/{boardId}/stacks', 'verb' => 'GET'],
-        ['name' => 'stack_api#create', 'url' => '/api/v1.0/board/{boardId}/stack', 'verb' => 'POST'],
-        ['name' => 'stack_api#delete', 'url' => '/api/v1.0/board/{boardId}/stack/{id}', 'verb' => 'DELETE'],
+		['name' => 'stack_api#index', 'url' => '/api/v1.0/board/{boardId}/stacks', 'verb' => 'GET'],
+		['name' => 'stack_api#create', 'url' => '/api/v1.0/board/{boardId}/stack', 'verb' => 'POST'],
+		['name' => 'stack_api#delete', 'url' => '/api/v1.0/board/{boardId}/stack/{id}', 'verb' => 'DELETE'],
 
-        ['name' => 'board_api#preflighted_cors', 'url' => '/api/v1.0/{path}',
-         'verb' => 'OPTIONS', 'requirements' => ['path' => '.+']],
+		['name' => 'board_api#preflighted_cors', 'url' => '/api/v1.0/{path}',
+		 'verb' => 'OPTIONS', 'requirements' => ['path' => '.+']],
 	]
 ];

lib/Db/Card.php

Spacing +6 added lines, -6 removed lines

@@ -59,7 +59,7 @@ 
 	}
 
 	public function getDuedate() {
-		if($this->duedate === null)
+		if ($this->duedate === null)
 			return null;
 		$dt = new DateTime($this->duedate);
 		return $dt->format('c');
@@ -71,16 +71,16 @@ 
 		$due = strtotime($this->duedate);
 
 		$today = new DateTime();
-		$today->setTime( 0, 0, 0 );
+		$today->setTime(0, 0, 0);
 
 		$match_date = new DateTime($this->duedate);
 
-		$match_date->setTime( 0, 0, 0 );
+		$match_date->setTime(0, 0, 0);
 
-		$diff = $today->diff( $match_date );
-		$diffDays = (integer)$diff->format( "%R%a" ); // Extract days count in interval
+		$diff = $today->diff($match_date);
+		$diffDays = (integer) $diff->format("%R%a"); // Extract days count in interval
 
-		if($due !== false) {
+		if ($due !== false) {
 			if ($diffDays === 1) {
 				$json['overdue'] = self::DUEDATE_NEXT;
 			}

Braces +3 added lines, -2 removed lines

@@ -59,8 +59,9 @@
 	}
 
 	public function getDuedate() {
-		if($this->duedate === null)
-			return null;
+		if($this->duedate === null) {
+					return null;
+		}
 		$dt = new DateTime($this->duedate);
 		return $dt->format('c');
 	}