kleijnweb / jwt-bundle

src/Tests/Functional/App/TestBundle.php

Doc Comments +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
 class TestBundle extends Bundle
 {
     /**
-     * @return ExtensionInterface
+     * @return boolean
      */
     public function getContainerExtension()
     {

src/Tests/Functional/App/app/TestKernel.php

Spacing +1 added lines, -1 removed lines

@@ -25,6 +25,6 @@
      */
     public function registerContainerConfiguration(LoaderInterface $loader)
     {
-        $loader->load(__DIR__ . '/config.yml');
+        $loader->load(__DIR__.'/config.yml');
     }
 }

src/Authenticator/JwtKey.php

Spacing +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
         if ($this->requiredClaims) {
             $missing = array_diff_key(array_flip($this->requiredClaims), $claims);
             if (count($missing)) {
-                throw new \InvalidArgumentException("Missing claims: " . implode(', ', $missing));
+                throw new \InvalidArgumentException("Missing claims: ".implode(', ', $missing));
             }
         }
         if ($this->issuer && !isset($claims['iss'])) {

src/DependencyInjection/KleijnWebJwtExtension.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
     public function load(array $configs, ContainerBuilder $container)
     {
         $config = $this->processConfiguration(new Configuration(), $configs);
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.yml');
 
         $keysDefinition = new Definition('jwt.keys');
@@ -35,7 +35,7 @@ 
         foreach ($config['keys'] as $keyId => $keyConfig) {
 
             $keyConfig['kid'] = $keyId;
-            $keyDefinition    = new Definition('jwt.keys.' . $keyId);
+            $keyDefinition    = new Definition('jwt.keys.'.$keyId);
             $keyDefinition->setClass('KleijnWeb\JwtBundle\Authenticator\JwtKey');
 
             if (isset($keyConfig['loader'])) {

src/Tests/Functional/FunctionalTest.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 use KleijnWeb\JwtBundle\Tests\Authenticator\AuthenticatorTest;
 use Symfony\Bundle\FrameworkBundle\Test\WebTestCase;
-use Symfony\Bundle\FrameworkBundle\Client;
 
 /**
  * @author John Kleijn <john@kleijnweb.nl>

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
         $client = $this->createClient();
         $server = [];
         if ($token) {
-            $server = ['HTTP_AUTHORIZATION' => 'Bearer ' . $token];
+            $server = ['HTTP_AUTHORIZATION' => 'Bearer '.$token];
         }
         $client->request('GET', $url, $parameters = [], $files = [], $server);
 

src/Tests/Authenticator/AuthenticatorTest.php

Spacing +2 added lines, -2 removed lines

@@ -175,7 +175,7 @@ 
     {
         $authenticator = new Authenticator($this->keys);
         $request       = new Request();
-        $request->headers->set('Authorization', 'Bearer ' . self::TEST_TOKEN);
+        $request->headers->set('Authorization', 'Bearer '.self::TEST_TOKEN);
         $token = $authenticator->createToken($request, 'myprovider');
 
         $expected = new JwtToken(self::TEST_TOKEN);
@@ -209,7 +209,7 @@ 
 
         $result = $authenticator->setUserRolesFromAudienceClaims($user, $token);
 
-        $this->assertEquals(['guests','users'], $result->getRoles());
+        $this->assertEquals(['guests', 'users'], $result->getRoles());
     }
 
     /**

src/Authenticator/Authenticator.php

Spacing +5 added lines, -5 removed lines

@@ -129,14 +129,14 @@
         /** @var JwtToken $credentials */
         $credentials = $token->getCredentials();
 
-        foreach($credentials->getClaims() as $claimKey => $claimValue)
+        foreach ($credentials->getClaims() as $claimKey => $claimValue)
         {
-            if($claimKey === 'aud' && method_exists($user, 'addRole')) {
-                if(is_array($claimValue)) {
-                    foreach($claimValue as $role) {
+            if ($claimKey === 'aud' && method_exists($user, 'addRole')) {
+                if (is_array($claimValue)) {
+                    foreach ($claimValue as $role) {
                         $user->addRole($role);
                     }
-                } elseif(is_string($claimValue)) {
+                } elseif (is_string($claimValue)) {
                     $user->addRole($claimValue);
                 }
             }