nextcloud / twofactor_gateway

lib/Command/Remove.php

Spacing +5 added lines, -5 removed lines

@@ -20,7 +20,7 @@ 
 
 class Remove extends Command {
 	/** @var AGateway[] */
-	private array $gateways = [];
+	private array $gateways = [ ];
 
 	public function __construct(
 		private Factory $gatewayFactory,
@@ -30,24 +30,24 @@ 
 		$fqcn = $this->gatewayFactory->getFqcnList();
 		foreach ($fqcn as $fqcn) {
 			$gateway = $this->gatewayFactory->get($fqcn);
-			$this->gateways[$gateway->getSettings()->id] = $gateway;
+			$this->gateways[ $gateway->getSettings()->id ] = $gateway;
 		}
 
 		$this->addArgument(
 			'gateway',
 			InputArgument::OPTIONAL,
-			'The name of the gateway: ' . implode(', ', array_keys($this->gateways))
+			'The name of the gateway: '.implode(', ', array_keys($this->gateways))
 		);
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function execute(InputInterface $input, OutputInterface $output) {
 		$gatewayName = strtolower((string)$input->getArgument('gateway'));
 		if (!array_key_exists($gatewayName, $this->gateways)) {
 			$helper = new QuestionHelper();
 			$choiceQuestion = new ChoiceQuestion('Please choose a provider:', array_keys($this->gateways));
 			$selected = $helper->ask($input, $output, $choiceQuestion);
-			$gateway = $this->gateways[$selected];
+			$gateway = $this->gateways[ $selected ];
 		}
 
 		$gateway->remove();

lib/Command/Test.php

Spacing +5 added lines, -5 removed lines

@@ -25,16 +25,16 @@ 
 		parent::__construct('twofactorauth:gateway:test');
 
 		$fqcn = $this->gatewayFactory->getFqcnList();
-		$gateways = [];
+		$gateways = [ ];
 		foreach ($fqcn as $fqcn) {
 			$gateway = $this->gatewayFactory->get($fqcn);
-			$gateways[$gateway->getSettings()->id] = $gateway;
+			$gateways[ $gateway->getSettings()->id ] = $gateway;
 		}
 
 		$this->addArgument(
 			'gateway',
 			InputArgument::REQUIRED,
-			'The name of the gateway: ' . implode(', ', array_keys($gateways))
+			'The name of the gateway: '.implode(', ', array_keys($gateways))
 		);
 		$this->addArgument(
 			'identifier',
@@ -43,7 +43,7 @@ 
 		);
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function execute(InputInterface $input, OutputInterface $output) {
 		$gatewayName = $input->getArgument('gateway');
 		$identifier = $input->getArgument('identifier');
@@ -55,7 +55,7 @@ 
 			return 1;
 		}
 
-		$gateway->send($identifier, 'Test', ['code' => '123456']);
+		$gateway->send($identifier, 'Test', [ 'code' => '123456' ]);
 		return 0;
 	}
 }

lib/Provider/Channel/SMS/Factory.php

Spacing +11 added lines, -11 removed lines

@@ -16,37 +16,37 @@
 /** @extends AFactory<AProvider> */
 class Factory extends AFactory {
 	/** @var array<AProvider> */
-	private array $instancesByFqcn = [];
-	#[\Override]
+	private array $instancesByFqcn = [ ];
+	#[\Override ]
 	protected function getPrefix(): string {
 		return 'OCA\\TwoFactorGateway\\Provider\\Channel\\SMS\\Provider\\Drivers\\';
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function getSuffix(): string {
 		return '';
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function getBaseClass(): string {
 		return AProvider::class;
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function get(string $name): object {
-		if (isset($this->instancesByFqcn[$name])) {
-			return $this->instancesByFqcn[$name];
+		if (isset($this->instancesByFqcn[ $name ])) {
+			return $this->instancesByFqcn[ $name ];
 		}
-		if (isset($this->instances[$name])) {
-			return $this->instances[$name];
+		if (isset($this->instances[ $name ])) {
+			return $this->instances[ $name ];
 		}
 		foreach ($this->getFqcnList() as $fqcn) {
 			$instance = \OCP\Server::get($fqcn);
 			$settings = $instance->getSettings();
 			if ($fqcn === $name || $settings->id === $name) {
 				$instance->setAppConfig(\OCP\Server::get(\OCP\IAppConfig::class));
-				$this->instances[$settings->id] = $instance;
-				$this->instancesByFqcn[$fqcn] = $instance;
+				$this->instances[ $settings->id ] = $instance;
+				$this->instancesByFqcn[ $fqcn ] = $instance;
 				return $instance;
 			}
 		}

lib/Provider/Channel/SMS/Gateway.php

Spacing +14 added lines, -14 removed lines

@@ -29,20 +29,20 @@ 
 		parent::__construct($appConfig);
 	}
 
-	#[\Override]
-	public function send(string $identifier, string $message, array $extra = []): void {
+	#[\Override ]
+	public function send(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->smsProviderFactory->getFqcnList();
-		$names = [];
-		$providers = [];
+		$names = [ ];
+		$providers = [ ];
 		foreach ($namespaces as $ns) {
 			$provider = $this->smsProviderFactory->get($ns);
-			$providers[] = $provider;
-			$names[] = $provider->getSettings()->name;
+			$providers[ ] = $provider;
+			$names[ ] = $provider->getSettings()->name;
 		}
 
 		$helper = new QuestionHelper();
@@ -50,29 +50,29 @@ 
 		$name = $helper->ask($input, $output, $choiceQuestion);
 		$selectedIndex = array_search($name, $names);
 
-		$provider = $providers[$selectedIndex];
+		$provider = $providers[ $selectedIndex ];
 
 		foreach ($provider->getSettings()->fields as $field) {
 			$id = $field->field;
-			$prompt = $field->prompt . ' ';
+			$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]
+	#[\Override ]
 	public function createSettings(): Settings {
 		try {
 			$settings = $this->getProvider()->getSettings();
@@ -84,7 +84,7 @@ 
 		return $settings;
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function isComplete(?Settings $settings = null): bool {
 		if ($settings === null) {
 			try {
@@ -97,7 +97,7 @@ 
 		return parent::isComplete($settings);
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function remove(?Settings $settings = null): void {
 		foreach ($this->smsProviderFactory->getFqcnList() as $fqcn) {
 			$provider = $this->smsProviderFactory->get($fqcn);

lib/Provider/Channel/Telegram/Factory.php

Spacing +11 added lines, -11 removed lines

@@ -15,37 +15,37 @@
 
 /** @extends AFactory<AProvider> */
 class Factory extends AFactory {
-	private array $instancesByFqcn = [];
-	#[\Override]
+	private array $instancesByFqcn = [ ];
+	#[\Override ]
 	protected function getPrefix(): string {
 		return 'OCA\\TwoFactorGateway\\Provider\\Channel\\Telegram\\Provider\\Drivers\\';
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function getSuffix(): string {
 		return '';
 	}
 
-	#[\Override]
+	#[\Override ]
 	protected function getBaseClass(): string {
 		return AProvider::class;
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function get(string $name): object {
-		if (isset($this->instancesByFqcn[$name])) {
-			return $this->instancesByFqcn[$name];
+		if (isset($this->instancesByFqcn[ $name ])) {
+			return $this->instancesByFqcn[ $name ];
 		}
-		if (isset($this->instances[$name])) {
-			return $this->instances[$name];
+		if (isset($this->instances[ $name ])) {
+			return $this->instances[ $name ];
 		}
 		foreach ($this->getFqcnList() as $fqcn) {
 			$instance = \OCP\Server::get($fqcn);
 			$settings = $instance->getSettings();
 			if ($fqcn === $name || $settings->id === $name) {
 				$instance->setAppConfig(\OCP\Server::get(\OCP\IAppConfig::class));
-				$this->instances[$name] = $instance;
-				$this->instancesByFqcn[$fqcn] = $instance;
+				$this->instances[ $name ] = $instance;
+				$this->instancesByFqcn[ $fqcn ] = $instance;
 				return $instance;
 			}
 		}

lib/Provider/Channel/Telegram/Gateway.php

Spacing +11 added lines, -11 removed lines

@@ -29,20 +29,20 @@ 
 		parent::__construct($appConfig);
 	}
 
-	#[\Override]
-	public function send(string $identifier, string $message, array $extra = []): void {
+	#[\Override ]
+	public function send(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->telegramProviderFactory->getFqcnList();
-		$names = [];
-		$providers = [];
+		$names = [ ];
+		$providers = [ ];
 		foreach ($namespaces as $ns) {
 			$provider = $this->telegramProviderFactory->get($ns);
-			$providers[] = $provider;
-			$names[] = $provider->getSettings()->name;
+			$providers[ ] = $provider;
+			$names[ ] = $provider->getSettings()->name;
 		}
 
 		$helper = new QuestionHelper();
@@ -50,11 +50,11 @@ 
 		$name = $helper->ask($input, $output, $choiceQuestion);
 		$selectedIndex = array_search($name, $names);
 
-		$providers[$selectedIndex]->cliConfigure($input, $output);
+		$providers[ $selectedIndex ]->cliConfigure($input, $output);
 		return 0;
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function createSettings(): Settings {
 		try {
 			$settings = $this->getProvider()->getSettings();
@@ -66,7 +66,7 @@ 
 		return $settings;
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function isComplete(?Settings $settings = null): bool {
 		if ($settings === null) {
 			try {
@@ -79,7 +79,7 @@ 
 		return parent::isComplete($settings);
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function remove(?Settings $settings = null): void {
 		foreach ($this->telegramProviderFactory->getFqcnList() as $fqcn) {
 			$provider = $this->telegramProviderFactory->get($fqcn);

lib/Provider/Channel/Telegram/Provider/Drivers/Bot.php

Spacing +5 added lines, -5 removed lines

@@ -54,10 +54,10 @@ 
 		);
 	}
 
-	#[\Override]
-	public function send(string $identifier, string $message, array $extra = []): void {
+	#[\Override ]
+	public function send(string $identifier, string $message, array $extra = [ ]): void {
 		if (empty($message)) {
-			$message = $this->l10n->t('`%s` is your Nextcloud verification code.', [$extra['code']]);
+			$message = $this->l10n->t('`%s` is your Nextcloud verification code.', [ $extra[ 'code' ] ]);
 		}
 		$this->logger->debug("sending telegram message to $identifier, message: $message");
 		$token = $this->getToken();
@@ -76,11 +76,11 @@ 
 		$this->logger->debug("telegram message to chat $identifier sent");
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function cliConfigure(InputInterface $input, OutputInterface $output): int {
 		$helper = new QuestionHelper();
 		$settings = $this->getSettings();
-		$tokenQuestion = new Question($settings->fields[0]->prompt . ' ');
+		$tokenQuestion = new Question($settings->fields[ 0 ]->prompt.' ');
 		$token = $helper->ask($input, $output, $tokenQuestion);
 		$this->setToken($token);
 		$output->writeln("Using $token.");

lib/Provider/Gateway/AGateway.php

Spacing +12 added lines, -12 removed lines

@@ -18,7 +18,7 @@ 
 
 abstract class AGateway implements IGateway {
 	use TConfigurable;
-	public const SCHEMA = [];
+	public const SCHEMA = [ ];
 	protected ?Settings $settings = null;
 
 	public function __construct(
@@ -29,25 +29,25 @@ 
 	/**
 	 * @throws MessageTransmissionException
 	 */
-	#[\Override]
-	abstract public function send(string $identifier, string $message, array $extra = []): void;
+	#[\Override ]
+	abstract public function send(string $identifier, string $message, array $extra = [ ]): void;
 
-	#[\Override]
+	#[\Override ]
 	public function isComplete(?Settings $settings = null): bool {
 		if (!is_object($settings)) {
 			$settings = $this->getSettings();
 		}
 		$savedKeys = $this->appConfig->getKeys(Application::APP_ID);
 		$providerId = $settings->id ?? $this->getProviderId();
-		$fields = [];
+		$fields = [ ];
 		foreach ($settings->fields as $field) {
-			$fields[] = $providerId . '_' . $field->field;
+			$fields[ ] = $providerId.'_'.$field->field;
 		}
 		$intersect = array_intersect($fields, $savedKeys);
 		return count($intersect) === count($fields);
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function getSettings(): Settings {
 		if ($this->settings !== null) {
 			return $this->settings;
@@ -57,10 +57,10 @@ 
 		return $this->settings;
 	}
 
-	#[\Override]
+	#[\Override ]
 	abstract public function cliConfigure(InputInterface $input, OutputInterface $output): int;
 
-	#[\Override]
+	#[\Override ]
 	public function remove(?Settings $settings = null): void {
 		if (!is_object($settings)) {
 			$settings = $this->getSettings();
@@ -68,7 +68,7 @@ 
 			$this->settings = $settings;
 		}
 		foreach ($settings->fields as $field) {
-			$method = 'delete' . $this->toCamel($field->field);
+			$method = 'delete'.$this->toCamel($field->field);
 			$this->{$method}();
 		}
 	}
@@ -77,11 +77,11 @@ 
 		return str_replace(' ', '', ucwords(str_replace('_', ' ', $field)));
 	}
 
-	#[\Override]
+	#[\Override ]
 	public function getProviderId(): string {
 		$id = self::deriveIdFromFqcn(static::class);
 		if ($id === null) {
-			throw new \LogicException('Cannot derive gateway id from FQCN: ' . static::class);
+			throw new \LogicException('Cannot derive gateway id from FQCN: '.static::class);
 		}
 		return $id;
 	}

lib/Provider/Gateway/TConfigurable.php

Spacing +8 added lines, -8 removed lines

@@ -20,28 +20,28 @@ 
 	 */
 	public function __call(string $name, array $args) {
 		if (!preg_match('/^(?<operation>get|set|delete)(?<field>[A-Z][A-Za-z0-9_]*)$/', $name, $matches)) {
-			throw new ConfigurationException('Invalid method ' . $name);
+			throw new ConfigurationException('Invalid method '.$name);
 		}
-		$field = strtolower(preg_replace('/(?<!^)[A-Z]/', '_$0', $matches['field']));
+		$field = strtolower(preg_replace('/(?<!^)[A-Z]/', '_$0', $matches[ 'field' ]));
 		$key = $this->keyFromField($field);
 
-		switch ($matches['operation']) {
+		switch ($matches[ 'operation' ]) {
 			case 'get':
 				$val = (string)$this->getAppConfig()->getValueString(Application::APP_ID, $key, '');
 				if ($val === '') {
-					throw new ConfigurationException('No value set for ' . $field);
+					throw new ConfigurationException('No value set for '.$field);
 				}
 				return $val;
 
 			case 'set':
-				$this->getAppConfig()->setValueString(Application::APP_ID, $key, (string)($args[0] ?? ''));
+				$this->getAppConfig()->setValueString(Application::APP_ID, $key, (string)($args[ 0 ] ?? ''));
 				return $this;
 
 			case 'delete':
 				$this->getAppConfig()->deleteKey(Application::APP_ID, $key);
 				return $this;
 		}
-		throw new ConfigurationException('Invalid operation ' . $matches['operation']);
+		throw new ConfigurationException('Invalid operation '.$matches[ 'operation' ]);
 	}
 
 	/**
@@ -52,10 +52,10 @@ 
 		$fields = $settings->fields;
 		foreach ($fields as $field) {
 			if ($field->field === $fieldName) {
-				return $settings->id . '_' . $fieldName;
+				return $settings->id.'_'.$fieldName;
 			}
 		}
-		throw new ConfigurationException('Invalid configuration field: ' . $fieldName . ', check SCHEMA at ' . static::class);
+		throw new ConfigurationException('Invalid configuration field: '.$fieldName.', check SCHEMA at '.static::class);
 	}
 
 	/**