nextcloud / circles

lib/ShareByCircleProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -879,7 +879,7 @@
 	/**
 	 * get database row of a give share
 	 *
-	 * @param $id
+	 * @param integer $id
 	 *
 	 * @return array
 	 * @throws ShareNotFound

Spacing +4 added lines, -4 removed lines

@@ -173,11 +173,11 @@ 
 
 		$data = $exists->fetch();
 		$exists->closeCursor();
-		$this->misc->log('______   ' . var_export($data, true));
+		$this->misc->log('______   '.var_export($data, true));
 
-		$this->misc->log('______   ' . $qb->getSQL());
+		$this->misc->log('______   '.$qb->getSQL());
 		$this->misc->log(
-			'______   ' . \OCP\Share::SHARE_TYPE_CIRCLE . '   ' . $share->getSharedWith() . '   '
+			'______   '.\OCP\Share::SHARE_TYPE_CIRCLE.'   '.$share->getSharedWith().'   '
 			.
 				$share->getNode()
 					  ->getId());
@@ -689,7 +689,7 @@ 
 		   )
 		   ->leftJoin(
 			   's', 'share', 's2', $qb->expr()
-									  ->eq('s.id', 's2.parent') . ' AND ' . $qb->expr()
+									  ->eq('s.id', 's2.parent').' AND '.$qb->expr()
 																			   ->eq(
 																				   's2.share_with',
 																				   $qb->createNamedParameter(

lib/AppInfo/Application.php

Spacing +16 added lines, -16 removed lines

@@ -64,14 +64,14 @@ 
 		 * Services
 		 */
 		$container->registerService(
-			'MiscService', function ($c) {
+			'MiscService', function($c) {
 			return new MiscService($c->query('Logger'), $c->query('AppName'));
 		}
 		);
 
 
 		$container->registerService(
-			'ConfigService', function ($c) {
+			'ConfigService', function($c) {
 			return new ConfigService(
 				$c->query('AppName'), $c->query('CoreConfig'), $c->query('UserId'),
 				$c->query('MiscService')
@@ -80,7 +80,7 @@ 
 		);
 
 		$container->registerService(
-			'DatabaseService', function ($c) {
+			'DatabaseService', function($c) {
 			return new DatabaseService(
 				$c->query('CirclesMapper'),
 				$c->query('MembersMapper')
@@ -89,7 +89,7 @@ 
 		);
 
 		$container->registerService(
-			'CirclesService', function ($c) {
+			'CirclesService', function($c) {
 			return new CirclesService(
 				$c->query('UserId'), $c->query('L10N'),
 				$c->query('ConfigService'),
@@ -100,7 +100,7 @@ 
 		);
 
 		$container->registerService(
-			'MembersService', function ($c) {
+			'MembersService', function($c) {
 			return new MembersService(
 				$c->query('UserId'), $c->query('L10N'),
 				$c->query('UserManager'),
@@ -133,7 +133,7 @@ 
 //		);
 
 		$container->registerService(
-			'NavigationController', function ($c) {
+			'NavigationController', function($c) {
 			return new NavigationController(
 				$c->query('AppName'), $c->query('Request'), $c->query('UserId'), $c->query('L10N'),
 				$c->query('ConfigService'),
@@ -143,7 +143,7 @@ 
 		);
 
 		$container->registerService(
-			'CirclesController', function ($c) {
+			'CirclesController', function($c) {
 			return new CirclesController(
 				$c->query('AppName'), $c->query('Request'), $c->query('UserId'), $c->query('L10N'),
 				$c->query('ConfigService'),
@@ -154,7 +154,7 @@ 
 		);
 
 		$container->registerService(
-			'MembersController', function ($c) {
+			'MembersController', function($c) {
 			return new MembersController(
 				$c->query('AppName'), $c->query('Request'), $c->query('UserId'), $c->query('L10N'),
 				$c->query('ConfigService'),
@@ -168,7 +168,7 @@ 
 		 * Mapper
 		 */
 		$container->registerService(
-			'CirclesMapper', function ($c) {
+			'CirclesMapper', function($c) {
 			return new CirclesMapper(
 				$c->query('ServerContainer')
 				  ->getDatabaseConnection(), $c->query('MiscService')
@@ -177,7 +177,7 @@ 
 		);
 
 		$container->registerService(
-			'MembersMapper', function ($c) {
+			'MembersMapper', function($c) {
 			return new MembersMapper(
 				$c->query('ServerContainer')
 				  ->getDatabaseConnection(), $c->query('MiscService')
@@ -188,7 +188,7 @@ 
 
 		// Translates
 		$container->registerService(
-			'L10N', function ($c) {
+			'L10N', function($c) {
 			return $c->query('ServerContainer')
 					 ->getL10N($c->query('AppName'));
 		}
@@ -198,20 +198,20 @@ 
 		 * Core
 		 */
 		$container->registerService(
-			'Logger', function ($c) {
+			'Logger', function($c) {
 			return $c->query('ServerContainer')
 					 ->getLogger();
 		}
 		);
 		$container->registerService(
-			'CoreConfig', function ($c) {
+			'CoreConfig', function($c) {
 			return $c->query('ServerContainer')
 					 ->getConfig();
 		}
 		);
 
 		$container->registerService(
-			'UserId', function ($c) {
+			'UserId', function($c) {
 			$user = $c->query('ServerContainer')
 					  ->getUserSession()
 					  ->getUser();
@@ -221,7 +221,7 @@ 
 		);
 
 		$container->registerService(
-			'UserManager', function ($c) {
+			'UserManager', function($c) {
 			return $c->query('ServerContainer')
 					 ->getUserManager();
 		}
@@ -247,7 +247,7 @@ 
 			 ->getServer()
 			 ->getNavigationManager()
 			 ->add(
-				 function () {
+				 function() {
 					 return [
 						 'id'    => $this->appName,
 						 'order' => 5,

lib/Service/CirclesService.php

Doc Comments +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
 	 * Create circle using this->userId as owner
 	 *
 	 * @param $type
-	 * @param $name
+	 * @param string $name
 	 *
 	 * @return Circle
 	 * @throws CircleTypeDisabledException

lib/Db/CirclesMapper.php

Spacing +1 added lines, -1 removed lines

@@ -239,7 +239,7 @@
 	 * @throws CircleAlreadyExistsException
 	 * @throws CircleCreationException
 	 */
-	public function create(Circle &$circle, Member &$owner) {
+	public function create(Circle & $circle, Member & $owner) {
 
 		if ($circle->getType() === Circle::CIRCLES_PERSONAL) {
 

lib/Model/Circle.php

Doc Comments +16 added lines, -1 removed lines

@@ -74,6 +74,9 @@ 
 		return $this;
 	}
 
+	/**
+	 * @return string
+	 */
 	public function getName() {
 		return $this->name;
 	}
@@ -83,6 +86,9 @@ 
 		return $this->owner;
 	}
 
+	/**
+	 * @param Member|null $owner
+	 */
 	public function setOwner($owner) {
 		$this->owner = $owner;
 	}
@@ -92,6 +98,9 @@ 
 		return $this->user;
 	}
 
+	/**
+	 * @param Member|null $user
+	 */
 	public function setUser($user) {
 		$this->user = $user;
 	}
@@ -121,6 +130,9 @@ 
 		return $this->type;
 	}
 
+	/**
+	 * @param string $str
+	 */
 	public function setTypeString($str) {
 		$this->typeString = $str;
 
@@ -131,6 +143,9 @@ 
 		return $this->typeString;
 	}
 
+	/**
+	 * @param string $str
+	 */
 	public function setTypeLongString($str) {
 		$this->typeLongString = $str;
 	}
@@ -223,7 +238,7 @@ 
 	 *
 	 * @param $array
 	 *
-	 * @return mixed
+	 * @return Member|null
 	 */
 	private static function getOwnerMemberFromArray($array) {
 		if (key_exists('owner', $array)) {

lib/Service/MembersService.php

Doc Comments +2 added lines, -2 removed lines

@@ -84,7 +84,7 @@ 
 
 	/**
 	 * @param $circleId
-	 * @param $name
+	 * @param string $name
 	 *
 	 * @return array
 	 * @throws CircleDoesNotExistException
@@ -165,7 +165,7 @@ 
 	/**
 	 * Invite a Member to a private Circle, or accept his request.
 	 *
-	 * @param $member
+	 * @param null|Member $member
 	 */
 	public static function InviteMemberToPrivateCircle(&$member) {
 		if ($member->getStatus() === Member::STATUS_REQUEST) {