Compiler - Code Metrics - Inspection of "Fix Watch command inability to compile updated par..." - stakx-io/stakx - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#46)

by Vladimir

created 2017-04-15 08:14 UTC

Compiler B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	416
Duplicated Lines	7.21 %

Coupling/Cohesion

Components	1
Dependencies	15

Test Coverage

Coverage

80.15%

Importance

Changes

Metric	Value
wmc	37
lcom	1
cbo	15
dl	30
loc	416
ccs	109
cts	136
cp	0.8015
rs	7.8833
c	0
b	0
f	0

20 Methods

Rating	Name	Duplication	Size	Complexity
A	isParentTemplate()	0	4	1
A	compileAll()	0	7	2
A	__construct()	0	7	1
A	setRedirectTemplate()	0	4	1
A	setTargetFolder()	0	4	1
A	setPageViews()	0	5	1
A	setThemeName()	0	4	1
A	refreshParent()	0	7	2
A	compileSome()	0	11	3
B	compilePageView()	0	25	4
A	compileStaticPageView()	0	8	1
B	compileDynamicPageViews()	0	23	4
A	compileRepeaterPageViews()	17	17	2
A	compileContentItem()	13	13	1
A	compileStandardRedirects()	0	15	2
A	compileExpandedRedirects()	0	22	3
A	renderRepeaterPageView()	0	14	1
A	renderDynamicPageView()	0	9	1
A	renderStaticPageView()	0	10	1
B	createTwigTemplate()	0	33	4

How to fix Duplicated Code

<?php

/**
 * @copyright 2017 Vladimir Jimenez
 * @license   https://github.com/allejo/stakx/blob/master/LICENSE.md MIT
 */

namespace allejo\stakx;

use allejo\stakx\Command\BuildableCommand;
use allejo\stakx\Document\ContentItem;
use allejo\stakx\Document\DynamicPageView;
use allejo\stakx\Document\PageView;
use allejo\stakx\Document\RepeaterPageView;
use allejo\stakx\FrontMatter\ExpandedValue;
use allejo\stakx\Manager\BaseManager;
use allejo\stakx\Manager\ThemeManager;
use allejo\stakx\Manager\TwigManager;
use allejo\stakx\System\Folder;
use Twig_Environment;
use Twig_Error_Syntax;
use Twig_Source;
use Twig_Template;

/**
 * This class takes care of rendering the Twig body of PageViews with the respective information and it also takes care
 * of writing the rendered Twig to the filesystem.
 *
 * @internal
 *
 * @since 0.1.1
 */
class Compiler extends BaseManager
{
    /** @var string|false */
    private $redirectTemplate;

    /** @var Twig_Template[] */
    private $templateDependencies;

    /** @var PageView[] */
    private $pageViewsFlattened;

    /** @var PageView[][] */
    private $pageViews;

    /** @var Folder */
    private $folder;

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

    /** @var Twig_Environment */
    private $twig;

    public function __construct()
    {
        parent::__construct();

        $this->twig = TwigManager::getInstance();
        $this->theme = '';
    }

    /**
     * @param string|false $template
     */
    public function setRedirectTemplate($template)
    {
        $this->redirectTemplate = $template;
    }

    /**
     * @param Folder $folder
     */
    public function setTargetFolder(Folder $folder)
    {
        $this->folder = $folder;
    }

    /**
     * @param PageView[][] $pageViews
     * @param PageView[]   $pageViewsFlattened
     */
    public function setPageViews(array &$pageViews, array &$pageViewsFlattened)
    {
        $this->pageViews = &$pageViews;
        $this->pageViewsFlattened = &$pageViewsFlattened;
    }

    public function setThemeName($themeName)
    {
        $this->theme = $themeName;
    }

    ///
    // Twig parent templates
    ///

    public function isParentTemplate($filePath)
    {
        return isset($this->templateDependencies[$filePath]);
    }

    public function refreshParent($filePath)
    {
        foreach ($this->templateDependencies[$filePath] as &$parentTemplate)

        {
            $this->compilePageView($parentTemplate);
        }
    }

    ///
    // IO Functionality
    ///

