owncloud / core

apps/files_external/lib/amazons3.php

Doc Comments +3 added lines

@@ -230,6 +230,9 @@
 		return false;
 	}
 
+	/**
+	 * @param string $path
+	 */
 	private function batchDelete ($path = null) {
 		$params = array(
 			'Bucket' => $this->bucket

apps/files_external/lib/config.php

Doc Comments +2 added lines, -1 removed lines

@@ -244,6 +244,7 @@ 
 	 *
 	 * @param string $class backend class name
 	 * @param array $options backend configuration options
+	 * @param boolean $isPersonal
 	 * @return int see self::STATUS_*
 	 */
 	public static function getBackendStatus($class, $options, $isPersonal) {
@@ -356,7 +357,7 @@ 
 	 * @param \OCP\IL10N $l
 	 * @param string $module
 	 * @param string $backend
-	 * @return string
+	 * @return OC_L10N_String
 	 */
 	private static function getSingleDependencyMessage(\OCP\IL10N $l, $module, $backend) {
 		switch (strtolower($module)) {

apps/files_external/lib/config/configadapter.php

Unused Use Statements -1 removed lines

@@ -32,7 +32,6 @@
 use OCA\Files_external\Service\UserStoragesService;
 use OCA\Files_External\Service\UserGlobalStoragesService;
 use OCA\Files_External\Lib\StorageConfig;
-use OCP\Files\StorageNotAvailableException;
 use OCA\Files_External\Lib\FailedStorage;
 
 /**

apps/files_external/lib/sftp.php

Doc Comments +1 added lines, -1 removed lines

@@ -195,7 +195,7 @@
 	}
 
 	/**
-	 * @return bool|string
+	 * @return string|false
 	 */
 	private function hostKeysPath() {
 		try {

Unused Use Statements -1 removed lines

@@ -31,7 +31,6 @@
  */
 namespace OC\Files\Storage;
 use Icewind\Streams\IteratorDirectory;
-
 use phpseclib\Net\SFTP\Stream;
 
 /**

apps/files_external/service/storagesservice.php

Doc Comments +1 added lines, -1 removed lines

@@ -394,7 +394,7 @@
 	/**
 	 * Add new storage to the configuration
 	 *
-	 * @param array $newStorage storage attributes
+	 * @param StorageConfig $newStorage storage attributes
 	 *
 	 * @return StorageConfig storage config, with added id
 	 */

apps/files_external/tests/amazons3migration.php

Doc Comments +3 added lines

@@ -123,6 +123,9 @@
 		return $storages;
 	}
 
+	/**
+	 * @param string $id
+	 */
 	public function deleteStorage($id) {
 		$stmt = \OC::$server->getDatabaseConnection()->prepare(
 			'DELETE FROM `*PREFIX*storages` WHERE `id` = ?'

apps/files_external/tests/controller/storagescontrollertest.php

Doc Comments +6 added lines

@@ -48,6 +48,9 @@ 
 		\OC_Mount_Config::$skipTest = false;
 	}
 
+	/**
+	 * @return \OCA\Files_External\Lib\Backend\Backend
+	 */
 	protected function getBackendMock($class = '\OCA\Files_External\Lib\Backend\SMB', $storageClass = '\OC\Files\Storage\SMB') {
 		$backend = $this->getMockBuilder('\OCA\Files_External\Lib\Backend\Backend')
 			->disableOriginalConstructor()
@@ -59,6 +62,9 @@ 
 		return $backend;
 	}
 
+	/**
+	 * @return \OCA\Files_External\Lib\Auth\AuthMechanism
+	 */
 	protected function getAuthMechMock($scheme = 'null', $class = '\OCA\Files_External\Lib\Auth\NullMechanism') {
 		$authMech = $this->getMockBuilder('\OCA\Files_External\Lib\Auth\AuthMechanism')
 			->disableOriginalConstructor()

apps/files_external/tests/service/backendservicetest.php

Doc Comments +5 added lines

@@ -35,6 +35,11 @@
 		$this->l10n = $this->getMock('\OCP\IL10N');
 	}
 
+	/**
+	 * @param string $class
+	 *
+	 * @return \OCA\Files_External\Lib\Backend\Backend
+	 */
 	protected function getBackendMock($class) {
 		$backend = $this->getMockBuilder('\OCA\Files_External\Lib\Backend\Backend')
 			->disableOriginalConstructor()

apps/files_sharing/api/server2server.php

Doc Comments +1 added lines, -1 removed lines

@@ -246,7 +246,7 @@
 	 *
 	 * @param int $id
 	 * @param string $token
-	 * @return array
+	 * @return string
 	 */
 	private function getShare($id, $token) {
 		$query = \OCP\DB::prepare('SELECT * FROM `*PREFIX*share` WHERE `id` = ? AND `token` = ? AND `share_type` = ?');