query_builder::fetch_topic_poster() - Code Metrics - Inspection of "Little corrections" - blitze/phpBB-ext-sitemaker - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( c767cb...92ad0e )

by Daniel

created 2016-04-25 04:46 UTC

query_builder::fetch_topic_poster() A

↳ Parent: query_builder

Complexity

Conditions	1
Paths	1

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	3
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	6
ccs	3
cts	3
cp	1
rs	9.4285
cc	1
eloc	3
nc	1
nop	1
crap	1

<?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\sitemaker\services\forum;

class query_builder
{
	/** @var \phpbb\auth\auth */
	protected $auth;

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

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

	/** @var \phpbb\db\driver\driver_interface */
	protected $db;

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

	/** @var array */
	protected $store;

	/** @var array */
	protected $ex_fid_ary;

	/** @var integer */
	protected $cache_time;

	/**
	 * Constructor
	 *
	 * @param \phpbb\auth\auth					$auth					Auth object
	 * @param \phpbb\config\config				$config					Config object
	 * @param \phpbb\content_visibility			$content_visibility		Content visibility
	 * @param \phpbb\db\driver\driver_interface	$db     				Database connection
	 * @param \phpbb\user						$user					User object
	 * @param integer							$cache_time				Cache results for 3 hours by default
	 */
	public function __construct(\phpbb\auth\auth $auth, \phpbb\config\config $config, \phpbb\content_visibility $content_visibility, \phpbb\db\driver\driver_interface $db, \phpbb\user $user, $cache_time = 10800)
	{
		$this->auth = $auth;
		$this->config = $config;
		$this->content_visibility = $content_visibility;
		$this->db = $db;
		$this->user = $user;
		$this->cache_time = $cache_time;

		$this->ex_fid_ary = array_unique(array_keys($this->auth->acl_getf('!f_read', true)));
	}

	/**
	 * Begin query
	 *
	 * @param bool $track_topics
	 * @return $this
	 */
	public function query($track_topics = true)
	{
		$this->_reset();

		$this->store['sql_array'] = array(
			'SELECT'	=> array('t.*, f.*'),

			'FROM'		=> array(FORUMS_TABLE => 'f'),

			'LEFT_JOIN'	=> array(),

			'WHERE'		=> array(),
		);

		// Topics table need to be the last in the chain
		$this->store['sql_array']['FROM'][TOPICS_TABLE] = 't';

		if ($track_topics)
		{
			$this->fetch_tracking_info();
		}

		return $this;
	}

	/**
	 * Fetch Forum by id(s)
	 *
	 * @param $forum_id
	 * @return $this
	 */
	public function fetch_forum($forum_id)
	{
		$this->_fetch($forum_id, 'f.forum_id');

		return $this;
	}

	/**
	 * Fetch Topic by id(s)
	 *
	 * @param mixed $topic_id	Limit by topic id: single id or array of topic ids
	 * @return $this
	 */
	public function fetch_topic($topic_id)
	{
		$this->_fetch($topic_id, 't.topic_id');

		return $this;
	}

	/**
	 * Fetch Topic by Poster id(s)
	 *
	 * @param mixed $user_id	User id of topic poster: single id or array of user ids
	 * @return $this
	 */
	public function fetch_topic_poster($user_id)
	{
		$this->_fetch($user_id, 't.topic_poster');

		return $this;
	}

	/**
	 * Fetch by Topic Type
	 *
	 * @param array $topic_type
	 * @return $this
	 */
	public function fetch_topic_type(array $topic_type)
	{
		if (sizeof($topic_type))
		{
			$this->store['sql_array']['WHERE'][] = $this->db->sql_in_set('t.topic_type', $topic_type);
		}

		return $this;
	}