    /**
     * Compile all of the PageViews registered with the compiler.
     *
     * @since 0.1.0
     */
    public function compileAll()
    {
        foreach ($this->pageViewsFlattened as &$pageView)
        {
            $this->compilePageView($pageView);
        }
    }

    public function compileSome($filter = array())
    {
        /** @var PageView $pageView */
        foreach ($this->pageViewsFlattened as &$pageView)
        {
            if ($pageView->hasTwigDependency($filter['namespace'], $filter['dependency']))
            {
                $this->compilePageView($pageView);
            }
        }
    }

    /**
     * Compile an individual PageView item.
     *
     * This function will take care of determining *how* to treat the PageView and write the compiled output to a the
     * respective target file.
     *
     * @param DynamicPageView|RepeaterPageView|PageView $pageView The PageView that needs to be compiled
     *
     * @since 0.1.1
     */
    public function compilePageView(&$pageView)
    {
        $this->output->debug('Compiling {type} PageView: {pageview}', array(
            'pageview' => $pageView->getRelativeFilePath(),
            'type' => $pageView->getType()
        ));

        switch ($pageView->getType())
        {
            case PageView::STATIC_TYPE:
                $this->compileStaticPageView($pageView);
                $this->compileStandardRedirects($pageView);
                break;

            case PageView::DYNAMIC_TYPE:
                $this->compileDynamicPageViews($pageView);

                $this->compileStandardRedirects($pageView);
                break;

            case PageView::REPEATER_TYPE:
                $this->compileRepeaterPageViews($pageView);

                $this->compileExpandedRedirects($pageView);
                break;
        }
    }

    /**
     * Write the compiled output for a static PageView.
     *
     * @param PageView $pageView
     *
     * @since 0.1.1
     */
    private function compileStaticPageView(&$pageView)
    {
        $targetFile = $pageView->getTargetFile();
        $output = $this->renderStaticPageView($pageView);

        $this->output->notice('Writing file: {file}', array('file' => $targetFile));
        $this->folder->writeFile($targetFile, $output);
    }

    /**
     * Write the compiled output for a dynamic PageView.
     *
     * @param DynamicPageView $pageView
     *
     * @since 0.1.1
     */
    private function compileDynamicPageViews(&$pageView)
    {
        $contentItems = $pageView->getContentItems();
        $template = $this->createTwigTemplate($pageView);

        foreach ($contentItems as &$contentItem)
        {
            if ($contentItem->isDraft() && !Service::getParameter(BuildableCommand::USE_DRAFTS))
            {
                $this->output->debug('{file}: marked as a draft', array(
                    'file' => $contentItem->getRelativeFilePath()
                ));

                continue;
            }

            $targetFile = $contentItem->getTargetFile();
            $output = $this->renderDynamicPageView($template, $pageView, $contentItem);

            $this->output->notice('Writing file: {file}', array('file' => $targetFile));
            $this->folder->writeFile($targetFile, $output);
        }
    }

    /**
     * Write the compiled output for a repeater PageView.
     *
     * @param RepeaterPageView $pageView
     *
     * @since 0.1.1
     */
    private function compileRepeaterPageViews(&$pageView)

    {
        $pageView->rewindPermalink();

        $template = $this->createTwigTemplate($pageView);
        $permalinks = $pageView->getRepeaterPermalinks();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

        foreach ($permalinks as $permalink)
        {
            $pageView->bumpPermalink();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            $targetFile = $pageView->getTargetFile();
            $output = $this->renderRepeaterPageView($template, $pageView, $permalink);

            $this->output->notice('Writing repeater file: {file}', array('file' => $targetFile));
            $this->folder->writeFile($targetFile, $output);
        }
    }

    /**
     * @deprecated
     *
     * @todo This function needs to be rewritten or removed. Something
     *
     * @param ContentItem $contentItem
     */
    public function compileContentItem(&$contentItem)

    {
        $pageView = $contentItem->getPageView();
        $template = $this->createTwigTemplate($pageView);

        $contentItem->evaluateFrontMatter($pageView->getFrontMatter(false));

        $targetFile = $contentItem->getTargetFile();
        $output = $this->renderDynamicPageView($template, $pageView, $contentItem);

        $this->output->notice('Writing file: {file}', array('file' => $targetFile));
        $this->folder->writeFile($targetFile, $output);
    }

