[RESOLVED] UPDATE works in FireFox, but not IE8 or Chrome

robkir

Something is weird. My update statement works fine in FireFox, but the client is using IE8 and the UPDATE fails. It also fails in Chrome. I am suspecting that something is wrong in my FOREACH statement--but I can't spot it. Any ideas?

//---| UPDATE
	if($_POST['submit']=="Save")
	{
		foreach($_POST['teamGroup_ud'] as $coachID => $value) 
		{
    		$teamGroup = (protect($_POST['teamGroup_ud'][$coachID]));
    		$teamPool = (protect($_POST['teamPool_ud'][$coachID])) ;
    		$teamFlight = (protect($_POST['teamFlight_ud'][$coachID])) ;
			$regionID = (protect($_POST['regionID_ud'][$coachID])) ;

			$query = "UPDATE team_info SET
			regionID ='$regionID' ,
			teamGroup ='$teamGroup' ,
			teamPool ='$teamPool' ,
			teamFlight ='$teamFlight'
			WHERE coachID = '$coachID' LIMIT 1";
	    	mysql_query($query) or die ("Error in query: $query");

	}			
}

Here's the entire code, including the FOREACH statement just in case you have any other thoughts.

<?php
session_start();
require("../validateAdmin.php");
require("../functions/checkEntries.php");
include_once "../DBconMOD.php";
include_once "../style_sheet.php";

//---| UPDATE
	if($_POST['submit']=="Save")
	{
		foreach($_POST['teamGroup_ud'] as $coachID => $value) 
		{
    		$teamGroup = (protect($_POST['teamGroup_ud'][$coachID]));
    		$teamPool = (protect($_POST['teamPool_ud'][$coachID])) ;
    		$teamFlight = (protect($_POST['teamFlight_ud'][$coachID])) ;
			$regionID = (protect($_POST['regionID_ud'][$coachID])) ;

			$query = "UPDATE team_info SET
			regionID ='$regionID' ,
			teamGroup ='$teamGroup' ,
			teamPool ='$teamPool' ,
			teamFlight ='$teamFlight'
			WHERE coachID = '$coachID' LIMIT 1";
	    	mysql_query($query) or die ("Error in query: $query");

	}			
}
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Language" content="en-us" />
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<?php echo "<title>Control Panel </title>";?>
<link rel="stylesheet" type="text/css" href="../../templates/master_style_sheet.css" />
<link rel="stylesheet" type="text/css" href="../templates/master_style_sheet.css">
<link rel="stylesheet" type="text/css" href="../style_sheets/tournament_style_sheet.css">

<style type="text/css">
.table_small_type {
	border: 1px solid #008000;
	text-align: center;
	font-family: Arial, Helvetica, sans-serif;
	font-size: xx-small;
	text-decoration: underline;
}
.coach_left_just {
	border: 1px solid #008000;
	text-align: left;
	font-family: Arial, Helvetica, sans-serif;
	font-size: small;
}
</style>

</head>

<body> <span class="red_italic_text_medium">
<img src="../artwork/CupLogo.png" width="265" height="94">Assign Groups and 
Pools<hr></span>
<?php 
	include('../navigation/admin_menu.htm');
?>
<!-- DROP DOWN MENU to choose division -->	
	 	<form method="post" name="divSearch"><label class="small_type"><strong>Division:</strong></label>
	 			<select name="showWhat" onchange="javascript:document.divSearch.submit()"> 
				<option value="View All"> Choose </option>
				<option value="U10 Boys"> U10 Boys</option>
				<option value="U12 Boys">U12 Boys</option>
				<option value="U14 Boys">U14 Boys</option>
				<option value="U16 Boys">U16 Boys</option>
				<option value="U19 Boys">U19 Boys</option>
				<option value="U10 Girls">U10 Girls</option>
				<option value="U12 Girls">U12 Girls</option>
				<option value="U14 Girls">U14 Girls</option>
				<option value="U16 Girls">U16 Girls</option>
				<option value="U19 Girls">U19 Girls</option>

			</select> 

				<input name="Submit1" type="submit" value="See Division" />&nbsp;
		<span class="small_type">PRINT </span><span class="cell_borderLeft">
		&nbsp;<a target="_self" href="../pools/display_pools.php">Pools</a>&nbsp;</span><br>
