Compiler - Code Metrics - Inspection of "Add 'import' keyword for _config.yml" - stakx-io/stakx - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#42)

by Vladimir

created 2017-03-28 23:06 UTC

Compiler A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	358
Duplicated Lines	12.29 %

Coupling/Cohesion

Components	1
Dependencies	13

Test Coverage

Coverage

78.74%

Importance

Changes

Metric	Value
dl	44
loc	358
ccs	100
cts	127
cp	0.7874
rs	10
c	0
b	0
f	0
wmc	29
lcom	1
cbo	13

17 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	setRedirectTemplate()	0	4	1
A	setTargetFolder()	0	4	1
A	setPageViews()	0	5	1
A	compileAll()	0	7	2
A	compileSome()	0	11	3
A	compileStaticPageView()	0	8	1
A	compileRepeaterPageViews()	17	17	2
A	compilePageView()	0	20	4
A	compileDynamicPageViews()	14	14	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
A	createTwigTemplate()	0	18	2

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\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\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 PageView[] */
    private $pageViewsFlattened;

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

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

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

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

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

    /**
     * @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;
    }

    ///
    // 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
     */
    private function compilePageView(&$pageView)
    {
        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)
        {
            $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
        {
            return $this->twig->createTemplate($pageView->getContent());
        }
        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 (#42)

Compiler A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

17 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\Document\ContentItem;
11			use allejo\stakx\Document\DynamicPageView;
12			use allejo\stakx\Document\PageView;
13			use allejo\stakx\Document\RepeaterPageView;
14			use allejo\stakx\FrontMatter\ExpandedValue;
15			use allejo\stakx\Manager\BaseManager;
16			use allejo\stakx\Manager\TwigManager;
17			use allejo\stakx\System\Folder;
18			use Twig_Environment;
19			use Twig_Error_Syntax;
20			use Twig_Source;
21			use Twig_Template;
22
23			/**
24			* This class takes care of rendering the Twig body of PageViews with the respective information and it also takes care
25			* of writing the rendered Twig to the filesystem.
26			*
27			* @internal
28			*
29			* @since 0.1.1
30			*/
31			class Compiler extends BaseManager
32			{
33			/** @var string\|false */
34			private $redirectTemplate;
35
36			/** @var PageView[] */
37			private $pageViewsFlattened;
38
39			/** @var PageView[][] */
40			private $pageViews;
41
42			/** @var Folder */
43			private $folder;
44
45			/** @var Twig_Environment */
46			private $twig;
47
48	13		public function __construct()
49			{
50	13		parent::__construct();
51
52	13		$this->twig = TwigManager::getInstance();
53	13		}
54
55			/**
56			* @param string\|false $template
57			*/
58			public function setRedirectTemplate($template)
59			{
60			$this->redirectTemplate = $template;
61			}
62
63			/**
64			* @param Folder $folder
65			*/
66	13		public function setTargetFolder(Folder $folder)
67			{
68	13		$this->folder = $folder;
69	13		}
70
71			/**
72			* @param PageView[][] $pageViews
73			* @param PageView[] $pageViewsFlattened
74			*/
75	13		public function setPageViews(array &$pageViews, array &$pageViewsFlattened)
76			{
77	13		$this->pageViews = &$pageViews;
78	13		$this->pageViewsFlattened = &$pageViewsFlattened;
79	13		}
80
81			///
82			// IO Functionality
83			///
84
85			/**
86			* Compile all of the PageViews registered with the compiler.
87			*
88			* @since 0.1.0
89			*/
90	13		public function compileAll()
91			{
92	13		foreach ($this->pageViewsFlattened as &$pageView)
93			{
94	13		$this->compilePageView($pageView);
95	13		}
96	13		}
97
98			public function compileSome($filter = array())
99			{
100			/** @var PageView $pageView */
101			foreach ($this->pageViewsFlattened as &$pageView)
102			{
103			if ($pageView->hasTwigDependency($filter['namespace'], $filter['dependency']))
104			{
105			$this->compilePageView($pageView);
106			}
107			}
108			}
109
110			/**
111			* Compile an individual PageView item.
112			*
113			* This function will take care of determining how to treat the PageView and write the compiled output to a the
114			* respective target file.
115			*
116			* @param DynamicPageView\|RepeaterPageView\|PageView $pageView The PageView that needs to be compiled
117			*
118			* @since 0.1.1
119			*/
120	13		private function compilePageView(&$pageView)
121			{
122	13		switch ($pageView->getType())
123	1		{
124	13		case PageView::STATIC_TYPE:
125	10		$this->compileStaticPageView($pageView);
126	10		$this->compileStandardRedirects($pageView);
127	10		break;
128
129	3		case PageView::DYNAMIC_TYPE:
130	1		$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...
131	1		$this->compileStandardRedirects($pageView);
132	1		break;
133
134	2		case PageView::REPEATER_TYPE:
135	2		$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...
136	2		$this->compileExpandedRedirects($pageView);
137	2		break;
138	13		}
139	13		}
140
141			/**
142			* Write the compiled output for a static PageView.
143			*
144			* @param PageView $pageView
145			*
146			* @since 0.1.1
147			*/
148	10		private function compileStaticPageView(&$pageView)
149			{
150	10		$targetFile = $pageView->getTargetFile();
151	10		$output = $this->renderStaticPageView($pageView);
152
153	10		$this->output->notice('Writing file: {file}', array('file' => $targetFile));
154	10		$this->folder->writeFile($targetFile, $output);
155	10		}
156
157			/**
158			* Write the compiled output for a dynamic PageView.
159			*
160			* @param DynamicPageView $pageView
161			*
162			* @since 0.1.1
163			*/
164	1	View Code Duplication	private function compileDynamicPageViews(&$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...
165			{
166	1		$contentItems = $pageView->getContentItems();
167	1		$template = $this->createTwigTemplate($pageView);
168
169	1		foreach ($contentItems as &$contentItem)
170			{
171	1		$targetFile = $contentItem->getTargetFile();
172	1		$output = $this->renderDynamicPageView($template, $pageView, $contentItem);
173
174	1		$this->output->notice('Writing file: {file}', array('file' => $targetFile));
175	1		$this->folder->writeFile($targetFile, $output);
176	1		}
177	1		}
178
179			/**
180			* Write the compiled output for a repeater PageView.
181			*
182			* @param RepeaterPageView $pageView
183			*
184			* @since 0.1.1
185			*/
186	2	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...
187			{
188	2		$pageView->rewindPermalink();
189
190	2		$template = $this->createTwigTemplate($pageView);
191	2		$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...
192
193	2		foreach ($permalinks as $permalink)
194			{
195	2		$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...
196	2		$targetFile = $pageView->getTargetFile();
197	2		$output = $this->renderRepeaterPageView($template, $pageView, $permalink);
198
199	2		$this->output->notice('Writing repeater file: {file}', array('file' => $targetFile));
200	2		$this->folder->writeFile($targetFile, $output);
201	2		}
202	2		}
203
204			/**
205			* @deprecated
206			*
207			* @todo This function needs to be rewritten or removed. Something
208			*
209			* @param ContentItem $contentItem
210			*/
211		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...
212			{
213			$pageView = $contentItem->getPageView();
214			$template = $this->createTwigTemplate($pageView);
215
216			$contentItem->evaluateFrontMatter($pageView->getFrontMatter(false));
217
218			$targetFile = $contentItem->getTargetFile();
219			$output = $this->renderDynamicPageView($template, $pageView, $contentItem);
220
221			$this->output->notice('Writing file: {file}', array('file' => $targetFile));
222			$this->folder->writeFile($targetFile, $output);
223			}
224
225			///
226			// Redirect handling
227			///
228
229			/**
230			* Write redirects for standard redirects.
231			*
232			* @param PageView $pageView
233			*
234			* @since 0.1.1
235			*/
236	11		private function compileStandardRedirects(&$pageView)
237			{
238	11		$redirects = $pageView->getRedirects();
239
240	11		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...
241			{
242	4		$redirectPageView = PageView::createRedirect(
243	4		$redirect,
244	4		$pageView->getPermalink(),
245	4		$this->redirectTemplate
246	4		);
247
248	4		$this->compileStaticPageView($redirectPageView);
249	11		}
250	11		}
251
252			/**
253			* Write redirects for expanded redirects.
254			*
255			* @param RepeaterPageView $pageView
256			*
257			* @since 0.1.1
258			*/
259	2		private function compileExpandedRedirects(&$pageView)
260			{
261	2		$permalinks = $pageView->getRepeaterPermalinks();
262
263			/** @var ExpandedValue[] $repeaterRedirect */
264	2		foreach ($pageView->getRepeaterRedirects() as $repeaterRedirect)
265			{
266			/**
267			* @var int $index
268			* @var ExpandedValue $redirect
269			*/
270			foreach ($repeaterRedirect as $index => $redirect)
271			{
272			$redirectPageView = PageView::createRedirect(
273			$redirect->getEvaluated(),
274			$permalinks[$index]->getEvaluated(),
275			$this->redirectTemplate
276			);
277			$this->compilePageView($redirectPageView);
278			}
279	2		}
280	2		}
281
282			///
283			// Twig Functionality
284			///
285
286			/**
287			* Get the compiled HTML for a specific iteration of a repeater PageView.
288			*
289			* @param Twig_Template $template
290			* @param PageView $pageView
291			* @param ExpandedValue $expandedValue
292			*
293			* @since 0.1.1
294			*
295			* @return string
296			*/
297	2		private function renderRepeaterPageView(&$template, &$pageView, &$expandedValue)
298			{
299	2		$this->twig->addGlobal('__currentTemplate', $pageView->getFilePath());
300
301	2		$pageView->setFrontMatter(array(
302	2		'permalink' => $expandedValue->getEvaluated(),
303	2		'iterators' => $expandedValue->getIterators(),
304	2		));
305
306			return $template
307	2		->render(array(
308	2		'this' => $pageView->createJail(),
309	2		));
310			}
311
312			/**
313			* Get the compiled HTML for a specific ContentItem.
314			*
315			* @param Twig_Template $template
316			* @param PageView $pageView
317			* @param ContentItem $contentItem
318			*
319			* @since 0.1.1
320			*
321			* @return string
322			*/
323	1		private function renderDynamicPageView(&$template, &$pageView, &$contentItem)
324			{
325	1		$this->twig->addGlobal('__currentTemplate', $pageView->getFilePath());
326
327			return $template
328	1		->render(array(
329	1		'this' => $contentItem->createJail(),
330	1		));
331			}
332
333			/**
334			* Get the compiled HTML for a static PageView.
335			*
336			* @param PageView $pageView
337			*
338			* @since 0.1.1
339			*
340			* @throws \Exception
341			* @throws \Throwable
342			* @throws Twig_Error_Syntax
343			*
344			* @return string
345			*/
346	10		private function renderStaticPageView(&$pageView)
347			{
348	10		$this->twig->addGlobal('__currentTemplate', $pageView->getFilePath());
349
350	10		return $this
351	10		->createTwigTemplate($pageView)
352	10		->render(array(
353	10		'this' => $pageView->createJail(),
354	10		));
355			}
356
357			/**
358			* Create a Twig template that just needs an array to render.
359			*
360			* @param PageView $pageView The PageView whose body will be used for Twig compilation
361			*
362			* @since 0.1.1
363			*
364			* @throws \Exception
365			* @throws \Throwable
366			* @throws Twig_Error_Syntax
367			*
368			* @return Twig_Template
369			*/
370	13		private function createTwigTemplate(&$pageView)
371			{
372			try
373			{
374	13		return $this->twig->createTemplate($pageView->getContent());
375			}
376			catch (Twig_Error_Syntax $e)
377			{
378			$e->setTemplateLine($e->getTemplateLine() + $pageView->getLineOffset());
379			$e->setSourceContext(new Twig_Source(
380			$pageView->getContent(),
381			$pageView->getName(),
382			$pageView->getRelativeFilePath()
383			));
384
385			throw $e;
386			}
387			}
388			}
389

stakx-io / stakx

Pull Request — master (#42)

Compiler A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

17 Methods

How to fix Duplicated Code

Duplicated Code

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like