Explorar el Código

php 8.2+ fixes (#611)

* php 8.2+ fixes

* php 8.2+ fixes
OwN-3m-All hace 2 años
padre
commit
ca97cf507d

+ 1 - 1
home.php

@@ -375,7 +375,7 @@ function ogpHome()
 			// Custom Tabs
 			if( isset($settings['custom_tab']) && $settings['custom_tab'] == "1" && isset($settings['custom_tab_name']) && $settings['custom_tab_name'] != "" )
 			{
-				echo "<li><a href='$settings[custom_tab_link]' target='$settings[custom_tab_target_blank]'><span class='customtab'>$settings[custom_tab_name]</span></a>";
+				echo "<li><a href='" . $settings['custom_tab_link'] . "' target='" . $settings['custom_tab_target_blank'] . "'><span class='customtab'>" . $settings['custom_tab_name'] . "</span></a>";
 
 				if( isset($settings['custom_tab_sub']) && $settings['custom_tab_sub'] == "1" )
 				{

+ 1 - 1
includes/database_mysqli.php

@@ -3666,7 +3666,7 @@ class OGPDatabaseMySQL extends OGPDatabase
 											 "disable";
 		$firewall_settings['default_allowed'] = isset($firewall_settings['default_allowed']) ?
 													  $firewall_settings['default_allowed'] :
-													  "$agent_ip:$remote_server[agent_port],$ftp_ip:$remote_server[ftp_port],22,25,80";
+													  "$agent_ip:" . $remote_server['agent_port'] . ",$ftp_ip:" . $remote_server['ftp_port'] . ",22,25,80";
 		$firewall_settings['allow_port_command'] = isset($firewall_settings['allow_port_command']) ?
 														 $firewall_settings['allow_port_command'] :
 														 "ufw allow %PORT%";

+ 1 - 1
includes/functions.php

@@ -210,7 +210,7 @@ function create_home_selector_address($module, $subpage, $server_homes, $extra_i
 	{
 		foreach($extra_inputs as $input)
 		{
-			echo "<input type='$input[type]' name='$input[name]' value='$input[value]' />\n";
+			echo "<input type='" . $input['type'] . "' name='" . $input['name'] . "' value='" . $input['value'] . "' />\n";
 		}
 	}
 	echo "<select onchange=\"this.form.submit();\" name='home_id-mod_id-ip-port'>\n";

+ 1 - 1
modules/addonsmanager/addons_installer.php

@@ -255,7 +255,7 @@ function exec_ogp_module() {
             <tr><td align='right'><?php print_lang('game_name'); ?>: </td><td align='left'><?php  echo $home_info['game_name']; ?></td></tr>
             <tr><td align='right'><?php print_lang('directory'); ?>: </td><td align='left'><?php  echo $home_info['home_path']; ?></td></tr>
             <tr><td align='right'><?php print_lang('remote_server'); ?>: </td>
-            <td align='left'><?php  echo "$home_info[remote_server_name] ($home_info[agent_ip]:$home_info[agent_port])"; ?></td></tr>
+            <td align='left'><?php  echo $home_info['remote_server_name'] . " (" . $home_info['agent_ip'] . ":" . $home_info['agent_port'] . ")"; ?></td></tr>
             <tr><td align='right'><?php print_lang('select_addon'); ?>: </td>
             <td align='left'>
 			<select name="addon_id">

+ 2 - 2
modules/extras/extras.php

@@ -585,7 +585,7 @@ function exec_ogp_module()
 		$installed_str = $on_disk ? $installed ? "<a class='uninstall' style='color:blue;' data-module-folder='$folder' data-module-id='".
 												 $installed_modules_by_folder[$folder]."' href='#uninstall_$folder' >".get_lang("uninstall")."</a>" : 
 												 "<a class='install' style='color:blue;' data-module-folder='$folder' href='#install_$folder' >".get_lang("install")."</a> - ".
-												 "<a class='remove' style='color:red;' data-module-folder='$module[title]' data-remove-mode='modules' href='#remove_$folder' >".get_lang("remove")."</a>" : 
+												 "<a class='remove' style='color:red;' data-module-folder='" . $module['title'] . "' data-remove-mode='modules' href='#remove_$folder' >".get_lang("remove")."</a>" : 
 												 "<b style='color:red;' >".get_lang("not_installed")."</b>";
 		$uptodate = FALSE;
 		if($on_disk)
@@ -625,7 +625,7 @@ function exec_ogp_module()
 		$on_disk = file_exists($install_nfo);
 		$is_old = $on_disk && (strtotime('+1 hour', filemtime($local_repo_file)) <= time());
 		$installed_str = $on_disk ? "<b style='color:green;' >".get_lang("installed")."</b> - ".
-									"<a class='remove' style='color:red;' data-module-folder='$theme[title]' data-remove-mode='themes' href='#remove_$folder' >".get_lang("remove")."</a>": 
+									"<a class='remove' style='color:red;' data-module-folder='" . $theme['title'] . "' data-remove-mode='themes' href='#remove_$folder' >".get_lang("remove")."</a>": 
 									"<b style='color:red;' >".get_lang("not_installed")."</b>";
 		$uptodate = FALSE;
 		if($on_disk)

+ 1 - 1
modules/gamemanager/home_handling_functions.php

@@ -455,7 +455,7 @@ function get_monitor_buttons($server_home, $server_xml)
 	$buttons = array();
 	foreach($db->getInstalledModules() as $installed_module)
 	{
-		$buttons_file = "modules/$installed_module[folder]/monitor_buttons.php";
+		$buttons_file = "modules/" . $installed_module['folder'] . "/monitor_buttons.php";
 		if(file_exists($buttons_file))
 		{
 			require($buttons_file);

+ 2 - 2
modules/gamemanager/mini_start.php

@@ -100,8 +100,8 @@ if( !isset( $_POST['start_server'] ) )
 	echo get_lang_f('select_params_and_start',  get_lang("start_server") );
 
 	echo "<form action='home.php?m=gamemanager&amp;p=game_monitor&amp;home_id-mod_id-ip-port=". $server_home['home_id'] . "-". $server_home['mod_id'] . "-" . $server_home['ip'] . "-" . $server_home['port'] . "' method='post'>\n
-		<input type='hidden' name='mod_id' value='$server_home[mod_id]' />\n
-		<input type='hidden' name='home_id' value='$server_home[home_id]' />\n
+		<input type='hidden' name='mod_id' value='" . $server_home['mod_id'] . "' />\n
+		<input type='hidden' name='home_id' value='" . $server_home['home_id'] . "' />\n
 		<input type='hidden' name='remote_server_id' value='".$server_home['remote_server_id']."' />\n
 		<table class='start-server'>
 		<tr><td class='right'>". get_lang("ogp_agent_ip") .

+ 8 - 8
modules/gamemanager/rsync_install.php

@@ -186,7 +186,7 @@ function exec_ogp_module() {
 	elseif( preg_match("/linux/", $server_xml->game_key) )
 		$os = "linux";
 		
-	echo "<h2>Update $home_info[home_name]</h2>";
+	echo "<h2>Update " . $home_info['home_name'] . "</h2>";
 	
 	$screenRunningForHome = $remote->is_screen_running(OGP_SCREEN_TYPE_HOME,$home_id);
 	if ($screenRunningForHome === 1)
@@ -374,10 +374,10 @@ function exec_ogp_module() {
 				 <input type='hidden' name='mod_id' value='$mod_id' />
 				 <input type='hidden' name='lgslname' value='$lgslname' />
 				 <input type='hidden' name='state' value='start' />
-				 <tr><td align='right'>Game name:</td><td align='left'>$home_info[game_name]</td></tr>
-				 <tr><td align='right'>Directory:</td><td align='left'>$home_info[home_path]</td></tr>
+				 <tr><td align='right'>Game name:</td><td align='left'>" . $home_info['game_name'] . "</td></tr>
+				 <tr><td align='right'>Directory:</td><td align='left'>" . $home_info['home_path'] . "</td></tr>
 				 <tr><td align='right'>Remoteserver:</td>
-				 <td align='left'>$home_info[remote_server_name] ($home_info[agent_ip]:$home_info[agent_port])</td></tr>
+				 <td align='left'>" . $home_info['remote_server_name'] . " (" . $home_info['agent_ip'] . ":" . $home_info['agent_port'] . ")</td></tr>
 				 <tr><td align='right'>Rsync Server:</td>
 				 <td align='left'>".create_drop_box_from_array_rsync($rsync_sites,"url_id")."<br>";
 			if( $master_server_home_id != FALSE AND $master_server_home_id != $home_id )
@@ -398,10 +398,10 @@ function exec_ogp_module() {
 				 <input type='hidden' name='mod_id' value='$mod_id' />
 				 <input type='hidden' name='lgslname' value='$lgslname' />
 				 <input type='hidden' name='state' value='start' />
-				 <tr><td align='right'>Game name:</td><td align='left'>$home_info[game_name]</td></tr>
-				 <tr><td align='right'>Master Server Name:</td><td align='left'>$ms_home_info[home_name]</td></tr>
-				 <tr><td align='right'>Master Server Directory:</td><td align='left'>$ms_home_info[home_path]</td></tr>
-				 <tr><td align='right'>Local Directory:</td><td align='left'>$home_info[home_path]</td></tr>".
+				 <tr><td align='right'>Game name:</td><td align='left'>" . $home_info['game_name'] . "</td></tr>
+				 <tr><td align='right'>Master Server Name:</td><td align='left'>" . $ms_home_info['home_name'] . "</td></tr>
+				 <tr><td align='right'>Master Server Directory:</td><td align='left'>" . $ms_home_info['home_path'] . "</td></tr>
+				 <tr><td align='right'>Local Directory:</td><td align='left'>" . $home_info['home_path'] . "</td></tr>".
 				 "<input type='hidden' name='master_server_home_id' value='$master_server_home_id' />".
 				 "</td></tr></table><p><input type='submit' name='update' value='". get_lang("update_from_local_master_server") ."' /></p>
 				 </form>";

+ 1 - 1
modules/gamemanager/server_monitor.php

@@ -199,7 +199,7 @@ function exec_ogp_module() {
 
 		if ( $isAdmin )
 		{
-			echo "<p><a href='?m=user_games&amp;p=assign&amp;user_id=$_SESSION[user_id]'>".
+			echo "<p><a href='?m=user_games&amp;p=assign&amp;user_id=" . $_SESSION['user_id'] . "'>".
 				 get_lang("assign_game_homes") ."</a></p>";
 		}
 		return;

+ 4 - 4
modules/gamemanager/update_server_manual.php

@@ -67,7 +67,7 @@ function exec_ogp_module() {
 	$pid = isset($_REQUEST['pid']) ? $_REQUEST['pid'] : -1;
 	$filename = isset($_REQUEST['filename']) ? $_REQUEST['filename'] : "";
 
-	echo "<h2>". get_lang("install_update_manual") ." $home_info[home_name]</h2>";
+	echo "<h2>". get_lang("install_update_manual") ." " . $home_info['home_name'] . "</h2>";
 
 	if ( !empty($state) )
 	{
@@ -139,10 +139,10 @@ function exec_ogp_module() {
 			<input type='hidden' name='home_id' value='$home_id' />
 			<input type='hidden' name='mod_id' value='$mod_id' />
 			<input type='hidden' name='state' value='start' />
-			<tr><td align='right'>". get_lang("game_name") .":</td><td align='left'>$home_info[game_name]</td></tr>
-			<tr><td align='right'>". get_lang("dest_dir") .":</td><td align='left'>$home_info[home_path]</td></tr>
+			<tr><td align='right'>". get_lang("game_name") .":</td><td align='left'>" . $home_info['game_name'] . "</td></tr>
+			<tr><td align='right'>". get_lang("dest_dir") .":</td><td align='left'>" . $home_info['home_path'] . "</td></tr>
 			<tr><td align='right'>". get_lang("remote_server") .":</td>
-			<td align='left'>$home_info[remote_server_name] ($home_info[agent_ip]:$home_info[agent_port])</td></tr>
+			<td align='left'>" . $home_info['remote_server_name'] . " (" . $home_info['agent_ip'] . ":" . $home_info['agent_port'] . ")</td></tr>
 			<tr><td align='right'>". get_lang("file_url") .":</td>
 			<td align='left'><input type='text' id='url' name='url' value='' onChange='setFilename(this.value)' size='50' /></td></tr>
 			<tr><td colspan='2' class='info'>". get_lang("file_url_info") ."</td></tr>

+ 2 - 2
modules/mysql/edit_server.php

@@ -70,7 +70,7 @@ function exec_ogp_module() {
 	
 	foreach($server_homes as $server_home)
 	{
-		$homes_array["$server_home[home_id]"] = "(ID ".$server_home['home_id'].") ".$server_home['home_name'];
+		$homes_array[$server_home['home_id']] = "(ID ".$server_home['home_id'].") ".$server_home['home_name'];
 	}
 	
     if ( $mysql_server === FALSE )
@@ -571,7 +571,7 @@ function exec_ogp_module() {
 			foreach ( $mysql_server_dbs as $mysql_db )
 			{
 				$home_info = $db->getGameHomeWithoutMods($mysql_db['home_id']);
-				$db_array["$mysql_db[db_id]"] = $mysql_db['db_name']." (".$home_info['home_name'].")";
+				$db_array[$mysql_db['db_id']] = $mysql_db['db_name']." (".$home_info['home_name'].")";
 			}
 			
 			$ft = new FormTable();

+ 1 - 1
modules/mysql/user_db.php

@@ -67,7 +67,7 @@ function exec_ogp_module() {
 	$db_array["0"] = get_lang('select_db');
 	foreach ( $home_dbs as $home_db )
 	{
-		$db_array["$home_db[db_id]"] = $home_db['db_name'];
+		$db_array[$home_db['db_id']] = $home_db['db_name'];
 	}
 		
 	$ft = new FormTable();

+ 1 - 1
modules/server/servers.php

@@ -94,7 +94,7 @@ function exec_ogp_module() {
 						"'>[". get_lang("restart") ."]</a>\n".
 						"<a href='?m=server&amp;p=log&amp;rhost_id=".$server_row['remote_server_id']."'>[". get_lang("view_log") ."]</a>\n";
 			$tittle .= "<span class='success'>". get_lang("online") ."</span>";
-			$booble .= "<img src='images/magnifglass.png' data-url='home.php?m=server&p=mon_stats&remote_server_id=$server_row[remote_server_id]&type=cleared' weight='8' class='center' />";
+			$booble .= "<img src='images/magnifglass.png' data-url='home.php?m=server&p=mon_stats&remote_server_id=" . $server_row['remote_server_id'] . "&type=cleared' weight='8' class='center' />";
 		}
 		elseif( $host_stat === -1 )
 		{

+ 4 - 4
modules/user_admin/show_groups.php

@@ -113,10 +113,10 @@ function exec_ogp_module() {
 					": <select name=\"user_to_add\">";
 				foreach ($available_users as $user_row )
 				{
-					echo "<option value=\"$user_row[user_id]\">" . htmlentities($user_row[users_login]) . "</option>";
+					echo "<option value=\"" . $user_row['user_id'] . "\">" . htmlentities($user_row['users_login']) . "</option>";
 				}
 				echo "</select>\n";
-				echo "<input type='hidden' name='group_id' value='$row[group_id]' />";
+				echo "<input type='hidden' name='group_id' value='" . $row['group_id'] . "' />";
 				echo "<input type='submit' name='add_user_to_group' value='".get_lang('add_user')."' />";
 				echo "</form>\n";
 			}else{
@@ -140,7 +140,7 @@ function exec_ogp_module() {
                 $user_info = $db->getUserById($user_id['user_id']);
                 echo "<li><a href='?m=user_admin&amp;p=del_from_group&amp;group_id=".
                     $row['group_id']."&amp;user_id=".$user_id['user_id']."'>[".get_lang('remove_from_group').
-                    "]</a> $user_info[users_login]</li>";
+                    "]</a> " . $user_info['users_login'] . "</li>";
             }
             echo "</ul>";
         }
@@ -161,4 +161,4 @@ function exec_ogp_module() {
 	}
 	echo paginationPages($count_groups[0]['total'], $page_user, $limit_user, $uri, 3, 'userGroups');	
 }
-?>
+?>

+ 3 - 3
modules/user_admin/show_users.php

@@ -115,10 +115,10 @@ function exec_ogp_module() {
 			print "ownedby='" . $ownedBy . "'";
 		}
         print ">";
-        print "<td class='actions'><a href='?m=user_games&amp;p=assign&amp;user_id=$row[user_id]'>[".
+        print "<td class='actions'><a href='?m=user_games&amp;p=assign&amp;user_id=" . $row['user_id'] . "'>[".
             get_lang('assign_homes')."]</a><br />
-            <a href='?m=user_admin&amp;p=del&amp;user_id=$row[user_id]'>[".get_lang('delete')."]</a><br />
-            <a href='?m=user_admin&amp;p=edit_user&amp;user_id=$row[user_id]'>[".get_lang('edit_profile')."]</a></td>
+            <a href='?m=user_admin&amp;p=del&amp;user_id=" . $row['user_id'] . "'>[".get_lang('delete')."]</a><br />
+            <a href='?m=user_admin&amp;p=edit_user&amp;user_id=" . $row['user_id'] . "'>[".get_lang('edit_profile')."]</a></td>
             <td>".htmlentities($row['users_login'])."</td><td>".htmlentities($row['users_role'])."</td>
             <td>".htmlentities($row['users_email'])."</td>
             <td>$user_expires</td>";

+ 3 - 3
modules/user_games/assign_home.php

@@ -288,10 +288,10 @@ function exec_ogp_module()
 			$access_rights = empty($row['access_rights']) ? "-" : $row['access_rights'];
 			$type = $id_type == "group" ? "user_group_expiration_date" : "user_expiration_date";
 			$expiration = $row[$type] == "X" ? "X" : date('d/m/Y H:i:s', $row[$type]);
-			echo "<tr><td><input type=checkbox class='change_access_rights' data-home_id='$row[home_id]' >$row[home_id]</td>
+			echo "<tr><td><input type=checkbox class='change_access_rights' data-home_id='" . $row['home_id'] . "' >" . $row['home_id'] . "</td>
 				<td>".$row['agent_ip']." (Agent)</td>
-				<td>$row[game_name]</td>
-				<td>$row[home_path]</td>
+				<td>" . $row['game_name'] . "</td>
+				<td>" . $row['home_path'] . "</td>
 				<td>" . htmlentities($row["home_name"]) . "</td>
 				<td>$access_rights</td>
 				<td>$expiration</td>

+ 3 - 3
modules/user_games/check_expire.php

@@ -30,7 +30,7 @@ function exec_ogp_module()
 	{
 		foreach($expired_servers as $expired_server)
 		{
-			$db->logger(date('d/m/Y H:i:s', $expired_server['server_expiration_date'])." : SERVER EXPIRED: HOME ID:$expired_server[home_id] ($expired_server[home_name])");
+			$db->logger(date('d/m/Y H:i:s', $expired_server['server_expiration_date'])." : SERVER EXPIRED: HOME ID:" . $expired_server['home_id'] . " (" . $expired_server['home_name'] . ")");
 			$db->check_expire_date(0, $expired_server['home_id'], array('server'));
 		}
 	}
@@ -40,7 +40,7 @@ function exec_ogp_module()
 	{
 		foreach($expired_users as $expired_user)
 		{
-			$db->logger(date('d/m/Y H:i:s', $expired_user['user_expiration_date'])." : USER ASSIGNATION EXPIRED : HOME ID:$expired_user[home_id] TO USER ID:$expired_user[user_id]");
+			$db->logger(date('d/m/Y H:i:s', $expired_user['user_expiration_date'])." : USER ASSIGNATION EXPIRED : HOME ID:" . $expired_user['home_id'] . " TO USER ID:" . $expired_user['user_id']);
 			$db->check_expire_date($expired_user['user_id'], $expired_user['home_id'], array('user'));
 		}
 	}
@@ -55,7 +55,7 @@ function exec_ogp_module()
 	{
 		foreach($expired_groups as $expired_group)
 		{
-			$db->logger(date('d/m/Y H:i:s', $expired_group['user_group_expiration_date'])." : GROUP ASSIGNATION EXPIRED : HOME ID:$expired_group[home_id] TO GROUP ID:$expired_group[group_id]");
+			$db->logger(date('d/m/Y H:i:s', $expired_group['user_group_expiration_date'])." : GROUP ASSIGNATION EXPIRED : HOME ID:" . $expired_group['home_id'] . " TO GROUP ID:" . $expired_group['group_id']);
 			$db->check_expire_date($expired_group['user_id'], $expired_group['home_id'], array('user_group'));
 		}
 	}

+ 1 - 1
modules/user_games/edit_home.php

@@ -604,7 +604,7 @@ function exec_ogp_module()
 			if ( $db->updateExpirationDate($home_id, $_POST['expiration_date'], 'server') === TRUE )
 			{
 				echo json_encode(array('result' => 'success', 'info' => get_lang('expiration_date_changed')));
-				$db->logger( get_lang('expiration_date_changed') ." [ HOME ID:$home_id NEW DATE:$_POST[expiration_date] ]");
+				$db->logger( get_lang('expiration_date_changed') ." [ HOME ID:$home_id NEW DATE:" . $_POST['expiration_date'] ." ]");
 			}
 			else
 				echo json_encode(array('result' => 'failure', 'info' => get_lang('expiration_date_could_not_be_changed')));

+ 2 - 2
modules/user_games/home_mods.php

@@ -114,7 +114,7 @@ function exec_ogp_module()
 			 "<td><b>". get_lang("nice_level") ."</b></td><td></td>".
 			 "</tr>\n";
 		foreach ( $enabled_mods as $enabled_rows ) {
-			echo "<tr id='mod_cfg_id_$enabled_rows[mod_cfg_id]'>".
+			echo "<tr id='mod_cfg_id_" . $enabled_rows['mod_cfg_id'] . "'>".
 				 "<td><a href='?m=user_games&amp;p=edit&amp;mod_id=".$enabled_rows['mod_id'].
 				 "&amp;home_id=$home_id&amp;submit=delete_mod'>[ ". get_lang("remove_mod") ." ]</a><br>".
 				 "<a href='?m=user_games&p=install_cmds&home_id=$home_id&mod_id=".$enabled_rows['mod_id'].
@@ -134,7 +134,7 @@ function exec_ogp_module()
 				 echo create_drop_box_from_array(array_merge(range(-19,19)),
 				 'nice',$enabled_rows['nice']).
 				 "</td><td>\n".
-				 "<button class='set_options' id='$enabled_rows[mod_cfg_id]' >". get_lang("set_options") ."</button>\n".
+				 "<button class='set_options' id='" . $enabled_rows['mod_cfg_id'] . "' >". get_lang("set_options") ."</button>\n".
 				 "</td></tr>\n";
 
 		}

+ 5 - 5
modules/user_games/show_homes.php

@@ -84,15 +84,15 @@ function exec_ogp_module()
 
 		$os_arch = preg_match('/win/',$row['game_key']) ? "(Windows" : "(Linux";
 		$os_arch .= preg_match('/(win|linux)64/',$row['game_key']) ? " 64bit)" : ")";
-		echo "<tr class='tr".($i++%2)."'><td class='tdh'>$row[home_id]</td><td>".$display_ip."</td>".
-			 "<td class='tdh'>$row[game_name] $os_arch</td><td>$row[home_path]<br><div class='size' id='".$row["home_id"].
+		echo "<tr class='tr".($i++%2)."'><td class='tdh'>" . $row['home_id'] . "</td><td>".$display_ip."</td>".
+			 "<td class='tdh'>" . $row['game_name'] . $os_arch . "</td><td>" . $row['home_path'] . "<br><div class='size' id='".$row["home_id"].
 			 "' style='cursor:pointer;' >[".get_lang('get_size')."]</div></td><td class='tdh'>";
 		echo empty($row['home_name']) ? get_lang('not_available') : htmlentities($row['home_name']);
 		$expiration_date = $row['server_expiration_date'] == "X" ? "X" : date('d/m/Y H:i:s', $row['server_expiration_date']);
 		echo "</td><td>".$expiration_date."</td><td>
-			<a href='?m=user_games&amp;p=del&amp;home_id=$row[home_id]'>[".get_lang('delete')."]</a>
-			<a href='?m=user_games&amp;p=edit&amp;home_id=$row[home_id]'>[".get_lang('edit')."]</a>
-			<a href='?m=user_games&amp;p=clone&amp;home_id=$row[home_id]'>[".get_lang('clone')."]</a>
+			<a href='?m=user_games&amp;p=del&amp;home_id=" . $row['home_id'] . "'>[".get_lang('delete')."]</a>
+			<a href='?m=user_games&amp;p=edit&amp;home_id=" . $row['home_id'] . "'>[".get_lang('edit')."]</a>
+			<a href='?m=user_games&amp;p=clone&amp;home_id=" . $row['home_id'] . "'>[".get_lang('clone')."]</a>
 			</td></tr>";
 	}
 	

+ 2 - 2
ogp_api.php

@@ -862,7 +862,7 @@ function api_user_admin()
 			if ($db->unassignHomeFrom("user",$account['user_id'],$game_home['home_id']) === TRUE)
 			{
 				$status = "200";
-				$message = "Home id#$game_home[home_id] has been unnassigned from $email successfully.";
+				$message = "Home id#" . $game_home['home_id'] . " has been unnassigned from $email successfully.";
 			}
 			else
 			{
@@ -1375,7 +1375,7 @@ function api_gamemanager()
 				
 				$remote->masterServerUpdate($home_id,$home_info['home_path'],$ms_home_id,$ms_info['home_path'],$exec_folder_path,$exec_path,$preInstallCMD,$postInstallCMD);
 				$status = "200";
-				$message = "Installation from master server ($home_info[home_name]) started";
+				$message = "Installation from master server (" . $home_info['home_name'] . ") started";
 			}
 			else
 			{

+ 2 - 2
test_api.php

@@ -35,11 +35,11 @@ if(isset($_POST['function']))
 <form method='<?php echo $_POST['method']; ?>' action="ogp_api.php<?php echo $get_req; ?>" target="_blank">
 <?php
 if($_POST['method'] == 'GET')
-	echo "<input type=hidden name='$_POST[function]'>";
+	echo "<input type=hidden name='" . $_POST['function'] . "'>";
 ?>
 <table>
 <?php
-$inputs = $functions["$_POST[function]"];
+$inputs = $functions[$_POST['function']];
 foreach($inputs as $input => $mandatory)
 {
 	$is_mandatory = $mandatory?"":" (optional)";