Resolver::resolve() - Code Metrics - Inspection of "Merge pull request #1 from eclipxe13/development" - eclipxe13/engineworks-templates - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9fea99...819409 )

by Carlos C

created 2016-09-28 00:30 UTC

Resolver::resolve() A

↳ Parent: Resolver

Complexity

Conditions	3
Paths	2

Size

Total Lines	7
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	3

Importance

Changes

Metric	Value
dl	0
loc	7
ccs	4
cts	4
cp	1
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	4
nc	2
nop	1
crap	3

<?php
namespace EngineWorks\Templates;

class Resolver
{
    /** @var string */
    private $directory;

    /** @var string */
    private $extension;

    /**
     * Templates constructor.
     *
     * @param string $directory Locations where templates are
     * @param string $extension Templates extension
     */
    public function __construct($directory = '', $extension = 'php')
    {
        $this->setDirectory($directory);
        $this->setExtension($extension);
    }

    /**
     * @return string
     */
    public function getDirectory()
    {
        return $this->directory;
    }

    /**
     * @param string $directory
     */
    public function setDirectory($directory)
    {
        $this->directory = $directory;
    }

    /**
     * @return string
     */
    public function getExtension()
    {
        return $this->extension;
    }

    /**
     * @param string $extension
     */
    public function setExtension($extension)
    {
        $this->extension = $extension;
    }

    /**
     * Resolve a filename by its friendly name, the real name will be
     * directory + template + extension
     *
     * @param string $template
     * @return string
     */
    public function resolve($template)
    {
        if (0 === strpos($template, '../') || false !== strpos($template, '/../')) {
            throw new \InvalidArgumentException('The filename try to escape the current path');
        }
        return $this->directory . '/' . $template . '.' . $this->extension;
    }
}


1		<?php
2		namespace EngineWorks\Templates;
3
4		class Resolver
5		{
6		/** @var string */
7		private $directory;
8
9		/** @var string */
10		private $extension;
11
12		/**
13		* Templates constructor.
14		*
15		* @param string $directory Locations where templates are
16		* @param string $extension Templates extension
17		*/
18	24	public function __construct($directory = '', $extension = 'php')
19		{
20	24	$this->setDirectory($directory);
21	24	$this->setExtension($extension);
22	24	}
23
24		/**
25		* @return string
26		*/
27	3	public function getDirectory()
28		{
29	3	return $this->directory;
30		}
31
32		/**
33		* @param string $directory
34		*/
35	24	public function setDirectory($directory)
36		{
37	24	$this->directory = $directory;
38	24	}
39
40		/**
41		* @return string
42		*/
43	3	public function getExtension()
44		{
45	3	return $this->extension;
46		}
47
48		/**
49		* @param string $extension
50		*/
51	24	public function setExtension($extension)
52		{
53	24	$this->extension = $extension;
54	24	}
55
56		/**
57		* Resolve a filename by its friendly name, the real name will be
58		* directory + template + extension
59		*
60		* @param string $template
61		* @return string
62		*/
63	12	public function resolve($template)
64		{
65	12	if (0 === strpos($template, '../') \|\| false !== strpos($template, '/../')) {
66	3	throw new \InvalidArgumentException('The filename try to escape the current path');
67		}
68	9	return $this->directory . '/' . $template . '.' . $this->extension;
69		}
70		}
71

eclipxe13 / engineworks-templates

Push — master ( 9fea99...819409 )

Resolver::resolve() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like