owncloud / core

apps/federation/lib/dbhandler.php

Doc Comments +2 added lines, -2 removed lines

@@ -271,8 +271,8 @@
 	}
 
 	/**
-	 * @param $username
-	 * @param $password
+	 * @param string $username
+	 * @param string $password
 	 * @return bool
 	 */
 	public function auth($username, $password) {

apps/files_sharing/api/sharees.php

Doc Comments +1 added lines, -1 removed lines

@@ -327,7 +327,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) {

lib/private/Server.php

Doc Comments +1 added lines, -1 removed lines

@@ -856,7 +856,7 @@
 	 * Get the certificate manager for the user
 	 *
 	 * @param string $userId (optional) if not specified the current loggedin user is used
-	 * @return \OCP\ICertificateManager | null if $uid is null and no user is logged in
+	 * @return null|CertificateManager | null if $uid is null and no user is logged in
 	 */
 	public function getCertificateManager($userId = null) {
 		if (is_null($userId)) {

lib/private/Share20/Manager.php

Unused Use Statements -2 removed lines

@@ -25,7 +25,6 @@ 
 use OCP\IUserManager;
 use OCP\Share\IManager;
 use OCP\Share\IProviderFactory;
-use OC\Share20\Exception\BackendError;
 use OCP\IConfig;
 use OCP\IL10N;
 use OCP\ILogger;
@@ -35,7 +34,6 @@ 
 use OCP\IGroupManager;
 use OCP\Files\File;
 use OCP\Files\Folder;
-
 use OCP\Share\Exceptions\ShareNotFound;
 use OCP\Share\Exceptions\GenericShareException;
 

core/Command/Security/ListCertificates.php

Unused Use Statements -1 removed lines

@@ -28,7 +28,6 @@
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Helper\Table;
 use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class ListCertificates extends Base {

core/Controller/LostController.php

Doc Comments +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
 	}
 
 	/**
-	 * @param $message
+	 * @param string $message
 	 * @param array $additional
 	 * @return array
 	 */

settings/Controller/AppSettingsController.php

Doc Comments +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
 	}
 
 	/**
-	 * @param string|int $category
+	 * @param string $category
 	 * @return int
 	 */
 	protected function getCategory($category) {

settings/Controller/CertificateController.php

Doc Comments +3 added lines, -3 removed lines

@@ -70,7 +70,7 @@ 
 	 *
 	 * @NoAdminRequired
 	 * @NoSubadminRequired
-	 * @return array
+	 * @return DataResponse
 	 */
 	public function addPersonalRootCertificate() {
 		return $this->addCertificate($this->userCertificateManager);
@@ -80,7 +80,7 @@ 
 	 * Add a new root certificate to a trust store
 	 *
 	 * @param ICertificateManager $certificateManager
-	 * @return array
+	 * @return DataResponse
 	 */
 	private function addCertificate(ICertificateManager $certificateManager) {
 		$headers = [];
@@ -158,7 +158,7 @@ 
 	/**
 	 * Add a new personal root certificate to the system's trust store
 	 *
-	 * @return array
+	 * @return DataResponse
 	 */
 	public function addSystemRootCertificate() {
 		return $this->addCertificate($this->systemCertificateManager);

settings/Controller/CheckSetupController.php

Doc Comments +1 added lines, -1 removed lines

@@ -268,7 +268,7 @@
 
 	/**
 	 * @NoCSRFRequired
-	 * @return DataResponse
+	 * @return DataDisplayResponse
 	 */
 	public function getFailedIntegrityCheckFiles() {
 		$completeResults = $this->checker->getResults();