nextcloud / deck

lib/Service/BoardService.php

Unused Use Statements -3 removed lines

@@ -27,12 +27,9 @@
 use OCA\Deck\Db\AclMapper;
 use OCA\Deck\Db\Group;
 use OCA\Deck\Db\Label;
-
-
 use OCA\Deck\Db\User;
 use OCP\IGroupManager;
 use OCP\IL10N;
-
 use OCA\Deck\Db\Board;
 use OCA\Deck\Db\BoardMapper;
 use OCA\Deck\Db\LabelMapper;

Spacing +7 added lines, -7 removed lines

@@ -62,10 +62,10 @@ 
 		$groupBoards = $this->boardMapper->findAllByGroups($userInfo['user'], $userInfo['groups']);
 		$complete = array_merge($userBoards, $groupBoards);
 		$result = [];
-		foreach($complete as &$item) {
-			if(!array_key_exists($item->getId(), $result)) {
+		foreach ($complete as &$item) {
+			if (!array_key_exists($item->getId(), $result)) {
 				$item = $this->mapOwner($item);
-				if($item->getAcl() !== null) {
+				if ($item->getAcl() !== null) {
 					foreach ($item->getAcl() as &$acl) {
 						$acl = $this->mapAcl($acl);
 					}
@@ -81,21 +81,21 @@ 
 		$board = $this->boardMapper->find($boardId, true, true);
 		$board = $this->mapOwner($board);
 		foreach ($board->getAcl() as &$acl) {
-			if($acl !== null) {
+			if ($acl !== null) {
 				$this->mapAcl($acl);
 			}
 		}
 		return $board;
 	}
 
-	private function mapAcl(Acl &$acl) {
+	private function mapAcl(Acl & $acl) {
 		$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) {
 				return new User($userManager->get($acl->getParticipant($participant)));
 			}
-			if($acl->getType() === Acl::PERMISSION_TYPE_GROUP) {
+			if ($acl->getType() === Acl::PERMISSION_TYPE_GROUP) {
 				return new Group($groupManager->get($acl->getParticipant($participant)));
 			}
 		});

lib/Controller/ShareController.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
 	 * @return array
 	 */
 	public function searchUser($search) {
-		if($search==='%') {
+		if ($search === '%') {
 			$search = '';
 		}
 		$limit = 5;

lib/Db/RelationalEntity.php

Doc Comments +2 added lines, -2 removed lines

@@ -30,7 +30,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)) {
@@ -40,7 +40,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 +9 added lines, -9 removed lines

@@ -83,30 +83,30 @@
 	 */
 	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(strpos($methodName, 'resolve') === 0 && array_key_exists($attr, $this->_resolvedProperties)) {
-			if($this->_resolvedProperties[$attr] !== null) {
+	public function __call($methodName, $args) {
+		$attr = lcfirst(substr($methodName, 7));
+		if (strpos($methodName, 'resolve') === 0 && array_key_exists($attr, $this->_resolvedProperties)) {
+			if ($this->_resolvedProperties[$attr] !== null) {
 				return $this->_resolvedProperties[$attr];
 			} else {
 				return $this->getter($attr);
 			}
 		}
 
-		$attr = lcfirst( substr($methodName, 3) );
-		if(strpos($methodName, 'set') === 0 && array_key_exists($attr, $this->_resolvedProperties)) {
-			if(!is_scalar($args[0])) {
+		$attr = lcfirst(substr($methodName, 3));
+		if (strpos($methodName, 'set') === 0 && array_key_exists($attr, $this->_resolvedProperties)) {
+			if (!is_scalar($args[0])) {
 				$args[0] = $args[0]['primaryKey'];
 			}
 			return parent::setter($attr, $args);

lib/Db/RelationalObject.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
 	 * This method should be overwritten if object doesn't implement \JsonSerializable
 	 */
 	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));