</form>
<p><span class="small_type">Use this page to assign the Group and Pool 
information to each team. <strong>ONLY </strong>teams who have been approved 
by TD will appear. If Team Type, Team Rating and or Team Age does not 
appear--it means that the TD approved the tournament, but the coach did not 
fill those fields in on their applications. Note: Be aware that this version does not take 
into account Coed Teams. We may have to manually modify those very few teams 
to to fit the programs needs. </span></p>

<form method="post">
<table style="width: 100%" class="cell_border">
	<tr class="background_color">
	<td class="cell_border" width="5%"><strong>Group</strong></td>
	<td class="cell_border" width="5%"><strong>Team #</strong></td>
	<td class="cell_border" width="5%"><strong>Flight</strong></td>
	<td class="cell_border" width="5%"><strong>Region</strong></td>
	<td class="cell_border" width="15%"><strong>Reg Name</strong></td>
	<td class="cell_border" width="5%"><strong>Reg ID</strong></td>		
	<td class="cell_border" width="20%"><strong>Coach</strong></td>
	<td class="cell_border" width="25%"><strong>Team Type</strong></td>
	<td class="cell_border" width="5%"><strong>Team Rate</strong></td>

	<td class="cell_border" width="5%"><strong>Team ID#</strong></td>

	</tr>
<?php 

//---|Assign Flight colors
	$red= "Red"; 
	$blue = "Blue";
	$green = "Green"; 
	$orange= "Orange"; 

