keeko\core\package\AbstractResponse

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	2
A	getServiceContainer()	0	3	1
A	setData()	0	3	1
A	getTwig()	0	3	1
	run()	0	1	?

<?php
namespace keeko\core\package;

use Symfony\Component\HttpFoundation\Request;
use keeko\core\utils\TwigRenderTrait;

abstract class AbstractResponse {
	
	use TwigRenderTrait;

	protected $data = [];

	protected $twig;

	protected $module;

	public function __construct(AbstractModule $module, $format) {
		$this->module = $module;
		$templatePath = sprintf('%s/%s/templates/%s', KEEKO_PATH_MODULES, $module->getModel()->getName(), $format);
		
		if (file_exists($templatePath)) {
			$loader = new \Twig_Loader_Filesystem($templatePath);

Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space

			$this->twig = new \Twig_Environment($loader);
		}
	}

	/**
	 * Returns the service container
	 *
	 * @return ServiceContainer
	 */
	protected function getServiceContainer() {
		return $this->module->getServiceContainer();
	}

	public function setData($data) {
		$this->data = $data;
	}
	
	protected function getTwig() {

The return type could not be reliably inferred; please add a @return annotation.

		return $this->module->getTwig();
	}

	abstract public function run(Request $request, $data = null);

For interfaces and abstract methods it is generally a good practice to add a @return annotation even if it is just @return void or @return null, so that implementors know what to do in the overridden method.

}