	/**
	 * Fetch Topic Watch info
	 *
	 * @return $this
	 *
	public function fetch_watch_status()
	{
		if ($this->user->data['is_registered'])
		{
			$this->store['sql_array']['SELECT'][] = 'tw.notify_status';
			$this->store['sql_array']['LEFT_JOIN'][] = array(
				'FROM'	=> array(TOPICS_WATCH_TABLE => 'tw'),
				'ON'	=> 'tw.user_id = ' . $this->user->data['user_id'] . ' AND t.topic_id = tw.topic_id'
			);

			$this->store['sql_array']['SELECT'][] = 'fw.notify_status';
			$this->store['sql_array']['LEFT JOIN'][] = array(
				'FROM'	=> array(FORUMS_WATCH_TABLE => 'fw'),
				'ON'	=> '(fw.forum_id = f.forum_id AND fw.user_id = ' . $this->user->data['user_id'] . ')',
			);
		}

		return $this;
	}

	/**
	 * Fetch Topic Bookmark Info
	 *
	 * @return $this
	 *
	public function fetch_bookmark_status()
	{
		if ($this->user->data['is_registered'] && $this->config['allow_bookmarks'])
		{
			$this->store['sql_array']['SELECT'][] = 'bm.topic_id as bookmarked';
			$this->store['sql_array']['LEFT_JOIN'][] = array(
				'FROM'	=> array(BOOKMARKS_TABLE => 'bm'),
				'ON'	=> 'bm.user_id = ' . $this->user->data['user_id'] . ' AND t.topic_id = bm.topic_id'
			);
		}

		return $this;
	}
	*/

	/**
	 * Fetch Topic Tracking Info
	 *
	 * @return $this
	 */
	public function fetch_tracking_info()
	{
		if ($this->user->data['is_registered'] && $this->config['load_db_lastread'])
		{
			$this->cache_time = 0;

			$this->store['sql_array']['SELECT'][] = 'tt.mark_time, ft.mark_time as forum_mark_time';
			$this->store['sql_array']['LEFT_JOIN'][] = array(
				'FROM'	=> array(TOPICS_TRACK_TABLE => 'tt'),
				'ON'	=> 'tt.user_id = ' . $this->user->data['user_id'] . ' AND t.topic_id = tt.topic_id'
			);

			$this->store['sql_array']['LEFT_JOIN'][] = array(
				'FROM'	=> array(FORUMS_TRACK_TABLE => 'ft'),
				'ON'	=> 'ft.user_id = ' . $this->user->data['user_id'] . ' AND t.forum_id = ft.forum_id'
			);
		}

		return $this;
	}

	/**
	 * Fetch by Date Range
	 *
	 * @param int $unix_start_time
	 * @param int $unix_stop_time
	 * @param string $mode
	 * @return $this
	 */
	public function fetch_date_range($unix_start_time, $unix_stop_time, $mode = 'topic')
	{
		if ($unix_start_time && $unix_stop_time)
		{
			$this->store['sql_array']['WHERE'][] = (($mode == 'topic') ? 't.topic_time' : 'p.post_time') . " BETWEEN $unix_start_time AND $unix_stop_time";
		}

		return $this;
	}

	/**
	 * Fetch by Custom Query
	 *
	 * @param array	$sql_array		Array of elements to merge into query
	 * 										array(
	 * 											'SELECT'	=> array('p.*'),
	 * 											'WHERE'		=> array('p.post_id = 2'),
	 * 										)
	 * @return $this
	 */
	public function fetch_custom(array $sql_array)
	{
		$this->store['sql_array'] = array_merge_recursive($this->store['sql_array'], $sql_array);

		return $this;
	}

	/**
	 * Set Sorting Order
	 *
	 * @param string $sort_key		The sorting key e.g. t.topic_time
	 * @param string $sort_dir		Sort direction: ASC/DESC
	 * @return $this
	 */
	public function set_sorting($sort_key, $sort_dir = 'DESC')
	{
		$this->store['sql_array']['ORDER_BY'] = $sort_key . ' ' . $sort_dir;

		return $this;
	}

	/**
	 * Build the query
	 *
	 * @param bool|true $check_visibility		Should we only return data from forums the user is allowed to see?
	 * @param bool|true $enable_caching			Should the query be cached where possible?
	 * @param bool|true $exclude_hidden_forums	Leave out hidden forums?
	 * @return $this
	 */
	public function build($check_visibility = true, $enable_caching = true, $exclude_hidden_forums = true)
	{
		$this->_set_cache_time($enable_caching);
		$this->_set_topic_visibility($check_visibility);

		if ($exclude_hidden_forums)
		{
			$this->store['sql_array']['WHERE'][] = 'f.hidden_forum = 0';
		}

		$this->store['sql_array']['WHERE'][] = 'f.forum_id = t.forum_id';
		$this->store['sql_array']['WHERE'][] = 't.topic_moved_id = 0';

		$this->store['sql_array']['SELECT'] = join(', ', array_filter($this->store['sql_array']['SELECT']));
		$this->store['sql_array']['WHERE'] = join(' AND ', array_filter($this->store['sql_array']['WHERE']));

		return $this;
	}

