blitze\content\services\actions\topic\view

Complexity

Total Complexity

3

Size/Duplication

Total Lines	76
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

<?php
class view implements action_interface
{
	/** @var \phpbb\request\request_interface */
	protected $request;

	/** @var \phpbb\template\template */
	protected $template;

	/** @var \blitze\content\services\types */
	protected $content_types;

	/** @var \blitze\content\services\views\views_factory */
	protected $views;

	/** @var string */
	protected $phpbb_root_path;

	/** @var string */
	protected $php_ext;

	/**
	 * Constructor
	 *
	 * @param \phpbb\request\request_interface				$request				Request object
	 * @param \phpbb\template\template						$template				Template object
	 * @param \blitze\content\services\types				$content_types			Content types object
	 * @param \blitze\content\services\views\views_factory	$views					Views factory object
	 * @param string										$phpbb_root_path		Path to the phpbb includes directory.
	 * @param string										$php_ext				php file extension
	 */
	public function __construct(\phpbb\request\request_interface $request, \phpbb\template\template $template, \blitze\content\services\types $content_types, \blitze\content\services\views\views_factory $views, $phpbb_root_path, $php_ext)
	{
		$this->request = $request;
		$this->template = $template;
		$this->content_types = $content_types;
		$this->views = $views;
		$this->phpbb_root_path = $phpbb_root_path;
		$this->php_ext = $php_ext;
	}

	/**
	 * @inheritdoc
	 */
	public function execute($u_action, $mode = '')
	{
		$topic_id = $this->request->variable('t', 0);
		$type = $this->request->variable('type', '');

		$view_tpl = '';
		if ($entity = $this->content_types->get_type($type)

Avoid IF statements that are always true or false

		{
			$entity->set_show_poster_info(false);

This code did not parse for me. Apparently, there is an error somewhere around this line:

Syntax error, unexpected ';'

			$entity->set_show_poster_contents(false);
			$entity->set_allow_comments(false);

			$update_count = array();
			$overwrite = $this->get_data_overwrite($mode, $u_action, $type, $topic_id);

			/** @var \blitze\content\services\views\driver\views_interface $view_handler */
			$view_handler = $this->views->get($entity->get_content_view());
			$view_handler->render_detail($entity, $topic_id, $update_count, $overwrite);
			$view_tpl = $view_handler->get_detail_template();
		}

		$this->template->assign_vars(array(
			'MODE'				=> $mode,
			'S_HIDE_HEADERS'	=> true,
			'S_VIEWING'			=> $view_tpl,
		));
	}

	/**
	 * Overwrite template data
	 *
	 * @param string $mode
	 * @param string $u_action
	 * @param string $type