framgia / laravel-jwt

src/Storage/CacheStorage.php

Doc Comments +3 added lines, -3 removed lines

@@ -18,7 +18,7 @@ 
     protected $tag;
 
     /**
-     * @param \Illuminate\Cache\CacheManager  $cache
+     * @param Repository  $cache
      * @param string $tag
      */
     public function __construct(Repository $cache, $tag = 'jwt')
@@ -31,7 +31,7 @@ 
      * Add a new item into storage.
      *
      * @param  string  $key
-     * @param  mixed  $value
+     * @param  string  $value
      * @param  int  $minutes
      * @return void
      */
@@ -42,7 +42,7 @@ 
 
     /**
      * @param  string  $key
-     * @param  mixed  $value
+     * @param  string  $value
      * @return void
      */
     public function forever($key, $value)

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
      */
     protected function cache()
     {
-        if (! method_exists($this->cache, 'tags')) {
+        if (!method_exists($this->cache, 'tags')) {
             return $this->cache;
         }
 

tests/BlacklistTest.php

Doc Comments +4 added lines

@@ -163,6 +163,10 @@
         $this->assertEquals(500, $property->getValue($list));
     }
 
+    /**
+     * @param string $id
+     * @param integer $exp
+     */
     protected function tokenStub($id = null, $exp = null, $claims = [])
     {
         $builder = new \Lcobucci\JWT\Builder();

tests/ServiceProviderTest.php

Doc Comments +4 added lines

@@ -9,6 +9,10 @@
 }
 
 if (!function_exists('config_path')) {
+
+    /**
+     * @param string $path
+     */
     function config_path($path) {
         return $path;
     }

Spacing +3 added lines, -3 removed lines

@@ -51,13 +51,13 @@
     {
         $app = m::mock(ApplicationStub::class)->makePartial();
 
-        $app->singleton('config', function() {
+        $app->singleton('config', function () {
             $repo = new \Illuminate\Config\Repository();
-            $repo->set('jwt', include __DIR__.'/../config/jwt.php');
+            $repo->set('jwt', include __DIR__ . '/../config/jwt.php');
             return $repo;
         });
 
-        $app->singleton(\Illuminate\Contracts\Cache\Repository::class, function() {
+        $app->singleton(\Illuminate\Contracts\Cache\Repository::class, function () {
             return m::mock(\Illuminate\Contracts\Cache\Repository::class);
         });
 

Braces +4 added lines, -2 removed lines

@@ -3,13 +3,15 @@
 use Mockery as m;
 
 if (!function_exists('env')) {
-    function env($name, $default = null) {
+    function env($name, $default = null)
+    {
         return $default;
     }
 }
 
 if (!function_exists('config_path')) {
-    function config_path($path) {
+    function config_path($path)
+    {
         return $path;
     }
 }

tests/SigningTokensTest.php

Doc Comments +3 added lines

@@ -37,6 +37,9 @@
 {
     use \Framgia\Jwt\SigningTokens;
 
+    /**
+     * @param string $type
+     */
     public function signer($algorithm = 'sha256', $type = null)
     {
         // Pass call to protected method

src/Guard.php

Spacing +3 added lines, -3 removed lines

@@ -66,7 +66,7 @@ 
         // If we've already retrieved the user for the current request we can just
         // return it back immediately. We do not want to fetch the user data on
         // every call to this method because that would be tremendously slow.
-        if (! is_null($this->user)) {
+        if (!is_null($this->user)) {
             return $this->user;
         }
 
@@ -74,7 +74,7 @@ 
 
         $token = $this->getTokenForRequest();
 
-        if (! is_null($token)) {
+        if (!is_null($token)) {
             $user = $this->provider->retrieveById($token->getClaim('sub'));
         }
 
@@ -140,7 +140,7 @@ 
         $builder->setSubject($id);
 
         if ($this->user instanceof ProvidesCredentials) {
-            foreach($this->user->getCredentials() as $key => $value) {
+            foreach ($this->user->getCredentials() as $key => $value) {
                 $builder->set($key, $value);
             }
         }

Braces +1 added lines, -2 removed lines

@@ -48,8 +48,7 @@
         Request $request,
         Blacklist $blacklist,
         Signer $signer
-    )
-    {
+    ) {
         $this->request = $request;
         $this->provider = $provider;
         $this->blacklist = $blacklist;

src/Blacklist.php

Spacing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     {
         // if there is no exp claim then add the jwt to
         // the blacklist indefinitely
-        if (! $token->hasClaim('exp')) {
+        if (!$token->hasClaim('exp')) {
             return $this->addForever($token);
         }
 
@@ -114,7 +114,7 @@ 
         }
 
         // check whether the expiry + grace has past
-        return $val !== null && ! Carbon::createFromTimestamp($val['valid_until'])->isFuture();
+        return $val !== null && !Carbon::createFromTimestamp($val['valid_until'])->isFuture();
     }
 
     /**

tests/CacheStorageTest.php

Braces +3 added lines, -1 removed lines

@@ -49,4 +49,6 @@
     }
 }
 
-abstract class TaggedStoreStub extends \Illuminate\Cache\TaggableStore implements \Illuminate\Contracts\Cache\Store {}
+abstract class TaggedStoreStub extends \Illuminate\Cache\TaggableStore implements \Illuminate\Contracts\Cache\Store
+{
+}

tests/GuardTest.php

Spacing +3 added lines, -3 removed lines

@@ -43,7 +43,7 @@ 
 
         $signer->shouldReceive('verify')->once()->andReturn(true);
 
-        $request->headers->set('Authorization', 'Bearer '.$token);
+        $request->headers->set('Authorization', 'Bearer ' . $token);
 
         $guard = new Framgia\Jwt\Guard($provider, $request, $blacklist, $signer);
 
@@ -65,7 +65,7 @@ 
 
         $token = $signer->sign((new \Lcobucci\JWT\Builder())->setSubject(1)->setExpiration(time() + 3600))->getToken();
 
-        $request->headers->set('Authorization', 'Bearer '.$token);
+        $request->headers->set('Authorization', 'Bearer ' . $token);
 
         $guard = new Framgia\Jwt\Guard($provider, $request, $blacklist, $signer);
 
@@ -257,7 +257,7 @@ 
         $request = Illuminate\Http\Request::create('/');
         $signer->shouldReceive('verify')->once()->andReturn(true);
 
-        $request->headers->set('Authorization', 'Bearer '.$token);
+        $request->headers->set('Authorization', 'Bearer ' . $token);
 
         $guard = new Framgia\Jwt\Guard($provider, $request, $blacklist, $signer);
 

src/SigningTokens.php

Spacing +2 added lines, -2 removed lines

@@ -20,10 +20,10 @@
 
         // Check if provided class exists and extends BaseVerifier
         // to avoid possible code injections
-        if (is_subclass_of($class = Signer::class.'\\'.$type.'\\'.$algorithm, Signer\BaseSigner::class)) {
+        if (is_subclass_of($class = Signer::class . '\\' . $type . '\\' . $algorithm, Signer\BaseSigner::class)) {
             return new $class;
         }
 
-        throw new InvalidArgumentException('['.$algorithm.'] is not supported in ['.$type.'] verifier.');
+        throw new InvalidArgumentException('[' . $algorithm . '] is not supported in [' . $type . '] verifier.');
     }
 }