live627 / smf-ultimate-menu

ManageUltimateMenu.php

Spacing +1 added lines, -1 removed lines

@@ -333,7 +333,7 @@
 		{
 			// I see you made it to the final stage, my young padawan.
 			if (!empty($id))
-				$smcFunc['db_query']('','
+				$smcFunc['db_query']('', '
 					UPDATE {db_prefix}um_menu
 					SET name = {string:name}, type = {string:type}, target = {string:target}, position = {string:position}, link = {string:link}, status = {string:status}, permissions = {string:permissions}, parent = {string:parent}
 					WHERE id_button = {int:id}',

Braces +39 added lines, -29 removed lines

@@ -8,8 +8,9 @@ 
  * @license http://opensource.org/licenses/MIT MIT
  */
 
-if (!defined('SMF'))
+if (!defined('SMF')) {
 	die('Hacking attempt...');
+}
 
 function Menu()
 {
@@ -24,8 +25,9 @@ 
 	);
 
 	// Default to sub action 'manmenu'
-	if (!isset($_GET['sa']) || !isset($subActions[$_GET['sa']]))
-		$_GET['sa'] = 'manmenu';
+	if (!isset($_GET['sa']) || !isset($subActions[$_GET['sa']])) {
+			$_GET['sa'] = 'manmenu';
+	}
 
 	// Have you got the proper permissions?
 	isAllowedTo('admin_forum');
@@ -73,8 +75,9 @@ 
 		checkSession();
 
 		// Make sure every entry is a proper integer.
-		foreach ($_POST['remove'] as $index => $page_id)
-			$_POST['remove'][(int) $index] = (int) $page_id;
+		foreach ($_POST['remove'] as $index => $page_id) {
+					$_POST['remove'][(int) $index] = (int) $page_id;
+		}
 
 		// Delete the page!
 		$smcFunc['db_query']('', '
@@ -95,8 +98,8 @@ 
 		foreach (total_getMenu() as $item)
 		{
 			$status = !empty($_POST['status'][$item['id_button']]) ? 'active' : 'inactive';
-			if ($status != $item['status'])
-				$smcFunc['db_query']('', '
+			if ($status != $item['status']) {
+							$smcFunc['db_query']('', '
 					UPDATE {db_prefix}um_menu
 					SET status = {string:status}
 					WHERE id_button = {int:item}',
@@ -105,14 +108,16 @@ 
 						'item' => $item['id_button'],
 					)
 				);
+			}
 		}
 		rebuild_um_menu();
 		redirectexit('action=admin;area=umen');
 	}
 
 	// New item?
-	if (isset($_POST['new']))
-		redirectexit('action=admin;area=umen;sa=addbutton');
+	if (isset($_POST['new'])) {
+			redirectexit('action=admin;area=umen;sa=addbutton');
+	}
 
 	loadLanguage('ManageBoards');
 
@@ -302,13 +307,15 @@ 
 		$target = isset($_REQUEST['target']) ? $_REQUEST['target'] : '_self';
 
 		// These fields are required!
-		foreach ($required_fields as $required_field)
-			if ($_POST[$required_field] == '')
+		foreach ($required_fields as $required_field) {
+					if ($_POST[$required_field] == '')
 				$post_errors[$required_field] = 'um_menu_empty_' . $required_field;
+		}
 
 		// Stop making numeric names!
-		if (is_numeric($name))
-			$post_errors['name'] = 'um_menu_numeric';
+		if (is_numeric($name)) {
+					$post_errors['name'] = 'um_menu_numeric';
+		}
 
 		// Let's make sure you're not trying to make a name that's already taken.
 		$request = $smcFunc['db_query']('', '
@@ -326,14 +333,15 @@ 
 
 		$smcFunc['db_free_result']($request);
 
-		if ($check > 0)
-			$post_errors['name'] = 'um_menu_mysql';
+		if ($check > 0) {
+					$post_errors['name'] = 'um_menu_mysql';
+		}
 
 		if (empty($post_errors))
 		{
 			// I see you made it to the final stage, my young padawan.
-			if (!empty($id))
-				$smcFunc['db_query']('','
+			if (!empty($id)) {
+							$smcFunc['db_query']('','
 					UPDATE {db_prefix}um_menu
 					SET name = {string:name}, type = {string:type}, target = {string:target}, position = {string:position}, link = {string:link}, status = {string:status}, permissions = {string:permissions}, parent = {string:parent}
 					WHERE id_button = {int:id}',
@@ -349,8 +357,8 @@ 
 						'parent' => $parent,
 					)
 				);
-			else
-				$smcFunc['db_insert']('insert',
+			} else {
+							$smcFunc['db_insert']('insert',
 					'{db_prefix}um_menu',
 						array(
 							'slug' => 'string', 'name' => 'string', 'type' => 'string', 'target' => 'string', 'position' => 'string', 'link' => 'string', 'status' => 'string', 'permissions' => 'string', 'parent' => 'string',
@@ -360,6 +368,7 @@ 
 						),
 						array('id_button')
 					);
+			}
 
 			rebuild_um_menu();
 
@@ -368,8 +377,7 @@ 
 			clean_cache('menu_buttons');
 
 			redirectexit('action=admin;area=umen');
-		}
-		else
+		} else
 		{
 			$context['post_error'] = $post_errors;
 			$context['error_title'] = empty($id) ? 'um_menu_errors_create' : 'um_menu_errors_modify';
@@ -430,8 +438,7 @@ 
 			'status' => $row['status'],
 			'parent' => $row['parent'],
 		);
-	}
-	else
+	} else
 	{
 		$context['button_data'] = array(
 			'name' => '',
@@ -458,8 +465,9 @@ 
 		FROM {db_prefix}um_menu');
 
 	$um_menu = array();
-	while ($row = $smcFunc['db_fetch_assoc']($request))
-		$um_menu[] = $row;
+	while ($row = $smcFunc['db_fetch_assoc']($request)) {
+			$um_menu[] = $row;
+	}
 
 	return $um_menu;
 }
@@ -481,8 +489,9 @@ 
 	);
 
 	$um_menu = array();
-	while ($row = $smcFunc['db_fetch_assoc']($request))
-		$um_menu[] = $row;
+	while ($row = $smcFunc['db_fetch_assoc']($request)) {
+			$um_menu[] = $row;
+	}
 
 	return $um_menu;
 }
@@ -510,8 +519,9 @@ 
 		FROM {db_prefix}um_menu');
 
 	$db_buttons = array();
-	while ($row = $smcFunc['db_fetch_assoc']($request))
-		$db_buttons[$row['id_button']] = $row;
+	while ($row = $smcFunc['db_fetch_assoc']($request)) {
+			$db_buttons[$row['id_button']] = $row;
+	}
 
 	$smcFunc['db_free_result']($request);
 	updateSettings(