BaseUrlFunction::__invoke() - Code Metrics - Inspection of "Make url() more robust" - stakx-io/stakx - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3657ae...e63a18 )

by Vladimir

created 2017-01-17 02:30 UTC

BaseUrlFunction::__invoke() B

↳ Parent: BaseUrlFunction

Complexity

Conditions	6
Paths	16

Size

Total Lines	22
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	6
eloc	12
nc	16
nop	2
dl	0
loc	22
ccs	0
cts	15
cp	0
crap	42
rs	8.6737
c	1
b	0
f	0

<?php

namespace allejo\stakx\Twig;

use allejo\stakx\Object\JailObject;
use Twig_Environment;

class BaseUrlFunction
{
    public function __invoke (Twig_Environment $env, $assetPath)
    {
        $globals = $env->getGlobals();

        if ($assetPath instanceof JailObject) {
            $assetPath = $assetPath->getPermalink();
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }
        }
        else if (is_array($assetPath)) {
            $assetPath = $assetPath['permalink'];
        }
        else if (is_null($assetPath)) {
            $assetPath = '/';
        }

        // @TODO 1.0.0 Remove support for 'base' as it's been deprecated

        $base = (array_key_exists('base', $globals['site'])) ? $globals['site']['base'] : $globals['site']['baseurl'];

        $baseURL = (empty($base)) ? '/' : '/' . trim($base, '/') . '/';
        $url     = ltrim($assetPath, '/');

        return ($baseURL . $url);
    }

    public static function get ()
    {
        return new \Twig_SimpleFunction('url', new self(), array(
            'needs_environment' => true
        ));
    }
}

1		<?php
2
3		namespace allejo\stakx\Twig;
4
5		use allejo\stakx\Object\JailObject;
6		use Twig_Environment;
7
8		class BaseUrlFunction
9		{
10		public function __invoke (Twig_Environment $env, $assetPath)
11		{
12		$globals = $env->getGlobals();
13
14		if ($assetPath instanceof JailObject) {
15		$assetPath = $assetPath->getPermalink();
		0 ignored issues – show Documentation Bug introduced 2017-01-17 02:32 UTC by Report Bug Copy Issue Report The method `getPermalink` does not exist on `object<allejo\stakx\Object\JailObject>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
16		}
17		else if (is_array($assetPath)) {
18		$assetPath = $assetPath['permalink'];
19		}
20		else if (is_null($assetPath)) {
21		$assetPath = '/';
22		}
23
24		// @TODO 1.0.0 Remove support for 'base' as it's been deprecated
		0 ignored issues – show Coding Style Best Practice introduced 2016-08-21 19:38 UTC by Report Bug Copy Issue Report Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker. Loading history...
25		$base = (array_key_exists('base', $globals['site'])) ? $globals['site']['base'] : $globals['site']['baseurl'];
26
27		$baseURL = (empty($base)) ? '/' : '/' . trim($base, '/') . '/';
28		$url = ltrim($assetPath, '/');
29
30		return ($baseURL . $url);
31		}
32
33	1	public static function get ()
34		{
35	1	return new \Twig_SimpleFunction('url', new self(), array(
36		'needs_environment' => true
37	1	));
38		}
39		}

stakx-io / stakx

Push — master ( 3657ae...e63a18 )

BaseUrlFunction::__invoke() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like