owncloud / core

lib/private/DB/MigrationService.php

Doc Comments +1 added lines

@@ -313,6 +313,7 @@
 	}
 
 	/**
+	 * @param string $version
 	 * @return string
 	 */
 	private function getClass($version) {

apps/files_trashbin/lib/Trashbin.php

Doc Comments +6 added lines, -2 removed lines

@@ -213,6 +213,10 @@ 
 
 	/**
 	 *
+	 * @param string $user
+	 * @param string $targetFilename
+	 * @param string $targetLocation
+	 * @param integer $timestamp
 	 */
 	public static function insertTrashEntry($user, $targetFilename, $targetLocation, $timestamp) {
 		$query = \OC_DB::prepare("INSERT INTO `*PREFIX*files_trash` (`id`,`timestamp`,`location`,`user`) VALUES (?,?,?,?)");
@@ -756,7 +760,7 @@ 
 	 * if the size limit for the trash bin is reached, we delete the oldest
 	 * files in the trash bin until we meet the limit again
 	 *
-	 * @param array $files
+	 * @param \OCP\Files\FileInfo[] $files
 	 * @param string $user
 	 * @param int $availableSpace available disc space
 	 * @return int size of deleted files
@@ -784,7 +788,7 @@ 
 	/**
 	 * delete files older then max storage time
 	 *
-	 * @param array $files list of files sorted by mtime
+	 * @param \OCP\Files\FileInfo[] $files list of files sorted by mtime
 	 * @param string $user
 	 * @return integer[] size of deleted files and number of deleted files
 	 */

lib/private/App/AppManager.php

Doc Comments +6 added lines, -3 removed lines

@@ -120,7 +120,7 @@ 
 	/**
 	 * List all installed apps
 	 *
-	 * @return string[]
+	 * @return integer[]
 	 */
 	public function getInstalledApps() {
 		return array_keys($this->getInstalledAppsValues());
@@ -347,6 +347,9 @@ 
 		return in_array($appId, $this->shippedApps);
 	}
 
+	/**
+	 * @param string $appId
+	 */
 	private function isAlwaysEnabled($appId) {
 		$alwaysEnabled = $this->getAlwaysEnabledApps();
 		return in_array($appId, $alwaysEnabled);
@@ -374,7 +377,7 @@ 
 
 	/**
 	 * @param string $package
-	 * @return mixed
+	 * @return integer
 	 * @since 10.0
 	 */
 	public function installApp($package) {
@@ -386,7 +389,7 @@ 
 
 	/**
 	 * @param string $package
-	 * @return mixed
+	 * @return boolean
 	 * @since 10.0
 	 */
 	public function updateApp($package) {

lib/private/User/Manager.php

Doc Comments -1 removed lines

@@ -395,7 +395,6 @@
 
 	/**
 	 * @param \Closure $callback
-	 * @param string $search
 	 * @since 10.0
 	 */
 	public function callForSeenUsers (\Closure $callback) {

lib/public/IUserManager.php

Doc Comments +6 added lines, -2 removed lines

@@ -45,6 +45,7 @@ 
 	 *
 	 * @param \OCP\UserInterface $backend
 	 * @since 8.0.0
+	 * @return void
 	 */
 	public function registerBackend($backend);
 
@@ -60,12 +61,14 @@ 
 	 *
 	 * @param \OCP\UserInterface $backend
 	 * @since 8.0.0
+	 * @return void
 	 */
 	public function removeBackend($backend);
 
 	/**
 	 * remove all user backends
 	 * @since 8.0.0
+	 * @return void
 	 */
 	public function clearBackends() ;
 
@@ -131,7 +134,7 @@ 
 	/**
 	 * returns how many users per backend exist (if supported by backend)
 	 *
-	 * @return array an array of backend class as key and count number as value
+	 * @return integer an array of backend class as key and count number as value
 	 * @since 8.0.0
 	 */
 	public function countUsers();
@@ -140,6 +143,7 @@ 
 	 * @param \Closure $callback
 	 * @param string $search
 	 * @since 9.0.0
+	 * @return void
 	 */
 	public function callForAllUsers (\Closure $callback, $search = '');
 
@@ -153,8 +157,8 @@ 
 
 	/**
 	 * @param \Closure $callback
-	 * @param string $search
 	 * @since 10.0
+	 * @return void
 	 */
 	public function callForSeenUsers (\Closure $callback);
 

lib/private/legacy/app.php

Doc Comments -3 removed lines

@@ -807,9 +807,6 @@
 	/**
 	 * List all apps, this is used in apps.php
 	 *
-	 * @param bool $onlyLocal
-	 * @param bool $includeUpdateInfo Should we check whether there is an update
-	 *                                in the app store?
 	 * @return array
 	 */
 	public static function listAllApps() {

lib/private/AppFramework/Http/Request.php

Doc Comments +1 added lines, -1 removed lines

@@ -448,7 +448,7 @@
 
 	/**
 	 * Checks if the CSRF check was correct
-	 * @return bool true if CSRF check passed
+	 * @return boolean|null true if CSRF check passed
 	 */
 	public function passesCSRFCheck() {
 		if($this->csrfTokenManager === null) {

apps/dav/lib/CalDAV/Publishing/Xml/Publisher.php

Unused Use Statements -1 removed lines

@@ -20,7 +20,6 @@
  */
 namespace OCA\DAV\CalDAV\Publishing\Xml;
 
-use OCA\DAV\CalDAV\Publishing\PublishPlugin as Plugin;
 use Sabre\Xml\Writer;
 use Sabre\Xml\XmlSerializable;
 

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

Doc Comments +4 added lines, -1 removed lines

@@ -236,6 +236,9 @@ 
 		return '"' . $this->info->getEtag() . '"';
 	}
 
+	/**
+	 * @param string $path
+	 */
 	private function getPartFileBasePath($path) {
 		$partFileInStorage = \OC::$server->getConfig()->getSystemValue('part_file_in_storage', true);
 		if ($partFileInStorage) {
@@ -598,7 +601,7 @@ 
 	/**
 	 * Set $algo to get a specific checksum, leave null to get all checksums
 	 * (space seperated)
-	 * @param null $algo
+	 * @param string $algo
 	 * @return string
 	 */
 	public function getChecksum($algo = null) {