//--| load records from team_info 

			if ($_POST['showWhat'])
			{
				$showWhat = $_POST['showWhat']; 
				$_SESSION['old'] = $showWhat; 
					}
						else
					{
						$showWhat = $_SESSION['old'];
			}			

		print "<span class=\"background_color\">Displaying<strong> " . $showWhat . "</strong></span><br/>";


			$result=mysql_query("SELECT * 
			FROM team_info 
			WHERE whatDiv= '".$showWhat."' && packetApproved = 1
			ORDER BY teamRating + 0 DESC");

//ORDER BY teamFlight, teamGroup, teamPool ASC");

				while ($contact = mysql_fetch_array($result)) 
				{

?>	
		<tr>

		<td class="cell_border">
			<select name="
			<?php echo '
			teamGroup_ud[' . $contact['coachID'] . ']'; ?> ">
			<option value=""></option>
			<option value="1">1</option>
			<option value="2">2</option>
			<option value="3">3</option>
			<option value="4">4</option>
			<option value="5">5</option>
			<option value="6">6</option>
			<option value="7">7</option>
			<option value="8">8</option>
			<option value ="<?php echo $contact['teamGroup'];?>"
    		selected><?php echo $contact['teamGroup'];?></option>
			</select> 

			<td class="cell_border">
			<select name="
			<?php echo '
			teamPool_ud[' . $contact['coachID'] . ']'; ?> ">
			<option value=""></option>
			<option value="A">A</option>
			<option value="AA">AA</option>
			<option value="B">B</option>
			<option value="BB">BB</option>
			<option value="C">C</option>
			<option value="D">D</option>
			<option value="E">E</option>
			<option value="F">F</option>
			<option value="G">G</option>
			<option value="H">H</option>
			<option value="I">I</option>
			<option value="J">J</option>
			<option value="K">K</option>
			<option value="L">L</option>
			<option value="M">M</option>
			<option value="N">N</option>
			<option value="O">O</option>
			<option value="P">P</option>
			<option value="Q">Q</option>
			<option value="R">R</option>
			<option value="S">S</option>
			<option value="T">T</option>
			<option value="U">U</option>
			<option value="V">V</option>
			<option value="W">W</option>
			<option value="X">X</option>
			<option value="Y">Y</option>
			<option value="Z">Z</option>
			<option value ="<?php echo (htmlentities($contact['teamPool']));?>"
    		selected><?php echo (htmlentities($contact['teamPool']));?></option>
			</select> 
<!-- Team Flight Color -->					
				<td class="cell_border">
				<select name="
				<?php echo '
				teamFlight_ud[' . $contact['coachID'] . ']'; ?> ">
				<option value=""></option>
				<option value="Red">Red</option>
				<option value="Blue">Blue</option>
				<option value="Green">Green</option>
				<option value="Orange">Orange</option>

			<option value ="<?php echo (htmlentities($contact['teamFlight']));?>"
    		selected><?php echo (htmlentities($contact['teamFlight']));?></option>
			</select> 



		<td class="cell_border"><?php echo $contact['region'] ;?></td>
		<td class="cell_border"><?php echo $contact['regionName'] ;?></td>


		<td class="cell_border">
			<?php echo '<input type ="text"
				style="background-color: #81F7F3;" 
			    name ="regionID_ud[' . $contact['coachID'] . ']" 
				value="' . (htmlentities($contact['regionID'])) . '">'; ?><style="background-color: #81F7F3;">
				</td>

		<td class="cell_border"><?php echo $contact['coachLast'] . ", " . $contact['coachFirst']  ;?></td>
		<?php 
				$teamType_convert = $contact['teamType'];
					if ($teamType_convert == 1)
						$teamType = "Select Team" ;
					if ($teamType_convert == 2)
						$teamType = "One of several Select Teams" ;
					if ($teamType_convert == 3)
						$teamType = "Fall Season" ;
		?>
		<td class="cell_border"><?php echo $teamType ;?></td>
		<td class="cell_border"><?php echo $contact['teamRating'] ;?></td>

		<td class="cell_border"><?php echo $contact['coachID'] ;?></td>

	</tr>


<?php 
						} // end while loop
?>
			</table>
			<input name="submit" type="submit" value="Save"></form>
</body>
</html>

Thanks.
Robkir

bradgrafelman

Since PHP is server-side, it must be your client-side code (a.k.a. HTML/Javascript) that's causing a problem.

I would try removing all of the whitespace (line breaks, spaces, etc.) from the SELECT element's name values first.

robkir

Hi bradgrafelman,

Your explanation makes sense. I tried it, but the problem does not appear to be in either the SELECT or UPDATE code. I'm not sure what else I should fiddle with and any other suggestions would be most welcome.

In the meantime, since I am under pressure to deliver, and since it works with Firefox, I got the client to install Firefox. Still...I don't like it when things go astray and I can't figure out why.

Thanks,
robkir

cretaceous

I've noticed before that Fireox seems to expose Post variables which have not been explicitly accessed
( a bit like doing extract($request) -pseudocode)

IE does not - so must be a bug in FF I think

I can see you explicitly access your POST variables but that may be a route to investigate

robkir

cretaceous;10946338 wrote:
I can see you explicitly access your POST variables but that may be a route to investigate

Hi cretaceous,

I am not sure what 'explicitly accessing the POST' refers to--is there another way to do it? Should I be using $REQUEST rather than $POST.

Your observation that it might be a bug in FF seems spot on-- especially since the UPDATE failed in Chrome as well as IE.

Thanks for your input.

cretaceous

stick with $_POST
the best thing is to print your variables to screen, to see if they are what you expect

put this above the /-- Update and see what you get:
echo 'updating<br>';
echo $POST['submit'];
print_r($POST['teamGroup_ud']); //its an array so this will show its contents

but it could be an earlier script that is not working correctly

NOTE EDIT ON ECHO LINES IF YOU LOOK AT THE EMAIL FROM FORUM!

robkir

I didn't get any data from your suggested code, except, of course, the echoed word 'updating'. I don't think a previous script could be causing it, since it is the starting thread for the module set. This is so weird. I might need to have a second cup of coffee to wrap my brain around it.

Thanks

bradgrafelman

Does this code consistently not work in IE and Chrome?

If you have access to one of the incompatible browsers, try doing a print_r($_POST); at the top of the script (and perhaps do a die() as well, just for debugging purposes - all we want to see are the POST values and nothing else). That should dump all POST'ed elements to screen.

Copy and paste the output here so we can see what's being POST'ed from the browsers that don't seem to be working. I still can't help but wonder if the problem isn't caused by all of the whitespace you have in the name attributes (unless you removed this...?).

robkir

Both IE and Chrome repeat the same problem. Here's the output from array:

Array ( [submit] => Save [ teamGroup_ud] => Array ( [78] => 1 [110] => 2 [139] => 1 [194] => 1 [227] => 2 [265] => 3 [101] => 3 [167] => [175] => 1 [201] => 2 [259] => 3 [109] => 2 [119] => 5 [157] => 4 [238] => 3 [44] => 6 [59] => 4 [86] => 5 [106] => 6 [164] => 7 [276] => 7 [61] => 4 [100] => 7 [208] => 6 [217] => 6 [236] => 7 [46] => 5 [271] => 4 [62] => 5 ) [ teamPool_ud] => Array ( [78] => C [110] => F [139] => A [194] => B [227] => E [265] => J [101] => K [167] => [175] => D [201] => G [259] => L [109] => H [119] => Q [157] => M [238] => I [44] => U [59] => N [86] => S [106] => V [164] => Y [276] => Z [61] => O [100] => BB [208] => X [217] => W [236] => AA [46] => T [271] => P [62] => R ) [ teamFlight_ud] => Array ( [78] => Red [110] => Red [139] => Red [194] => Red [227] => Red [265] => Red [101] => Red [167] => [175] => Red [201] => Red [259] => Red [109] => Red [119] => Blue [157] => Blue [238] => Red [44] => Blue [59] => Blue [86] => Blue [106] => Blue [164] => Blue [276] => Blue [61] => Blue [100] => Blue [208] => Blue [217] => Blue [236] => Blue [46] => Blue [271] => Blue [62] => Blue ) [regionID_ud] => Array ( [78] => PV #2dd [110] => [139] => NT #2 [194] => SM #4 [227] => [265] => [101] => ST #2 [167] => [175] => WT #2 [201] => CT #1 [259] => PV #5 [109] => SM #2 [119] => NT #1 [157] => [238] => [44] => [59] => PV #4 [86] => SM #1 [106] => [164] => [276] => CT #2 [61] => [100] => ST #1 [208] => WT #1 [217] => P Pali #2 [236] => PV #3 [46] => P Pali #1 [271] => SM #3 [62] => ST #3 ) )

As you suggested I removed all of the whitespace (line breaks, spaces, etc.) from the SELECT element's name values. Perhaps there are other areas I missed?

I'll forward a copy of the code in another thread reply, otherwise this post will be too long.

robkir

Here's the complete code

Thanks,

<?php
session_start();
require("../validateAdmin.php");
require("../functions/checkEntries.php");
include_once "../DBconMOD.php";
include_once "../style_sheet.php";
print_r($_POST) or die('no data!');
/* (cretaceous' code below)
echo 'updating<br>';
echo $_POST['teamGroup_ud'][$coachID];
echo $_POST['submit'];
print_r($_POST['teamGroup_ud']); //its an array so this will show its contents
*/
//---| UPDATE
	if($_POST['submit']=="Save")
	{
		foreach($_POST['teamGroup_ud'] as $coachID => $value) 
		{
    		$teamGroup = (protect($_POST['teamGroup_ud'][$coachID]));
    		$teamPool = (protect($_POST['teamPool_ud'][$coachID])) ;
    		$teamFlight = (protect($_POST['teamFlight_ud'][$coachID])) ;
			$regionID = (protect($_POST['regionID_ud'][$coachID])) ;

$query = "UPDATE team_info SET regionID ='$regionID', teamGroup='$teamGroup', teamPool='$teamPool',teamFlight='$teamFlight' WHERE coachID='$coachID' LIMIT 1";
mysql_query($query) or die ("Error in query: $query");
		}			
	}
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Language" content="en-us" />
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<link rel="stylesheet" type="text/css" href="../style_sheets/tournament_style_sheet.css">
<style type="text/css">
.table_small_type {
	border: 1px solid #008000;
	text-align: center;
	font-family: Arial, Helvetica, sans-serif;
	font-size: xx-small;
	text-decoration: underline;
}
.coach_left_just {
	border: 1px solid #008000;
	text-align: left;
	font-family: Arial, Helvetica, sans-serif;
	font-size: small;
}
</style>

</head>

<body> <span class="red_italic_text_medium">
<img src="../artwork/CupLogo.png" width="265" height="94">Assign Groups and 
Pools<hr></span>
<?php 
	include('../navigation/admin_menu.htm');
?>
<!-- DROP DOWN MENU to choose division -->	
	 	<form method="post" name="divSearch"><label class="small_type"><strong>Division:</strong></label>
	 			<select name="showWhat" onchange="javascript:document.divSearch.submit()"> 
				<option value="View All"> Choose </option>
				<option value="U10 Boys"> U10 Boys</option>
				<option value="U12 Boys">U12 Boys</option>
				<option value="U14 Boys">U14 Boys</option>
				<option value="U16 Boys">U16 Boys</option>
				<option value="U19 Boys">U19 Boys</option>
				<option value="U10 Girls">U10 Girls</option>
				<option value="U12 Girls">U12 Girls</option>
				<option value="U14 Girls">U14 Girls</option>
				<option value="U16 Girls">U16 Girls</option>
				<option value="U19 Girls">U19 Girls</option>

			</select> 

				<input name="Submit1" type="submit" value="See Division" />&nbsp;
		<span class="small_type">PRINT </span><span class="cell_borderLeft">
		&nbsp;<a target="_self" href="../pools/display_pools.php">Pools</a>&nbsp;</span><br>
</form>
<p><span class="small_type">Use this page to assign the Group and Pool 
information to each team. <strong>ONLY </strong>teams who have been approved 
by TD will appear. If Team Type, Team Rating and or Team Age does not 
appear--it means that the TD approved the tournament, but the coach did not 
fill those fields in on their applications. Note: Be aware that this version does not take 
into account Coed Teams. We may have to manually modify those very few teams 
to to fit the programs needs. </span></p>

<form method="post">
<input name="submit" type="submit" value="Save">

<table style="width: 100%" class="cell_border">
	<tr class="background_color">
	<td class="cell_border" width="5%"><strong>Group</strong></td>
	<td class="cell_border" width="5%"><strong>Team #</strong></td>
	<td class="cell_border" width="5%"><strong>Flight</strong></td>
	<td class="cell_border" width="5%"><strong>Region</strong></td>
	<td class="cell_border" width="15%"><strong>Reg Name</strong></td>
	<td class="cell_border" width="5%"><strong>Reg ID</strong></td>		
	<td class="cell_border" width="20%"><strong>Coach</strong></td>
	<td class="cell_border" width="25%"><strong>Team Type</strong></td>
	<td class="cell_border" width="5%"><strong>Team Rate</strong></td>

	<td class="cell_border" width="5%"><strong>Team ID#</strong></td>

	</tr>
<?php 

//---|Assign Flight colors
	$red= "Red"; 
	$blue = "Blue";
	$green = "Green"; 
	$orange= "Orange"; 

//--| load records from team_info 

			if ($_POST['showWhat'])
			{
				$showWhat = $_POST['showWhat']; 
				$_SESSION['old'] = $showWhat; 
					}
						else
					{
						$showWhat = $_SESSION['old'];
			}			

		print "<span class=\"background_color\">Displaying<strong> " . $showWhat . "</strong></span><br/>";



$result=mysql_query("SELECT * FROM team_info WHERE whatDiv='".$showWhat."'&&packetApproved = 1 ORDER BY teamRating +0 DESC");

				while ($contact = mysql_fetch_array($result)) 
				{					
?>	
		<tr>

		<td class="cell_border">
			<select name="
			<?php echo '
			teamGroup_ud[' . $contact['coachID'] . ']'; ?> ">
			<option value=""></option>
			<option value="1">1</option>
			<option value="2">2</option>
			<option value="3">3</option>
			<option value="4">4</option>
			<option value="5">5</option>
			<option value="6">6</option>
			<option value="7">7</option>
			<option value="8">8</option>
			<option value ="<?php echo $contact['teamGroup'];?>"
    		selected><?php echo $contact['teamGroup'];?></option>
			</select> 

			<td class="cell_border">
			<select name="
			<?php echo '
			teamPool_ud[' . $contact['coachID'] . ']'; ?> ">
			<option value=""></option>
			<option value="A">A</option>
			<option value="AA">AA</option>
			<option value="B">B</option>
			<option value="BB">BB</option>
			<option value="C">C</option>
			<option value="D">D</option>
			<option value="E">E</option>
			<option value="F">F</option>
			<option value="G">G</option>
			<option value="H">H</option>
			<option value="I">I</option>
			<option value="J">J</option>
			<option value="K">K</option>
			<option value="L">L</option>
			<option value="M">M</option>
			<option value="N">N</option>
			<option value="O">O</option>
			<option value="P">P</option>
			<option value="Q">Q</option>
			<option value="R">R</option>
			<option value="S">S</option>
			<option value="T">T</option>
			<option value="U">U</option>
			<option value="V">V</option>
			<option value="W">W</option>
			<option value="X">X</option>
			<option value="Y">Y</option>
			<option value="Z">Z</option>
			<option value ="<?php echo (htmlentities($contact['teamPool']));?>"
    		selected><?php echo (htmlentities($contact['teamPool']));?></option>
			</select> 
<!-- Team Flight Color -->					
				<td class="cell_border">
				<select name="
				<?php echo '
				teamFlight_ud[' . $contact['coachID'] . ']'; ?> ">
				<option value=""></option>
				<option value="Red">Red</option>
				<option value="Blue">Blue</option>
				<option value="Green">Green</option>
				<option value="Orange">Orange</option>

			<option value ="<?php echo (htmlentities($contact['teamFlight']));?>"
    		selected><?php echo (htmlentities($contact['teamFlight']));?></option>
			</select> 

		<td class="cell_border"><?php echo $contact['region'] ;?></td>
		<td class="cell_border"><?php echo $contact['regionName'] ;?></td>

		<td class="cell_border">
			<?php echo '<input type ="text"
				style="background-color: #81F7F3;" 
			    name ="regionID_ud[' . $contact['coachID'] . ']" 
				value="' . (htmlentities($contact['regionID'])) . '">'; ?><style="background-color: #81F7F3;">
				</td>

		<td class="cell_border"><?php echo $contact['coachLast'] . ", " . $contact['coachFirst']  ;?></td>
		<?php 
				$teamType_convert = $contact['teamType'];
					if ($teamType_convert == 1)
						$teamType = "Select Team" ;
					if ($teamType_convert == 2)
						$teamType = "One of several Select Teams" ;
					if ($teamType_convert == 3)
						$teamType = "Fall Season" ;
		?>
		<td class="cell_border"><?php echo $teamType ;?></td>
		<td class="cell_border"><?php echo $contact['teamRating'] ;?></td>

		<td class="cell_border"><?php echo $contact['coachID'] ;?></td>

	</tr>

<?php 
						} // end while loop
?>
			</table>
			<input name="submit" type="submit" value="Save"></form>
</body>
</html>

bradgrafelman

The problem is indeed with whitespace - look at the output you posted: there's a space after the opening '[' bracket for the array index.

You still haven't removed all of the whitespace here, though:

<select name=" 
                <?php echo ' 
                teamGroup_ud[' . $contact['coachID'] . ']'; ?> ">

robkir

And that was it!!!! Problem solved.

I learned a couple of valuable pieces of information today-- thanks. I very much appreciate your help.

Cheers,
robkir

bradgrafelman

Glad I could help!

Don't forget to mark this thread resolved using the link on the Thread Tools menu.