JwtTokenEndpointAuthMethodCompilerPass::process() - Code Metrics - Inspection of "Metadata" - OAuth2-Framework/oauth2-framework - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions

Pull Request — master (#37)

by Florent

created 2017-04-12 12:39 UTC

JwtTokenEndpointAuthMethodCompilerPass::process() A

↳ Parent: JwtTokenEndpointAuthMethodCompilerPass

Complexity

Conditions	3
Paths	2

Size

Total Lines	12
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	8
nc	2
nop	1

<?php

declare(strict_types=1);

/*
 * The MIT License (MIT)
 *
 * Copyright (c) 2014-2017 Spomky-Labs
 *
 * This software may be modified and distributed under the terms
 * of the MIT license.  See the LICENSE file for details.
 */

namespace OAuth2Framework\Bundle\Server\DependencyInjection\Compiler;

use OAuth2Framework\Bundle\Server\Service\MetadataBuilder;
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;

final class JwtTokenEndpointAuthMethodCompilerPass implements CompilerPassInterface
{
    /**
     * {@inheritdoc}
     */
    public function process(ContainerBuilder $container)
    {
        if (!$container->hasDefinition(MetadataBuilder::class) || !$container->hasParameter('oauth2_server.endpoint.jwks_uri.route_name')) {
            return;
        }

        $routeName = $container->getParameter('oauth2_server.endpoint.jwks_uri.route_name');
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $definition = $container->getDefinition(MetadataBuilder::class);
        $definition->addMethodCall('', ['token_endpoint_auth_signing_alg_values_supported', $this->getJWTLoader()->getSupportedSignatureAlgorithms()]);

        $definition->addMethodCall('', ['token_endpoint_auth_encryption_alg_values_supported', $this->getJWTLoader()->getSupportedKeyEncryptionAlgorithms()]);

        $definition->addMethodCall('', ['token_endpoint_auth_encryption_enc_values_supported', $this->getJWTLoader()->getSupportedContentEncryptionAlgorithms()]);

    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			/*
6			* The MIT License (MIT)
7			*
8			* Copyright (c) 2014-2017 Spomky-Labs
9			*
10			* This software may be modified and distributed under the terms
11			* of the MIT license. See the LICENSE file for details.
12			*/
13
14			namespace OAuth2Framework\Bundle\Server\DependencyInjection\Compiler;
15
16			use OAuth2Framework\Bundle\Server\Service\MetadataBuilder;
17			use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
18			use Symfony\Component\DependencyInjection\ContainerBuilder;
19
20			final class JwtTokenEndpointAuthMethodCompilerPass implements CompilerPassInterface
21			{
22			/**
23			* {@inheritdoc}
24			*/
25			public function process(ContainerBuilder $container)
26			{
27			if (!$container->hasDefinition(MetadataBuilder::class) \|\| !$container->hasParameter('oauth2_server.endpoint.jwks_uri.route_name')) {
28			return;
29			}
30
31			$routeName = $container->getParameter('oauth2_server.endpoint.jwks_uri.route_name');
			0 ignored issues – show Unused Code introduced 2017-04-12 12:03 UTC by Report Bug Copy Issue Report `$routeName` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
32			$definition = $container->getDefinition(MetadataBuilder::class);
33			$definition->addMethodCall('', ['token_endpoint_auth_signing_alg_values_supported', $this->getJWTLoader()->getSupportedSignatureAlgorithms()]);
			0 ignored issues – show Bug introduced 2017-04-12 12:03 UTC by Report Bug Copy Issue Report The method `getJWTLoader()` does not seem to exist on `object<OAuth2Framework\B...AuthMethodCompilerPass>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
34			$definition->addMethodCall('', ['token_endpoint_auth_encryption_alg_values_supported', $this->getJWTLoader()->getSupportedKeyEncryptionAlgorithms()]);
			0 ignored issues – show Bug introduced 2017-04-12 12:03 UTC by Report Bug Copy Issue Report The method `getJWTLoader()` does not seem to exist on `object<OAuth2Framework\B...AuthMethodCompilerPass>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
35			$definition->addMethodCall('', ['token_endpoint_auth_encryption_enc_values_supported', $this->getJWTLoader()->getSupportedContentEncryptionAlgorithms()]);
			0 ignored issues – show Bug introduced 2017-04-12 12:03 UTC by Report Bug Copy Issue Report The method `getJWTLoader()` does not seem to exist on `object<OAuth2Framework\B...AuthMethodCompilerPass>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
36			}
37			}
38

OAuth2-Framework / oauth2-framework

Pull Request — master (#37)

JwtTokenEndpointAuthMethodCompilerPass::process() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like