• Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 68
  • Last Modified:

php problem?

I have a very simple php program (attached). It is linked to from another place.

In Firefox, I get the attached jpg.

In Chrome I get a blank screen.

I ran the php syntax through http://phpcodechecker.com/. No issues.

What's wrong?
clear_sess.php
php_issue.jpg
0
Richard Korts
Asked:
Richard Korts
  • 3
  • 2
1 Solution
 
Ray PaseurCommented:
Please use the code snippet feature to post code samples here at E-E, thanks.
<?php
// clear_sess.php
function conv_date($x) {
		$d = explode("-", $x);
		$r = $d[1] . "/" . $d[2] . "/" . $d[0];
		return $r;
	}
include "db_connect.php";
session_start();
$qry = "SELECT * from users where uid = '" . $_SESSION['uid'] . "'";
$uid = $_SESSION['uid'];
$res = mysqli_query($link, $qry);
$nr = mysqli_num_rows($res);
$u = mysqli_fetch_array($res,MYSQLI_ASSOC);
session_unset();
$_SESSION['uid'] = $u['uid'];
//echo $u['firm_name'] . "<br>";
$_SESSION['firmname'] = $u['firm_name'];
$_SESSION['addr1'] = $u['addr1'];
$_SESSION['addr2'] = $u['addr2'];
$_SESSION['phone'] = $u['phone'];
$_SESSION['tester'] = $u['tester'];
$_SESSION['tg_make_model'] = $u['tg_make_model'];
$_SESSION['tg_sn'] = $u['tg_sn'];
$_SESSION['tg_date_cal'] = conv_date($u['tg_date_cal']);
$_SESSION['city_tester_no'] = $u['city_tester_no'];
if(! empty($_SERVER['HTTP_USER_AGENT'])){
		$useragent = $_SERVER['HTTP_USER_AGENT'];
		if( preg_match('@(iPad|iPod|iPhone|Android|BlackBerry|SymbianOS|SCH-M\d+|Opera Mini|Windows CE|Nokia|SonyEricsson|webOS|PalmOS)@', $useragent) ){
			$loc = "phone.php";
		}
	} else {
	$loc = "form.php";	
	}
header("Location: " . $loc);
exit;
?>

Open in new window

0
 
Ray PaseurCommented:
Questions...

1. Did the session start correctly?  How do you know this (either yes or no)?

2. Did the query work?  How do you know this (either yes or no)?

3. What is the purpose of session_unset() on line 15?

4. Was there a reason to not  use session_write_close() in this script?

5. Is there any meaning that we should understand from the comment on line 2?
0
 
Richard KortsAuthor Commented:
Ray,

I found the issue; the variable $loc was not set in the non iphone, etc. devices case. Therefore, the header("Location: " . $loc) could not work. Makes PERFECT sense that Chrome displays a blank page.

Because of your questions I put in a bunch of echos, they led me to the solution.

Richard
0
 
Ray PaseurCommented:
Great!

A "standard" design pattern in almost any programming language is to assign the default variable values before any conditional code block that can change the variable values.  So something like this would be wise...
// SET DEFAULT VALUE TO REDIRECT TO THE HOME PAGE
$loc = '/';

// TEST USER AGENT FOR "PHONE" INDICATORS, CHOOSE REDIRECT PATH
$useragent = !empty($_SERVER['HTTP_USER_AGENT']) ? $_SERVER['HTTP_USER_AGENT'] : NULL;
if ( preg_match('@(iPad|iPod|iPhone|Android|BlackBerry|SymbianOS|SCH-M\d+|Opera Mini|Windows CE|Nokia|SonyEricsson|webOS|PalmOS)@', $useragent) ) {
    $loc = "phone.php";
} else {
    $loc = "form.php";  
}

// GOTO THE APPROPRIATE SCRIPT LOCATION
header("Location: " . $loc);

Open in new window

0
 
Richard KortsAuthor Commented:
That's EXACTLY what I did.

I often do that but not consistently.

R
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

Join & Write a Comment

Featured Post

Free Tool: Port Scanner

Check which ports are open to the outside world. Helps make sure that your firewall rules are working as intended.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

  • 3
  • 2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now