ManageUltimateMenu::__construct() - Code Metrics - Inspection of "do a sprintf" - live627/smf-ultimate-menu - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 4c81df...f4108d )

by John

created 2020-11-13 06:49 UTC

ManageUltimateMenu::__construct() A

↳ Parent: ManageUltimateMenu

Complexity

Conditions	3
Paths	2

Size

Total Lines	31
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	21
c	1
b	0
f	0
dl	0
loc	31
rs	9.584
cc	3
nc	2
nop	0

<?php
/**
 * @package   Ultimate Menu mod
 * @version   1.1
 * @author    John Rayes <[email protected]>
 * @copyright Copyright (c) 2014, John Rayes
 * @license   http://opensource.org/licenses/MIT MIT
 */

class ManageUltimateMenu
{
	private $um;

	function __construct()
	{
		global $context, $sourcedir, $txt;

		isAllowedTo('admin_forum');

		$context['page_title'] = $txt['admin_menu_title'];
		$context[$context['admin_menu_name']]['tab_data'] = array(
			'title' => $txt['admin_menu'],
			'description' => $txt['admin_menu_desc'],
			'tabs' => array(
				'manmenu' => array(
					'description' => $txt['admin_manage_menu_desc'],
				),
				'addbutton' => array(
					'description' => $txt['admin_menu_add_button_desc'],
				),
			),
		);
		loadTemplate('ManageUltimateMenu');
		require_once $sourcedir . '/Class-UltimateMenu.php';
		$this->um = new UltimateMenu;

		$subActions = array(
			'manmenu' => 'ManageUltimateMenu',
			'addbutton' => 'PrepareContext',
			'savebutton' => 'SaveButton',
		);
		if (!isset($_GET['sa']) || !isset($subActions[$_GET['sa']]))
			$_GET['sa'] = 'manmenu';
		$this->$subActions[$_GET['sa']]();
	}

	function ManageUltimateMenu()

