owncloud / core

lib/private/User/TokenAuthModule.php

Unused Use Statements -1 removed lines

@@ -30,7 +30,6 @@
 use OCP\Authentication\IAuthModule;
 use OCP\IRequest;
 use OCP\ISession;
-use OCP\IUser;
 use OCP\IUserManager;
 use OCP\Session\Exceptions\SessionNotAvailableException;
 

core/Command/User/SyncBackend.php

Doc Comments +4 added lines, -4 removed lines

@@ -172,7 +172,7 @@ 
 	 * @param SyncService $syncService
 	 * @param UserInterface $backend
 	 * @param string $missingAccountsAction
-	 * @param array $validActions
+	 * @param string[] $validActions
 	 */
 	private function syncMultipleUsers (
 		InputInterface $input,
@@ -220,7 +220,7 @@ 
 	 * @param UserInterface $backend
 	 * @param string $uid
 	 * @param string $missingAccountsAction
-	 * @param array $validActions
+	 * @param string[] $validActions
 	 */
 	private function syncSingleUser(
 		InputInterface $input,
@@ -255,7 +255,7 @@ 
 	}
 
 	/**
-	 * @param array $uids a list of uids to the the action
+	 * @param string[] $uids a list of uids to the the action
 	 * @param callable $callbackExists the callback used if the account for the uid exists. The
 	 * uid and the specific account will be passed as parameter to the callback in that order
 	 * @param callable $callbackMissing the callback used if the account doesn't exists. The uid (not
@@ -276,7 +276,7 @@ 
 	 * @param string[] $unknownUsers
 	 * @param InputInterface $input
 	 * @param OutputInterface $output
-	 * @param $missingAccountsAction
+	 * @param string $missingAccountsAction
 	 * @param $validActions
 	 */
 	private function handleUnknownUsers(array $unknownUsers, InputInterface $input, OutputInterface $output, $missingAccountsAction, $validActions) {

lib/private/Notification/Manager.php

Doc Comments +2 added lines

@@ -113,6 +113,8 @@
 	 * INTERNAL USE ONLY!! This method isn't part of the IManager interface
 	 * @internal This should only be used by the RegisterNotifierEventImpl (the real implementation).
 	 * Do NOT use this method outside as it might not work as expected.
+	 * @param string $id
+	 * @param string $name
 	 */
 	public function registerBuiltNotifier(INotifier $notifier, $id, $name) {
 		if (!isset($this->builtNotifiersHolder[$id]) && !isset($this->notifiersInfo[$id])) {

lib/private/DB/MigrationService.php

Doc Comments +2 added lines, -2 removed lines

@@ -275,7 +275,7 @@ 
 	 * Return the explicit version for the aliases; current, next, prev, latest
 	 *
 	 * @param string $alias
-	 * @return mixed|null|string
+	 * @return string
 	 */
 	public function getMigration($alias) {
 		switch($alias) {
@@ -361,7 +361,7 @@ 
 
 	/**
 	 * @param string $version
-	 * @return mixed
+	 * @return \Doctrine\DBAL\Schema\Schema
 	 * @throws \Exception
 	 */
 	protected function createInstance($version) {

settings/Controller/CorsController.php

Doc Comments -1 removed lines

@@ -134,7 +134,6 @@
 	 * @NoAdminRequired
 	 * @NoSubadminRequired
 	 *
-	 * @param string $domain Domain to remove
 	 * @return RedirectResponse Redirection to the settings page.
 	 */
 	public function removeDomain($id) {

core/Command/Db/ConvertType.php

Doc Comments +2 added lines, -2 removed lines

@@ -280,7 +280,7 @@ 
 	/**
 	 * @param Connection $fromDB
 	 * @param Connection $toDB
-	 * @param $app
+	 * @param string $app
 	 * @throws \Exception
 	 * @throws \OC\NeedsUpdateException
 	 */
@@ -410,7 +410,7 @@ 
 
 	/**
 	 * @param Table $table
-	 * @return mixed
+	 * @return string[]
 	 */
 	protected function getColumnTypes(Table $table) {
 		$tableName = $table->getName();

core/Middleware/TwoFactorMiddleware.php

Doc Comments +2 added lines, -2 removed lines

@@ -109,8 +109,8 @@
 	}
 
 	/**
-	 * @param $controller
-	 * @param $methodName
+	 * @param Controller $controller
+	 * @param string $methodName
 	 * @throws TwoFactorAuthRequiredException
 	 * @throws UserAlreadyLoggedInException
 	 */

lib/private/legacy/response.php

Doc Comments +2 added lines, -2 removed lines

@@ -274,7 +274,7 @@ 
 	 * @param string $userId
 	 * @param string $domain
 	 * @param \OCP\IConfig $config
-	 * @param array $headers
+	 * @param string[] $headers
 	 *
 	 * Format of $headers:
 	 * Array [
@@ -318,7 +318,7 @@ 
 	 * This function adds the CORS headers for all domains
 	 *
 	 * @param Sabre\HTTP\ResponseInterface $response
-	 * @param Array $headers
+	 * @param string[] $headers
 	 *
 	 * Format of $headers:
 	 * Array [

settings/Application.php

Unused Use Statements -3 removed lines

@@ -39,11 +39,8 @@
 use OC\Settings\Controller\AuthSettingsController;
 use OC\Settings\Controller\CertificateController;
 use OC\Settings\Controller\CheckSetupController;
-use OC\Settings\Controller\GroupsController;
 use OC\Settings\Controller\LogSettingsController;
 use OC\Settings\Controller\MailSettingsController;
-use OC\Settings\Controller\UsersController;
-use OC\Settings\Middleware\SubadminMiddleware;
 use OCP\AppFramework\App;
 use OCP\IContainer;
 use OCP\Util;