nextcloud / deck

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) {

Spacing +1 added lines, -1 removed lines

@@ -178,7 +178,7 @@
 		$userManager = $this->userManager;
 		$card->resolveRelation('owner', function($owner) use (&$userManager) {
 			$user = $userManager->get($owner);
-			if($user !== null) {
+			if ($user !== null) {
 				return new User($user);
 			}
 			return null;

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;

Spacing +10 added lines, -10 removed lines

@@ -77,7 +77,7 @@ 
 			}
 		}
 		foreach ($this->_resolvedProperties as $property => $value) {
-			if($value !== null) {
+			if ($value !== null) {
 				$json[$property] = $value;
 			}
 		}
@@ -107,29 +107,29 @@ 
 	 */
 	public function resolveRelation($property, $resolver) {
 		$result = null;
-		if($property !== null && $this->$property !== null) {
+		if ($property !== null && $this->$property !== null) {
 			$result = $resolver($this->$property);
 		}
 
-		if($result instanceof RelationalObject || $result === null) {
+		if ($result instanceof RelationalObject || $result === null) {
 			$this->_resolvedProperties[$property] = $result;
 		} else {
 			throw new \Exception('resolver must return an instance of RelationalObject');
 		}
 	}
 
-	public function __call($methodName, $args){
-		$attr = lcfirst( substr($methodName, 7) );
-		if(array_key_exists($attr, $this->_resolvedProperties) && strpos($methodName, 'resolve') === 0) {
-			if($this->_resolvedProperties[$attr] !== null) {
+	public function __call($methodName, $args) {
+		$attr = lcfirst(substr($methodName, 7));
+		if (array_key_exists($attr, $this->_resolvedProperties) && strpos($methodName, 'resolve') === 0) {
+			if ($this->_resolvedProperties[$attr] !== null) {
 				return $this->_resolvedProperties[$attr];
 			}
 			return $this->getter($attr);
 		}
 
-		$attr = lcfirst( substr($methodName, 3) );
-		if(array_key_exists($attr, $this->_resolvedProperties) && strpos($methodName, 'set') === 0) {
-			if(!is_scalar($args[0])) {
+		$attr = lcfirst(substr($methodName, 3));
+		if (array_key_exists($attr, $this->_resolvedProperties) && strpos($methodName, 'set') === 0) {
+			if (!is_scalar($args[0])) {
 				$args[0] = $args[0]['primaryKey'];
 			}
 			parent::setter($attr, $args);

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

Spacing +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
 			$notification = $this->notificationManager->createNotification();
 			$notification
 				->setApp('deck')
-				->setUser((string)$user->getUID())
+				->setUser((string) $user->getUID())
 				->setObject('card', $card->getId())
 				->setSubject('card-overdue', [
 					$card->getTitle(), $board->getTitle()
@@ -132,7 +132,7 @@ 
 		$notification = $this->notificationManager->createNotification();
 		$notification
 			->setApp('deck')
-			->setUser((string)$userId)
+			->setUser((string) $userId)
 			->setDateTime(new DateTime())
 			->setObject('board', $board->getId())
 			->setSubject('board-shared', [$board->getTitle(), $this->currentUser]);

lib/Db/RelationalObject.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 	 * @throws \Exception
 	 */
 	public function getObjectSerialization() {
-		if($this->object instanceof \JsonSerializable) {
+		if ($this->object instanceof \JsonSerializable) {
 			$this->object->jsonSerialize();
 		} else {
 			throw new \Exception('jsonSerialize is not implemented on ' . get_class($this));

lib/Db/Card.php

Spacing +6 added lines, -6 removed lines

@@ -64,7 +64,7 @@ 
 	}
 
 	public function getDuedate($isoFormat = false) {
-		if($this->duedate === null) {
+		if ($this->duedate === null) {
 			return null;
 		}
 		$dt = new DateTime($this->duedate);
@@ -80,16 +80,16 @@ 
 		$due = strtotime($this->duedate);
 
 		$today = new DateTime();
-		$today->setTime( 0, 0);
+		$today->setTime(0, 0);
 
 		$match_date = new DateTime($this->duedate);
 
-		$match_date->setTime( 0, 0);
+		$match_date->setTime(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;
 			}

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());

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 		$userManager = $this->userManager;
 		$assignment->resolveRelation('participant', function() use (&$userManager, &$assignment) {
 			$user = $userManager->get($assignment->getParticipant());
-			if($user !== null) {
+			if ($user !== null) {
 				return new User($user);
 			}
 			return null;

templates/main.php

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,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
@@ -50,9 +50,9 @@ 
 		'filters' => ['boardFilterAcl', 'cardFilter', 'cardSearchFilter', 'iconWhiteFilter', 'lightenColorFilter', 'orderObjectBy', 'dateFilters', 'textColorFilter', 'withoutAssignedUsers'],
 		'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/Db/BoardMapper.php

Spacing +6 added lines, -6 removed lines

@@ -140,7 +140,7 @@ 
 
 	public function findToDelete() {
 		// add buffer of 5 min
-		$timeLimit = time()-(60*5);
+		$timeLimit = time() - (60 * 5);
 		$sql = 'SELECT id, title, owner, color, archived, deleted_at FROM `*PREFIX*deck_boards` ' .
 			'WHERE `deleted_at` > 0 AND `deleted_at` < ?';
 		return $this->findEntities($sql, [$timeLimit]);
@@ -181,17 +181,17 @@ 
 		$userManager = $this->userManager;
 		$groupManager = $this->groupManager;
 		$acl->resolveRelation('participant', function($participant) use (&$acl, &$userManager, &$groupManager) {
-			if($acl->getType() === Acl::PERMISSION_TYPE_USER) {
+			if ($acl->getType() === Acl::PERMISSION_TYPE_USER) {
 				$user = $userManager->get($participant);
-				if($user !== null) {
+				if ($user !== null) {
 					return new User($user);
 				}
 				\OC::$server->getLogger()->debug('User ' . $acl->getId() . ' not found when mapping acl ' . $acl->getParticipant());
 				return null;
 			}
-			if($acl->getType() === Acl::PERMISSION_TYPE_GROUP) {
+			if ($acl->getType() === Acl::PERMISSION_TYPE_GROUP) {
 				$group = $groupManager->get($participant);
-				if($group !== null) {
+				if ($group !== null) {
 					return new Group($group);
 				}
 				\OC::$server->getLogger()->debug('Group ' . $acl->getId() . ' not found when mapping acl ' . $acl->getParticipant());
@@ -208,7 +208,7 @@ 
 		$userManager = $this->userManager;
 		$board->resolveRelation('owner', function($owner) use (&$userManager) {
 			$user = $userManager->get($owner);
-			if($user !== null) {
+			if ($user !== null) {
 				return new User($user);
 			}
 			return null;