SymfonyFramework - Code Metrics - Inspection of "Merge pull request #1044 from robfrawley/feature-d..." - liip/LiipImagineBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 2.0 ( 4eaf55...2f0468 )

by Lukas Kahwe

created 2018-01-30 06:45 UTC

SymfonyFramework A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	8
lcom	1
cbo	0
dl	0
loc	55
rs	10
c	1
b	0
f	0

5 Methods

Rating	Name	Size	Complexity
A	hasDirectContainerBuilderLogging()	4	1
A	getContainerResolvableRootWebPath()	4	2
A	isKernelGreaterThanOrEqualTo()	4	1
A	isKernelLessThan()	4	1
A	kernelVersionCompare()	4	3

<?php

/*
 * This file is part of the `liip/LiipImagineBundle` project.
 *
 * (c) https://github.com/liip/LiipImagineBundle/graphs/contributors
 *
 * For the full copyright and license information, please view the LICENSE.md
 * file that was distributed with this source code.
 */

namespace Liip\ImagineBundle\Utility\Framework;

use Symfony\Component\HttpKernel\Kernel;

class SymfonyFramework
{
    /**
     * @return bool
     */
    public static function hasDirectContainerBuilderLogging(): bool
    {
        return method_exists('\Symfony\Component\DependencyInjection\ContainerBuilder', 'log');
    }

    /**
     * @return string
     */
    public static function getContainerResolvableRootWebPath(): string
    {
        return sprintf('%%kernel.project_dir%%/%s', self::isKernelLessThan(4) ? 'web' : 'public');
    }

    /**
     * @param int      $major
     * @param int|null $minor
     * @param int|null $patch
     *
     * @return bool
     */
    public static function isKernelGreaterThanOrEqualTo(int $major, int $minor = null, int $patch = null): bool
    {
        return static::kernelVersionCompare('>=', $major, $minor, $patch);
class YourClass
{
    private static function getTemperature() {
        return "3422 °C";
}

public static function getSomeVariable()
{
    return static::getTemperature();
}
    }

    /**
     * @param int      $major
     * @param int|null $minor
     * @param int|null $patch
     *
     * @return bool
     */
    public static function isKernelLessThan(int $major, int $minor = null, int $patch = null): bool
    {
        return static::kernelVersionCompare('<', $major, $minor, $patch);
class YourClass
{
    private static function getTemperature() {
        return "3422 °C";
}

public static function getSomeVariable()
{
    return static::getTemperature();
}
    }

    /**
     * @param string   $operator
     * @param int      $major
     * @param int|null $minor
     * @param int|null $patch
     *
     * @return bool
     */
    private static function kernelVersionCompare(string $operator, int $major, int $minor = null, int $patch = null): bool
    {
        return version_compare(Kernel::VERSION_ID, sprintf("%d%'.02d%'.02d", $major, $minor ?: 0, $patch ?: 0), $operator);
    }
}


1			<?php
2
3			/*
4			* This file is part of the `liip/LiipImagineBundle` project.
5			*
6			* (c) https://github.com/liip/LiipImagineBundle/graphs/contributors
7			*
8			* For the full copyright and license information, please view the LICENSE.md
9			* file that was distributed with this source code.
10			*/
11
12			namespace Liip\ImagineBundle\Utility\Framework;
13
14			use Symfony\Component\HttpKernel\Kernel;
15
16			class SymfonyFramework
17			{
18			/**
19			* @return bool
20			*/
21			public static function hasDirectContainerBuilderLogging(): bool
22			{
23			return method_exists('\Symfony\Component\DependencyInjection\ContainerBuilder', 'log');
24			}
25
26			/**
27			* @return string
28			*/
29			public static function getContainerResolvableRootWebPath(): string
30			{
31			return sprintf('%%kernel.project_dir%%/%s', self::isKernelLessThan(4) ? 'web' : 'public');
32			}
33
34			/**
35			* @param int $major
36			* @param int\|null $minor
37			* @param int\|null $patch
38			*
39			* @return bool
40			*/
41			public static function isKernelGreaterThanOrEqualTo(int $major, int $minor = null, int $patch = null): bool
42			{
43			return static::kernelVersionCompare('>=', $major, $minor, $patch);
			0 ignored issues – show Bug introduced 2017-02-12 09:11 UTC by Report Bug Copy Issue Report Since `kernelVersionCompare()` is declared private, calling it with `static` will lead to errors in possible sub-classes. You can either use `self`, or increase the visibility of `kernelVersionCompare()` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return static::getTemperature(); } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { private static function getTemperature() { return "-182 °C"; } } print YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return self::getTemperature(); } } Loading history...
44			}
45
46			/**
47			* @param int $major
48			* @param int\|null $minor
49			* @param int\|null $patch
50			*
51			* @return bool
52			*/
53			public static function isKernelLessThan(int $major, int $minor = null, int $patch = null): bool
54			{
55			return static::kernelVersionCompare('<', $major, $minor, $patch);
			0 ignored issues – show Bug introduced 2017-02-12 09:11 UTC by Report Bug Copy Issue Report Since `kernelVersionCompare()` is declared private, calling it with `static` will lead to errors in possible sub-classes. You can either use `self`, or increase the visibility of `kernelVersionCompare()` to at least protected. Let’s assume you have a class which uses late-static binding: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return static::getTemperature(); } } The code above will run fine in your PHP runtime. However, if you now create a sub-class and call the `getSomeVariable()` on that sub-class, you will receive a runtime error: class YourSubClass extends YourClass { private static function getTemperature() { return "-182 °C"; } } print YourSubClass::getSomeVariable(); // Will cause an access error. In the case above, it makes sense to update `SomeClass` to use `self` instead: class YourClass { private static function getTemperature() { return "3422 °C"; } public static function getSomeVariable() { return self::getTemperature(); } } Loading history...
56			}
57
58			/**
59			* @param string $operator
60			* @param int $major
61			* @param int\|null $minor
62			* @param int\|null $patch
63			*
64			* @return bool
65			*/
66			private static function kernelVersionCompare(string $operator, int $major, int $minor = null, int $patch = null): bool
67			{
68			return version_compare(Kernel::VERSION_ID, sprintf("%d%'.02d%'.02d", $major, $minor ?: 0, $patch ?: 0), $operator);
69			}
70			}
71

liip / LiipImagineBundle

Push — 2.0 ( 4eaf55...2f0468 )

SymfonyFramework A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

5 Methods

Duplication Side-by-Side

Filter issues like