app\controllers\Examples

Complexity

Total Complexity

14

Size/Duplication

Total Lines	205
Duplicated Lines	56.1 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	17	17	1
A	getOnecolumn()	9	9	1
A	getTwocolumn()	9	9	1
A	getThreecolumn()	9	9	1
A	getJohn()	13	13	1
A	getMarkup()	13	13	1
A	getCarousel()	13	13	1
A	getTheme()	0	9	1
A	getTheme2()	0	11	1
A	getFlash()	0	19	1
B	getGrid()	4	31	2
A	getFullpage()	13	13	1
A	getException()	15	15	1

<?php
namespace app\controllers;

class Examples extends \erdiko\controllers\Web
{
    use \erdiko\theme\traits\Controller; // Add theme engine suport (for convenience)

    public function get($request, $response, $args)

This method seems to be duplicated in your project.

    {
        // $this->container->logger->debug("examples");

67% of this comment could be valid code. Did you maybe forget this after debugging?

        $view = 'examples/list.html';

        // Get erdiko config, this gets application.json and loads the theme specified
        // $themeData = \erdiko\theme\Config::get();

46% of this comment could be valid code. Did you maybe forget this after debugging?

        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        // $themeData['args'] = $args; // optional

55% of this comment could be valid code. Did you maybe forget this after debugging?

        $themeData['page'] = [
            'title' => "Erdiko Web Example",
            'hello' => "world"
            ];

        return $this->container->theme->render($response, $view, $themeData);
    }

    public function getOnecolumn($request, $response, $args)

This method seems to be duplicated in your project.

    {
        $this->container->logger->debug("route: /config");
        $view = 'layouts/1column.html';
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        $themeData['page']['title'] = "1 Column Layout";

        return $this->container->theme->render($response, $view, $themeData);
    }

    public function getTwocolumn($request, $response, $args)

This method seems to be duplicated in your project.

    {
        $this->container->logger->debug("route: /config");
        $view = 'layouts/2column.html';
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        $themeData['page']['title'] = "2 Column Layout";

        return $this->container->theme->render($response, $view, $themeData);
    }

    public function getThreecolumn($request, $response, $args)

This method seems to be duplicated in your project.

    {
        $this->container->logger->debug("route: /config");
        $view = 'layouts/3column.html';
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        $themeData['page']['title'] = "3 Column Layout";

        return $this->container->theme->render($response, $view, $themeData);
    }

    public function getJohn($request, $response, $args)

This method seems to be duplicated in your project.

    {
        $view = 'pages/example.html';

        // Get erdiko config, this gets application.json and loads the theme specified
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        $themeData['args'] = $args; // optional
        $themeData['page'] = [
            'title' => "Erdiko Web Example"
            ];

        return $this->container->theme->render($response, $view, $themeData);
    }

    public function getMarkup($request, $response, $args)

This method seems to be duplicated in your project.

    {
        $view = 'examples/markup.html';

        // Get erdiko config, this gets application.json and loads the theme specified
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        $themeData['args'] = $args; // optional
        $themeData['page'] = [
            'title' => "Markup Example"
            ];

        return $this->container->theme->render($response, $view, $themeData);
    }

    public function getCarousel($request, $response, $args)

This method seems to be duplicated in your project.

    {
        $view = 'examples/carousel.html';

        // Get erdiko config, this gets application.json and loads the theme specified
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        $themeData['page'] = [
            'title' => "Fullpage Example",
            'description' => "This is the description of the page."
            ];

        return $this->container->theme->render($response, $view, $themeData);
    }

    /**
     * Leverage the theme trait to easily add content to an action
     */
    public function getTheme($request, $response, $args)
    {
        $this->getThemeEngine();
        $this->theme->title = "Theme Engine Example";
        $this->theme->description = "This page is rendered using the erdiko theme engine.
            \\erdiko\\theme\\Engine";

        return $this->render($response, null, $theme);

The variable $theme does not exist. Did you forget to declare it?

    }

    /**
     * Alternative way to use the theme Engine (explicit)
     * In this approach it does not rely on the theme trait
     */
    public function getTheme2($request, $response, $args)
    {
        $theme = new \erdiko\theme\Engine( $this->container->get('settings')['theme'] );

The call to Engine::__construct() has too many arguments starting with $this->container->get('settings')['theme'].

        $theme->title = "Theme Engine Example";
        $theme->description = "This page is rendered with the erdiko theme engine.
            It does not leverage the erdiko theme trait.
            \\erdiko\\theme\\Engine";

        return $this->container->theme->render($response, $theme->getDefaultView(), $theme->toArray());
        // return $this->render($response, null, $theme);

69% of this comment could be valid code. Did you maybe forget this after debugging?

    }

    public function getFlash($request, $response, $args)
    {
        $view = 'page.html';

        // Add some flash messages
        $this->container->flash->addMessage('success', 'This is a success message');
        $this->container->flash->addMessage('info', 'This is an info message');
        $this->container->flash->addMessage('warning', 'This is a warning message');
        $this->container->flash->addMessage('danger', 'This is a danger (error) message');

        // Get erdiko config, this gets application.json and loads the theme specified
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        $themeData['args'] = $args;
        $themeData['page'] = [
            'title' => "Flash Message Example"
            ];

        return $this->container->theme->render($response, $view, $themeData);
    }

    public function getGrid($request, $response, $args)
    {
        $this->container->logger->debug("/controller");
        $view = 'examples/grid.html';

        // Get erdiko config, this gets application.json and loads the theme specified
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        // $this->container->logger->debug("config: ".print_r($config, true));

65% of this comment could be valid code. Did you maybe forget this after debugging?

        // Generate data for grid
        $item = [
            'url' => "#",
            'image' => "/images/grid-item.png",
            'name' => "Item"
        ];
        $items = array();
        $max = (int)$args['param'];
        $this->container->logger->debug("param: ".$max);

        for($i = 0; $i < $max; $i++) {

This code seems to be duplicated across your project.

            $item['name'] = "Item $i";
            $items[] = $item;
        }

        $themeData['page'] = [
            'title' => "Grid Example",
            'items' => $items
            ];

        return $this->container->theme->render($response, $view, $themeData);
    }

    public function getFullpage($request, $response, $args)

This method seems to be duplicated in your project.

    {
        $view = 'fullpage.html';

        // Get erdiko config, this gets application.json and loads the theme specified
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

$themeData was never initialized. Although not strictly required by PHP, it is generally a good practice to add $themeData = array(); before regardless.

        $themeData['page'] = [
            'title' => "Fullpage Example",
            'description' => "This is the description of the page."
            ];

        return $this->container->theme->render($response, $view, $themeData);
    }

    public function getException($request, $response, $args)

This method seems to be duplicated in your project.

    {
        $view = 'page.html';

$view is not used, you could remove the assignment.

        throw new \Exception("This is an exception");

        // Get erdiko config, this gets application.json and loads the theme specified
        $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']);

// Get erdiko config, this gets application.json and loads the theme specified $themeData['theme'] = \erdiko\theme\Config::get($this->container->get('settings')['theme']); does not seem to be reachable.

        $themeData['page'] = [
            'title' => "Fullpage Example",
            'description' => "This is the description of the page."
            ];

        return $this->container->theme->render($response, $view, $themeData);
    }
}