Popup issue in IE6 and 7 v Firefox

[mkyle1]

Hi,

Mr Gates has a lot to answer for. I am using a popup window for my site below. When I select Team 1992 on the left and then click on a fixture on Opponents (Cumbernauld United) in firefox it pops up and displays the full match report (pulled from the database). However if I do this in IE6 or 7 the full report is not displayed. The popup script is below. I believe it is something to do with the 2k limit for the GET command but I have not idea how to fix it

Martin

http://www.steinsthistle.co.uk

<?
//$report 	= $var1 = $_GET["var1"];
//$opponent 	= $var2 = $_GET["var2"];



//print("Opponent = <u>$opponent</u><br><br>"); 
//echo stripslashes($var1); ?>
<?
$report 	= $var1 = $_GET["var1"];
$opponent 	= $var2 = $_GET["var2"];

include("dbinfo_steins.php");
$link = mysql_connect(localhost,$username,$password);
@mysql_select_db($database) or die( "Unable to select database .....");

		$query = "SELECT * FROM fixtures WHERE opponents = \"$report\"";
		$mysql_result = mysql_query($query);
		if($mysql_result == 0)
		{	
			//print("<br><br><br><B>NO NEW DVDs at this time<B><br><br>");
		}
		else
		{
			while($row = mysql_fetch_row($mysql_result))
			{
				$item_00 	= $row[0];
				$item_01 	= $row[1];
				$item_02 	= $row[2];
				$item_03 	= $row[3];
				$item_04 	= $row[4];
				$item_05 	= $row[5];
				$item_06 	= $row[6];

			}

print("Opponent = <u>$var2</u><br><br>");			
echo stripslashes($var1);
//print("<td><center>$report</center></td>");

		}
?>

[jimbovsat]

Well, before you decide that Mr. Gates is at fault, I'd start by cleaning all that unencoded HTML off the GET request.

"... php?var1=<p>Allandale 7:00 pm 8th ... "

[mkyle1]

Hi,

please can you give me an idea of the best way to clean up the below as suggested:

?php    

include("../dbinfo_steins.php");
mysql_connect(localhost,$username,$password);
@mysql_select_db($database) or die( "Unable to select database");

$query_team = "SELECT * FROM fixtures WHERE team = \"1992s\"";
$mysql_result = mysql_query($query_team);

$query_teaminfo = "SELECT * FROM fixtures_info WHERE team = \"1992s\"";
$mysql_resulti = mysql_query($query_teaminfo);

	if($mysql_result == 0)
	{	
	}
	else
	{
		while($row = mysql_fetch_row($mysql_result))
		{
			$date 	= $row[1];
			$opponent 	= $row[2];
			$fixture	= $row[3];
			$result	= $row[4];
			$report	= $row[5];

			print("<tr>");
			print("<td><div align=\"center\">$date</div></td>");

			print("<td><div align=\"center\"><a href=javascript:popcontact(\"pop-up.php?var1=$report&var2=$opponent\")>$opponent</a></div></td>");

			/*print("<td><div align=\"center\"><a href=\"pop-up.php?var1=$report&var2=$opponent\" target=\"_blank\">$opponent</a></div></td>");*/
			print("<td><div align=\"center\">$fixture</div></td>");
			print("<td><div align=\"center\">$result</div></td>");
			print("</tr>");
		}
	}

	if($mysql_resulti == 0)
	{	
	}
	else
	{
		while($row = mysql_fetch_row($mysql_resulti))
		{
			$link 	= $row[1];
			$news		= $row[2];
			$linkname	= $row[3];

			print("<tr><td colspan=\"4\"><div align=\"center\">League Link: <span class=\"style11\"><a href=\"$link\" target=\"_blank\">$linkname</a></span></div></td></tr></table>");
			print("<blockquote><p align=\"center\" class=\"style15\">TEAM NEWS");
			print("<p class=\"style13\">$news</p>");
		}
	}

mysql_close();
?>

Thanks, Martin

[NogDog]

You should not be trying to transmit text like that via GET. Besides the fact that -- as you have already discovered -- there are limits to the amount of data that can be transmitted that way (and there is no defined limit so each client will be different), but it is also a waste of bandwidth. A more practical approach would be to just send a few variables and values with which the called PHP page can then generate the desired content. E.g.: the link might be something like:

http://www.example.com/pop_up.php?year=1992&team=team_id

Then pop_up.php would use the variables $GET['year'] and $GET['team_id'] to query the database to get the desired text (or otherwise grab the output from a file or however you need to do it).