Refreshing problems

mparab

Hello,

Anyone know how to stop duplicate values from being entered into a database if someone presses the refresh button for the browser.

Thanks in advance,

Manik

kevinsequeira

after the data is enterd into the db, use header("location:some_page.php"); to redirect the user to another page

reg
kevin

mparab

Hello,

Thanks for the suggestion but I dont' really want to go to another page, I just want to clear the url, so that I get the orginal page back, without any variables in the url.

Any ideas,
M

stolzyboy

you can go back to that same page

header("Location: samepagethatyouwereat.php");

mparab

That goes before any output. Like before the <html> tag right?

stolzyboy

right

mparab

I don't know if I'm doing something wrong, but the page doesn't load with the header pointed to itself, it just doesn't load

stolzyboy

well you need to do something like:

if they submit the form
//do the header
else
spit out the form
<form>

</form>

you can't just put the header at the top before any other conditional code, otherwise, it will redirect to itself forever

kevinsequeira

post your code

reg
kevin

mparab

This is the my code:

<html>
<head>
<title>Northstar Technical Timesheet Application</title>
<script>
function popUp(URL) {
day = new Date();
id = day.getTime();
eval("page" + id + " = window.open(URL, '" + id + "', 'toolbar=0,scrollbars=0,location=0,statusbar=0,menubar=0,resizable=1,width=320,height=375');");
}
</script>
</head>
<body>
<?php

if (array_key_exists("id",$GET)&& $GET['op']=="delete"){
$id=$GET['id'];
delete($id);
}
else if
(array_key_exists("weekends",$GET)&& $GET['op']=="display"){
$week=$GET['weekends'];
display($week);
}

if($POST["submit"]) {
submit_data($POST);
}
else if($POST["update"]) {
update($POST);
}
else {
entryform();
}