    ///
    // Redirect handling
    ///

    /**
     * Write redirects for standard redirects.
     *
     * @param PageView $pageView
     *
     * @since 0.1.1
     */
    private function compileStandardRedirects(&$pageView)
    {
        $redirects = $pageView->getRedirects();

        foreach ($redirects as $redirect)
$collection = json_decode($data, true);
if ( ! is_array($collection)) {
    throw new \RuntimeException('$collection must be an array.');
}

foreach ($collection as $item) { /** ... */ }
        {
            $redirectPageView = PageView::createRedirect(
                $redirect,
                $pageView->getPermalink(),
                $this->redirectTemplate
            );

            $this->compileStaticPageView($redirectPageView);
        }
    }

    /**
     * Write redirects for expanded redirects.
     *
     * @param RepeaterPageView $pageView
     *
     * @since 0.1.1
     */
    private function compileExpandedRedirects(&$pageView)
    {
        $permalinks = $pageView->getRepeaterPermalinks();

        /** @var ExpandedValue[] $repeaterRedirect */
        foreach ($pageView->getRepeaterRedirects() as $repeaterRedirect)
        {
            /**
             * @var int           $index
             * @var ExpandedValue $redirect
             */
            foreach ($repeaterRedirect as $index => $redirect)
            {
                $redirectPageView = PageView::createRedirect(
                    $redirect->getEvaluated(),
                    $permalinks[$index]->getEvaluated(),
                    $this->redirectTemplate
                );
                $this->compilePageView($redirectPageView);
            }
        }
    }

    ///
    // Twig Functionality
    ///

    /**
     * Get the compiled HTML for a specific iteration of a repeater PageView.
     *
     * @param Twig_Template $template
     * @param PageView      $pageView
     * @param ExpandedValue $expandedValue
     *
     * @since  0.1.1
     *
     * @return string
     */
    private function renderRepeaterPageView(&$template, &$pageView, &$expandedValue)
    {
        $this->twig->addGlobal('__currentTemplate', $pageView->getFilePath());

        $pageView->setFrontMatter(array(
            'permalink' => $expandedValue->getEvaluated(),
            'iterators' => $expandedValue->getIterators(),
        ));

        return $template
            ->render(array(
                'this' => $pageView->createJail(),
            ));
    }

    /**
     * Get the compiled HTML for a specific ContentItem.
     *
     * @param Twig_Template $template
     * @param PageView      $pageView
     * @param ContentItem   $contentItem
     *
     * @since  0.1.1
     *
     * @return string
     */
    private function renderDynamicPageView(&$template, &$pageView, &$contentItem)
    {
        $this->twig->addGlobal('__currentTemplate', $pageView->getFilePath());

        return $template
            ->render(array(
                'this' => $contentItem->createJail(),
            ));
    }

    /**
     * Get the compiled HTML for a static PageView.
     *
     * @param PageView $pageView
     *
     * @since  0.1.1
     *
     * @throws \Exception
     * @throws \Throwable
     * @throws Twig_Error_Syntax
     *
     * @return string
     */
    private function renderStaticPageView(&$pageView)
    {
        $this->twig->addGlobal('__currentTemplate', $pageView->getFilePath());

        return $this
            ->createTwigTemplate($pageView)
            ->render(array(
                'this' => $pageView->createJail(),
            ));
    }

    /**
     * Create a Twig template that just needs an array to render.
     *
     * @param PageView $pageView The PageView whose body will be used for Twig compilation
     *
     * @since  0.1.1
     *
     * @throws \Exception
     * @throws \Throwable
     * @throws Twig_Error_Syntax
     *
     * @return Twig_Template
     */
    private function createTwigTemplate(&$pageView)
    {
        try
        {
            $template = $this->twig->createTemplate($pageView->getContent());

            if (Service::getParameter(BuildableCommand::WATCHING))
            {
                $parent = $template->getParent(array());

                while (false !== $parent)
                {
                    $path = str_replace('@theme', $this->fs->appendPath(ThemeManager::THEME_FOLDER, $this->theme), $parent->getTemplateName());
                    $this->templateDependencies[$path][$pageView->getName()] = &$pageView;

                    $parent = $parent->getParent(array());
                }
            }

            return $template;
        }
        catch (Twig_Error_Syntax $e)
        {
            $e->setTemplateLine($e->getTemplateLine() + $pageView->getLineOffset());
            $e->setSourceContext(new Twig_Source(
                $pageView->getContent(),
                $pageView->getName(),
                $pageView->getRelativeFilePath()
            ));

            throw $e;
        }
    }
}


