chillerlan / php-oauth-core

src/Storage/SessionStorage.php

Braces +2 added lines, -4 removed lines

@@ -83,8 +83,7 @@ 
 
 		if(isset($_SESSION[$this->sessionVar]) && is_array($_SESSION[$this->sessionVar])){
 			$_SESSION[$this->sessionVar][$service] = $token;
-		}
-		else{
+		} else{
 			$_SESSION[$this->sessionVar] = [$service => $token];
 		}
 
@@ -153,8 +152,7 @@ 
 
 		if(isset($_SESSION[$this->stateVar]) && is_array($_SESSION[$this->stateVar])){
 			$_SESSION[$this->stateVar][$service] = $state;
-		}
-		else{
+		} else{
 			$_SESSION[$this->stateVar] = [$service => $state];
 		}
 

Spacing +16 added lines, -16 removed lines

@@ -15,7 +15,7 @@ 
 use chillerlan\OAuth\Core\AccessToken;
 use chillerlan\Traits\ImmutableSettingsInterface;
 
-class SessionStorage extends OAuthStorageAbstract{
+class SessionStorage extends OAuthStorageAbstract {
 
 	/**
 	 * @var bool
@@ -37,21 +37,21 @@ 
 	 *
 	 * @param \chillerlan\Traits\ImmutableSettingsInterface|null $options
 	 */
-	public function __construct(ImmutableSettingsInterface $options = null){
+	public function __construct(ImmutableSettingsInterface $options = null) {
 		parent::__construct($options);
 
 		$this->sessionVar = $this->options->sessionTokenVar;
 		$this->stateVar = $this->options->sessionStateVar;
 
-		if($this->options->sessionStart && !$this->sessionIsActive()){
+		if ($this->options->sessionStart && !$this->sessionIsActive()) {
 			session_start();
 		}
 
-		if(!isset($_SESSION[$this->sessionVar])){
+		if (!isset($_SESSION[$this->sessionVar])) {
 			$_SESSION[$this->sessionVar] = [];
 		}
 
-		if(!isset($_SESSION[$this->stateVar])){
+		if (!isset($_SESSION[$this->stateVar])) {
 			$_SESSION[$this->stateVar] = [];
 		}
 
@@ -62,8 +62,8 @@ 
 	 *
 	 * @codeCoverageIgnore
 	 */
-	public function __destruct(){
-		if($this->options->sessionStart){
+	public function __destruct() {
+		if ($this->options->sessionStart) {
 			session_write_close();
 		}
 	}
@@ -77,10 +77,10 @@ 
 	public function storeAccessToken(string $service, AccessToken $token):OAuthStorageInterface{
 		$token = $token->__toJSON();
 
-		if(isset($_SESSION[$this->sessionVar]) && is_array($_SESSION[$this->sessionVar])){
+		if (isset($_SESSION[$this->sessionVar]) && is_array($_SESSION[$this->sessionVar])) {
 			$_SESSION[$this->sessionVar][$service] = $token;
 		}
-		else{
+		else {
 			$_SESSION[$this->sessionVar] = [$service => $token];
 		}
 
@@ -95,7 +95,7 @@ 
 	 */
 	public function getAccessToken(string $service):AccessToken{
 
-		if($this->hasAccessToken($service)){
+		if ($this->hasAccessToken($service)) {
 			return (new AccessToken)->__fromJSON($_SESSION[$this->sessionVar][$service]);
 		}
 
@@ -118,7 +118,7 @@ 
 	 */
 	public function clearAccessToken(string $service):OAuthStorageInterface{
 
-		if(array_key_exists($service, $_SESSION[$this->sessionVar])){
+		if (array_key_exists($service, $_SESSION[$this->sessionVar])) {
 			unset($_SESSION[$this->sessionVar][$service]);
 		}
 
@@ -130,7 +130,7 @@ 
 	 */
 	public function clearAllAccessTokens():OAuthStorageInterface{
 
-		foreach(array_keys($_SESSION[$this->sessionVar]) as $service){
+		foreach (array_keys($_SESSION[$this->sessionVar]) as $service) {
 			unset($_SESSION[$this->sessionVar][$service]); // trigger the memzero destructor
 		}
 
@@ -147,10 +147,10 @@ 
 	 */
 	public function storeCSRFState(string $service, string $state):OAuthStorageInterface{
 
-		if(isset($_SESSION[$this->stateVar]) && is_array($_SESSION[$this->stateVar])){
+		if (isset($_SESSION[$this->stateVar]) && is_array($_SESSION[$this->stateVar])) {
 			$_SESSION[$this->stateVar][$service] = $state;
 		}
-		else{
+		else {
 			$_SESSION[$this->stateVar] = [$service => $state];
 		}
 
@@ -165,7 +165,7 @@ 
 	 */
 	public function getCSRFState(string $service):string{
 
-		if($this->hasCSRFState($service)){
+		if ($this->hasCSRFState($service)) {
 			return $_SESSION[$this->stateVar][$service];
 		}
 
@@ -188,7 +188,7 @@ 
 	 */
 	public function clearCSRFState(string $service):OAuthStorageInterface{
 
-		if(array_key_exists($service, $_SESSION[$this->stateVar])){
+		if (array_key_exists($service, $_SESSION[$this->stateVar])) {
 			unset($_SESSION[$this->stateVar][$service]);
 		}
 

src/Storage/OAuthStorageInterface.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 use chillerlan\OAuth\Core\AccessToken;
 
-interface OAuthStorageInterface{
+interface OAuthStorageInterface {
 
 	public const TOKEN_NONCE = "\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x01";
 

examples/oauth-example-common.php

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
 ];
 
 /** @var \chillerlan\Traits\ContainerInterface $options */
-$options = new class($options_arr) extends OAuthOptions{
+$options = new class($options_arr) extends OAuthOptions {
 	use DatabaseOptionsTrait, LogOptionsTrait;
 
 	protected $sleep;
@@ -71,11 +71,11 @@ 
 $logger->addInstance(new ConsoleLog($options), 'console');
 
 /** @var \chillerlan\HTTP\HTTPClientInterface $http */
-$http = new class($options) extends HTTPClientAbstract{
+$http = new class($options) extends HTTPClientAbstract {
 
 	protected $client;
 
-	public function __construct(ContainerInterface $options){
+	public function __construct(ContainerInterface $options) {
 		parent::__construct($options);
 		$this->client = new CurlClient($this->options);
 	}

src/OAuthOptions.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
  * @property string     $ca_info
  * @property int        $max_redirects
  */
-class OAuthOptions implements ImmutableSettingsInterface{
+class OAuthOptions implements ImmutableSettingsInterface {
 	use OAuthOptionsTrait, HTTPOptionsTrait, MemzeroDestructorTrait, ImmutableSettingsContainer{
 		__construct as protected containerConstruct;
 	}
@@ -65,7 +65,7 @@ 
 	 *
 	 * @param iterable|null $properties
 	 */
-	public function __construct(iterable $properties = null){
+	public function __construct(iterable $properties = null) {
 		// enable encryption by default if possible...
 		$this->useEncryption = extension_loaded('sodium');
 

src/Core/OAuthProvider.php

Spacing +2 added lines, -2 removed lines

@@ -90,7 +90,7 @@ 
 	 * @param \chillerlan\Traits\ImmutableSettingsInterface           $options
 	 * @param \Psr\Log\LoggerInterface|null                   $logger
 	 */
-	public function __construct(HTTPClientInterface $http, OAuthStorageInterface $storage, ImmutableSettingsInterface $options, LoggerInterface $logger = null){
+	public function __construct(HTTPClientInterface $http, OAuthStorageInterface $storage, ImmutableSettingsInterface $options, LoggerInterface $logger = null) {
 		$this->setHTTPClient($http);
 
 		$this->storage     = $storage;
@@ -99,7 +99,7 @@ 
 
 		$this->serviceName = (new ReflectionClass($this))->getShortName();
 
-		if($this instanceof ApiClientInterface){
+		if ($this instanceof ApiClientInterface) {
 			$this->loadEndpoints();
 		}
 

src/Storage/DBStorage.php

Spacing +8 added lines, -8 removed lines

@@ -16,7 +16,7 @@ 
 use chillerlan\OAuth\Core\AccessToken;
 use chillerlan\Traits\ImmutableSettingsInterface;
 
-class DBStorage extends OAuthStorageAbstract{
+class DBStorage extends OAuthStorageAbstract {
 
 	/**
 	 * @var \chillerlan\Database\Database
@@ -31,10 +31,10 @@ 
 	 *
 	 * @throws \chillerlan\OAuth\Storage\OAuthStorageException
 	 */
-	public function __construct(ImmutableSettingsInterface $options, Database $db){
+	public function __construct(ImmutableSettingsInterface $options, Database $db) {
 		parent::__construct($options);
 
-		if(!$this->options->dbTokenTable || !$this->options->dbProviderTable){
+		if (!$this->options->dbTokenTable || !$this->options->dbProviderTable) {
 			throw new OAuthStorageException('invalid table config');
 		}
 
@@ -62,7 +62,7 @@ 
 	public function storeAccessToken(string $service, AccessToken $token):OAuthStorageInterface{
 		$providers = $this->getProviders();
 
-		if(empty($providers) || !isset($providers[$service])){
+		if (empty($providers) || !isset($providers[$service])) {
 			throw new OAuthStorageException('unknown service');
 		}
 
@@ -73,7 +73,7 @@ 
 			$this->options->dbTokenTableExpires    => $token->expires,
 		];
 
-		if($this->hasAccessToken($service) === true){
+		if ($this->hasAccessToken($service) === true) {
 			$this->db->update
 				->table($this->options->dbTokenTable)
 				->set($values)
@@ -107,7 +107,7 @@ 
 			->where($this->options->dbTokenTableLabel, $this->getLabel($service))
 			->query();
 
-		if(is_bool($r) || $r->length < 1){
+		if (is_bool($r) || $r->length < 1) {
 			throw new OAuthStorageException('token not found');
 		}
 
@@ -187,7 +187,7 @@ 
 			->where($this->options->dbTokenTableLabel, $this->getLabel($service))
 			->query();
 
-		if(is_bool($r) || $r->length < 1){
+		if (is_bool($r) || $r->length < 1) {
 			throw new OAuthStorageException('state not found');
 		}
 
@@ -207,7 +207,7 @@ 
 			->where($this->options->dbTokenTableLabel, $this->getLabel($service))
 			->query();
 
-		if(is_bool($r) || $r->length < 1 || empty(trim($r[0]->state))){
+		if (is_bool($r) || $r->length < 1 || empty(trim($r[0]->state))) {
 			return false;
 		}
 

src/Storage/OAuthStorageAbstract.php

Spacing +5 added lines, -5 removed lines

@@ -34,7 +34,7 @@ 
 	 * @param \chillerlan\Traits\ImmutableSettingsInterface|null $options
 	 * @param \Psr\Log\LoggerInterface|null              $logger
 	 */
-	public function __construct(ImmutableSettingsInterface $options = null, LoggerInterface $logger = null){
+	public function __construct(ImmutableSettingsInterface $options = null, LoggerInterface $logger = null) {
 		$this->options = $options ?? new OAuthOptions;
 		$this->logger  = $logger ?? new NullLogger;
 	}
@@ -49,7 +49,7 @@ 
 
 		unset($token);
 
-		if($this->options->useEncryption === true){
+		if ($this->options->useEncryption === true) {
 			return $this->encrypt($data);
 		}
 
@@ -63,7 +63,7 @@ 
 	 */
 	public function fromStorage(string $data):AccessToken{
 
-		if($this->options->useEncryption === true){
+		if ($this->options->useEncryption === true) {
 			$data = $this->decrypt($data);
 		}
 
@@ -78,7 +78,7 @@ 
 	 */
 	protected function encrypt(string &$data):string {
 
-		if(function_exists('sodium_crypto_secretbox')){
+		if (function_exists('sodium_crypto_secretbox')) {
 			$box = sodium_crypto_secretbox($data, $this::TOKEN_NONCE, sodium_hex2bin($this->options->storageCryptoKey));
 
 			sodium_memzero($data);
@@ -97,7 +97,7 @@ 
 	 */
 	protected function decrypt(string $box):string {
 
-		if(function_exists('sodium_crypto_secretbox_open')){
+		if (function_exists('sodium_crypto_secretbox_open')) {
 			return sodium_crypto_secretbox_open(sodium_hex2bin($box), $this::TOKEN_NONCE, sodium_hex2bin($this->options->storageCryptoKey));
 		}