	/**
	 * Get the query array
	 *
	 * @return array	The sql array that can be used with sql_build_query
	 */
	public function get_sql_array()
	{
		return $this->store['sql_array'];
	}

	/**
	 * @param bool $enable_caching
	 */
	protected function _set_cache_time($enable_caching)
	{
		if ($enable_caching === false)
		{
			$this->cache_time = 0;
		}
	}

	/**
	 * @param int $column_id
	 * @param string $column
	 */
	private function _fetch($column_id, $column)
	{
		if (!empty($column_id))
		{
			$this->store['sql_array']['WHERE'][] = (is_array($column_id)) ? $this->db->sql_in_set($column, $column_id) : $column . ' = ' . (int) $column_id;
		}
	}

	/**
	 * @param bool $check_visibility
	 */
	private function _set_topic_visibility($check_visibility)
	{
		if ($check_visibility)
		{
			$this->store['sql_array']['WHERE'][] = 't.topic_time <= ' . time();
			$this->store['sql_array']['WHERE'][] = $this->content_visibility->get_global_visibility_sql('topic', $this->ex_fid_ary, 't.');
		}
	}

	/**
	 * Reset data
	 */
	private function _reset()
	{
		$this->store = array(
			'attachments'	=> array(),
			'post_ids'		=> array(),
			'poster_ids'	=> array(),
			'sql_array'		=> array(),
			'topic'			=> array(),
			'tracking'		=> array(),
		);
	}
}


