Charcoal\App\Template\AbstractTemplate

Complexity

Total Complexity

9

Size/Duplication

Total Lines	84
Duplicated Lines	11.9 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes	1
Bugs	1	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	set_app()	0	5	1
A	app()	0	4	1
A	setLogger()	0	4	1
A	set_logger()	0	5	1
A	logger()	0	4	1
A	create_view()	10	10	2

<?php

namespace Charcoal\App\Template;

// slim/slim dependencies
use \Slim\App as SlimApp;

// PSR-3 logger
use \Psr\Log\LoggerInterface;
use \Psr\Log\LoggerAwareInterface;

// Module `charcoal-view` dependencies
use \Charcoal\View\GenericView;
use \Charcoal\View\ViewableInterface;
use \Charcoal\View\ViewableTrait;

// Local namespace dependencies
use \Charcoal\App\Template\TemplateInterface;

/**
*
*/
abstract class AbstractTemplate implements
    LoggerAwareInterface,
    TemplateInterface,
    ViewableInterface
{

    use ViewableTrait;

    /**
    * @var LoggerInterface $logger
    */
    private $logger;

    public function __construct(array $data = null)
    {
        if (isset($data['logger'])) {
            $this->set_logger($data['logger']);
        }

        $this->set_app($data['app']);
    }

    /**
    * @param SlimApp $app
    * @return App Chainable
    */
    public function set_app(SlimApp $app)
    {
        $this->app = $app;

The property app does not exist. Did you maybe forget to declare it?

        return $this;
    }

    public function app($app)

The parameter $app is not used and could be removed.

    {
        return $this->app;
    }

    /**
    * > LoggerAwareInterface > setLogger()
    *
    * Fulfills the PSR-1 style LoggerAwareInterface
    *
    * @param LoggerInterface $logger
    * @return AbstractEngine Chainable
    */
    public function setLogger(LoggerInterface $logger)
    {
        return $this->set_logger($logger);
    }

    /**
    * @param LoggerInterface $logger
    * @return AbstractEngine Chainable
    */
    public function set_logger(LoggerInterface $logger = null)
    {
        $this->logger = $logger;
        return $this;
    }

    /**
    * @erturn LoggerInterface
    */
    public function logger()
    {
        return $this->logger;
    }

    /**
    * The default Template View is a simple GenericView.
    *
    * @return \Charcoal\View\ViewInterface
    */
    public function create_view(array $data = null)

This method seems to be duplicated in your project.

    {
        $view = new GenericView([
            'logger'=>null
        ]);
        if ($data !== null) {
            $view->set_data($data);
        }
        return $view;
    }
}