owncloud / core

lib/private/Encryption/Util.php

Unused Use Statements -1 removed lines

@@ -31,7 +31,6 @@
 use OC\Files\Filesystem;
 use OC\Files\View;
 use OCP\Encryption\IEncryptionModule;
-use OCP\Files\Storage;
 use OCP\IConfig;
 use OCP\IUser;
 

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

Doc Comments -1 removed lines

@@ -68,7 +68,6 @@
 	/**
 	 * @param \Sabre\DAV\Tree $tree tree
 	 * @param IUserSession $userSession user session
-	 * @param \OCP\Files\Folder $userFolder user home folder
 	 * @param \OCP\Share\IManager $shareManager share manager
 	 */
 	public function __construct(

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

Doc Comments +3 added lines, -2 removed lines

@@ -152,7 +152,7 @@ 
 	 * This will be used in the {DAV:}supported-report-set property.
 	 *
 	 * @param string $uri
-	 * @return array
+	 * @return string[]
 	 */
 	public function getSupportedReportSet($uri) {
 		return [self::REPORT_NAME];
@@ -164,7 +164,7 @@ 
 	 * @param string $reportName
 	 * @param mixed $report
 	 * @param string $uri
-	 * @return bool
+	 * @return null|false
 	 * @throws BadRequest
 	 * @throws PreconditionFailed
 	 * @internal param $ [] $report
@@ -350,6 +350,7 @@ 
 	 * with a leading slash but no trailing slash
 	 * @param string[] $requestedProps requested properties
 	 * @param Node[] nodes nodes for which to fetch and prepare responses
+	 * @param Node[] $nodes
 	 * @return Response[]
 	 */
 	public function prepareResponses($filesUri, $requestedProps, $nodes) {

apps/files_sharing/lib/Controller/ShareesController.php

Doc Comments +2 added lines, -2 removed lines

@@ -358,7 +358,7 @@ 
 	 * split user and remote from federated cloud id
 	 *
 	 * @param string $address federated share address
-	 * @return array [user, remoteURL]
+	 * @return string[] [user, remoteURL]
 	 * @throws \Exception
 	 */
 	public function splitUserRemote($address) {
@@ -567,7 +567,7 @@ 
 	 * trusted domain entries
 	 *
 	 * @param string $target target
-	 * @return true if one match was found, false otherwise
+	 * @return boolean if one match was found, false otherwise
 	 */
 	protected function isInstanceDomain($target) {
 		if (strpos($target, '/') !== false) {

lib/private/DB/Migrator.php

Doc Comments +4 added lines

@@ -235,6 +235,10 @@
 		return '/^' . preg_quote($this->config->getSystemValue('dbtableprefix', 'oc_')) . '/';
 	}
 
+	/**
+	 * @param integer $step
+	 * @param integer $max
+	 */
 	protected function emit($sql, $step, $max) {
 		if ($this->noEmit) {
 			return;

lib/private/Share20/DefaultShareProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -1051,7 +1051,7 @@
 	/**
 	 * Check whether the share object fits the expectations of this provider
 	 *
-	 * @param IShare $share share
+	 * @param \OCP\Share\IShare $share share
 	 *
 	 * @throws InvalidArgumentException if the share validation failed
 	 */

lib/private/Files/Stream/Checksum.php

Doc Comments +6 added lines, -3 removed lines

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

lib/public/Files.php

Doc Comments +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
 	/**
 	 * Search for files by mimetype
 	 * @param string $mimetype
-	 * @return array
+	 * @return \OC\Files\FileInfo[]
 	 * @since 6.0.0
 	 */
 	static public function searchByMime( $mimetype ) {

lib/private/Theme/ThemeService.php

Doc Comments +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
 	}
 
 	/**
-	 * @param $themeName
+	 * @param string $themeName
 	 * @param bool $appTheme
 	 * @return false|string
 	 */