admin_helper::prepare_ads_locations() - Code Metrics - Inspection of "ACP Ad Management Improvements" - phpbb-extensions/ad-management - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#74)

by Jakub

created 2017-08-12 15:38 UTC

admin_helper::prepare_ads_locations() A

↳ Parent: admin_helper

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

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

<?php
/**
 *
 * Advertisement management. An extension for the phpBB Forum Software package.
 *
 * @copyright (c) 2017 phpBB Limited <https://www.phpbb.com>
 * @license GNU General Public License, version 2 (GPL-2.0)
 *
 */

namespace phpbb\ads\controller;

use phpbb\ads\controller\admin_input as input;

/**
 * Admin helper
 */
class admin_helper
{
	/** @var \phpbb\user */
	protected $user;

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

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

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

	/** @var \phpbb\ads\location\manager */
	protected $location_manager;

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

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

	/**
	 * Constructor
	 *
	 * @param \phpbb\user						$user				User object
	 * @param \phpbb\language\language          $language           Language object
	 * @param \phpbb\template\template			$template			Template object
	 * @param \phpbb\log\log					$log				The phpBB log system
	 * @param \phpbb\ads\location\manager		$location_manager	Template location manager object
	 * @param string							$root_path			phpBB root path
	 * @param string							$php_ext			PHP extension
	 */
	public function __construct(\phpbb\user $user, \phpbb\language\language $language, \phpbb\template\template $template, \phpbb\log\log $log, \phpbb\ads\location\manager $location_manager, $root_path, $php_ext)
	{
		$this->user = $user;
		$this->language = $language;
		$this->template = $template;
		$this->log = $log;
		$this->location_manager = $location_manager;
		$this->root_path = $root_path;
		$this->php_ext = $php_ext;
	}

	/**
	 * Assign template locations data to the template.
	 *
	 * @param	mixed	$ad_locations	The form data or nothing.
	 * @return	void
	 */
	public function assign_locations($ad_locations = false)
	{
		foreach ($this->location_manager->get_all_locations() as $location_id => $location_data)
		{
			$this->template->assign_block_vars('ad_locations', array(
				'LOCATION_ID'   => $location_id,
				'LOCATION_DESC' => $location_data['desc'],
				'LOCATION_NAME' => $location_data['name'],
				'S_SELECTED'    => $ad_locations ? in_array($location_id, $ad_locations) : false,
			));
		}
	}

	/**
	 * Assign form data to the template.
	 *
	 * @param	array	$data	The form data.
	 * @return	void
	 */
	public function assign_form_data($data)
	{
		$this->template->assign_vars(array(
			'AD_NAME'         => $data['ad_name'],
			'AD_NOTE'         => $data['ad_note'],
			'AD_CODE'         => $data['ad_code'],
			'AD_ENABLED'      => $data['ad_enabled'],
			'AD_END_DATE'     => $this->prepare_end_date($data['ad_end_date']),
			'AD_PRIORITY'     => $data['ad_priority'],
			'AD_VIEWS_LIMIT'  => $data['ad_views_limit'],
			'AD_CLICKS_LIMIT' => $data['ad_clicks_limit'],
			'AD_OWNER'        => $this->prepare_ad_owner($data['ad_owner']),
		));
	}

	public function assign_errors(array $errors)
	{
		$errors = array_map(array($this->language, 'lang'), $errors);

		$this->template->assign_vars(array(
			'S_ERROR'   => (bool) count($errors),
			'ERROR_MSG' => count($errors) ? implode('<br />', $errors) : '',
		));
	}

	/**
	 * Log action
	 *
	 * @param	string	$action		Performed action in uppercase
	 * @param	string	$ad_name	Advertisement name
	 * @return	void
	 */
	public function log($action, $ad_name)
	{
		$this->log->add('admin', $this->user->data['user_id'], $this->user->ip, 'ACP_PHPBB_ADS_' . $action . '_LOG', time(), array($ad_name));
	}

