CS auto fixes

Signed-off-by: Joas Schilling <coding@schilljs.com>

lib/AppInfo/Application.php

@@ -26,8 +26,7 @@ namespace OCA\Registration\AppInfo;
 use OCP\AppFramework\App;
 
 class Application extends App {
-
+	public function __construct(array $urlParams = []) {
-	public function __construct(array $urlParams = array()) {
 		parent::__construct('registration', $urlParams);
 
 		$container = $this->getContainer();
@@ -39,5 +38,4 @@ class Application extends App {
 			$container->registerCapability(\OCA\Registration\Capabilities::class);
 		}
 	}
-
 }

lib/Capabilities.php

@@ -23,7 +23,6 @@
 
 namespace OCA\Registration;
 
-use OCP\Capabilities\ICapability;
 use OCP\Capabilities\IPublicCapability;
 use OCP\IURLGenerator;
 
@@ -47,5 +46,4 @@ class Capabilities implements IPublicCapability {
 			]
 		];
 	}
-
 }

lib/Controller/ApiController.php

@@ -47,9 +47,9 @@ class ApiController extends OCSController {
 	/** @var Defaults */
 	private $defaults;
 
-	const REGISTRATION_STATUS_COMPLETE = 0;
+	public const REGISTRATION_STATUS_COMPLETE = 0;
-	const REGISTRATION_STATUS_PENDING = 1;
+	public const REGISTRATION_STATUS_PENDING = 1;
-	const REGISTRATION_STATUS_EXISTING = 2;
+	public const REGISTRATION_STATUS_EXISTING = 2;
 
 	public function __construct($appName,
 								IRequest $request,

lib/Controller/RegisterController.php

@@ -59,10 +59,10 @@ class RegisterController extends Controller {
 	 * @return TemplateResponse
 	 */
 	public function askEmail($errormsg, $entered) {
-		$params = array(
+		$params = [
 			'errormsg' => $errormsg ? $errormsg : $this->request->getParam('errormsg'),
 			'entered' => $entered ? $entered : $this->request->getParam('entered')
-		);
+		];
 		return new TemplateResponse('registration', 'register', $params, 'guest');
 	}
 
@@ -88,20 +88,20 @@ class RegisterController extends Controller {
 			} else {
 				$this->registrationService->generateNewToken($reg);
 				$this->mailService->sendTokenByMail($reg);
-				return new TemplateResponse('registration', 'message', array('msg' =>
+				return new TemplateResponse('registration', 'message', ['msg' =>
 					$this->l10n->t('There is already a pending registration with this email, a new verification email has been sent to the address.')
-				), 'guest');
+				], 'guest');
 			}
 		} catch (RegistrationException $e) {
-			return new TemplateResponse('registration', 'message', array('msg' =>
+			return new TemplateResponse('registration', 'message', ['msg' =>
 				$e->getMessage().'<br/>'.$e->getHint()
-			), 'guest');
+			], 'guest');
 		}
 
 
-		return new TemplateResponse('registration', 'message', array('msg' =>
+		return new TemplateResponse('registration', 'message', ['msg' =>
 			$this->l10n->t('Verification email successfully sent.')
-		), 'guest');
+		], 'guest');
 	}
 
 	/**
@@ -130,7 +130,6 @@ class RegisterController extends Controller {
 		} catch (RegistrationException $exception) {
 			return $this->renderError($exception->getMessage(), $exception->getHint());
 		}
-
 	}
 
 	/**
@@ -152,8 +151,8 @@ class RegisterController extends Controller {
 			return new TemplateResponse('registration', 'form',
 				[
 					'email' => $registration->getEmail(),
-					'entered_data' => array('user' => $username),
+					'entered_data' => ['user' => $username],
-					'errormsgs' => array($exception->getMessage()),
+					'errormsgs' => [$exception->getMessage()],
 					'token' => $token
 				], 'guest');
 		}
@@ -166,18 +165,17 @@ class RegisterController extends Controller {
 			return new TemplateResponse(
 				'registration',
 				'message',
-				array('msg' => $this->l10n->t("Your account has been successfully created, but it still needs approval from an administrator.")),
+				['msg' => $this->l10n->t("Your account has been successfully created, but it still needs approval from an administrator.")],
 				'guest');
 		}
 	}
 
 	private function renderError($error, $hint="") {
-		return new TemplateResponse('', 'error', array(
+		return new TemplateResponse('', 'error', [
-			'errors' => array(array(
+			'errors' => [[
 				'error' => $error,
 				'hint' => $hint
-			))
+			]]
-		), 'error');
+		], 'error');
 	}
-
 }

lib/Controller/SettingsController.php

@@ -52,7 +52,7 @@ class SettingsController extends Controller {
 	 */
 	public function admin($registered_user_group, $allowed_domains, $admin_approval_required) {
 		// handle domains
-		if ( ( $allowed_domains==='' ) || ( $allowed_domains === NULL ) ){
+		if (($allowed_domains==='') || ($allowed_domains === null)) {
 			$this->config->deleteAppValue($this->appName, 'allowed_domains');
 		} else {
 			$this->config->setAppValue($this->appName, 'allowed_domains', $allowed_domains);
@@ -63,34 +63,34 @@ class SettingsController extends Controller {
 
 		// handle groups
 		$groups = $this->groupmanager->search('');
-		$group_id_list = array();
+		$group_id_list = [];
 		foreach ($groups as $group) {
 			$group_id_list[] = $group->getGid();
 		}
 		if ($registered_user_group === 'none') {
 			$this->config->deleteAppValue($this->appName, 'registered_user_group');
-			return new DataResponse(array(
+			return new DataResponse([
-				'data' => array(
+				'data' => [
 					'message' => (string) $this->l10n->t('Saved'),
-				),
+				],
 				'status' => 'success'
 				
-			));
+			]);
 		} elseif (in_array($registered_user_group, $group_id_list)) {
 			$this->config->setAppValue($this->appName, 'registered_user_group', $registered_user_group);
-			return new DataResponse(array(
+			return new DataResponse([
-				'data' => array(
+				'data' => [
 					'message' => (string) $this->l10n->t('Saved'),
-				),
+				],
 				'status' => 'success'
-			));
+			]);
 		} else {
-			return new DataResponse(array(
+			return new DataResponse([
-				'data' => array(
+				'data' => [
 					'message' => (string) $this->l10n->t('No such group'),
-				),
+				],
 				'status' => 'error'
-			), Http::STATUS_NOT_FOUND);
+			], Http::STATUS_NOT_FOUND);
 		}
 	}
 	/**

lib/Db/Registration.php

@@ -26,7 +26,6 @@ namespace OCA\Registration\Db;
 use OCP\AppFramework\Db\Entity;
 
 class Registration extends Entity {
-
 	public $id;
 	protected $email;
 	protected $username;
@@ -40,5 +39,4 @@ class Registration extends Entity {
 	public function __construct() {
 		$this->addType('emailConfirmed', 'boolean');
 	}
-
 }

lib/Db/RegistrationMapper.php

@@ -97,5 +97,4 @@ class RegistrationMapper extends Mapper {
 		//FIXME eqivalent to ISecureRandom::CHAR_HUMAN_READABLE introduced in https://github.com/nextcloud/server/commit/f2a2b34e4639e88f8d948a388a51f010212b42a3 but not supported in ownCloud yet. We'll just use the string for now then switch to constants when supported.
 		$registration->setClientSecret($token);
 	}
-
 }

lib/Service/MailService.php

@@ -79,7 +79,7 @@ class MailService {
 	 * @throws RegistrationException
 	 */
 	public function sendTokenByMail(Registration $registration) {
-		$link = $this->urlGenerator->linkToRoute('registration.register.verifyToken', array('token' => $registration->getToken()));
+		$link = $this->urlGenerator->linkToRoute('registration.register.verifyToken', ['token' => $registration->getToken()]);
 		$link = $this->urlGenerator->getAbsoluteURL($link);
 		$template_var = [
 			'link' => $link,
@@ -125,7 +125,7 @@ class MailService {
 			}
 		}
 
-		$to_arr = array();
+		$to_arr = [];
 		foreach ($admin_users as $au) {
 			$au_email = $au->getEMailAddress();
 			if ($au_email && $au->isEnabled()) {
@@ -182,8 +182,8 @@ class MailService {
 		$message->setPlainBody($plaintext_part);
 		$message->setHtmlBody($html_part);
 		$failed_recipients = $this->mailer->send($message);
-		if ( !empty($failed_recipients) )
+		if (!empty($failed_recipients)) {
 			throw new RegistrationException('Failed recipients: '.print_r($failed_recipients, true));
 		}
-
+	}
 }

lib/Service/RegistrationException.php

@@ -24,7 +24,6 @@
 namespace OCA\Registration\Service;
 
 class RegistrationException extends \Exception {
-
 	protected $hint;
 
 	/**
@@ -45,5 +44,4 @@ class RegistrationException extends \Exception {
 	public function getHint() {
 		return $this->hint;
 	}
-
 }

lib/Service/RegistrationService.php

@@ -139,8 +139,9 @@ class RegistrationService {
 			$registration->setPassword($password);
 		}
 		$this->registrationMapper->generateNewToken($registration);
-		if ( $password !== '' && $username !== '' )
+		if ($password !== '' && $username !== '') {
 			$this->registrationMapper->generateClientSecret($registration);
+		}
 		$this->registrationMapper->insert($registration);
 		return $registration;
 	}
@@ -151,13 +152,13 @@ class RegistrationService {
 	 * @throws RegistrationException
 	 */
 	public function validateEmail($email) {
-
 		$this->mailService->validateEmail($email);
 
 		// check for pending registrations
 		try {
 			return $this->registrationMapper->find($email);//if not found DB will throw a exception
-		} catch (DoesNotExistException $e) {}
+		} catch (DoesNotExistException $e) {
+		}
 
 		if ($this->userManager->getByEmail($email)) {
 			throw new RegistrationException(
@@ -434,7 +435,6 @@ class RegistrationService {
 			['msg' => $this->l10n->t('Your account has been successfully created, you can <a href="%s">log in now</a>.', [$this->urlGenerator->getAbsoluteURL('/')])]
 			, 'guest'
 		);
-
 	}
 
 	/**
@@ -451,5 +451,4 @@ class RegistrationService {
 			}
 		}
 	}
-
 }

lib/Settings/RegistrationSettings.php

@@ -1,7 +1,7 @@
 <?php
+
 namespace OCA\Registration\Settings;
 
-use OCP\AppFramework\Http\TemplateResponse;
 use OCP\Settings\ISettings;
 
 use OCA\Registration\Controller\SettingsController;
@@ -31,5 +31,3 @@ class RegistrationSettings implements ISettings {
 		return $this->getSection();
 	}
 }
-
-?>

lib/Util/CoreBridge.php

@@ -2,8 +2,7 @@
 
 namespace OCA\Registration\Util;
 
-class CoreBridge
+class CoreBridge {
-{
 	/**
 	 * This function maps an exception class to the available exception class of the core
 	 * in order to provide cross-core and cross-version compatibility.
@@ -12,8 +11,7 @@ class CoreBridge
 	 * @return string
 	 * @throws \LogicException
 	 */
-    public static function exceptionClass($className)
+	public static function exceptionClass($className) {
-    {
 		static $classes = [
 			'OCSException'           => [
 				'OCP\AppFramework\OCS\OCSException',
@@ -52,8 +50,7 @@ class CoreBridge
 	 * @param null|int    $code
 	 * @return \Exception
 	 */
-    public static function createException($className, $message = null, $code = null)
+	public static function createException($className, $message = null, $code = null) {
-    {
 		$exceptionClassName = self::exceptionClass($className);
 
 		$reflection = new \ReflectionClass($exceptionClassName);

templates/admin.php

@@ -25,7 +25,9 @@ foreach ( $_['groups'] as $group ) {
 
 	<p>
 	<label for="admin_approval_required"><?php p($l->t('Require admin approval?')); ?>
-	<input type="checkbox" id="admin_approval_required" name="admin_approval_required" <?php if($_['approval_required'] === "yes" ) echo " checked"; ?>>
+	<input type="checkbox" id="admin_approval_required" name="admin_approval_required" <?php if ($_['approval_required'] === "yes") {
+	echo " checked";
+} ?>>
 	</label>
 	</p>
 </form>

templates/email.newuser.disabled_html.php

@@ -1,3 +1,4 @@
 <?php
+
 echo $l->t('A new user "%s" has created an account on %s and awaits admin approbation', [$_['user'], $_['sitename']]);
 echo str_replace('{link}', $_['link'], '<br/><br/><a href="{link}">{link}</a>');

templates/email.newuser.disabled_plaintext.php

@@ -1,3 +1,4 @@
 <?php
+
 echo $l->t('A new user "%s" has created an account on %s and awaits admin approbation', [$_['user'], $_['sitename']]);
 echo "\n\n".$_['link'];

templates/email.newuser_html.php

@@ -1,2 +1,3 @@
 <?php
+
 echo $l->t('A new user "%s" has created an account on %s', [$_['user'], $_['sitename']]);

templates/email.newuser_plaintext.php

@@ -1,2 +1,3 @@
 <?php
+
 echo $l->t('A new user "%s" has created an account on %s', [$_['user'], $_['sitename']]);

templates/email.validate_html.php

@@ -1,3 +1,4 @@
 <?php
+
 echo $l->t('To create a new account on %s, just click the following link:', [$_['sitename']]);
 echo str_replace('{link}', $_['link'], '<br/><br/><a href="{link}">{link}</a>');

templates/email.validate_plaintext.php

@@ -1,3 +1,4 @@
 <?php
+
 echo $l->t("To create a new account on %s, just click the following link:", [$_['sitename']]);
 echo "\n\n".$_['link'];

templates/form.php

@@ -1,9 +1,10 @@
 <?php
 \OCP\Util::addStyle('registration', 'style');
 \OCP\Util::addScript('registration', 'form');
-if ( \OCP\Util::getVersion()[0] >= 12 )
+if (\OCP\Util::getVersion()[0] >= 12) {
 	\OCP\Util::addStyle('core', 'guest');
-?><form action="<?php print_unescaped(\OC::$server->getURLGenerator()->linkToRoute('registration.register.createAccount', array('token'=>$_['token']))) ?>" method="post">
+}
+?><form action="<?php print_unescaped(\OC::$server->getURLGenerator()->linkToRoute('registration.register.createAccount', ['token'=>$_['token']])) ?>" method="post">
 	<input type="hidden" name="requesttoken" value="<?php p($_['requesttoken']) ?>" />
 	<fieldset>
 		<?php if (!empty($_['errormsgs'])) {?>
@@ -24,7 +25,9 @@ if ( \OCP\Util::getVersion()[0] >= 12 )
 		</p>
 
 		<p class="groupmiddle">
-			<input type="text" name="username" id="username" value="<?php if (!empty($_['entered_data']['user'])) { p($_['entered_data']['user']); } ?>" placeholder="<?php p($l->t('Username')); ?>" />
+			<input type="text" name="username" id="username" value="<?php if (!empty($_['entered_data']['user'])) {
+	p($_['entered_data']['user']);
+} ?>" placeholder="<?php p($l->t('Username')); ?>" />
 			<label for="username" class="infield"><?php p($l->t('Username')); ?></label>
 			<img id="username-icon" class="svg" src="<?php print_unescaped(image_path('', 'actions/user.svg')); ?>" alt=""/>
 		</p>

templates/message.php

@@ -1,7 +1,8 @@
 <?php
 \OCP\Util::addStyle('registration', 'style');
-if ( \OCP\Util::getVersion()[0] >= 12 )
+if (\OCP\Util::getVersion()[0] >= 12) {
-        \OCP\Util::addStyle('core', 'guest')
+	\OCP\Util::addStyle('core', 'guest');
+}
 ?>
 <ul class="msg error-wide nc-theming-main-text">
 	<li><?php print_unescaped($_['msg'])?></li>

templates/register.php

@@ -1,7 +1,8 @@
 <?php
 \OCP\Util::addStyle('registration', 'style');
-if ( \OCP\Util::getVersion()[0] >= 12 )
+if (\OCP\Util::getVersion()[0] >= 12) {
 	\OCP\Util::addStyle('core', 'guest');
+}
 if ($_['entered']): ?>
 	<?php if (empty($_['errormsg'])): ?>
 		<ul class="success">

tests/Integration/Controller/RegisterControllerTest.php

@@ -2,13 +2,9 @@
 
 namespace OCA\Registration\Tests\Integration\Controller;
 
-use OCA\Registration\Db\Registration;
 use OCA\Registration\Db\RegistrationMapper;
 use OCA\Registration\Service\MailService;
 use OCA\Registration\Service\RegistrationService;
-use OCA\Registration\Util\CoreBridge;
-use OCP\AppFramework\Http;
-use OCP\AppFramework\Http\DataResponse;
 use OCP\Defaults;
 use OCP\IConfig;
 use OCP\IGroupManager;
@@ -19,7 +15,6 @@ use OCP\IRequest;
 use OCP\Security\ISecureRandom;
 use OCP\Security\ICrypto;
 use OCP\ISession;
-use OCP\IUser;
 use OCP\IURLGenerator;
 use OCP\IUserManager;
 use OCP\IUserSession;
@@ -28,6 +23,7 @@ use \OCP\AppFramework\Http\TemplateResponse;
 
 use ChristophWurst\Nextcloud\Testing\DatabaseTransaction;
 use ChristophWurst\Nextcloud\Testing\TestCase;
+
 /**
  * class RegistrationControllerTest
  *
@@ -134,9 +130,9 @@ class RegisterControllerTest extends TestCase {
 
 		$ret = $this->controller->validateEmail($email);
 
-		$expected = new TemplateResponse('registration', 'message', array('msg' =>
+		$expected = new TemplateResponse('registration', 'message', ['msg' =>
 			$this->l10n->t('Verification email successfully sent.')
-		), 'guest');
+		], 'guest');
 
 
 		$this->assertEquals($expected, $ret, print_r($ret, true));

tests/Integration/Service/RegistrationServiceTest.php

@@ -6,9 +6,6 @@ use OCA\Registration\Db\Registration;
 use OCA\Registration\Db\RegistrationMapper;
 use OCA\Registration\Service\MailService;
 use OCA\Registration\Service\RegistrationService;
-use OCA\Registration\Util\CoreBridge;
-use OCP\AppFramework\Http;
-use OCP\AppFramework\Http\DataResponse;
 use OCP\Defaults;
 use OCP\IConfig;
 use OCP\IGroupManager;
@@ -26,6 +23,7 @@ use OCP\IUserSession;
 
 use ChristophWurst\Nextcloud\Testing\DatabaseTransaction;
 use ChristophWurst\Nextcloud\Testing\TestCase;
+
 /**
  * class RegistrationServiceTest
  *
@@ -202,7 +200,7 @@ class RegistrationServiceTest extends TestCase {
 
 		$this->config->expects($this->atLeastOnce())
 			->method('getAppValue')
-			->will($this->returnCallback(array($this, 'settingsCallback1')));
+			->will($this->returnCallback([$this, 'settingsCallback1']));
 
 
 		$form_input_username = 'alice1';