nextcloud / twofactor_gateway

lib/Command/Status.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
 		parent::__construct('twofactorauth:gateway:status');
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function execute(InputInterface $input, OutputInterface $output) {
 		$fqcn = $this->gatewayFactory->getFqcnList();
 		foreach ($fqcn as $fqcn) {
@@ -30,7 +30,7 @@ 
 			$gateway = $this->gatewayFactory->get($fqcn::getProviderId());
 			$isConfigured = $gateway->isComplete();
 			$settings = $gateway->getSettings();
-			$output->writeln($settings['name'] . ': ' . ($isConfigured ? 'configured' : 'not configured'));
+			$output->writeln($settings[ 'name' ].': '.($isConfigured ? 'configured' : 'not configured'));
 		}
 		return 0;
 	}

lib/Command/Remove.php

Spacing +5 added lines, -5 removed lines

@@ -20,7 +20,7 @@ 
 use Symfony\Component\Console\Question\ChoiceQuestion;
 
 class Remove extends Command {
-	private array $ids = [];
+	private array $ids = [ ];
 
 	public function __construct(
 		private Factory $gatewayFactory,
@@ -29,17 +29,17 @@ 
 
 		$fqcn = $this->gatewayFactory->getFqcnList();
 		foreach ($fqcn as $fqcn) {
-			$this->ids[] = $fqcn::getProviderId();
+			$this->ids[ ] = $fqcn::getProviderId();
 		}
 
 		$this->addArgument(
 			'gateway',
 			InputArgument::OPTIONAL,
-			'The name of the gateway: ' . implode(', ', $this->ids)
+			'The name of the gateway: '.implode(', ', $this->ids)
 		);
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function execute(InputInterface $input, OutputInterface $output) {
 		$gatewayName = strtolower((string)$input->getArgument('gateway'));
 		if (!in_array($gatewayName, $this->ids, true)) {
@@ -52,7 +52,7 @@ 
 			/** @var AGateway */
 			$gateway = $this->gatewayFactory->get($gatewayName);
 		} catch (Exception $e) {
-			$output->writeln('<error>' . $e->getMessage() . '</error>');
+			$output->writeln('<error>'.$e->getMessage().'</error>');
 			return 1;
 		}
 

lib/Command/Configure.php

Spacing +4 added lines, -4 removed lines

@@ -20,7 +20,7 @@ 
 use Symfony\Component\Console\Question\ChoiceQuestion;
 
 class Configure extends Command {
-	private array $ids = [];
+	private array $ids = [ ];
 
 	public function __construct(
 		private Factory $gatewayFactory,
@@ -29,17 +29,17 @@ 
 
 		$fqcn = $this->gatewayFactory->getFqcnList();
 		foreach ($fqcn as $fqcn) {
-			$this->ids[] = $fqcn::getProviderId();
+			$this->ids[ ] = $fqcn::getProviderId();
 		}
 
 		$this->addArgument(
 			'gateway',
 			InputArgument::OPTIONAL,
-			'The name of the gateway: ' . implode(', ', $this->ids)
+			'The name of the gateway: '.implode(', ', $this->ids)
 		);
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function execute(InputInterface $input, OutputInterface $output) {
 		$gatewayName = strtolower((string)$input->getArgument('gateway'));
 		if (!in_array($gatewayName, $this->ids, true)) {

lib/Controller/SettingsController.php

Spacing +16 added lines, -16 removed lines

@@ -43,17 +43,17 @@ 
 	 * 400: User not found
 	 * 503: Gateway wasn't configured yed
 	 */
-	#[NoAdminRequired]
-	#[ApiRoute(verb: 'GET', url: '/settings/{gateway}/verification')]
+	#[NoAdminRequired ]
+	#[ApiRoute(verb: 'GET', url: '/settings/{gateway}/verification') ]
 	public function getVerificationState(string $gateway): JSONResponse {
 		$user = $this->userSession->getUser();
 
 		if (is_null($user)) {
-			return new JSONResponse(['message' => 'User not found'], Http::STATUS_BAD_REQUEST);
+			return new JSONResponse([ 'message' => 'User not found' ], Http::STATUS_BAD_REQUEST);
 		}
 
 		if (!$this->gatewayFactory->get($gateway)->isComplete()) {
-			return new JSONResponse([], Http::STATUS_SERVICE_UNAVAILABLE);
+			return new JSONResponse([ ], Http::STATUS_SERVICE_UNAVAILABLE);
 		}
 
 		return new JSONResponse($this->setup->getState($user, $gateway)->jsonSerialize());
@@ -70,19 +70,19 @@ 
 	 * 200: OK
 	 * 400: User not found
 	 */
-	#[NoAdminRequired]
-	#[ApiRoute(verb: 'POST', url: '/settings/{gateway}/verification/start')]
+	#[NoAdminRequired ]
+	#[ApiRoute(verb: 'POST', url: '/settings/{gateway}/verification/start') ]
 	public function startVerification(string $gateway, string $identifier): JSONResponse {
 		$user = $this->userSession->getUser();
 
 		if (is_null($user)) {
-			return new JSONResponse(['message' => 'User not found'], Http::STATUS_BAD_REQUEST);
+			return new JSONResponse([ 'message' => 'User not found' ], Http::STATUS_BAD_REQUEST);
 		}
 
 		try {
 			$state = $this->setup->startSetup($user, $gateway, $identifier);
 		} catch (VerificationException $e) {
-			return new JSONResponse(['message' => $e->getMessage()], Http::STATUS_BAD_REQUEST);
+			return new JSONResponse([ 'message' => $e->getMessage() ], Http::STATUS_BAD_REQUEST);
 		}
 
 		return new JSONResponse([
@@ -101,22 +101,22 @@ 
 	 * 200: OK
 	 * 400: User not found
 	 */
-	#[NoAdminRequired]
-	#[ApiRoute(verb: 'POST', url: '/settings/{gateway}/verification/finish')]
+	#[NoAdminRequired ]
+	#[ApiRoute(verb: 'POST', url: '/settings/{gateway}/verification/finish') ]
 	public function finishVerification(string $gateway, string $verificationCode): JSONResponse {
 		$user = $this->userSession->getUser();
 
 		if (is_null($user)) {
-			return new JSONResponse(['message' => 'User not found'], Http::STATUS_BAD_REQUEST);
+			return new JSONResponse([ 'message' => 'User not found' ], Http::STATUS_BAD_REQUEST);
 		}
 
 		try {
 			$this->setup->finishSetup($user, $gateway, $verificationCode);
 		} catch (VerificationException) {
-			return new JSONResponse([], Http::STATUS_BAD_REQUEST);
+			return new JSONResponse([ ], Http::STATUS_BAD_REQUEST);
 		}
 
-		return new JSONResponse([]);
+		return new JSONResponse([ ]);
 	}
 
 	/**
@@ -128,13 +128,13 @@ 
 	 * 200: OK
 	 * 400: User not found
 	 */
-	#[NoAdminRequired]
-	#[ApiRoute(verb: 'DELETE', url: '/settings/{gateway}/verification')]
+	#[NoAdminRequired ]
+	#[ApiRoute(verb: 'DELETE', url: '/settings/{gateway}/verification') ]
 	public function revokeVerification(string $gateway): JSONResponse {
 		$user = $this->userSession->getUser();
 
 		if (is_null($user)) {
-			return new JSONResponse(['message' => 'User not found'], Http::STATUS_BAD_REQUEST);
+			return new JSONResponse([ 'message' => 'User not found' ], Http::STATUS_BAD_REQUEST);
 		}
 
 		return new JSONResponse($this->setup->disable($user, $gateway));

lib/Settings/PersonalSettings.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 	) {
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function getBody(): ITemplate {
 		$template = Server::get(ITemplateManager::class)->getTemplate('twofactor_gateway', 'personal_settings');
 		$template->assign('gateway', $this->gateway);

lib/Service/StateStorage.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 	}
 
 	private function buildConfigKey(string $gatewayName, string $key): string {
-		return $gatewayName . "_$key";
+		return $gatewayName."_$key";
 	}
 
 	private function getUserValue(IUser $user, string $gatewayName, string $key, string $default = ''): string {

lib/Capabilities.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 	 *      twofactorgateway?: TwoFactorGatewayCapabilities,
 	 * }
 	 */
-	#[Override]
+	#[Override ]
 	public function getCapabilities(): array {
 		$capabilities = [
 			'features' => self::FEATURES,

lib/Provider/Channel/SMS/Provider/Drivers/Ovh.php

Spacing +27 added lines, -27 removed lines

@@ -34,12 +34,12 @@ 
 	public const SCHEMA = [
 		'name' => 'OVH',
 		'fields' => [
-			['field' => 'endpoint',        'prompt' => 'Please enter the endpoint (ovh-eu, ovh-us, ovh-ca, soyoustart-eu, soyoustart-ca, kimsufi-eu, kimsufi-ca, runabove-ca):'],
-			['field' => 'application_key', 'prompt' => 'Please enter your application key:'],
-			['field' => 'application_secret','prompt' => 'Please enter your application secret:'],
-			['field' => 'consumer_key',    'prompt' => 'Please enter your consumer key:'],
-			['field' => 'account',         'prompt' => 'Please enter your account (sms-*****):'],
-			['field' => 'sender',          'prompt' => 'Please enter your sender:'],
+			[ 'field' => 'endpoint', 'prompt' => 'Please enter the endpoint (ovh-eu, ovh-us, ovh-ca, soyoustart-eu, soyoustart-ca, kimsufi-eu, kimsufi-ca, runabove-ca):' ],
+			[ 'field' => 'application_key', 'prompt' => 'Please enter your application key:' ],
+			[ 'field' => 'application_secret', 'prompt' => 'Please enter your application secret:' ],
+			[ 'field' => 'consumer_key', 'prompt' => 'Please enter your consumer key:' ],
+			[ 'field' => 'account', 'prompt' => 'Please enter your account (sms-*****):' ],
+			[ 'field' => 'sender', 'prompt' => 'Please enter your sender:' ],
 		],
 	];
 	private IClient $client;
@@ -76,24 +76,24 @@ 
 		$this->client = $clientService->newClient();
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function send(string $identifier, string $message) {
 		$endpoint = $this->getEndpoint();
 		$sender = $this->getSender();
 		$smsAccount = $this->getAccount();
 
-		$this->attrs['AK'] = $this->getApplicationKey();
-		$this->attrs['AS'] = $this->getApplicationSecret();
-		$this->attrs['CK'] = $this->getConsumerKey();
-		if (!isset($this->endpoints[$endpoint])) {
+		$this->attrs[ 'AK' ] = $this->getApplicationKey();
+		$this->attrs[ 'AS' ] = $this->getApplicationSecret();
+		$this->attrs[ 'CK' ] = $this->getConsumerKey();
+		if (!isset($this->endpoints[ $endpoint ])) {
 			throw new InvalidProviderException("Endpoint $endpoint not found");
 		}
-		$this->attrs['endpoint'] = $this->endpoints[$endpoint];
+		$this->attrs[ 'endpoint' ] = $this->endpoints[ $endpoint ];
 
 		$this->getTimeDelta();
 
-		$header = $this->getHeader('GET', $this->attrs['endpoint'] . '/sms');
-		$response = $this->client->get($this->attrs['endpoint'] . '/sms', [
+		$header = $this->getHeader('GET', $this->attrs[ 'endpoint' ].'/sms');
+		$response = $this->client->get($this->attrs[ 'endpoint' ].'/sms', [
 			'headers' => $header,
 		]);
 		$body = (string)$response->getBody();
@@ -121,15 +121,15 @@ 
 		];
 		$body = json_encode($content);
 
-		$header = $this->getHeader('POST', $this->attrs['endpoint'] . "/sms/$smsAccount/jobs", $body);
-		$response = $this->client->post($this->attrs['endpoint'] . "/sms/$smsAccount/jobs", [
+		$header = $this->getHeader('POST', $this->attrs[ 'endpoint' ]."/sms/$smsAccount/jobs", $body);
+		$response = $this->client->post($this->attrs[ 'endpoint' ]."/sms/$smsAccount/jobs", [
 			'headers' => $header,
 			'json' => $content,
 		]);
 		$body = (string)$response->getBody();
 		$resultPostJob = json_decode($body, true);
 
-		if (count($resultPostJob['validReceivers']) === 0) {
+		if (count($resultPostJob[ 'validReceivers' ]) === 0) {
 			throw new MessageTransmissionException("Bad receiver $identifier");
 		}
 	}
@@ -140,16 +140,16 @@ 
 	 * @throws InvalidProviderException
 	 */
 	private function getTimeDelta(): void {
-		if (!isset($this->attrs['timedelta'])) {
-			if (!isset($this->attrs['endpoint'])) {
+		if (!isset($this->attrs[ 'timedelta' ])) {
+			if (!isset($this->attrs[ 'endpoint' ])) {
 				throw new InvalidProviderException('Need to set the endpoint');
 			}
 			try {
-				$response = $this->client->get($this->attrs['endpoint'] . '/auth/time');
+				$response = $this->client->get($this->attrs[ 'endpoint' ].'/auth/time');
 				$serverTimestamp = (int)$response->getBody();
-				$this->attrs['timedelta'] = $serverTimestamp - time();
+				$this->attrs[ 'timedelta' ] = $serverTimestamp - time();
 			} catch (Exception $ex) {
-				throw new InvalidProviderException('Unable to calculate time delta:' . $ex->getMessage());
+				throw new InvalidProviderException('Unable to calculate time delta:'.$ex->getMessage());
 			}
 		}
 	}
@@ -162,14 +162,14 @@ 
 	 * @return array $header Contains the data for the request need by OVH
 	 */
 	private function getHeader($method, $query, $body = '') {
-		$timestamp = time() + $this->attrs['timedelta'];
-		$prehash = $this->attrs['AS'] . '+' . $this->attrs['CK'] . '+' . $method . '+' . $query . '+' . $body . '+' . $timestamp;
+		$timestamp = time() + $this->attrs[ 'timedelta' ];
+		$prehash = $this->attrs[ 'AS' ].'+'.$this->attrs[ 'CK' ].'+'.$method.'+'.$query.'+'.$body.'+'.$timestamp;
 		$header = [
 			'Content-Type' => 'application/json; charset=utf-8',
-			'X-Ovh-Application' => $this->attrs['AK'],
+			'X-Ovh-Application' => $this->attrs[ 'AK' ],
 			'X-Ovh-Timestamp' => $timestamp,
-			'X-Ovh-Signature' => '$1$' . sha1($prehash),
-			'X-Ovh-Consumer' => $this->attrs['CK'],
+			'X-Ovh-Signature' => '$1$'.sha1($prehash),
+			'X-Ovh-Consumer' => $this->attrs[ 'CK' ],
 		];
 		return $header;
 	}

lib/Provider/Channel/SMS/Provider/Drivers/SerwerSMS.php

Spacing +5 added lines, -5 removed lines

@@ -27,9 +27,9 @@ 
 	public const SCHEMA = [
 		'name' => 'SerwerSMS',
 		'fields' => [
-			['field' => 'login',    'prompt' => 'Please enter your SerwerSMS.pl API login:'],
-			['field' => 'password', 'prompt' => 'Please enter your SerwerSMS.pl API password:'],
-			['field' => 'sender',   'prompt' => 'Please enter your SerwerSMS.pl sender name:'],
+			[ 'field' => 'login', 'prompt' => 'Please enter your SerwerSMS.pl API login:' ],
+			[ 'field' => 'password', 'prompt' => 'Please enter your SerwerSMS.pl API password:' ],
+			[ 'field' => 'sender', 'prompt' => 'Please enter your SerwerSMS.pl sender name:' ],
 		],
 	];
 	private IClient $client;
@@ -40,7 +40,7 @@ 
 		$this->client = $clientService->newClient();
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function send(string $identifier, string $message) {
 		$login = $this->getLogin();
 		$password = $this->getPassword();
@@ -61,7 +61,7 @@ 
 
 			$responseData = json_decode((string)$response->getBody(), true);
 
-			if ($responseData['success'] !== true) {
+			if ($responseData[ 'success' ] !== true) {
 				throw new MessageTransmissionException();
 			}
 		} catch (Exception $ex) {