owncloud / core

apps/dav/lib/DAV/SystemPrincipalBackend.php

Doc Comments +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
 	 * Returns the list of members for a group-principal
 	 *
 	 * @param string $principal
-	 * @return array
+	 * @return string[]
 	 */
 	public function getGroupMemberSet($principal) {
 		// TODO: for now the group principal has only one member, the user itself

apps/dav/lib/Upload/AssemblyStreamZsync.php

Doc Comments +5 added lines, -2 removed lines

@@ -183,7 +183,7 @@ 
 	/**
 	 * @param IFile[] $nodes
 	 * @param IFile $backingFile
-	 * @param $fileLength
+	 * @param string $fileLength
 	 * @return resource
 	 *
 	 * @throws \BadMethodCallException
@@ -207,6 +207,9 @@ 
 		return $wrapped;
 	}
 
+	/**
+	 * @param integer $current
+	 */
 	protected function getNextNodeStart($current) {
 		foreach ($this->sortedNodes as $node) {
 			if ($current >= $node['start'] && $current < $node['end']) {
@@ -220,7 +223,7 @@ 
 	}
 
 	/**
-	 * @param $pos
+	 * @param integer $pos
 	 */
 	protected function getNodeForPosition($pos) {
 		foreach ($this->sortedNodes as $node) {

apps/dav/lib/Upload/ChunkingPluginZsync.php

Doc Comments +1 added lines

@@ -107,6 +107,7 @@
 	 *
 	 * @param string &$zsyncMetadata actual metadata
 	 * @param string $destination destination path
+	 * @param string|null $zsyncMetadata
 	 */
 	private function postMoveZsync(&$zsyncMetadata, $destination) {
 		if (!$zsyncMetadata) {

apps/files_sharing/lib/External/Manager.php

Doc Comments +3 added lines

@@ -308,6 +308,9 @@
 		return $result;
 	}
 
+	/**
+	 * @param string $mountPoint
+	 */
 	public function removeShare($mountPoint) {
 		$mountPointObj = $this->mountManager->find($mountPoint);
 		$id = $mountPointObj->getStorage()->getCache()->getId('');

apps/files_trashbin/lib/Storage.php

Doc Comments +1 added lines, -1 removed lines

@@ -134,7 +134,7 @@
 	 * check if it is a file located in data/user/files only files in the
 	 * 'files' directory should be moved to the trash
 	 *
-	 * @param $path
+	 * @param string $path
 	 * @return bool
 	 */
 	protected function shouldMoveToTrash($path) {

core/Controller/OccController.php

Doc Comments +2 added lines, -2 removed lines

@@ -131,8 +131,8 @@
 
 	/**
 	 * Check if command is allowed and has a valid security token
-	 * @param $command
-	 * @param $token
+	 * @param string $command
+	 * @param string $token
 	 */
 	protected function validateRequest($command, $token) {
 		$allowedHosts = ['::1', '127.0.0.1', 'localhost'];

lib/private/Files/Node/Root.php

Doc Comments +4 added lines, -1 removed lines

@@ -175,7 +175,7 @@ 
 	 * @param string $path
 	 * @throws \OCP\Files\NotFoundException
 	 * @throws \OCP\Files\NotPermittedException
-	 * @return File|Folder
+	 * @return string
 	 */
 	public function get($path) {
 		$path = $this->normalizePath($path);
@@ -373,6 +373,9 @@ 
 		return $folder;
 	}
 
+	/**
+	 * @param string $fullPath
+	 */
 	private function resolveVirtualNode($fullPath) {
 		$pieces = \explode('/', $fullPath);
 		if ($pieces[1] !== 'meta') {

lib/private/legacy/api.php

Doc Comments +2 added lines, -2 removed lines

@@ -269,7 +269,7 @@ 
 	/**
 	 * authenticate the api call
 	 * @param array $action the action details as supplied to OC_API::register()
-	 * @return bool
+	 * @return boolean|string
 	 */
 	private static function isAuthorised($action) {
 		$level = $action['authlevel'];
@@ -318,7 +318,7 @@ 
 
 	/**
 	 * http basic auth
-	 * @return string|false (username, or false on failure)
+	 * @return string|boolean (username, or false on failure)
 	 */
 	private static function loginUser() {
 		if (self::$isLoggedIn === true) {

lib/private/Repair/RepairMismatchFileCachePath.php

Doc Comments +14 added lines, -1 removed lines

@@ -102,7 +102,7 @@ 
 	 * @param string $wrongPath wrong path of the entry to fix
 	 * @param int $correctStorageNumericId numeric idea of the correct storage
 	 * @param string $correctPath value to which to set the path of the entry
-	 * @return bool true for success
+	 * @return boolean|null true for success
 	 */
 	private function fixEntryPath(IOutput $out, $fileId, $wrongPath, $correctStorageNumericId, $correctPath) {
 		// delete target if exists
@@ -132,6 +132,9 @@ 
 		$out->advance(1, $text);
 	}
 
+	/**
+	 * @param \OCP\DB\QueryBuilder\IQueryBuilder $qb
+	 */
 	private function addQueryConditionsParentIdWrongPath($qb) {
 		// thanks, VicDeo!
 		if ($this->connection->getDatabasePlatform() instanceof MySqlPlatform) {
@@ -171,6 +174,9 @@ 
 		}
 	}
 
+	/**
+	 * @param \OCP\DB\QueryBuilder\IQueryBuilder $qb
+	 */
 	private function addQueryConditionsNonExistingParentIdEntry($qb, $storageNumericId = null) {
 		// Subquery for parent existence
 		$qbe = $this->connection->getQueryBuilder();
@@ -200,6 +206,9 @@ 
 		}
 	}
 
+	/**
+	 * @param integer $storageNumericId
+	 */
 	private function countResultsToProcessParentIdWrongPath($storageNumericId = null) {
 		$qb = $this->connection->getQueryBuilder();
 		$qb->select($qb->createFunction('COUNT(*)'));
@@ -210,6 +219,9 @@ 
 		return $count;
 	}
 
+	/**
+	 * @param integer $storageNumericId
+	 */
 	private function countResultsToProcessNonExistingParentIdEntry($storageNumericId = null) {
 		$qb = $this->connection->getQueryBuilder();
 		$qb->select($qb->createFunction('COUNT(*)'));
@@ -359,6 +371,7 @@ 
 	 *
 	 * @param int $storageId storage id
 	 * @param string $path path for which to create the parent entry
+	 * @param integer $reuseFileId
 	 * @return int file id of the newly created parent
 	 */
 	private function getOrCreateEntry($storageId, $path, $reuseFileId = null) {