nextcloud / twofactor_gateway

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/Service/SetupService.php

Spacing +2 added lines, -2 removed lines

@@ -62,8 +62,8 @@
 			$gateway->send(
 				$user,
 				$identifier,
-				$this->l10n->t('%s is your verification code.', [$verificationNumber]),
-				['code' => $verificationNumber],
+				$this->l10n->t('%s is your verification code.', [ $verificationNumber ]),
+				[ 'code' => $verificationNumber ],
 			);
 		} catch (MessageTransmissionException $ex) {
 			throw new VerificationException($ex->getMessage(), $ex->getCode(), $ex);

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/Gateway.php

Spacing +18 added lines, -18 removed lines

@@ -33,17 +33,17 @@ 
 		parent::__construct($appConfig);
 	}
 
-	#[\Override]
-	public function send(IUser $user, string $identifier, string $message, array $extra = []): void {
+	#[\Override ]
+	public function send(IUser $user, string $identifier, string $message, array $extra = [ ]): void {
 		$this->getProvider()->send($identifier, $message);
 	}
 
-	#[\Override]
+	#[\Override ]
 	final public function cliConfigure(InputInterface $input, OutputInterface $output): int {
 		$namespaces = $this->providerFactory->getFqcnList();
-		$schemas = [];
+		$schemas = [ ];
 		foreach ($namespaces as $ns) {
-			$schemas[] = $ns::SCHEMA;
+			$schemas[ ] = $ns::SCHEMA;
 		}
 		$names = array_column($schemas, 'name');
 
@@ -51,32 +51,32 @@ 
 		$choiceQuestion = new ChoiceQuestion('Please choose a SMS provider:', $names);
 		$name = $helper->ask($input, $output, $choiceQuestion);
 		$selectedIndex = array_search($name, $names);
-		$schema = $schemas[$selectedIndex];
+		$schema = $schemas[ $selectedIndex ];
 
-		$provider = $this->getProvider($namespaces[$selectedIndex]::getProviderId());
+		$provider = $this->getProvider($namespaces[ $selectedIndex ]::getProviderId());
 
-		foreach ($schema['fields'] as $field) {
-			$id = $field['field'];
-			$prompt = $field['prompt'] . ' ';
-			$defaultVal = $field['default'] ?? null;
-			$optional = (bool)($field['optional'] ?? false);
+		foreach ($schema[ 'fields' ] as $field) {
+			$id = $field[ 'field' ];
+			$prompt = $field[ 'prompt' ].' ';
+			$defaultVal = $field[ 'default' ] ?? null;
+			$optional = (bool)($field[ 'optional' ] ?? false);
 
 			$answer = (string)$helper->ask($input, $output, new Question($prompt, $defaultVal));
 
 			if ($optional && $answer === '') {
-				$method = 'delete' . $this->toCamel($id);
+				$method = 'delete'.$this->toCamel($id);
 				$provider->{$method}();
 				continue;
 			}
 
-			$method = 'set' . $this->toCamel($id);
+			$method = 'set'.$this->toCamel($id);
 			$provider->{$method}($answer);
 		}
 		return 0;
 	}
 
-	#[\Override]
-	public function isComplete(array $schema = []): bool {
+	#[\Override ]
+	public function isComplete(array $schema = [ ]): bool {
 		if (empty($schema)) {
 			try {
 				$provider = $this->getProvider();
@@ -88,8 +88,8 @@ 
 		return parent::isComplete($schema);
 	}
 
-	#[\Override]
-	public function remove(array $schema = []): void {
+	#[\Override ]
+	public function remove(array $schema = [ ]): void {
 		if (empty($schema)) {
 			$schema = static::SCHEMA;
 		}

lib/Command/Test.php

Spacing +5 added lines, -5 removed lines

@@ -25,10 +25,10 @@ 
 	) {
 		parent::__construct('twofactorauth:gateway:test');
 
-		$ids = [];
+		$ids = [ ];
 		$fqcn = $this->gatewayFactory->getFqcnList();
 		foreach ($fqcn as $fqcn) {
-			$ids[] = $fqcn::getProviderId();
+			$ids[ ] = $fqcn::getProviderId();
 		}
 
 		$this->addArgument(
@@ -39,7 +39,7 @@ 
 		$this->addArgument(
 			'gateway',
 			InputArgument::REQUIRED,
-			'The name of the gateway: ' . implode(', ', $ids)
+			'The name of the gateway: '.implode(', ', $ids)
 		);
 		$this->addArgument(
 			'identifier',
@@ -48,7 +48,7 @@ 
 		);
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function execute(InputInterface $input, OutputInterface $output) {
 		$uid = $input->getArgument('uid');
 		$user = $this->userManager->get($uid);
@@ -66,7 +66,7 @@ 
 			return 1;
 		}
 
-		$gateway->send($user, $identifier, 'Test', ['code' => '123456']);
+		$gateway->send($user, $identifier, 'Test', [ 'code' => '123456' ]);
 		return 0;
 	}
 }

lib/Provider/Channel/Signal/Gateway.php

Spacing +10 added lines, -10 removed lines

@@ -31,7 +31,7 @@ 
 		'name' => 'Signal',
 		'instructions' => 'The gateway can send authentication to your Signal mobile and deskop app.',
 		'fields' => [
-			['field' => 'url', 'prompt' => 'Please enter the URL of the Signal gateway (leave blank to use default):', 'default' => 'http://localhost:5000'],
+			[ 'field' => 'url', 'prompt' => 'Please enter the URL of the Signal gateway (leave blank to use default):', 'default' => 'http://localhost:5000' ],
 		],
 	];
 
@@ -43,29 +43,29 @@ 
 		parent::__construct($appConfig);
 	}
 
-	#[\Override]
-	public function send(IUser $user, string $identifier, string $message, array $extra = []): void {
+	#[\Override ]
+	public function send(IUser $user, string $identifier, string $message, array $extra = [ ]): void {
 		$client = $this->clientService->newClient();
 		// determine type of gateway
-		$response = $client->get($this->getUrl() . '/v1/about');
+		$response = $client->get($this->getUrl().'/v1/about');
 		if ($response->getStatusCode() === 200) {
 			// New style gateway https://gitlab.com/morph027/signal-cli-dbus-rest-api
 			$response = $client->post(
-				$this->getUrl() . '/v1/send/' . $identifier,
+				$this->getUrl().'/v1/send/'.$identifier,
 				[
 					'json' => [ 'message' => $message ],
 				]
 			);
 			$body = (string)$response->getBody();
 			$json = json_decode($body, true);
-			if ($response->getStatusCode() !== 201 || is_null($json) || !is_array($json) || !isset($json['timestamp'])) {
+			if ($response->getStatusCode() !== 201 || is_null($json) || !is_array($json) || !isset($json[ 'timestamp' ])) {
 				$status = $response->getStatusCode();
 				throw new MessageTransmissionException("error reported by Signal gateway, status=$status, body=$body}");
 			}
 		} else {
 			// Try old deprecated gateway https://gitlab.com/morph027/signal-web-gateway
 			$response = $client->post(
-				$this->getUrl() . '/v1/send/' . $identifier,
+				$this->getUrl().'/v1/send/'.$identifier,
 				[
 					'body' => [
 						'to' => $identifier,
@@ -77,17 +77,17 @@ 
 			$body = (string)$response->getBody();
 			$json = json_decode($body, true);
 
-			if ($response->getStatusCode() !== 200 || is_null($json) || !is_array($json) || !isset($json['success']) || $json['success'] !== true) {
+			if ($response->getStatusCode() !== 200 || is_null($json) || !is_array($json) || !isset($json[ 'success' ]) || $json[ 'success' ] !== true) {
 				$status = $response->getStatusCode();
 				throw new MessageTransmissionException("error reported by Signal gateway, status=$status, body=$body}");
 			}
 		}
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function cliConfigure(InputInterface $input, OutputInterface $output): int {
 		$helper = new QuestionHelper();
-		$urlQuestion = new Question(self::SCHEMA['fields'][0]['prompt'], self::SCHEMA['fields'][0]['default']);
+		$urlQuestion = new Question(self::SCHEMA[ 'fields' ][ 0 ][ 'prompt' ], self::SCHEMA[ 'fields' ][ 0 ][ 'default' ]);
 		$url = $helper->ask($input, $output, $urlQuestion);
 		$output->writeln("Using $url.");