blitze\content\event\mcp_topic

Complexity

Total Complexity

14

Size/Duplication

Total Lines	135
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

0%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	getSubscribedEvents()	0	11	1
A	modify_post_data()	0	6	1
A	modify_review_row()	0	14	4
A	modify_forum_sql()	0	9	2
A	modify_topic_queue_sql()	0	15	2
A	modify_forumlist()	0	4	1
A	get_forum_list()	0	7	2

<?php
/**
 *
 * @package sitemaker
 * @copyright (c) 2013 Daniel A. (blitze)
 * @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
 *
 */

namespace blitze\content\event;

use Symfony\Component\EventDispatcher\EventSubscriberInterface;

class mcp_topic implements EventSubscriberInterface
{
	/** @var \phpbb\db\driver\driver_interface */
	protected $db;

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

	/* @var \blitze\content\services\fields */
	protected $fields;

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

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

	/**
	 * Constructor
	 *
	 * @param \phpbb\db\driver\driver_interface		$db					Database object
	 * @param \blitze\content\services\types		$content_types		Content types object
	 * @param \blitze\content\services\fields		$fields				Content fields object
	 * @param string								$phpbb_root_path	Path to the phpbb includes directory.
	 * @param string								$php_ext			php file extension
	*/
	public function __construct(\phpbb\db\driver\driver_interface $db, \blitze\content\services\types $content_types, \blitze\content\services\fields $fields, $phpbb_root_path, $php_ext)
	{
		$this->db = $db;
		$this->content_types = $content_types;
		$this->fields = $fields;
		$this->phpbb_root_path = $phpbb_root_path;
		$this->php_ext = $php_ext;
	}

	/**
	 * @return array
	 */
	static public function getSubscribedEvents()

As per PSR2, the static declaration should come after the visibility declaration.

	{
		return array(
			'core.mcp_topic_modify_post_data'	=> 'modify_post_data',
			'core.mcp_topic_review_modify_row'	=> 'modify_review_row',
			'core.mcp_view_forum_modify_sql'	=> 'modify_forum_sql',
			'core.mcp_queue_get_posts_for_topics_query_before'	=> 'modify_topic_queue_sql',
			'core.mcp_front_queue_unapproved_total_before'		=> 'modify_forumlist',
			'core.mcp_front_view_queue_postid_list_after'		=> 'modify_forumlist',
		);
	}

	/**
	 * @param \phpbb\event\data $event
	 * @return void
	 */
	public function modify_post_data(\phpbb\event\data $event)
	{
		// We use $event['rowset'][0]['forum_id'] instead of $event['forum_id'] because forum_id is not guaranteed

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

		// as it is only retrieved from url parameters on topic view but links from pagination does not include forumid
		$this->type = $this->content_types->get_forum_type($event['rowset'][0]['forum_id']);
	}

	/**
	 * @param \phpbb\event\data $event
	 * @return void
	 */
	public function modify_review_row(\phpbb\event\data $event)
	{
		if ($event['row']['post_id'] === $event['topic_info']['topic_first_post_id'] && $this->type)
		{
			$entity = $this->content_types->get_type($this->type);
			$this->fields->prepare_to_show($this->type, 'summary', $entity->get_summary_tags(), $entity->get_content_fields(), $entity->get_summary_tpl());

'summary' is of type string, but the function expects a array.

			$post_row = $event['post_row'];
			$content = $this->fields->build_content($post_row);
			$post_row['MESSAGE'] = isset($content['SEQ_DISPLAY']) ? $content['SEQ_DISPLAY'] : $content['CUSTOM_DISPLAY'];

			$event['post_row'] = $post_row;
		}
	}

	/**
	 * @param \phpbb\event\data $event
	 * @return void
	 */
	public function modify_forum_sql(\phpbb\event\data $event)
	{
		$types = $this->content_types->get_forum_types();

		if ($types[$event['forum_id']])
		{
			redirect(append_sid("{$this->phpbb_root_path}mcp.$this->php_ext"));
		}
	}

	/**
	 * @param \phpbb\event\data $event
	 * @return void
	 */
	public function modify_topic_queue_sql(\phpbb\event\data $event)
	{
		$forum_list = $event['forum_list'];
		$forum_list = is_array($forum_list) ? $forum_list : array($forum_list);

		$sql = 'SELECT t.forum_id, t.topic_id, t.topic_title, t.topic_title AS post_subject, t.topic_time AS post_time, t.topic_poster AS poster_id, t.topic_first_post_id AS post_id, t.topic_attachment AS post_attachment, t.topic_first_poster_name AS username, t.topic_first_poster_colour AS user_colour
			FROM ' . TOPICS_TABLE . ' t
			WHERE ' . $this->db->sql_in_set('forum_id', $this->get_forum_list($forum_list)) . '
				AND  ' . $this->db->sql_in_set('topic_visibility', $event['visibility_const']) . "
				AND topic_delete_user <> 0
				{$event['limit_time_sql']}
			ORDER BY {$event['sort_order_sql']}";

		$event['sql'] = $sql;
	}

	/**
	 * @param \phpbb\event\data $event
	 * @return void
	 */
	public function modify_forumlist(\phpbb\event\data $event)
	{
		$event['forum_list'] = $this->get_forum_list($event['forum_list']);
	}

	/**
	 * @param array
	 * @return array
	 */
	protected function get_forum_list(array $forum_list)
	{
		$types = $this->content_types->get_forum_types();
		$forum_ids = array_diff($forum_list, array_keys($types));

		return sizeof($forum_ids) ? $forum_ids : array(0);
	}
}