Pull Request — master (#46)

Compiler B

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

20 Methods

How to fix Duplicated Code

Duplicated Code

Available Fixes

Available Fixes

1			<?php
2
3			/**
4			* @copyright 2017 Vladimir Jimenez
5			* @license https://github.com/allejo/stakx/blob/master/LICENSE.md MIT
6			*/
7
8			namespace allejo\stakx;
9
10			use allejo\stakx\Command\BuildableCommand;
11			use allejo\stakx\Document\ContentItem;
12			use allejo\stakx\Document\DynamicPageView;
13			use allejo\stakx\Document\PageView;
14			use allejo\stakx\Document\RepeaterPageView;
15			use allejo\stakx\FrontMatter\ExpandedValue;
16			use allejo\stakx\Manager\BaseManager;
17			use allejo\stakx\Manager\ThemeManager;
18			use allejo\stakx\Manager\TwigManager;
19			use allejo\stakx\System\Folder;
20			use Twig_Environment;
21			use Twig_Error_Syntax;
22			use Twig_Source;
23			use Twig_Template;
24
25			/**
26			* This class takes care of rendering the Twig body of PageViews with the respective information and it also takes care
27			* of writing the rendered Twig to the filesystem.
28			*
29			* @internal
30			*
31			* @since 0.1.1
32			*/
33			class Compiler extends BaseManager
34			{
35			/** @var string\|false */
36			private $redirectTemplate;
37
38			/** @var Twig_Template[] */
39			private $templateDependencies;
40
41			/** @var PageView[] */
42			private $pageViewsFlattened;
43
44			/** @var PageView[][] */
45			private $pageViews;
46
47			/** @var Folder */
48			private $folder;
49	14
50			/** @var string */
51	14		private $theme;
52
53	14		/** @var Twig_Environment */
54	14		private $twig;
55
56			public function __construct()
57			{
58			parent::__construct();
59
60			$this->twig = TwigManager::getInstance();
61			$this->theme = '';
62			}
63
64			/**
65			* @param string\|false $template
66			*/
67	14		public function setRedirectTemplate($template)
68			{
69	14		$this->redirectTemplate = $template;
70	14		}
71
72			/**
73			* @param Folder $folder
74			*/
75			public function setTargetFolder(Folder $folder)
76	14		{
77			$this->folder = $folder;
78	14		}
79	14
80	14		/**
81			* @param PageView[][] $pageViews
82			* @param PageView[] $pageViewsFlattened
83			*/
84			public function setPageViews(array &$pageViews, array &$pageViewsFlattened)
85			{
86			$this->pageViews = &$pageViews;
87			$this->pageViewsFlattened = &$pageViewsFlattened;
88			}
89
90			public function setThemeName($themeName)
91	14		{
92			$this->theme = $themeName;
93	14		}
94
95	14		///
96	14		// Twig parent templates
97	14		///
98
99			public function isParentTemplate($filePath)
100			{
101			return isset($this->templateDependencies[$filePath]);
102			}
103
104			public function refreshParent($filePath)
105			{
106			foreach ($this->templateDependencies[$filePath] as &$parentTemplate)
			0 ignored issues – show Bug introduced 2017-04-12 05:03 UTC by Report Bug Copy Issue Report The expression `$this->templateDependencies[$filePath]` of type `object<Twig_Template>` is not traversable. Loading history...
107			{
108			$this->compilePageView($parentTemplate);
109			}
110			}
111
112			///
113			// IO Functionality
114			///
115
116			/**
117			* Compile all of the PageViews registered with the compiler.
118			*
119			* @since 0.1.0
120			*/
121	14		public function compileAll()
122			{
123	14		foreach ($this->pageViewsFlattened as &$pageView)
124	14		{
125	14		$this->compilePageView($pageView);
126	14		}
127			}
128	14
129			public function compileSome($filter = array())
130	14		{
131	10		/** @var PageView $pageView */
132	10		foreach ($this->pageViewsFlattened as &$pageView)
133	10		{
134			if ($pageView->hasTwigDependency($filter['namespace'], $filter['dependency']))
135	4		{
136	2		$this->compilePageView($pageView);
137	2		}
138	2		}
139			}
140	2
141	2		/**
142	2		* Compile an individual PageView item.
143	2		*
144	14		* This function will take care of determining how to treat the PageView and write the compiled output to a the
145	14		* respective target file.
146			*
147			* @param DynamicPageView\|RepeaterPageView\|PageView $pageView The PageView that needs to be compiled
148			*
149			* @since 0.1.1
150			*/
151			public function compilePageView(&$pageView)
152			{
153			$this->output->debug('Compiling {type} PageView: {pageview}', array(
154	10		'pageview' => $pageView->getRelativeFilePath(),
155			'type' => $pageView->getType()
156	10		));
157	10
158			switch ($pageView->getType())
159	10		{
160	10		case PageView::STATIC_TYPE:
161	10		$this->compileStaticPageView($pageView);
162			$this->compileStandardRedirects($pageView);
163			break;
164
165			case PageView::DYNAMIC_TYPE:
166			$this->compileDynamicPageViews($pageView);
			1 ignored issue – show Compatibility introduced 2017-03-25 04:29 UTC by Report Bug Copy Issue Report `$pageView` of type `object<allejo\stakx\Document\PageView>` is not a sub-type of `object<allejo\stakx\Document\DynamicPageView>`. It seems like you assume a child class of the class `allejo\stakx\Document\PageView` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
167			$this->compileStandardRedirects($pageView);
168			break;
169
170	3		case PageView::REPEATER_TYPE:
171			$this->compileRepeaterPageViews($pageView);
			1 ignored issue – show Compatibility introduced 2017-03-25 04:29 UTC by Report Bug Copy Issue Report `$pageView` of type `object<allejo\stakx\Document\PageView>` is not a sub-type of `object<allejo\stakx\Document\RepeaterPageView>`. It seems like you assume a child class of the class `allejo\stakx\Document\PageView` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
172	2		$this->compileExpandedRedirects($pageView);
173	2		break;
174			}
175	2		}
176
177	2		/**
178	2		* Write the compiled output for a static PageView.
179	1		*
180	1		* @param PageView $pageView
181	1		*
182			* @since 0.1.1
183	1		*/
184			private function compileStaticPageView(&$pageView)
185			{
186	2		$targetFile = $pageView->getTargetFile();
187	3		$output = $this->renderStaticPageView($pageView);
188
189	2		$this->output->notice('Writing file: {file}', array('file' => $targetFile));
190	2		$this->folder->writeFile($targetFile, $output);
191	2		}
192	2
193			/**
194			* Write the compiled output for a dynamic PageView.
195			*
196			* @param DynamicPageView $pageView
197			*
198			* @since 0.1.1
199			*/
200			private function compileDynamicPageViews(&$pageView)
201	2		{
202			$contentItems = $pageView->getContentItems();
203	2		$template = $this->createTwigTemplate($pageView);
204
205	2		foreach ($contentItems as &$contentItem)
206	2		{
207			if ($contentItem->isDraft() && !Service::getParameter(BuildableCommand::USE_DRAFTS))
208	2		{
209			$this->output->debug('{file}: marked as a draft', array(
210	2		'file' => $contentItem->getRelativeFilePath()
211	2		));
212	2
213			continue;
214	2		}
215	2
216	2		$targetFile = $contentItem->getTargetFile();
217	2		$output = $this->renderDynamicPageView($template, $pageView, $contentItem);
218
219			$this->output->notice('Writing file: {file}', array('file' => $targetFile));
220			$this->folder->writeFile($targetFile, $output);
221			}
222			}
223
224			/**
225			* Write the compiled output for a repeater PageView.
226			*
227			* @param RepeaterPageView $pageView
228			*
229			* @since 0.1.1
230			*/
231		View Code Duplication	private function compileRepeaterPageViews(&$pageView)
			0 ignored issues – show Duplication introduced 2017-03-17 07:45 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
232			{
233			$pageView->rewindPermalink();
234
235			$template = $this->createTwigTemplate($pageView);
236			$permalinks = $pageView->getRepeaterPermalinks();
			0 ignored issues – show Bug introduced 2017-03-25 04:29 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `allejo\stakx\Document\PageView` as the method `getRepeaterPermalinks()` does only exist in the following sub-classes of `allejo\stakx\Document\PageView`: `allejo\stakx\Document\RepeaterPageView`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
237
238			foreach ($permalinks as $permalink)
239			{
240			$pageView->bumpPermalink();
			0 ignored issues – show Bug introduced 2017-03-25 04:29 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `allejo\stakx\Document\PageView` as the method `bumpPermalink()` does only exist in the following sub-classes of `allejo\stakx\Document\PageView`: `allejo\stakx\Document\RepeaterPageView`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
241			$targetFile = $pageView->getTargetFile();
242			$output = $this->renderRepeaterPageView($template, $pageView, $permalink);
243
244			$this->output->notice('Writing repeater file: {file}', array('file' => $targetFile));
245			$this->folder->writeFile($targetFile, $output);
246			}
247			}
248
249			/**
250			* @deprecated
251	12		*
252			* @todo This function needs to be rewritten or removed. Something
253	12		*
254			* @param ContentItem $contentItem
255	12		*/
256		View Code Duplication	public function compileContentItem(&$contentItem)
			0 ignored issues – show Duplication introduced 2017-03-24 08:42 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
257	4		{
258	4		$pageView = $contentItem->getPageView();
259	4		$template = $this->createTwigTemplate($pageView);
260	4
261	4		$contentItem->evaluateFrontMatter($pageView->getFrontMatter(false));
262
263	4		$targetFile = $contentItem->getTargetFile();
264	12		$output = $this->renderDynamicPageView($template, $pageView, $contentItem);
265	12
266			$this->output->notice('Writing file: {file}', array('file' => $targetFile));
267			$this->folder->writeFile($targetFile, $output);
268			}
269
270			///
271			// Redirect handling
272			///
273
274	2		/**
275			* Write redirects for standard redirects.
276	2		*
277			* @param PageView $pageView
278			*
279	2		* @since 0.1.1
280			*/
281			private function compileStandardRedirects(&$pageView)
282			{
283			$redirects = $pageView->getRedirects();
284
285			foreach ($redirects as $redirect)
			0 ignored issues – show Bug introduced 2017-03-21 01:32 UTC by Report Bug Copy Issue Report The expression `$redirects` of type `null\|array<integer,string>` is not guaranteed to be traversable. How about adding an additional type check? There are different options of fixing this problem. If you want to be on the safe side, you can add an additional type-check: $collection = json_decode($data, true); if ( ! is_array($collection)) { throw new \RuntimeException('$collection must be an array.'); } foreach ($collection as $item) { /** ... / } If you are sure that the expression is traversable, you might want to add a doc comment cast* to improve IDE auto-completion and static analysis: /** @var array $collection / $collection = json_decode($data, true); foreach ($collection as $item) { /* .. / } Mark the issue as a false-positive*: Just hover the remove button, in the top-right corner of this issue for more options. Loading history...
286			{
287			$redirectPageView = PageView::createRedirect(
288			$redirect,
289			$pageView->getPermalink(),
290			$this->redirectTemplate
291			);
292
293			$this->compileStaticPageView($redirectPageView);
294	2		}
295	2		}
296
297			/**
298			* Write redirects for expanded redirects.
299			*
300			* @param RepeaterPageView $pageView
301			*
302			* @since 0.1.1
303			*/
304			private function compileExpandedRedirects(&$pageView)
305			{
306			$permalinks = $pageView->getRepeaterPermalinks();
307
308			/** @var ExpandedValue[] $repeaterRedirect */
309			foreach ($pageView->getRepeaterRedirects() as $repeaterRedirect)
310			{
311			/**
312	2		* @var int $index
313			* @var ExpandedValue $redirect
314	2		*/
315			foreach ($repeaterRedirect as $index => $redirect)
316	2		{
317	2		$redirectPageView = PageView::createRedirect(
318	2		$redirect->getEvaluated(),
319	2		$permalinks[$index]->getEvaluated(),
320			$this->redirectTemplate
321			);
322	2		$this->compilePageView($redirectPageView);
323	2		}
324	2		}
325			}
326
327			///
328			// Twig Functionality
329			///
330
331			/**
332			* Get the compiled HTML for a specific iteration of a repeater PageView.
333			*
334			* @param Twig_Template $template
335			* @param PageView $pageView
336			* @param ExpandedValue $expandedValue
337			*
338	2		* @since 0.1.1
339			*
340	2		* @return string
341			*/
342			private function renderRepeaterPageView(&$template, &$pageView, &$expandedValue)
343	2		{
344	2		$this->twig->addGlobal('__currentTemplate', $pageView->getFilePath());
345	2
346			$pageView->setFrontMatter(array(
347			'permalink' => $expandedValue->getEvaluated(),
348			'iterators' => $expandedValue->getIterators(),
349			));
350
351			return $template
352			->render(array(
353			'this' => $pageView->createJail(),
354			));
355			}
356
357			/**
358			* Get the compiled HTML for a specific ContentItem.
359			*
360			* @param Twig_Template $template
361	10		* @param PageView $pageView
362			* @param ContentItem $contentItem
363	10		*
364			* @since 0.1.1
365	10		*
366	10		* @return string
367	10		*/
368	10		private function renderDynamicPageView(&$template, &$pageView, &$contentItem)
369	10		{
370			$this->twig->addGlobal('__currentTemplate', $pageView->getFilePath());
371
372			return $template
373			->render(array(
374			'this' => $contentItem->createJail(),
375			));
376			}
377
378			/**
379			* Get the compiled HTML for a static PageView.
380			*
381			* @param PageView $pageView
382			*
383			* @since 0.1.1
384			*
385	14		* @throws \Exception
386			* @throws \Throwable
387			* @throws Twig_Error_Syntax
388			*
389	14		* @return string
390			*/
391			private function renderStaticPageView(&$pageView)
392			{
393			$this->twig->addGlobal('__currentTemplate', $pageView->getFilePath());
394
395			return $this
396			->createTwigTemplate($pageView)
397			->render(array(
398			'this' => $pageView->createJail(),
399			));
400			}
401
402			/**
403			* Create a Twig template that just needs an array to render.
404			*
405			* @param PageView $pageView The PageView whose body will be used for Twig compilation
406			*
407			* @since 0.1.1
408			*
409			* @throws \Exception
410			* @throws \Throwable
411			* @throws Twig_Error_Syntax
412			*
413			* @return Twig_Template
414			*/
415			private function createTwigTemplate(&$pageView)
416			{
417			try
418			{
419			$template = $this->twig->createTemplate($pageView->getContent());
420
421			if (Service::getParameter(BuildableCommand::WATCHING))
422			{
423			$parent = $template->getParent(array());
424
425			while (false !== $parent)
426			{
427			$path = str_replace('@theme', $this->fs->appendPath(ThemeManager::THEME_FOLDER, $this->theme), $parent->getTemplateName());
428			$this->templateDependencies[$path][$pageView->getName()] = &$pageView;
429
430			$parent = $parent->getParent(array());
431			}
432			}
433
434			return $template;
435			}
436			catch (Twig_Error_Syntax $e)
437			{
438			$e->setTemplateLine($e->getTemplateLine() + $pageView->getLineOffset());
439			$e->setSourceContext(new Twig_Source(
440			$pageView->getContent(),
441			$pageView->getName(),
442			$pageView->getRelativeFilePath()
443			));
444
445			throw $e;
446			}
447			}
448			}
449

stakx-io / stakx

Pull Request — master (#46)

Compiler B

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

20 Methods

How to fix Duplicated Code

Duplicated Code

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like