owncloud / core

settings/ChangePassword/Controller.php

Doc Comments +3 added lines

@@ -67,6 +67,9 @@
 		}
 	}
 
+	/**
+	 * @param string|boolean $username
+	 */
 	private static function sendNotificationMail($username) {
 		$userObject = \OC::$server->getUserManager()->get($username);
 		$email = $userObject->getEMailAddress();

apps/dav/lib/CardDAV/SyncService.php

Doc Comments +1 added lines, -1 removed lines

@@ -245,7 +245,7 @@
 	}
 
 	/**
-	 * @return array|null
+	 * @return string
 	 */
 	public function getLocalSystemAddressBook() {
 		if ($this->localSystemAddressBook === null) {

apps/files_sharing/lib/Hooks.php

Unused Use Statements -1 removed lines

@@ -33,7 +33,6 @@
 use OCP\Share\IShare;
 use Symfony\Component\EventDispatcher\GenericEvent;
 use OCA\Files_Sharing\Service\NotificationPublisher;
-use OCP\Share\Exceptions\ShareNotFound;
 
 class Hooks {
 	/**

Doc Comments +4 added lines

@@ -138,6 +138,10 @@
 		);
 	}
 
+	/**
+	 * @param IShare[] $shares
+	 * @param string $fileId
+	 */
 	private function filterSharesByFileId($shares, $fileId) {
 		return \array_filter($shares, function (IShare $share) use ($fileId) {
 			return \strval($share->getNodeId()) === \strval($fileId);

settings/Application.php

Unused Use Statements -3 removed lines

@@ -39,12 +39,9 @@
 use OC\Settings\Controller\AuthSettingsController;
 use OC\Settings\Controller\CertificateController;
 use OC\Settings\Controller\CheckSetupController;
-use OC\Settings\Controller\GroupsController;
 use OC\Settings\Controller\LegalSettingsController;
 use OC\Settings\Controller\LogSettingsController;
 use OC\Settings\Controller\MailSettingsController;
-use OC\Settings\Controller\UsersController;
-use OC\Settings\Middleware\SubadminMiddleware;
 use OCP\AppFramework\App;
 use OCP\IContainer;
 use OCP\Util;

apps/dav/lib/Files/FileLocksBackend.php

Doc Comments +5 added lines, -2 removed lines

@@ -33,6 +33,9 @@ 
 	/** @var Tree */
 	private $tree;
 
+	/**
+	 * @param Tree $tree
+	 */
 	public function __construct($tree) {
 		$this->tree = $tree;
 	}
@@ -75,7 +78,7 @@ 
 	 *
 	 * @param string $uri
 	 * @param Locks\LockInfo $lockInfo
-	 * @return bool
+	 * @return boolean|null
 	 */
 	function lock($uri, Locks\LockInfo $lockInfo) {
 		// TODO: Implement lock() method.
@@ -86,7 +89,7 @@ 
 	 *
 	 * @param string $uri
 	 * @param Locks\LockInfo $lockInfo
-	 * @return bool
+	 * @return boolean|null
 	 */
 	function unlock($uri, Locks\LockInfo $lockInfo) {
 		// TODO: Implement unlock() method.

apps/dav/lib/Connector/Sabre/Node.php

Doc Comments +1 added lines, -1 removed lines

@@ -387,7 +387,7 @@
 	}
 
 	/**
-	 * @param $returnChildLocks
+	 * @param boolean $returnChildLocks
 	 * @return Lock[]
 	 */
 	public function getLocks($returnChildLocks) {

lib/private/Lock/Persistent/LockMapper.php

Doc Comments +4 added lines

@@ -30,6 +30,10 @@
 		parent::__construct($db, 'persistent_locks', null);
 	}
 
+	/**
+	 * @param integer $storageId
+	 * @param boolean $returnChildLocks
+	 */
 	public function getLocksByPath($storageId, $internalPath, $returnChildLocks) {
 		$query = $this->db->getQueryBuilder();
 		$pathPattern = $this->db->escapeLikeParameter($internalPath) . '%';