	{
		global $context, $txt, $scripturl;

		// Get rid of all of em!
		if (!empty($_POST['removeAll']))
		{
			checkSession();
			$this->um->deleteallButtons();
			$this->um->rebuildMenu();
			redirectexit('action=admin;area=umen');
		}
		// User pressed the 'remove selection button'.
		elseif (!empty($_POST['removeButtons']) && !empty($_POST['remove']) && is_array($_POST['remove']))
		{
			checkSession();

			// Make sure every entry is a proper integer.
			foreach ($_POST['remove'] as $index => $page_id)
				$_POST['remove'][(int) $index] = (int) $page_id;

			// Delete the page(s)!
			$this->um->deleteButton($_POST['remove']);
			$this->um->rebuildMenu();
			redirectexit('action=admin;area=umen');
		}
		// Changing the status?
		elseif (isset($_POST['save']))
		{
			checkSession();
			$this->um->updateButton($_POST);
			$this->um->rebuildMenu();
			redirectexit('action=admin;area=umen');
		}
		// New item?
		elseif (isset($_POST['new']))
			redirectexit('action=admin;area=umen;sa=addbutton');

		$button_names = $this->um->getButtonNames();
		$listOptions = array(
			'id' => 'menu_list',
			'items_per_page' => 20,
			'base_href' => $scripturl . '?action=admin;area=umen;sa=manmenu',
			'default_sort_col' => 'name',
			'default_sort_dir' => 'desc',
			'get_items' => array(
				'function' => array($this->um, 'list_getMenu'),
			),
			'get_count' => array(
				'function' => array($this->um, 'list_getNumButtons'),
			),
			'no_items_label' => $txt['um_menu_no_buttons'],
			'columns' => array(
				'name' => array(
					'header' => array(
						'value' => $txt['um_menu_button_name'],
					),
					'data' => array(
						'db_htmlsafe' => 'name',
					),
					'sort' => array(
						'default' => 'name',
						'reverse' => 'name DESC',
					),
				),
				'type' => array(
					'header' => array(
						'value' => $txt['um_menu_button_type'],
					),
					'data' => array(
						'function' => function($rowData) use ($txt)
						{
							return $txt['um_menu_' . $rowData['type'] . '_link'];
						},
					),
					'sort' => array(
						'default' => 'type',
						'reverse' => 'type DESC',
					),
				),
				'position' => array(
					'header' => array(
						'value' => $txt['um_menu_button_position'],
					),
					'data' => array(
						'function' => function($rowData) use ($txt, $button_names)
						{
							return sprintf(
								'%s %s',
								$txt['um_menu_' . $rowData['position']],
								isset($button_names[$rowData['parent']])
									? $button_names[$rowData['parent']]
									: ucwords($rowData['parent'])
							);
						},
					),
					'sort' => array(
						'default' => 'position',
						'reverse' => 'position DESC',
					),
				),
				'link' => array(
					'header' => array(
						'value' => $txt['um_menu_button_link'],
					),
					'data' => array(
						'db_htmlsafe' => 'link',
					),
					'sort' => array(
						'default' => 'link',
						'reverse' => 'link DESC',
					),
				),
				'status' => array(
					'header' => array(
						'value' => $txt['um_menu_button_active'],
						'class' => 'centertext',
					),
					'data' => array(
						'function' => function($rowData)
						{
							return sprintf(
								'<input type="checkbox" name="status[%1$s]" id="status_%1$s" value="%1$s"%2$s />',
								$rowData['id_button'],
								$rowData['status'] == 'inactive' ? '' : ' checked="checked"'
							);
						},
						'class' => 'centertext',
					),
					'sort' => array(
						'default' => 'status',
						'reverse' => 'status DESC',
					),
				),
				'actions' => array(
					'header' => array(
						'value' => $txt['um_menu_actions'],
						'class' => 'centertext',
					),
					'data' => array(
						'function' => function($rowData) use ($txt)
						{
							return sprintf(
								'<a href="%s?action=admin;area=umen;sa=addbutton;edit;in=%d">%s</a>',
								$scripturl,

								$rowData['id_button'],
								$txt['modify']
							);
						},
						'class' => 'centertext',
					),
				),
				'check' => array(
					'header' => array(
						'value' => '<input type="checkbox" onclick="invertAll(this, this.form);" class="input_check" />',
						'class' => 'centertext',
					),
					'data' => array(
						'sprintf' => array(
							'format' => '<input type="checkbox" name="remove[]" value="%d" class="input_check" />',
							'params' => array(
								'id_button' => false,
							),
						),
						'class' => 'centertext',
					),
				),
			),
			'form' => array(
				'href' => $scripturl . '?action=admin;area=umen;sa=manmenu',
			),
			'additional_rows' => array(
				array(
					'position' => 'below_table_data',
					'value' => '
						<input type="submit" name="removeButtons" value="' . $txt['um_menu_remove_selected'] . '" onclick="return confirm(\'' . $txt['um_menu_remove_confirm'] . '\');" class="button_submit" />
						<input type="submit" name="removeAll" value="' . $txt['um_menu_remove_all'] . '" onclick="return confirm(\'' . $txt['um_menu_remove_all_confirm'] . '\');" class="button_submit" />
						<input type="submit" name="new" value="' . $txt['um_admin_add_button'] . '" class="button_submit" />
						<input type="submit" name="save" value="' . $txt['save'] . '" class="button_submit" />',
					'class' => 'righttext',
				),
			),
		);
		require_once $sourcedir . '/Subs-List.php';

		createList($listOptions);
		$context['sub_template'] = 'show_list';
		$context['default_list'] = 'menu_list';
	}

	function SaveButton()

