owncloud / core

apps/files_sharing/lib/SharedMount.php

Doc Comments +2 added lines, -2 removed lines

@@ -109,7 +109,7 @@ 
 	 *
 	 * @param string $newPath
 	 * @param \OCP\Share\IShare $share
-	 * @return bool
+	 * @return boolean|null
 	 */
 	private function updateFileTarget($newPath, &$share) {
 		$share->setTarget($newPath);
@@ -125,7 +125,7 @@ 
 	 * @param string $path
 	 * @param View $view
 	 * @param SharedMount[] $mountpoints
-	 * @return mixed
+	 * @return string
 	 */
 	private function generateUniqueTarget($path, $view, array $mountpoints) {
 		$pathinfo = \pathinfo($path);

core/Command/Security/ListRoutes.php

Doc Comments +3 added lines

@@ -121,6 +121,9 @@
 		}
 	}
 
+	/**
+	 * @param integer $name
+	 */
 	private function buildController($name) {
 		$parts = \explode('.', $name);
 		if (\count($parts) === 4 && $parts[0] === 'ocs') {

lib/private/App/AppManager.php

Doc Comments +7 added lines, -4 removed lines

@@ -132,7 +132,7 @@ 
 	/**
 	 * List all installed apps
 	 *
-	 * @return string[]
+	 * @return integer[]
 	 */
 	public function getInstalledApps() {
 		return \array_keys($this->getInstalledAppsValues());
@@ -241,7 +241,7 @@ 
 	/**
 	 * Do not allow more than one active app-theme
 	 *
-	 * @param $appId
+	 * @param string $appId
 	 * @throws AppManagerException
 	 */
 	protected function canEnableTheme($appId) {
@@ -400,6 +400,9 @@ 
 		return \in_array($appId, $this->shippedApps);
 	}
 
+	/**
+	 * @param string $appId
+	 */
 	private function isAlwaysEnabled($appId) {
 		$alwaysEnabled = $this->getAlwaysEnabledApps();
 		return \in_array($appId, $alwaysEnabled);
@@ -428,7 +431,7 @@ 
 	/**
 	 * @param string $package package path
 	 * @param bool $skipMigrations whether to skip migrations, which would only install the code
-	 * @return string|false app id or false in case of error
+	 * @return integer app id or false in case of error
 	 * @since 10.0
 	 */
 	public function installApp($package, $skipMigrations = false) {
@@ -441,7 +444,7 @@ 
 
 	/**
 	 * @param string $package
-	 * @return mixed
+	 * @return boolean
 	 * @since 10.0
 	 */
 	public function updateApp($package) {

lib/private/App/CodeChecker/NodeVisitor.php

Doc Comments +15 added lines

@@ -241,6 +241,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $name
+	 */
 	private function checkBlackList($name, $errorCode, Node $node) {
 		$lowerName = \strtolower($name);
 
@@ -254,6 +257,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $constantName
+	 */
 	private function checkBlackListConstant($class, $constantName, Node $node) {
 		$name = $class . '::' . $constantName;
 		$lowerName = \strtolower($name);
@@ -268,6 +274,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $class
+	 */
 	private function checkBlackListFunction($class, $functionName, Node $node) {
 		$name = $class . '::' . $functionName;
 		$lowerName = \strtolower($name);
@@ -282,6 +291,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $class
+	 */
 	private function checkBlackListMethod($class, $functionName, Node $node) {
 		$name = $class . '::' . $functionName;
 		$lowerName = \strtolower($name);
@@ -296,6 +308,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $name
+	 */
 	private function buildReason($name, $errorCode) {
 		if (isset($this->errorMessages[$errorCode])) {
 			$desc = $this->list->getDescription($errorCode, $name);

lib/private/AvatarManager.php

Doc Comments +3 added lines

@@ -98,6 +98,9 @@
 		}
 	}
 
+	/**
+	 * @param string $userId
+	 */
 	private function buildAvatarPath($userId) {
 		$avatar = \substr_replace(\substr_replace(\md5($userId), '/', 4, 0), '/', 2, 0);
 		return \explode('/', $avatar);

lib/private/Comments/Manager.php

Doc Comments +1 added lines, -1 removed lines

@@ -206,7 +206,7 @@
 	/**
 	 * removes an entry from the comments run time cache
 	 *
-	 * @param mixed $id the comment's id
+	 * @param string $id the comment's id
 	 */
 	protected function uncache($id) {
 		$id = \strval($id);

lib/private/DB/Migrator.php

Doc Comments +4 added lines

@@ -234,6 +234,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/Files/Cache/Cache.php

Doc Comments +3 added lines

@@ -409,6 +409,9 @@
 		}
 	}
 
+	/**
+	 * @param string $path
+	 */
 	private function getParentPath($path) {
 		$parent = \dirname($path);
 		if ($parent === '.') {

lib/private/Files/Cache/Wrapper/CacheJail.php

Doc Comments +4 added lines, -1 removed lines

@@ -74,7 +74,7 @@ 
 
 	/**
 	 * @param array $entry
-	 * @return array
+	 * @return string
 	 */
 	protected function formatCacheEntry($entry) {
 		if (isset($entry['path'])) {
@@ -193,6 +193,9 @@ 
 		return $this->cache->getStatus($this->getSourcePath($file));
 	}
 
+	/**
+	 * @param \OCP\Files\Cache\ICacheEntry[] $results
+	 */
 	private function formatSearchResults($results) {
 		$results = \array_filter($results, [$this, 'filterCacheEntry']);
 		$results = \array_values($results);