nextcloud / deck

lib/Service/StackService.php

Doc Comments +3 added lines

@@ -85,6 +85,9 @@
 		return $stacks;
 	}
 
+	/**
+	 * @param integer $order
+	 */
 	public function create($title, $boardId, $order) {
 		$this->permissionService->checkPermission(null, $boardId, Acl::PERMISSION_MANAGE);
 		if($this->boardService->isArchived(null, $boardId)) {

Unused Use Statements -2 removed lines

@@ -27,9 +27,7 @@
 use OCA\Deck\Db\CardMapper;
 use OCA\Deck\Db\LabelMapper;
 use OCA\Deck\Db\AssignedUsersMapper;
-
 use OCA\Deck\Db\Stack;
-
 use OCA\Deck\Db\StackMapper;
 use OCA\Deck\StatusException;
 

lib/Service/CardService.php

Doc Comments +3 added lines

@@ -50,6 +50,9 @@
 		return $card;
 	}
 
+	/**
+	 * @param integer $order
+	 */
 	public function create($title, $stackId, $type, $order, $owner) {
 		$this->permissionService->checkPermission($this->stackMapper, $stackId, Acl::PERMISSION_EDIT);
 		if($this->boardService->isArchived($this->stackMapper, $stackId)) {

lib/Service/PermissionService.php

Doc Comments +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
 	/**
 	 * Get current user permissions for a board
 	 *
-	 * @param Board|Entity $board
+	 * @param Board $board
 	 * @return array|bool
 	 * @internal param $boardId
 	 */

lib/AppInfo/Application.php

Unused Use Statements -1 removed lines

@@ -29,7 +29,6 @@
 use OCP\AppFramework\App;
 use OCA\Deck\Middleware\SharingMiddleware;
 use OCP\IGroup;
-
 use OCP\IUser;
 use OCP\IUserManager;
 use OCP\IURLGenerator;

lib/Db/CardMapper.php

Doc Comments +4 added lines, -1 removed lines

@@ -92,7 +92,7 @@ 
 
 	/**
 	 * @param $id
-	 * @return RelationalEntity if not found
+	 * @return Entity if not found
 	 * @throws \OCP\AppFramework\Db\MultipleObjectsReturnedException
 	 * @throws \OCP\AppFramework\Db\DoesNotExistException
 	 */
@@ -136,6 +136,9 @@ 
 		return parent::delete($entity);
 	}
 
+	/**
+	 * @param integer $stackId
+	 */
 	public function deleteByStack($stackId) {
 		$cards = $this->findAllByStack($stackId);
 		foreach ($cards as $card) {

lib/Db/RelationalEntity.php

Doc Comments +2 added lines, -2 removed lines

@@ -32,7 +32,7 @@ 
 
 	/**
 	 * Mark a property as relation so it will not get updated using Mapper::update
-	 * @param $property string Name of the property
+	 * @param string $property string Name of the property
 	 */
 	public function addRelation($property) {
 		if (!in_array($property, $this->_relations, true)) {
@@ -42,7 +42,7 @@ 
 
 	/**
 	 * Mark a property as resolvable via resolveRelation()
-	 * @param $property string Name of the property
+	 * @param string $property string Name of the property
 	 */
 	public function addResolvable($property) {
 		$this->_resolvedProperties[$property] = null;

lib/Notification/NotificationHelper.php

Doc Comments +4 added lines, -1 removed lines

@@ -98,7 +98,7 @@ 
 	 * Send notifications that a board was shared with a user/group
 	 *
 	 * @param $boardId
-	 * @param $acl
+	 * @param Acl $acl
 	 * @throws \InvalidArgumentException
 	 */
 	public function sendBoardShared($boardId, $acl) {
@@ -128,6 +128,9 @@ 
 		return $this->boards[$boardId];
 	}
 
+	/**
+	 * @param Board $board
+	 */
 	private function generateBoardShared($board, $userId) {
 		$notification = $this->notificationManager->createNotification();
 		$notification

lib/Db/AssignedUsersMapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
 	 * Check if user exists before assigning it to a card
 	 *
 	 * @param Entity $entity
-	 * @return null|Entity
+	 * @return AssignedUsers|null
 	 */
 	public function insert(Entity $entity) {
 		$user = $this->userManager->get($entity->getParticipant());