ManageUltimateMenu::SaveButton() - Code Metrics - Inspection of "listGroups to require arrays now" - live627/smf-ultimate-menu - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 932680...d062ce )

by John

created 2020-11-13 09:52 UTC

ManageUltimateMenu::SaveButton() C

↳ Parent: Project

Complexity

Conditions	14
Paths	37

Size

Total Lines	121
Code Lines	76

Duplication

Lines	0
Ratio	0 %

Importance

Changes	4
Bugs	0	Features	0

Metric	Value
cc	14
eloc	76
c	4
b	0
f	0
nc	37
nop	0
dl	0
loc	121
rs	5.5369

How to fix Long Method Complexity

<?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;

	public 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']]();
	}

	public function ManageUltimateMenu()
	{
		// 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');

		$this->ListButtons();
	}

	public function ListButtons()
	{
		global $context, $txt, $scripturl, $sourcedir;

		$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 ($scripturl, $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' => sprintf('
						<input type="submit" name="removeButtons" value="%s" onclick="return confirm(\'%s\');" class="button_submit" />
						<input type="submit" name="removeAll" value="%s" onclick="return confirm(\'%s\');" class="button_submit" />
						<input type="submit" name="new" value="%s" class="button_submit" />
						<input type="submit" name="save" value="%s" class="button_submit" />',
						$txt['um_menu_remove_selected'],
						$txt['um_menu_remove_confirm'],
						$txt['um_menu_remove_all'],
						$txt['um_menu_remove_all_confirm'],
						$txt['um_admin_add_button'],
						$txt['save']
					),
					'class' => 'righttext',
				),
			),
		);
		require_once $sourcedir . '/Subs-List.php';
		createList($listOptions);
		$context['sub_template'] = 'show_list';
		$context['default_list'] = 'menu_list';
	}

	public function SaveButton()
	{
		global $context, $txt;

		if (isset($_POST['submit']))
		{
			$post_errors = array();
			$required_fields = array(
				'name',
				'link',
				'parent',
			);
			$member_groups = array_column($this->um->listGroups([-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->listGroups(
						array_filter($menu_entry['permissions'], 'strlen'),
						[1]
					),
					'status' => $menu_entry['status'],
					'id' => $menu_entry['id'],
				);
			}
		}
	}

	public function PrepareContext()
	{
		global $context, $txt;

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

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

live627 / smf-ultimate-menu

Push — master ( 932680...d062ce )

ManageUltimateMenu::SaveButton() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like