	{
		global $context, $txt;

		if (isset($_POST['submit']))
		{
			$post_errors = array();
			$required_fields = array(
				'name',
				'link',
				'parent',
			);
			$member_groups = array_column($this->um->list_groups('-3', 1), 'id');
			$button_names = $this->um->getButtonNames();
			$args = array(
				'in' => FILTER_VALIDATE_INT,
				'name' => FILTER_UNSAFE_RAW,
				'position' => array(
					'filter' => FILTER_CALLBACK,
					'options' => function ($v)
					{
						return in_array($v, ['before', 'child_of', 'after']) ? $v : false;
					},
				),
				'parent' => array(
					'filter' => FILTER_CALLBACK,
					'options' => function ($v) use ($button_names)
					{
						return isset($button_names[$v]) ? $v : false;
					},
				),
				'type' => array(
					'filter' => FILTER_CALLBACK,
					'options' => function ($v)
					{
						return in_array($v, ['forum', 'external']) ? $v : false;
					},
				),
				'link' => FILTER_UNSAFE_RAW,
				'permissions' => array(
					'filter' => FILTER_CALLBACK,
					'flags' => FILTER_REQUIRE_ARRAY,
					'options' => function ($v) use ($member_groups)
					{
						return in_array($v, $member_groups) ? $v : false;
					},
				),
				'status' => array(
					'filter' => FILTER_CALLBACK,
					'options' => function ($v)
					{
						return in_array($v, ['active', 'inactive']) ? $v : false;
					},
				),
				'target' => array(
					'filter' => FILTER_CALLBACK,
					'options' => function ($v)
					{
						return in_array($v, ['_self', '_blank']) ? $v : false;
					},
				),
			);

			// Make sure we grab all of the content
			$menu_entry = array_replace(
				array(
					'target' => '_self',
					'type' => 'forum',
					'position' => 'before',
					'status' => 'active',
					'parent' => 'home',
				),
				filter_input_array(INPUT_POST, $args)
			);

			// These fields are required!
			foreach ($required_fields as $required_field)
				if (empty($menu_entry[$required_field]))
					$post_errors[$required_field] = 'um_menu_empty_' . $required_field;

			// Stop making numeric names!
			if (is_numeric($menu_entry['name']))
				$post_errors['name'] = 'um_menu_numeric';

			// Let's make sure you're not trying to make a name that's already taken.
			$check = $this->um->checkButton($menu_entry['id'], $menu_entry['name']);
			if ($check > 0)
				$post_errors['name'] = 'um_menu_mysql';

			// I see you made it to the final stage, my young padawan.
			if (empty($post_errors))
			{
				$this->um->saveButton($menu_entry);
				$this->um->rebuildMenu();


				// Before we leave, we must clear the cache. See, SMF
				// caches its menu at level 2 or higher.
				clean_cache('menu_buttons');

				redirectexit('action=admin;area=umen');
			}
			else
			{
				$context['page_title'] = $txt['um_menu_edit_title'];
				$context['post_error'] = $post_errors;
				$context['error_title'] = empty($menu_entry['id'])
					? 'um_menu_errors_create'
					: 'um_menu_errors_modify';
				$context['button_data'] = array(
					'name' => $menu_entry['name'],
					'type' => $menu_entry['type'],
					'target' => $menu_entry['target'],
					'position' => $menu_entry['position'],
					'link' => $menu_entry['link'],
					'parent' => $menu_entry['parent'],
					'permissions' => $this->um->list_groups(
						implode(',', array_filter($menu_entry['permissions'], 'strlen')),
						1
					),
					'status' => $menu_entry['status'],
					'id' => $menu_entry['id'],
				);
			}
		}
	}

	function PrepareContext()

	{
		global $context, $txt;

		if (isset($_GET['in']))
		{
			$row = $this->um->fetchButton($_GET['in']);

			$context['button_data'] = array(
				'id' => $_GET['in'],
				'name' => $row['name'],
				'target' => $row['target'],
				'type' => $row['type'],
				'position' => $row['position'],
				'permissions' => $this->um->list_groups($row['permissions'], 1),
				'link' => $row['link'],
				'status' => $row['status'],
				'parent' => $row['parent'],
			);
		}
		else
		{
			$context['button_data'] = array(
				'name' => '',
				'link' => '',
				'target' => '_self',
				'type' => 'forum',
				'position' => 'before',
				'status' => 'active',
				'permissions' => $this->um->list_groups('-3', 1),
				'parent' => 'home',
				'id' => 0,
			);

			$context['page_title'] = $txt['um_menu_add_title'];
		}
	}
}