	public function get_find_username_link()
	{
		return append_sid("{$this->root_path}memberlist.{$this->php_ext}", 'mode=searchuser&amp;form=acp_admanagement_add&amp;field=ad_owner&amp;select_single=true');
	}


	/**
	 * Prepare end date for display
	 *
	 * @param	mixed	$end_date	End date.
	 * @return	string	End date prepared for display.
	 */
	public function prepare_end_date($end_date)
	{
		if (empty($end_date))
		{
			return '';
		}

		if (is_numeric($end_date))
		{
			return $this->user->format_date($end_date, input::DATE_FORMAT);
		}

		return (string) $end_date;
	}

	public function prepare_ads_locations($ads_locations)
	{
		$result = array();
		$location_types = $this->location_manager->get_all_locations();
		foreach ($ads_locations as $location)
		{
			$result[$location['ad_id']][] = $location_types[$location['location_id']]['name'];
		}

		return $result;
	}

	/**
	 * Prepare ad owner for display. Method takes user_id
	 * of the ad owner and returns his/her username.
	 *
	 * @param	int		$ad_owner	User ID
	 * @return	string	Username belonging to $ad_owner.
	 */
	protected function prepare_ad_owner($ad_owner)
	{
		$user_id = array($ad_owner);
		$user_name = array();

		// Returns false when no errors occur trying to find the user
		if (false === user_get_id_name($user_id, $user_name))
		{
			if (empty($user_name))
			{
				return $user_id[0];
			}
			return $user_name[(int) $user_id[0]];
		}
		return '';
	}
}


