owncloud / core

lib/private/MembershipManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -491,7 +491,7 @@ 
 	 * are users in the group (identified by backend group id
 	 * if $isBackendGroupId is true, or gid if $isBackendGroupId is false)
 	 *
-	 * @param string|int $groupId
+	 * @param string $groupId
 	 * @param bool $isBackendGroupId
 	 * @param string $pattern
 	 * @return int
@@ -569,7 +569,7 @@ 
 
 	/**
 	 * @param IQueryBuilder $qb
-	 * @return int
+	 * @return boolean
 	 */
 	private function getAffectedQuery(IQueryBuilder $qb) {
 		// If affected is equal or more then 1, it means operation was successful

Unused Use Statements -1 removed lines

@@ -23,7 +23,6 @@
 
 use OC\Group\BackendGroup;
 use OC\User\Account;
-use OCP\AppFramework\Db\Entity;
 use OCP\IConfig;
 use OCP\IDBConnection;
 use Doctrine\DBAL\Exception\UniqueConstraintViolationException;

lib/private/Group/GroupMapper.php

Doc Comments +4 added lines

@@ -71,6 +71,10 @@
 		return $this->findEntities($qb->getSQL(), $qb->getParameters(), $limit, $offset);
 	}
 
+	/**
+	 * @param \Closure $callback
+	 * @param string $search
+	 */
 	public function callForAllGroups($callback, $search) {
 		$qb = $this->db->getQueryBuilder();
 		$qb->select(['*'])

lib/private/Group/Manager.php

Doc Comments +1 added lines, -1 removed lines

@@ -263,7 +263,7 @@
 	 *
 	 * @param \OC\User\User|null|false $user
 	 * @param string|null $scope string
-	 * @return array with group ids
+	 * @return integer[] with group ids
 	 */
 	public function getUserGroupIds($user, $scope = null) {
 		/** @var \OCP\IGroup[] $groupsForUser */

Unused Use Statements -1 removed lines

@@ -46,7 +46,6 @@
 use OCP\AppFramework\Db\DoesNotExistException;
 use OCP\AppFramework\Db\MultipleObjectsReturnedException;
 use OCP\GroupInterface;
-use OCP\IDBConnection;
 use OCP\IGroup;
 use OCP\IGroupManager;
 

lib/private/User/User.php

Doc Comments +5 added lines, -1 removed lines

@@ -86,7 +86,7 @@ 
 	 * @param Account $account
 	 * @param AccountMapper $mapper
 	 * @param MembershipManager $membershipManager
-	 * @param null $emitter
+	 * @param Manager $emitter
 	 * @param IConfig|null $config
 	 * @param null $urlGenerator
 	 * @param EventDispatcher|null $eventDispatcher
@@ -479,6 +479,10 @@ 
 		return $url;
 	}
 
+	/**
+	 * @param string $feature
+	 * @param string $value
+	 */
 	public function triggerChange($feature, $value = null) {
 		if ($this->emitter && in_array($feature, $this->account->getUpdatedFields(), true)) {
 			$this->emitter->emit('\OC\User', 'changeUser', [$this, $feature, $value]);

lib/private/Group/SyncService.php

Doc Comments +1 added lines, -1 removed lines

@@ -282,7 +282,7 @@
 	 * Return gid->BackendGroup map of user groups for this backend
 	 *
 	 * @param GroupInterface $backend
-	 * @param $uid
+	 * @param string $uid
 	 */
 	private function getLocalBackendMemberships(GroupInterface $backend, $uid) {
 		if ($this->lastUser !== $uid) {