?>
</body>
</html>
<?php
function submit_data($arr) {
$db = @mysql_connect("localhost", "root")
or die("Failed to connect to database!");
mysql_select_db("nti_admin", $db)
or die("Failed to connect to database!");
if($arr[status]=='Vacation' || $arr[status]=='Sick' || $arr[status]=='Time Off w/o Pay' || $arr[status]=='Berevement' || $arr[status]=='Stat Holiday')
{$arr[project]=='n/a';
$arr[activity]=='n/a';
}

$sql = "INSERT INTO timesheet (uname,date,status,hours,project,activity,rdproj,comments)
VALUES (\"$arr[uname]\", \"$arr[date]\", \"$arr[status]\", \"$arr[hours]\", \"$arr[project]\", \"$arr[activity]\", \"$arr[rdproj]\",\"$arr[comments]\")";

$result = mysql_query($sql)or die("Failed to submit query!");

entryform();
}

function entryform()
{

//$uname=$GET['name'];
//$USER[uname]=$uname;
$uname='mparab';
$start='2003-01-01';
$end='2003-12-31';
$today=date("Y-m-d");
$self = $_POST['PHP_SELF'];

$db = @mysql_connect("localhost", "root")
or die("Failed to connect to database!");
mysql_select_db("nti_admin", $db)

    or die("Failed to connect to database!");

$result=mysql_query("SELECT * FROM employee WHERE uname='$uname'",$db);

$act=mysql_query("SELECT * FROM activities",$db);

$pro=mysql_query("SELECT * FROM projects",$db);

$stat=mysql_query("SELECT * FROM status",$db);

$query=mysql_query("SELECT DISTINCT * FROM timesheet WHERE authorized=0 AND uname='$uname' AND date BETWEEN (NOW( ) - INTERVAL 14 DAY) AND NOW( ) ORDER BY date",$db);

$sql=mysql_query("SELECT * FROM timesheet WHERE (uname='$uname') AND (date BETWEEN '$start' AND '$end') ",$db);

$week=mysql_query("SELECT DISTINCT week_ends.weekends FROM week_ends, timesheet WHERE timesheet.date >= '$start' AND weekends < NOW( ) + INTERVAL 28 DAY",$db);

$thisweek=mysql_query("SELECT DISTINCT week_ends.weekends FROM week_ends, timesheet WHERE timesheet.date >= '2003-01-01' AND weekends < NOW( ) + INTERVAL 28 DAY AND weekends BETWEEN NOW( ) AND ( NOW( ) + INTERVAL 7 DAY )",$db);

if ($myrow=mysql_fetch_array($result))
{

print"<form name=input method=post action=$self >";
print"<table width=25% align=left cellspacing=0>";
print"<tr>";
print"<td colspan=2><img src=images/NTLogo.png width=100 height=75></td>";

value=$row[uname]>$row[uname]</td>";
print"</tr>";
print"<tr>";
print"<table>";
print"<tr>";
print"<td width=32%>";
print"</td>";
print"<td width=68%></td>";
print"</tr>";
print"<td>";
print"<table align=left border=0 cellspacing=2 cellpadding=2>";
print" <tr>";
print" <th>Timesheet For:</th>";
print" <td>$myrow[first_name] $myrow[last_name]</td>";
print" </tr>";
print"<tr>";
print"<th>Week Ending:</th>";
print"<td><select name=week>";
if ($weeks=mysql_fetch_array($week)AND $this=mysql_fetch_array($thisweek))

  do{
    if ($weeks[weekends]==$this[weekends])
  {print"<option value=$weeks[weekends] selected='$this[weekends]'>$weeks[weekends]";}
      else{print"<option value='$weeks[weekends]'>$weeks[weekends]";}
   }
    while($weeks=mysql_fetch_array($week));
  }
    print"</td>";

print" </tr>";
print"</table>";
print"</td>";
print"<td>";

if ($vacstat=mysql_fetch_array($sql))
{
do
{
if($vacstat[status]=='Vacation')
{
$totalvac=$myrow[vacation_allowed]+$myrow[carry_forward];
$vacation=$vacation+$vacstat[hours];
$remainvac=$totalvac-$vacation;
}
if($vacstat[status]=='Sick')
{
$sickused=$sickused+$vacstat[hours];
$remainsick=$myrow[sick_allowed]-$sickused;
}
}
while($vacstat=mysql_fetch_array($sql));
print"<table width=50% border=0 cellspacing=2 cellpadding=2 align=right>";
print"<tr>";
print"<th colspan=2>Vacation Stats</th>";
print"<th colspan=2>Sick Stats</th>";
print"</tr>";
print"<tr>";
print"<th>Carry Forward</th>";
print"<td>$myrow[carry_forward]</td>";
print"<th>Allowed</th>";
print"<td>$myrow[sick_allowed]</td>";
print"</tr>";
print"<tr>";
print"<th>Allowed</th>";
print"<td>$myrow[vacation_allowed]</td>";
print"<th>Used</th>";
print"<td>$sickused</td>";
print"</tr>";
print"<tr>";
print"<th>Total</th>";
print"<td>$totalvac</td>";
print"<th>Remaining</th>";
print"<td>$remainsick</td>";
print"</tr>";
print"<tr>";
print"<th>Used</th>";
print"<td>$vacation</td>";
print"<td></td>";
print"<td></td>";
print"</tr>";
print"<tr>";
print"<th>Remaining</th>";
print"<td>$remainvac</td>";
print"<td></td>";
print"<td></td>";
print"</tr>";
print"</table>";

print"</table>";
print"</td>";
print"</tr>";
}
print"<table width=90% align=center border=0 cellspacing=0 cellpadding=1>";
print"<tr>";
print"<td colspan=10><hr size=2>";
print"</tr>";
print" <tr>";
print"<td colspan=10 align=center><font size=+2 color=navy>Timesheet Information Centre</font></td>";
print" </tr>";
if ($row=mysql_fetch_array($query))
{
print"<tr align=left>";
print"<th>";
print"<th colspan=2 >Options";
print"<th >Date";
print"<th >Status";
print"<th >Hours";
print"<th >Project";
print"<th >Activity";
print"<th >R&D";
print"<th >Comments";
print" </tr>";

do
{
print" <tr>";
print"<td><input type=hidden name=uname value=$uname></td>";
print"<td><a href=\"javascript:popUp('edit.php?id=$row[id]&name=$uname')\">Edit</a>";
print"<td><a href=\"$_SERVER[PHP_SELF]?id=$row[id]&name=$uname&op=delete\">Delete</a>";
print"<td >$row[date]";
print"<td >$row[status]";
print"<td >$row[hours]";
print"<td >$row[project]";
print"<td >$row[activity]";
print"<td>";
if($row[rdproj]==0)
{print"No";}
else {print"Yes";}
print"</td>";
print"<td colspan=4>$row[comments] ";
print"</tr>";

}
while ($row=mysql_fetch_array($query));
print" <tr>";
print"<td><input type=hidden name=uname value=$uname></td>";
print"<td><input type=Submit name=submit value=Save ></td>";
print"<td><input type=Reset name=reset value=Reset></td>";
print"<td align=center><input name=date size=12 maxlength=12 value=$today></td>";
print"<td><select name=status>";
if($status=mysql_fetch_array($stat))
{
do{
print"<option value='$status[statname]'>$status[statname]";
}
while($status=mysql_fetch_array($stat));
}
else{}
print"</td>";

print"<td><input type=Text size=5 name=hours></td>";
print"<td><select name=project>";
if($project=mysql_fetch_array($pro))
{
do{
print"<option value='$project[proj_name]'>$project[proj_name]";
}
while($project=mysql_fetch_array($pro));
}
else{}
print"</td>";
print"<td><select name=activity>";
if($activity=mysql_fetch_array($act))
{
do{
print"<option value='$activity[activity]'>$activity[activity]";
}
while($activity=mysql_fetch_array($act));
}
else{}
print"</td>";

print"<td><select name=rdproj>";
print"<option value=0>No";
print"<option value=1>Yes";

print"</td>";
print"<td colspan=3><input name=comments size=5 maxlength=100></td>";
print"</tr>";
print"</table>";
print"</tr>";
}
print"</table>";
print"</table>";
print"</form>";
}
function delete($delete)
{
$db=@mysql_connect("localhost","root");
mysql_select_db("nti_admin",$db);
mysql_query("DELETE FROM timesheet WHERE id='$delete'",$db);

}

Thanks for your help,
M

stolzyboy

at what point, or in what if do you want the redirect to happen

mparab

After I click the Save button(the submit) I would like to redirect. So would I put it in the beginning of that function, or that function call? I don't know. I've tried to place it in various places and I always get an error say that there's already output.

Thanks again,

stolzyboy

go ahead and put it back in there (wherever you had it when you got the output error), then put this

<?
ob_start();
?>

right before your <html> line

mparab

Hi again,

Sorry for this, but I've placed the header everywhere, depending on where it's placed I still get an error either before or after I hit save. The only place I don't get in error is when I place it with
<?
ob_start();
header("Location: entry.php");
?>

but if you hit refresh it enter's the record again. Sorry for all the questions, but I haven't got a clue what to do about this.

Thanks in advance,
M

stolzyboy

i would put the header in here

if($POST["submit"]) {
submit_data($POST);
//header here
}
else if($POST["update"]) {
update($POST);
//header here
}

and keep the

<?
ob_start();
?>

as the very first thing in your code

mparab

That gets rid of the information in the url but it still tells me that I have output before the header.

Any other ideas, i'm getting to the point where i'm gonna just tell people not to hit refresh

Thanks for everything,
M

mparab

This is what I have for the beginning now:

<?
ob_start();
?>
<html>
<head>
<title>Northstar Technical Timesheet Application</title>
<script>
function popUp(URL) {
day = new Date();
id = day.getTime();
eval("page" + id + " = window.open(URL, '" + id + "', 'toolbar=0,scrollbars=0,location=0,statusbar=0,menubar=0,resizable=1,width=320,height=375');");
}
</script>
</head>
<body>
<?php
if (array_key_exists("id",$GET)&& $GET['op']=="delete"){
$id=$GET['id'];
delete($id);
}
else if
(array_key_exists("weekends",$GET)&& $GET['op']=="display"){
$week=$GET['weekends'];
display($week);
}
if($POST["submit"]) {
submit_data($POST);
header("Location: entry.php");
}
else if($POST["update"]) {
update($POST);
header("Location: entry.php");
}
else {
entryform();

}
?>
</body>
</html>

is it something little and stupid I'm not doing?

seanjw

Wouldn't it be better to...

Modify you db structure to include a 'hash' field
Set a random hash that gets passed along as a var when the form is submitted, and iserted into the 'hash' column along with the rest of the record (if there are no records that already have the hash).
Prior to all insertions, check for any records that already contain the hash, if any match, the insertion is kicked out.

-OR-

To cut down on db overhead you could set/reset the hash as a session var each time the form is submitted, the hash comparison would be done prior to setting/restting the hash.