Completed
Push — master ( 1de3f3...2f3bca )
by Brian
19:56 queued 14:03
created
src/SocialiteWasCalled.php 2 patches
Doc Comments   +2 added lines, -4 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
 
75 75
     /**
76 76
      * @param \Laravel\Socialite\SocialiteManager $socialite
77
-     * @param                                     $providerName
77
+     * @param                                     string $providerName
78 78
      * @param string                              $providerClass
79 79
      * @param null|string                         $oauth1Server
80 80
      *
@@ -94,7 +94,6 @@  discard block
 block discarded – undo
94 94
      *
95 95
      * @param string $providerClass must extend Laravel\Socialite\One\AbstractProvider
96 96
      * @param string $oauth1Server  must extend League\OAuth1\Client\Server\Server
97
-     * @param array  $config
98 97
      * @param mixed  $providerName
99 98
      *
100 99
      * @return \Laravel\Socialite\One\AbstractProvider
@@ -121,7 +120,6 @@  discard block
 block discarded – undo
121 120
      *
122 121
      * @param SocialiteManager $socialite
123 122
      * @param string           $providerClass must extend Laravel\Socialite\Two\AbstractProvider
124
-     * @param array            $config
125 123
      * @param mixed            $providerName
126 124
      *
127 125
      * @return \Laravel\Socialite\Two\AbstractProvider
@@ -145,7 +143,7 @@  discard block
 block discarded – undo
145 143
      *
146 144
      * @throws \SocialiteProviders\Manager\Exception\MissingConfigException
147 145
      *
148
-     * @return array
146
+     * @return string
149 147
      */
150 148
     protected function getConfig($providerClass, $providerName)
151 149
     {
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 
62 62
         $socialite->extend(
63 63
             $providerName,
64
-            function () use ($socialite, $providerName, $providerClass, $oauth1Server) {
64
+            function() use ($socialite, $providerName, $providerClass, $oauth1Server) {
65 65
                 $provider = $this->buildProvider($socialite, $providerName, $providerClass, $oauth1Server);
66 66
                 if (defined('SOCIALITEPROVIDERS_STATELESS') && SOCIALITEPROVIDERS_STATELESS) {
67 67
                     return $provider->stateless();
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
      */
174 174
     private function isOAuth1($oauth1Server)
175 175
     {
176
-        return ! empty($oauth1Server);
176
+        return !empty($oauth1Server);
177 177
     }
178 178
 
179 179
     /**
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
      */
197 197
     private function classExists($providerClass)
198 198
     {
199
-        if (! class_exists($providerClass)) {
199
+        if (!class_exists($providerClass)) {
200 200
             throw new InvalidArgumentException("{$providerClass} doesn't exist");
201 201
         }
202 202
     }
Please login to merge, or discard this patch.
src/Helpers/ConfigRetriever.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
             $this->getFromServices('client_id'),
48 48
             $this->getFromServices('client_secret'),
49 49
             $this->getFromServices('redirect'),
50
-            $this->getConfigItems($additionalConfigKeys, function ($key) {
50
+            $this->getConfigItems($additionalConfigKeys, function($key) {
51 51
                 return $this->getFromServices(strtolower($key));
52 52
             })
53 53
         );
@@ -97,12 +97,12 @@  discard block
 block discarded – undo
97 97
         $keyExists = array_key_exists($key, $this->servicesArray);
98 98
 
99 99
         // ADDITIONAL value is empty
100
-        if (! $keyExists && $this->isAdditionalConfig($key)) {
100
+        if (!$keyExists && $this->isAdditionalConfig($key)) {
101 101
             return;
102 102
         }
103 103
 
104 104
         // REQUIRED value is empty
105
-        if (! $keyExists) {
105
+        if (!$keyExists) {
106 106
             throw new MissingConfigException("Missing services entry for {$this->providerName}.$key");
107 107
         }
108 108
 
Please login to merge, or discard this patch.