SocialiteProviders / Manager

src/SocialiteWasCalled.php

Doc Comments +5 added lines, -4 removed lines

@@ -72,7 +72,7 @@ 
 
     /**
      * @param SocialiteManager $socialite
-     * @param                  $providerName
+     * @param                  string $providerName
      * @param string           $providerClass
      * @param null|string      $oauth1Server
      *
@@ -92,7 +92,6 @@ 
      *
      * @param string $providerClass must extend Laravel\Socialite\One\AbstractProvider
      * @param string $oauth1Server  must extend League\OAuth1\Client\Server\Server
-     * @param array  $config
      *
      * @return \Laravel\Socialite\One\AbstractProvider
      */
@@ -118,7 +117,6 @@ 
      *
      * @param SocialiteManager $socialite
      * @param string           $providerClass must extend Laravel\Socialite\Two\AbstractProvider
-     * @param array            $config
      *
      * @return \Laravel\Socialite\Two\AbstractProvider
      */
@@ -141,7 +139,7 @@ 
      *
      * @throws MissingConfigException
      *
-     * @return array
+     * @return Contracts\ConfigInterface
      */
     protected function getConfig($providerClass, $providerName)
     {
@@ -202,6 +200,9 @@ 
         }
     }
 
+    /**
+     * @param string|null $providerClass
+     */
     private function classExists($providerClass)
     {
         if (! class_exists($providerClass)) {

Spacing +4 added lines, -4 removed lines

@@ -59,7 +59,7 @@ 
 
         $socialite->extend(
             $providerName,
-            function () use ($socialite, $providerName, $providerClass, $oauth1Server) {
+            function() use ($socialite, $providerName, $providerClass, $oauth1Server) {
                 $provider = $this->buildProvider($socialite, $providerName, $providerClass, $oauth1Server);
                 if (defined('SOCIALITEPROVIDERS_STATELESS') && SOCIALITEPROVIDERS_STATELESS) {
                     return $provider->stateless();
@@ -154,7 +154,7 @@ 
             // We will use the $spoofedConfig variable for now as a way to find out if there was no
             // configuration in the .env file which means we should not return anything and jump
             // to the service config check to check if something can be found there.
-            if (! static::$spoofedConfig) {
+            if (!static::$spoofedConfig) {
                 return $config;
             }
         } catch (MissingConfigException $e) {
@@ -185,7 +185,7 @@ 
      */
     private function isOAuth1($oauth1Server)
     {
-        return ! empty($oauth1Server);
+        return !empty($oauth1Server);
     }
 
     /**
@@ -204,7 +204,7 @@ 
 
     private function classExists($providerClass)
     {
-        if (! class_exists($providerClass)) {
+        if (!class_exists($providerClass)) {
             throw new InvalidArgumentException("$providerClass doesn't exist");
         }
     }

src/ServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -19,11 +19,11 @@
     {
         parent::register();
 
-        if (class_exists('Laravel\Lumen\Application') && ! defined('SOCIALITEPROVIDERS_STATELESS')) {
+        if (class_exists('Laravel\Lumen\Application') && !defined('SOCIALITEPROVIDERS_STATELESS')) {
             define('SOCIALITEPROVIDERS_STATELESS', true);
         }
 
-        $this->app->singleton(ConfigRetrieverInterface::class, function () {
+        $this->app->singleton(ConfigRetrieverInterface::class, function() {
             return new ConfigRetriever();
         });
     }

src/Helpers/ConfigRetriever.php

Spacing +5 added lines, -5 removed lines

@@ -47,7 +47,7 @@ 
             $this->getFromEnv('KEY'),
             $this->getFromEnv('SECRET'),
             $this->getFromEnv('REDIRECT_URI'),
-            $this->getConfigItems($additionalConfigKeys, function ($key) {
+            $this->getConfigItems($additionalConfigKeys, function($key) {
                 return $this->getFromEnv(strtoupper($key));
             }));
     }
@@ -71,7 +71,7 @@ 
             $this->getFromServices('client_id'),
             $this->getFromServices('client_secret'),
             $this->getFromServices('redirect'),
-            $this->getConfigItems($additionalConfigKeys, function ($key) {
+            $this->getConfigItems($additionalConfigKeys, function($key) {
                 return $this->getFromServices(strtolower($key));
             }));
     }
@@ -120,12 +120,12 @@ 
         $keyExists = array_key_exists($key, $this->servicesArray);
 
         // ADDITIONAL value is empty
-        if (! $keyExists && $this->isAdditionalConfig($key)) {
+        if (!$keyExists && $this->isAdditionalConfig($key)) {
             return;
         }
 
         // REQUIRED value is empty
-        if (! $keyExists) {
+        if (!$keyExists) {
             throw new MissingConfigException("Missing services entry for {$this->providerName}.$key");
         }
 
@@ -152,7 +152,7 @@ 
         // REQUIRED value is empty
         if (empty($item)) {
             // If we are running in console we should spoof values to make Socialite happy...
-            if (! app()->runningInConsole()) {
+            if (!app()->runningInConsole()) {
                 throw new MissingConfigException("Configuration for $providerKey is missing.");
             }
             $item = $providerKey;

src/ConfigTrait.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     {
         // check manually if a key is given and if it exists in the config
         // this has to be done to check for spoofed additional config keys so that null isn't returned
-        if (! empty($key) && empty($this->config[$key])) {
+        if (!empty($key) && empty($this->config[$key])) {
             return $default;
         }
 

src/OAuth1/AbstractProvider.php

Spacing +4 added lines, -4 removed lines

@@ -35,7 +35,7 @@ 
      */
     public function user()
     {
-        if (! $this->hasNecessaryVerifier()) {
+        if (!$this->hasNecessaryVerifier()) {
             throw new \InvalidArgumentException('Invalid request. Missing OAuth verifier.');
         }
 
@@ -49,7 +49,7 @@ 
         if ($user instanceof User) {
             parse_str($token['credentialsResponseBody'], $credentialsResponseBody);
 
-            if (! $credentialsResponseBody || ! is_array($credentialsResponseBody)) {
+            if (!$credentialsResponseBody || !is_array($credentialsResponseBody)) {
                 throw new CredentialsException('Unable to parse token credentials response.');
             }
 
@@ -83,7 +83,7 @@ 
      */
     public function redirect()
     {
-        if (! $this->isStateless()) {
+        if (!$this->isStateless()) {
             $this->request->getSession()->put(
                 'oauth.temp', $temp = $this->server->getTemporaryCredentials()
             );
@@ -102,7 +102,7 @@ 
      */
     protected function getToken()
     {
-        if (! $this->isStateless()) {
+        if (!$this->isStateless()) {
             $temp = $this->request->getSession()->get('oauth.temp');
 
             return $this->server->getTokenCredentials(