1			<?php
2			/**
3			*
4			* Advertisement management. An extension for the phpBB Forum Software package.
5			*
6			* @copyright (c) 2017 phpBB Limited <https://www.phpbb.com>
7			* @license GNU General Public License, version 2 (GPL-2.0)
8			*
9			*/
10
11			namespace phpbb\ads\controller;
12
13			use phpbb\ads\controller\admin_input as input;
14
15			/**
16			* Admin helper
17			*/
18			class admin_helper
19			{
20			/** @var \phpbb\user */
21			protected $user;
22
23			/** @var \phpbb\language\language */
24			protected $language;
25
26			/** @var \phpbb\template\template */
27			protected $template;
28
29			/** @var \phpbb\log\log */
30			protected $log;
31
32			/** @var \phpbb\ads\location\manager */
33			protected $location_manager;
34
35			/** @var string root_path */
36			protected $root_path;
37
38			/** @var string php_ext */
39			protected $php_ext;
40
41			/**
42			* Constructor
43			*
44			* @param \phpbb\user $user User object
45			* @param \phpbb\language\language $language Language object
46			* @param \phpbb\template\template $template Template object
47			* @param \phpbb\log\log $log The phpBB log system
48			* @param \phpbb\ads\location\manager $location_manager Template location manager object
49			* @param string $root_path phpBB root path
50			* @param string $php_ext PHP extension
51			*/
52	14	View Code Duplication	public function __construct(\phpbb\user $user, \phpbb\language\language $language, \phpbb\template\template $template, \phpbb\log\log $log, \phpbb\ads\location\manager $location_manager, $root_path, $php_ext)
53			{
54	14		$this->user = $user;
55	14		$this->language = $language;
56	14		$this->template = $template;
57	14		$this->log = $log;
58	14		$this->location_manager = $location_manager;
59	14		$this->root_path = $root_path;
60	14		$this->php_ext = $php_ext;
61	14		}
62
63			/**
64			* Assign template locations data to the template.
65			*
66			* @param mixed $ad_locations The form data or nothing.
67			* @return void
68			*/
69	2		public function assign_locations($ad_locations = false)
70			{
71	2	View Code Duplication	foreach ($this->location_manager->get_all_locations() as $location_id => $location_data)
72			{
73	2		$this->template->assign_block_vars('ad_locations', array(
74	2		'LOCATION_ID' => $location_id,
75	2		'LOCATION_DESC' => $location_data['desc'],
76	2		'LOCATION_NAME' => $location_data['name'],
77	2		'S_SELECTED' => $ad_locations ? in_array($location_id, $ad_locations) : false,
78	2		));
79	2		}
80	2		}
81
82			/**
83			* Assign form data to the template.
84			*
85			* @param array $data The form data.
86			* @return void
87			*/
88	5		public function assign_form_data($data)
89			{
90	5		$this->template->assign_vars(array(
91	5		'AD_NAME' => $data['ad_name'],
92	5		'AD_NOTE' => $data['ad_note'],
93	5		'AD_CODE' => $data['ad_code'],
94	5		'AD_ENABLED' => $data['ad_enabled'],
95	5		'AD_END_DATE' => $this->prepare_end_date($data['ad_end_date']),
96	5		'AD_PRIORITY' => $data['ad_priority'],
97	5		'AD_VIEWS_LIMIT' => $data['ad_views_limit'],
98	5		'AD_CLICKS_LIMIT' => $data['ad_clicks_limit'],
99	5		'AD_OWNER' => $this->prepare_ad_owner($data['ad_owner']),
100	5		));
101	5		}
102
103	3		public function assign_errors(array $errors)
104			{
105	3		$errors = array_map(array($this->language, 'lang'), $errors);
106
107	3		$this->template->assign_vars(array(
108	3		'S_ERROR' => (bool) count($errors),
109	3		'ERROR_MSG' => count($errors) ? implode('<br />', $errors) : '',
110	3		));
111	3		}
112
113			/**
114			* Log action
115			*
116			* @param string $action Performed action in uppercase
117			* @param string $ad_name Advertisement name
118			* @return void
119			*/
120	1		public function log($action, $ad_name)
121			{
122	1		$this->log->add('admin', $this->user->data['user_id'], $this->user->ip, 'ACP_PHPBB_ADS_' . $action . '_LOG', time(), array($ad_name));
123	1		}
124
125	1		public function get_find_username_link()
126			{
127	1		return append_sid("{$this->root_path}memberlist.{$this->php_ext}", 'mode=searchuser&form=acp_admanagement_add&field=ad_owner&select_single=true');
128			}
129
130
131			/**
132			* Prepare end date for display
133			*
134			* @param mixed $end_date End date.
135			* @return string End date prepared for display.
136			*/
137	5		public function prepare_end_date($end_date)
138			{
139	5		if (empty($end_date))
140	5		{
141	2		return '';
142			}
143
144	3		if (is_numeric($end_date))
145	3		{
146	2		return $this->user->format_date($end_date, input::DATE_FORMAT);
147			}
148
149	1		return (string) $end_date;
150			}
151
152	2		public function prepare_ads_locations($ads_locations)
153			{
154	2		$result = array();
155	2		$location_types = $this->location_manager->get_all_locations();
156	2		foreach ($ads_locations as $location)
157			{
158	1		$result[$location['ad_id']][] = $location_types[$location['location_id']]['name'];
159	2		}
160
161	2		return $result;
162			}
163
164			/**
165			* Prepare ad owner for display. Method takes user_id
166			* of the ad owner and returns his/her username.
167			*
168			* @param int $ad_owner User ID
169			* @return string Username belonging to $ad_owner.
170			*/
171	5		protected function prepare_ad_owner($ad_owner)
172			{
173	5		$user_id = array($ad_owner);
174	5		$user_name = array();
175
176			// Returns false when no errors occur trying to find the user
177	5		if (false === user_get_id_name($user_id, $user_name))
178	5		{
179	1		if (empty($user_name))
180	1		{
181			return $user_id[0];
182			}
183	1		return $user_name[(int) $user_id[0]];
184			}
185	4		return '';
186			}
187			}
188

phpbb-extensions / ad-management

Pull Request — master (#74)

admin_helper::prepare_ads_locations() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like