1			<?php
2			/**
3			* @package Ultimate Menu mod
4			* @version 1.1
5			* @author John Rayes <[email protected]>
6			* @copyright Copyright (c) 2014, John Rayes
7			* @license http://opensource.org/licenses/MIT MIT
8			*/
9
10			class ManageUltimateMenu
11			{
12			private $um;
13
14			function __construct()
15			{
16			global $context, $sourcedir, $txt;
17
18			isAllowedTo('admin_forum');
19
20			$context['page_title'] = $txt['admin_menu_title'];
21			$context[$context['admin_menu_name']]['tab_data'] = array(
22			'title' => $txt['admin_menu'],
23			'description' => $txt['admin_menu_desc'],
24			'tabs' => array(
25			'manmenu' => array(
26			'description' => $txt['admin_manage_menu_desc'],
27			),
28			'addbutton' => array(
29			'description' => $txt['admin_menu_add_button_desc'],
30			),
31			),
32			);
33			loadTemplate('ManageUltimateMenu');
34			require_once $sourcedir . '/Class-UltimateMenu.php';
35			$this->um = new UltimateMenu;
36
37			$subActions = array(
38			'manmenu' => 'ManageUltimateMenu',
39			'addbutton' => 'PrepareContext',
40			'savebutton' => 'SaveButton',
41			);
42			if (!isset($_GET['sa']) \|\| !isset($subActions[$_GET['sa']]))
43			$_GET['sa'] = 'manmenu';
44			$this->$subActions[$_GET['sa']]();
45			}
46
47			function ManageUltimateMenu()
			0 ignored issues – show Best Practice introduced 2020-11-12 14:05 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
48			{
49			global $context, $txt, $scripturl;
50
51			// Get rid of all of em!
52			if (!empty($_POST['removeAll']))
53			{
54			checkSession();
55			$this->um->deleteallButtons();
56			$this->um->rebuildMenu();
57			redirectexit('action=admin;area=umen');
58			}
59			// User pressed the 'remove selection button'.
60			elseif (!empty($_POST['removeButtons']) && !empty($_POST['remove']) && is_array($_POST['remove']))
61			{
62			checkSession();
63
64			// Make sure every entry is a proper integer.
65			foreach ($_POST['remove'] as $index => $page_id)
66			$_POST['remove'][(int) $index] = (int) $page_id;
67
68			// Delete the page(s)!
69			$this->um->deleteButton($_POST['remove']);
70			$this->um->rebuildMenu();
71			redirectexit('action=admin;area=umen');
72			}
73			// Changing the status?
74			elseif (isset($_POST['save']))
75			{
76			checkSession();
77			$this->um->updateButton($_POST);
78			$this->um->rebuildMenu();
79			redirectexit('action=admin;area=umen');
80			}
81			// New item?
82			elseif (isset($_POST['new']))
83			redirectexit('action=admin;area=umen;sa=addbutton');
84
85			$button_names = $this->um->getButtonNames();
86			$listOptions = array(
87			'id' => 'menu_list',
88			'items_per_page' => 20,
89			'base_href' => $scripturl . '?action=admin;area=umen;sa=manmenu',
90			'default_sort_col' => 'name',
91			'default_sort_dir' => 'desc',
92			'get_items' => array(
93			'function' => array($this->um, 'list_getMenu'),
94			),
95			'get_count' => array(
96			'function' => array($this->um, 'list_getNumButtons'),
97			),
98			'no_items_label' => $txt['um_menu_no_buttons'],
99			'columns' => array(
100			'name' => array(
101			'header' => array(
102			'value' => $txt['um_menu_button_name'],
103			),
104			'data' => array(
105			'db_htmlsafe' => 'name',
106			),
107			'sort' => array(
108			'default' => 'name',
109			'reverse' => 'name DESC',
110			),
111			),
112			'type' => array(
113			'header' => array(
114			'value' => $txt['um_menu_button_type'],
115			),
116			'data' => array(
117			'function' => function($rowData) use ($txt)
118			{
119			return $txt['um_menu_' . $rowData['type'] . '_link'];
120			},
121			),
122			'sort' => array(
123			'default' => 'type',
124			'reverse' => 'type DESC',
125			),
126			),
127			'position' => array(
128			'header' => array(
129			'value' => $txt['um_menu_button_position'],
130			),
131			'data' => array(
132			'function' => function($rowData) use ($txt, $button_names)
133			{
134			return sprintf(
135			'%s %s',
136			$txt['um_menu_' . $rowData['position']],
137			isset($button_names[$rowData['parent']])
138			? $button_names[$rowData['parent']]
139			: ucwords($rowData['parent'])
140			);
141			},
142			),
143			'sort' => array(
144			'default' => 'position',
145			'reverse' => 'position DESC',
146			),
147			),
148			'link' => array(
149			'header' => array(
150			'value' => $txt['um_menu_button_link'],
151			),
152			'data' => array(
153			'db_htmlsafe' => 'link',
154			),
155			'sort' => array(
156			'default' => 'link',
157			'reverse' => 'link DESC',
158			),
159			),
160			'status' => array(
161			'header' => array(
162			'value' => $txt['um_menu_button_active'],
163			'class' => 'centertext',
164			),
165			'data' => array(
166			'function' => function($rowData)
167			{
168			return sprintf(
169			'<input type="checkbox" name="status[%1$s]" id="status_%1$s" value="%1$s"%2$s />',
170			$rowData['id_button'],
171			$rowData['status'] == 'inactive' ? '' : ' checked="checked"'
172			);
173			},
174			'class' => 'centertext',
175			),
176			'sort' => array(
177			'default' => 'status',
178			'reverse' => 'status DESC',
179			),
180			),
181			'actions' => array(
182			'header' => array(
183			'value' => $txt['um_menu_actions'],
184			'class' => 'centertext',
185			),
186			'data' => array(
187			'function' => function($rowData) use ($txt)
188			{
189			return sprintf(
190			'<a href="%s?action=admin;area=umen;sa=addbutton;edit;in=%d">%s</a>',
191			$scripturl,
			0 ignored issues – show Comprehensibility Best Practice introduced 2020-11-13 06:51 UTC by Report Bug Copy Issue Report The variable `$scripturl` seems to be never defined. Loading history...
192			$rowData['id_button'],
193			$txt['modify']
194			);
195			},
196			'class' => 'centertext',
197			),
198			),
199			'check' => array(
200			'header' => array(
201			'value' => '<input type="checkbox" onclick="invertAll(this, this.form);" class="input_check" />',
202			'class' => 'centertext',
203			),
204			'data' => array(
205			'sprintf' => array(
206			'format' => '<input type="checkbox" name="remove[]" value="%d" class="input_check" />',
207			'params' => array(
208			'id_button' => false,
209			),
210			),
211			'class' => 'centertext',
212			),
213			),
214			),
215			'form' => array(
216			'href' => $scripturl . '?action=admin;area=umen;sa=manmenu',
217			),
218			'additional_rows' => array(
219			array(
220			'position' => 'below_table_data',
221			'value' => '
222			<input type="submit" name="removeButtons" value="' . $txt['um_menu_remove_selected'] . '" onclick="return confirm(\'' . $txt['um_menu_remove_confirm'] . '\');" class="button_submit" />
223			<input type="submit" name="removeAll" value="' . $txt['um_menu_remove_all'] . '" onclick="return confirm(\'' . $txt['um_menu_remove_all_confirm'] . '\');" class="button_submit" />
224			<input type="submit" name="new" value="' . $txt['um_admin_add_button'] . '" class="button_submit" />
225			<input type="submit" name="save" value="' . $txt['save'] . '" class="button_submit" />',
226			'class' => 'righttext',
227			),
228			),
229			);
230			require_once $sourcedir . '/Subs-List.php';
			0 ignored issues – show Comprehensibility Best Practice introduced 2020-11-12 14:05 UTC by Report Bug Copy Issue Report The variable `$sourcedir` seems to be never defined. Loading history...
231			createList($listOptions);
232			$context['sub_template'] = 'show_list';
233			$context['default_list'] = 'menu_list';
234			}
235
236			function SaveButton()
			0 ignored issues – show Best Practice introduced 2020-11-12 12:02 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
237			{
238			global $context, $txt;
239
240			if (isset($_POST['submit']))
241			{
242			$post_errors = array();
243			$required_fields = array(
244			'name',
245			'link',
246			'parent',
247			);
248			$member_groups = array_column($this->um->list_groups('-3', 1), 'id');
249			$button_names = $this->um->getButtonNames();
250			$args = array(
251			'in' => FILTER_VALIDATE_INT,
252			'name' => FILTER_UNSAFE_RAW,
253			'position' => array(
254			'filter' => FILTER_CALLBACK,
255			'options' => function ($v)
256			{
257			return in_array($v, ['before', 'child_of', 'after']) ? $v : false;
258			},
259			),
260			'parent' => array(
261			'filter' => FILTER_CALLBACK,
262			'options' => function ($v) use ($button_names)
263			{
264			return isset($button_names[$v]) ? $v : false;
265			},
266			),
267			'type' => array(
268			'filter' => FILTER_CALLBACK,
269			'options' => function ($v)
270			{
271			return in_array($v, ['forum', 'external']) ? $v : false;
272			},
273			),
274			'link' => FILTER_UNSAFE_RAW,
275			'permissions' => array(
276			'filter' => FILTER_CALLBACK,
277			'flags' => FILTER_REQUIRE_ARRAY,
278			'options' => function ($v) use ($member_groups)
279			{
280			return in_array($v, $member_groups) ? $v : false;
281			},
282			),
283			'status' => array(
284			'filter' => FILTER_CALLBACK,
285			'options' => function ($v)
286			{
287			return in_array($v, ['active', 'inactive']) ? $v : false;
288			},
289			),
290			'target' => array(
291			'filter' => FILTER_CALLBACK,
292			'options' => function ($v)
293			{
294			return in_array($v, ['_self', '_blank']) ? $v : false;
295			},
296			),
297			);
298
299			// Make sure we grab all of the content
300			$menu_entry = array_replace(
301			array(
302			'target' => '_self',
303			'type' => 'forum',
304			'position' => 'before',
305			'status' => 'active',
306			'parent' => 'home',
307			),
308			filter_input_array(INPUT_POST, $args)
309			);
310
311			// These fields are required!
312			foreach ($required_fields as $required_field)
313			if (empty($menu_entry[$required_field]))
314			$post_errors[$required_field] = 'um_menu_empty_' . $required_field;
315
316			// Stop making numeric names!
317			if (is_numeric($menu_entry['name']))
318			$post_errors['name'] = 'um_menu_numeric';
319
320			// Let's make sure you're not trying to make a name that's already taken.
321			$check = $this->um->checkButton($menu_entry['id'], $menu_entry['name']);
322			if ($check > 0)
323			$post_errors['name'] = 'um_menu_mysql';
324
325			// I see you made it to the final stage, my young padawan.
326			if (empty($post_errors))
327			{
328			$this->um->saveButton($menu_entry);
329			$this->um->rebuildMenu();
330
331
332			// Before we leave, we must clear the cache. See, SMF
333			// caches its menu at level 2 or higher.
334			clean_cache('menu_buttons');
335
336			redirectexit('action=admin;area=umen');
337			}
338			else
339			{
340			$context['page_title'] = $txt['um_menu_edit_title'];
341			$context['post_error'] = $post_errors;
342			$context['error_title'] = empty($menu_entry['id'])
343			? 'um_menu_errors_create'
344			: 'um_menu_errors_modify';
345			$context['button_data'] = array(
346			'name' => $menu_entry['name'],
347			'type' => $menu_entry['type'],
348			'target' => $menu_entry['target'],
349			'position' => $menu_entry['position'],
350			'link' => $menu_entry['link'],
351			'parent' => $menu_entry['parent'],
352			'permissions' => $this->um->list_groups(
353			implode(',', array_filter($menu_entry['permissions'], 'strlen')),
354			1
355			),
356			'status' => $menu_entry['status'],
357			'id' => $menu_entry['id'],
358			);
359			}
360			}
361			}
362
363			function PrepareContext()
			0 ignored issues – show Best Practice introduced 2020-11-12 14:05 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
364			{
365			global $context, $txt;
366
367			if (isset($_GET['in']))
368			{
369			$row = $this->um->fetchButton($_GET['in']);
370
371			$context['button_data'] = array(
372			'id' => $_GET['in'],
373			'name' => $row['name'],
374			'target' => $row['target'],
375			'type' => $row['type'],
376			'position' => $row['position'],
377			'permissions' => $this->um->list_groups($row['permissions'], 1),
378			'link' => $row['link'],
379			'status' => $row['status'],
380			'parent' => $row['parent'],
381			);
382			}
383			else
384			{
385			$context['button_data'] = array(
386			'name' => '',
387			'link' => '',
388			'target' => '_self',
389			'type' => 'forum',
390			'position' => 'before',
391			'status' => 'active',
392			'permissions' => $this->um->list_groups('-3', 1),
393			'parent' => 'home',
394			'id' => 0,
395			);
396
397			$context['page_title'] = $txt['um_menu_add_title'];
398			}
399			}
400			}
401

live627 / smf-ultimate-menu

Push — master ( 4c81df...f4108d )

ManageUltimateMenu::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like