RenderController::renderPage() - Code Metrics - Inspection of "redoing to `Page` models and more" - hiqdev/yii2-module-pages - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2038e8...a5b302 )

by Andrii

created 2017-01-13 17:25 UTC

RenderController::renderPage() A

↳ Parent: RenderController

Complexity

Conditions	3
Paths	4

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

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

<?php

namespace hiqdev\yii2\modules\pages\controllers;

use hiqdev\yii2\modules\pages\models\AbstractPage;
use hiqdev\yii2\modules\pages\models\PagesIndex;
use Yii;
use yii\helpers\Html;
use yii\web\NotFoundHttpException;
use yii\base\InvalidConfigException;

class RenderController extends \yii\web\Controller
{
    public function getViewPath()
    {
        return Yii::$app->getViewPath();
    }

    /**
     * Index action.
     * @param string $page
     * @return string rendered page
     */
    public function actionIndex($page = null)
    {
        if (!$page) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $page = 'index';
        }

        $path = $this->module->find($page);

        if ($path === null) {
            throw new NotFoundHttpException(Yii::t('yii', 'Page not found.'));
        }

        $meta = $this->module->getMetadata($page);

        if ($meta['type'] === 'dir') {
            $index = PagesIndex::createFromDir($path);

            return $this->render('index', $index);
        } else {
            $page = AbstractPage::createFromFile($path);

            return $this->renderPage($page);
        }
    }

    public function renderTwig($path)

    {
        throw new InvalidConfigException('not implemented twig handler');
    }

    public function renderPhp($path)
    {
        $content = $this->renderFile($this->module->localPath($path));
        return $this->renderContent($content);
    }

    public function renderPage($page, array $params = [])
    {
        if ($page->layout) {
            $this->layout = $page->layout;
        }

        if ($page->title) {
            $this->view->title = Html::encode($page->title);
        }

        $this->view->params = $page->getData();

        return $this->renderContent($page->render($params));
    }

}


1			<?php
2
3			namespace hiqdev\yii2\modules\pages\controllers;
4
5			use hiqdev\yii2\modules\pages\models\AbstractPage;
6			use hiqdev\yii2\modules\pages\models\PagesIndex;
7			use Yii;
8			use yii\helpers\Html;
9			use yii\web\NotFoundHttpException;
10			use yii\base\InvalidConfigException;
11
12			class RenderController extends \yii\web\Controller
13			{
14			public function getViewPath()
15			{
16			return Yii::$app->getViewPath();
17			}
18
19			/**
20			* Index action.
21			* @param string $page
22			* @return string rendered page
23			*/
24			public function actionIndex($page = null)
25			{
26			if (!$page) {
			0 ignored issues – show Bug Best Practice introduced 2016-09-22 16:00 UTC by Report Bug Copy Issue Report The expression `$page` of type `string\|null` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
27			$page = 'index';
28			}
29
30			$path = $this->module->find($page);
31
32			if ($path === null) {
33			throw new NotFoundHttpException(Yii::t('yii', 'Page not found.'));
34			}
35
36			$meta = $this->module->getMetadata($page);
37
38			if ($meta['type'] === 'dir') {
39			$index = PagesIndex::createFromDir($path);
40
41			return $this->render('index', $index);
42			} else {
43			$page = AbstractPage::createFromFile($path);
44
45			return $this->renderPage($page);
46			}
47			}
48
49			public function renderTwig($path)
			0 ignored issues – show Unused Code introduced 2016-09-22 16:00 UTC by Report Bug Copy Issue Report The parameter `$path` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
50			{
51			throw new InvalidConfigException('not implemented twig handler');
52			}
53
54			public function renderPhp($path)
55			{
56			$content = $this->renderFile($this->module->localPath($path));
57			return $this->renderContent($content);
58			}
59
60			public function renderPage($page, array $params = [])
61			{
62			if ($page->layout) {
63			$this->layout = $page->layout;
64			}
65
66			if ($page->title) {
67			$this->view->title = Html::encode($page->title);
68			}
69
70			$this->view->params = $page->getData();
71
72			return $this->renderContent($page->render($params));
73			}
74
75			}
76

hiqdev / yii2-module-pages

Push — master ( 2038e8...a5b302 )

RenderController::renderPage() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like