1		<?php
2		/**
3		*
4		* @package sitemaker
5		* @copyright (c) 2013 Daniel A. (blitze)
6		* @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
7		*
8		*/
9
10		namespace blitze\sitemaker\services\forum;
11
12		class query_builder
13		{
14		/** @var \phpbb\auth\auth */
15		protected $auth;
16
17		/** @var \phpbb\config\config */
18		protected $config;
19
20		/** @var \phpbb\content_visibility */
21		protected $content_visibility;
22
23		/** @var \phpbb\db\driver\driver_interface */
24		protected $db;
25
26		/** @var \phpbb\user */
27		protected $user;
28
29		/** @var array */
30		protected $store;
31
32		/** @var array */
33		protected $ex_fid_ary;
34
35		/** @var integer */
36		protected $cache_time;
37
38		/**
39		* Constructor
40		*
41		* @param \phpbb\auth\auth $auth Auth object
42		* @param \phpbb\config\config $config Config object
43		* @param \phpbb\content_visibility $content_visibility Content visibility
44		* @param \phpbb\db\driver\driver_interface $db Database connection
45		* @param \phpbb\user $user User object
46		* @param integer $cache_time Cache results for 3 hours by default
47		*/
48	29	public function __construct(\phpbb\auth\auth $auth, \phpbb\config\config $config, \phpbb\content_visibility $content_visibility, \phpbb\db\driver\driver_interface $db, \phpbb\user $user, $cache_time = 10800)
49		{
50	29	$this->auth = $auth;
51	29	$this->config = $config;
52	29	$this->content_visibility = $content_visibility;
53	29	$this->db = $db;
54	29	$this->user = $user;
55	29	$this->cache_time = $cache_time;
56
57	29	$this->ex_fid_ary = array_unique(array_keys($this->auth->acl_getf('!f_read', true)));
58	29	}
59
60		/**
61		* Begin query
62		*
63		* @param bool $track_topics
64		* @return $this
65		*/
66	20	public function query($track_topics = true)
67		{
68	20	$this->_reset();
69
70	20	$this->store['sql_array'] = array(
71	20	'SELECT' => array('t., f.'),
72
73	20	'FROM' => array(FORUMS_TABLE => 'f'),
74
75	20	'LEFT_JOIN' => array(),
76
77	20	'WHERE' => array(),
78		);
79
80		// Topics table need to be the last in the chain
81	20	$this->store['sql_array']['FROM'][TOPICS_TABLE] = 't';
82
83		if ($track_topics)
84	20	{
85	16	$this->fetch_tracking_info();
86	16	}
87
88	20	return $this;
89		}
90
91		/**
92		* Fetch Forum by id(s)
93		*
94		* @param $forum_id
95		* @return $this
96		*/
97	15	public function fetch_forum($forum_id)
98		{
99	15	$this->_fetch($forum_id, 'f.forum_id');
100
101	15	return $this;
102		}
103
104		/**
105		* Fetch Topic by id(s)
106		*
107		* @param mixed $topic_id Limit by topic id: single id or array of topic ids
108		* @return $this
109		*/
110	3	public function fetch_topic($topic_id)
111		{
112	3	$this->_fetch($topic_id, 't.topic_id');
113
114	3	return $this;
115		}
116
117		/**
118		* Fetch Topic by Poster id(s)
119		*
120		* @param mixed $user_id User id of topic poster: single id or array of user ids
121		* @return $this
122		*/
123	3	public function fetch_topic_poster($user_id)
124		{
125	3	$this->_fetch($user_id, 't.topic_poster');
126
127	3	return $this;
128		}
129
130		/**
131		* Fetch by Topic Type
132		*
133		* @param array $topic_type
134		* @return $this
135		*/
136	15	public function fetch_topic_type(array $topic_type)
137		{
138	15	if (sizeof($topic_type))
139	15	{
140	5	$this->store['sql_array']['WHERE'][] = $this->db->sql_in_set('t.topic_type', $topic_type);
141	5	}
142
143	15	return $this;
144		}
145
146		/**
147		* Fetch Topic Watch info
148		*
149		* @return $this
150		*
151		public function fetch_watch_status()
152		{
153		if ($this->user->data['is_registered'])
154		{
155		$this->store['sql_array']['SELECT'][] = 'tw.notify_status';
156		$this->store['sql_array']['LEFT_JOIN'][] = array(
157		'FROM' => array(TOPICS_WATCH_TABLE => 'tw'),
158		'ON' => 'tw.user_id = ' . $this->user->data['user_id'] . ' AND t.topic_id = tw.topic_id'
159		);
160
161		$this->store['sql_array']['SELECT'][] = 'fw.notify_status';
162		$this->store['sql_array']['LEFT JOIN'][] = array(
163		'FROM' => array(FORUMS_WATCH_TABLE => 'fw'),
164		'ON' => '(fw.forum_id = f.forum_id AND fw.user_id = ' . $this->user->data['user_id'] . ')',
165		);
166		}
167
168		return $this;
169		}
170
171		/**
172		* Fetch Topic Bookmark Info
173		*
174		* @return $this
175		*
176		public function fetch_bookmark_status()
177		{
178		if ($this->user->data['is_registered'] && $this->config['allow_bookmarks'])
179		{
180		$this->store['sql_array']['SELECT'][] = 'bm.topic_id as bookmarked';
181		$this->store['sql_array']['LEFT_JOIN'][] = array(
182		'FROM' => array(BOOKMARKS_TABLE => 'bm'),
183		'ON' => 'bm.user_id = ' . $this->user->data['user_id'] . ' AND t.topic_id = bm.topic_id'
184		);
185		}
186
187		return $this;
188		}
189		*/
190
191		/**
192		* Fetch Topic Tracking Info
193		*
194		* @return $this
195		*/
196	16	public function fetch_tracking_info()
197		{
198	16	if ($this->user->data['is_registered'] && $this->config['load_db_lastread'])
199	16	{
200	5	$this->cache_time = 0;
201
202	5	$this->store['sql_array']['SELECT'][] = 'tt.mark_time, ft.mark_time as forum_mark_time';
203	5	$this->store['sql_array']['LEFT_JOIN'][] = array(
204	5	'FROM' => array(TOPICS_TRACK_TABLE => 'tt'),
205	5	'ON' => 'tt.user_id = ' . $this->user->data['user_id'] . ' AND t.topic_id = tt.topic_id'
206	5	);
207
208	5	$this->store['sql_array']['LEFT_JOIN'][] = array(
209	5	'FROM' => array(FORUMS_TRACK_TABLE => 'ft'),
210	5	'ON' => 'ft.user_id = ' . $this->user->data['user_id'] . ' AND t.forum_id = ft.forum_id'
211	5	);
212	5	}
213
214	16	return $this;
215		}
216
217		/**
218		* Fetch by Date Range
219		*
220		* @param int $unix_start_time
221		* @param int $unix_stop_time
222		* @param string $mode
223		* @return $this
224		*/
225	12	public function fetch_date_range($unix_start_time, $unix_stop_time, $mode = 'topic')
226		{
227	12	if ($unix_start_time && $unix_stop_time)
228	12	{
229	2	$this->store['sql_array']['WHERE'][] = (($mode == 'topic') ? 't.topic_time' : 'p.post_time') . " BETWEEN $unix_start_time AND $unix_stop_time";
230	2	}
231
232	12	return $this;
233		}
234
235		/**
236		* Fetch by Custom Query
237		*
238		* @param array $sql_array Array of elements to merge into query
239		* array(
240		* 'SELECT' => array('p.*'),
241		* 'WHERE' => array('p.post_id = 2'),
242		* )
243		* @return $this
244		*/
245	8	public function fetch_custom(array $sql_array)
246		{
247	8	$this->store['sql_array'] = array_merge_recursive($this->store['sql_array'], $sql_array);
248
249	8	return $this;
250		}
251
252		/**
253		* Set Sorting Order
254		*
255		* @param string $sort_key The sorting key e.g. t.topic_time
256		* @param string $sort_dir Sort direction: ASC/DESC
257		* @return $this
258		*/
259	12	public function set_sorting($sort_key, $sort_dir = 'DESC')
260		{
261	12	$this->store['sql_array']['ORDER_BY'] = $sort_key . ' ' . $sort_dir;
262
263	12	return $this;
264		}
265
266		/**
267		* Build the query
268		*
269		* @param bool\|true $check_visibility Should we only return data from forums the user is allowed to see?
270		* @param bool\|true $enable_caching Should the query be cached where possible?
271		* @param bool\|true $exclude_hidden_forums Leave out hidden forums?
272		* @return $this
273		*/
274	20	public function build($check_visibility = true, $enable_caching = true, $exclude_hidden_forums = true)
275		{
276	20	$this->_set_cache_time($enable_caching);
277	20	$this->_set_topic_visibility($check_visibility);
278
279		if ($exclude_hidden_forums)
280	20	{
281	20	$this->store['sql_array']['WHERE'][] = 'f.hidden_forum = 0';
282	20	}
283
284	20	$this->store['sql_array']['WHERE'][] = 'f.forum_id = t.forum_id';
285	20	$this->store['sql_array']['WHERE'][] = 't.topic_moved_id = 0';
286
287	20	$this->store['sql_array']['SELECT'] = join(', ', array_filter($this->store['sql_array']['SELECT']));
288	20	$this->store['sql_array']['WHERE'] = join(' AND ', array_filter($this->store['sql_array']['WHERE']));
289
290	20	return $this;
291		}
292
293		/**
294		* Get the query array
295		*
296		* @return array The sql array that can be used with sql_build_query
297		*/
298	7	public function get_sql_array()
299		{
300	7	return $this->store['sql_array'];
301		}
302
303		/**
304		* @param bool $enable_caching
305		*/
306	20	protected function _set_cache_time($enable_caching)
307		{
308	20	if ($enable_caching === false)
309	20	{
310	5	$this->cache_time = 0;
311	5	}
312	20	}
313
314		/**
315		* @param int $column_id
316		* @param string $column
317		*/
318	15	private function _fetch($column_id, $column)
319		{
320	15	if (!empty($column_id))
321	15	{
322	10	$this->store['sql_array']['WHERE'][] = (is_array($column_id)) ? $this->db->sql_in_set($column, $column_id) : $column . ' = ' . (int) $column_id;
323	10	}
324	15	}
325
326		/**
327		* @param bool $check_visibility
328		*/
329	20	private function _set_topic_visibility($check_visibility)
330		{
331		if ($check_visibility)
332	20	{
333	20	$this->store['sql_array']['WHERE'][] = 't.topic_time <= ' . time();
334	20	$this->store['sql_array']['WHERE'][] = $this->content_visibility->get_global_visibility_sql('topic', $this->ex_fid_ary, 't.');
335	20	}
336	20	}
337
338		/**
339		* Reset data
340		*/
341	20	private function _reset()
342		{
343	20	$this->store = array(
344	20	'attachments' => array(),
345	20	'post_ids' => array(),
346	20	'poster_ids' => array(),
347	20	'sql_array' => array(),
348	20	'topic' => array(),
349	20	'tracking' => array(),
350		);
351	20	}
352		}
353

blitze / phpBB-ext-sitemaker

Push — develop ( c767cb...92ad0e )

query_builder::fetch_topic_poster() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like