owncloud / core

lib/private/User/User.php

Doc Comments +5 added lines, -1 removed lines

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

lib/private/Files/Utils/Scanner.php

Doc Comments +1 added lines, -1 removed lines

@@ -170,7 +170,7 @@
 	/**
 	 * Returns whether the given mount point should be scanned
 	 *
-	 * @param $mount mount point
+	 * @param \OC\Files\Mount\MountPoint|null $mount mount point
 	 * @return bool true to scan, false to skip
 	 */
 	private function shouldScan($mount) {

lib/private/Share/MailNotifications.php

Doc Comments +3 added lines, -3 removed lines

@@ -189,10 +189,10 @@
 	 * inform recipient about public link share
 	 *
 	 * @param string $recipient recipient email address
-	 * @param string $filename the shared file
-	 * @param string $link the public link
 	 * @param array $options allows ['cc'] and ['bcc'] recipients
-	 * @param int $expiration expiration date (timestamp)
+	 * @param string $subject
+	 * @param boolean|string $htmlBody
+	 * @param boolean|string $textBody
 	 * @return string[] $result of failed recipients
 	 */
 	public function sendLinkShareMailFromBody($recipient, $subject, $htmlBody, $textBody, $options = []) {

lib/public/Share/IShareProvider.php

Doc Comments +5 added lines, -1 removed lines

@@ -63,6 +63,7 @@ 
 	 *
 	 * @param \OCP\Share\IShare $share
 	 * @since 9.0.0
+	 * @return void
 	 */
 	public function delete(\OCP\Share\IShare $share);
 
@@ -74,6 +75,7 @@ 
 	 * @param \OCP\Share\IShare $share
 	 * @param string $recipient UserId of the recipient
 	 * @since 9.0.0
+	 * @return void
 	 */
 	public function deleteFromSelf(\OCP\Share\IShare $share, $recipient);
 
@@ -181,6 +183,7 @@ 
 	 * @param string $uid
 	 * @param int $shareType
 	 * @since 9.1.0
+	 * @return void
 	 */
 	public function userDeleted($uid, $shareType);
 
@@ -191,6 +194,7 @@ 
 	 *
 	 * @param string $gid
 	 * @since 9.1.0
+	 * @return void
 	 */
 	public function groupDeleted($gid);
 
@@ -202,6 +206,7 @@ 
 	 * @param string $uid
 	 * @param string $gid
 	 * @since 9.1.0
+	 * @return void
 	 */
 	public function userDeletedFromGroup($uid, $gid);
 
@@ -213,7 +218,6 @@ 
 	 *
 	 * @param \OCP\Share\IShare $share
 	 * @param string $recipient userId of recipient
-	 * @param int $state state to set
 	 * @return \OCP\Share\IShare
 	 * @since 10.0.9
 	 */

lib/private/legacy/template.php

Doc Comments +1 added lines

@@ -322,6 +322,7 @@
 		* @param string $error_msg The error message to show
 		* @param string $hint An optional hint message - needs to be properly escaped
 		* @param int HTTP Status Code
+		* @param integer $httpStatusCode
 		*/
 	public static function printErrorPage($error_msg, $hint = '', $httpStatusCode = null) {
 		if ($error_msg === $hint) {

Unused Use Statements -1 removed lines

@@ -37,7 +37,6 @@
  */
 
 use OC\TemplateLayout;
-use OCP\IL10N;
 use OCP\Theme\ITheme;
 
 require_once __DIR__.'/template/functions.php';

lib/private/AppFramework/Utility/ControllerMethodReflector.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 	}
 
 	/**
-	 * @param object|string $object an object or classname
+	 * @param \OCP\AppFramework\Controller $object an object or classname
 	 * @param string $method the method which we want to inspect
 	 * @throws \ReflectionException
 	 */

lib/private/Files/Stream/Checksum.php

Doc Comments +6 added lines, -3 removed lines

@@ -61,8 +61,8 @@ 
 	}
 
 	/**
-	 * @param $source
-	 * @param $path
+	 * @param resource $source
+	 * @param string $path
 	 * @return resource
 	 */
 	public static function wrap($source, $path) {
@@ -122,6 +122,9 @@ 
 		return parent::stream_write($data);
 	}
 
+	/**
+	 * @param string $data
+	 */
 	private function updateHashingContexts($data) {
 		foreach ($this->hashingContexts as $ctx) {
 			\hash_update($ctx, $data);
@@ -195,7 +198,7 @@ 
 	}
 
 	/**
-	 * @return mixed
+	 * @return string
 	 * @return string
 	 */
 	private function getPathFromStreamContext() {

core/Command/User/ListUsers.php

Doc Comments +2 added lines

@@ -77,6 +77,8 @@
 
 	/**
 	 * If only a single attribute should be listed omit the key to make it fit in one row
+	 * @param string $key
+	 * @param boolean $useKey
 	 */
 	private function add(&$row, $key, $val, $useKey) {
 		if ($useKey) {

apps/dav/lib/CalDAV/CalDavBackend.php

Doc Comments +7 added lines, -1 removed lines

@@ -268,6 +268,9 @@ 
 		return \array_values($calendars);
 	}
 
+	/**
+	 * @param string $principalUri
+	 */
 	public function getUsersOwnCalendars($principalUri) {
 		$principalUri = $this->convertPrincipal($principalUri, true);
 		$fields = \array_values($this->propertyMap);
@@ -1278,7 +1281,7 @@ 
 	 * @param string $principalUri
 	 * @param string $uri
 	 * @param array $properties
-	 * @return mixed
+	 * @return integer
 	 * @throws Forbidden
 	 */
 	public function createSubscription($principalUri, $uri, array $properties) {
@@ -1685,6 +1688,9 @@ 
 		return $this->sharingBackend->applyShareAcl($resourceId, $acl);
 	}
 
+	/**
+	 * @param boolean $toV2
+	 */
 	private function convertPrincipal($principalUri, $toV2 = null) {
 		if ($this->principalBackend->getPrincipalPrefix() === 'principals') {
 			list(, $name) = URLUtil::splitPath($principalUri);

Unused Use Statements -1 removed lines

@@ -30,7 +30,6 @@
 use OCA\DAV\DAV\Sharing\Backend;
 use OCA\DAV\DAV\Sharing\IShareable;
 use OCP\DB\QueryBuilder\IQueryBuilder;
-use OCP\IConfig;
 use OCP\IDBConnection;
 use OCP\Security\ISecureRandom;
 use Sabre\CalDAV\Backend\AbstractBackend;