RequestAwareProvider::url() - Code Metrics - Inspection of "maintenance work" - zicht/url-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — release/3.x (#37)

unknown

created 2018-06-07 08:07 UTC

RequestAwareProvider::url() A

↳ Parent: RequestAwareProvider

Complexity

Conditions	4
Paths	4

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	7
nc	4
nop	2
dl	0
loc	12
rs	9.2
c	0
b	0
f	0

<?php
/**
 * @author Gerard van Helden <[email protected]>
 * @copyright Zicht Online <http://zicht.nl>
 */

namespace Zicht\Bundle\UrlBundle\Url;

use Symfony\Component\HttpFoundation\Request;

/**
 * This provider is "request aware", so it can either render absolute URL's
 */
class RequestAwareProvider extends DelegatingProvider
{
    /**
     * Constructor
     *
     * @param \Symfony\Component\HttpFoundation\Request $request
     */
    public function __construct(Request $request)
    {
        parent::__construct();

        $this->baseUrl = $request->getBaseUrl();

        $this->prefix = $request->getSchemeAndHttpHost() . $this->baseUrl;

        $this->baseUrlLen = strlen($this->baseUrl);

    }

    /**
     * @{inheritDoc}
     */
    public function url($object, array $options = array())
    {
        $ret = parent::url($object, $options);

        if ($this->baseUrlLen && substr($ret, 0, $this->baseUrlLen) == $this->baseUrl) {
            $ret = substr($ret, $this->baseUrlLen);
        }
        $ret = ltrim($ret, '/');
        if (!empty($options['absolute'])) {
            $ret = $this->prefix . '/' . $ret;
        }
        return $ret;
    }
}


1			<?php
2			/**
3			* @author Gerard van Helden <[email protected]>
4			* @copyright Zicht Online <http://zicht.nl>
5			*/
6
7			namespace Zicht\Bundle\UrlBundle\Url;
8
9			use Symfony\Component\HttpFoundation\Request;
10
11			/**
12			* This provider is "request aware", so it can either render absolute URL's
13			*/
14			class RequestAwareProvider extends DelegatingProvider
15			{
16			/**
17			* Constructor
18			*
19			* @param \Symfony\Component\HttpFoundation\Request $request
20			*/
21			public function __construct(Request $request)
22			{
23			parent::__construct();
24
25			$this->baseUrl = $request->getBaseUrl();
			0 ignored issues – show Bug Best Practice introduced 2018-01-19 11:22 UTC by Report Bug Copy Issue Report The property `baseUrl` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
26			$this->prefix = $request->getSchemeAndHttpHost() . $this->baseUrl;
			0 ignored issues – show Bug Best Practice introduced 2018-01-19 11:22 UTC by Report Bug Copy Issue Report The property `prefix` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
27			$this->baseUrlLen = strlen($this->baseUrl);
			0 ignored issues – show Bug Best Practice introduced 2018-01-19 11:22 UTC by Report Bug Copy Issue Report The property `baseUrlLen` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
28			}
29
30			/**
31			* @{inheritDoc}
32			*/
33			public function url($object, array $options = array())
34			{
35			$ret = parent::url($object, $options);
36
37			if ($this->baseUrlLen && substr($ret, 0, $this->baseUrlLen) == $this->baseUrl) {
38			$ret = substr($ret, $this->baseUrlLen);
39			}
40			$ret = ltrim($ret, '/');
41			if (!empty($options['absolute'])) {
42			$ret = $this->prefix . '/' . $ret;
43			}
44			return $ret;
45			}
46			}
47

zicht / url-bundle

Pull Request — release/3.x (#37)

RequestAwareProvider::url() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like