[RESOLVED] why are date and time not getting passed ?

swatisonee

hi,

i need to create an attendance register that logs a user, the time into office, the time out of office (at end of day) and of course the date itself.

I initially wanted to combine this with the login script but that would not work because a user may log in and log out of his webpages several times a day . i only need a record of the time user gets in and leaves office.

i'm sure there's an easier way somewhere but having gone spare trying to figure it out, i just created a file attendance.php as a link on a users index.php page

But I find that the logout info doesnt get saved ( if attend ==0) in the same script given below

Could some one guide me pl ? Thanks and M/C.
Swati

<?php

$id = $HTTP_POST_VARS["id"];
$t1 = $HTTP_POST_VARS["t1"];
$t2 = $HTTP_POST_VARS["t2"];
$message = $HTTP_POST_VARS["message"];
?>

  <?php

mysql_connect("localhost", $dbname, $dbpasswd )
    	or die ("Unable to connect to server.");

mysql_select_db($database)
    	or die ("Unable to select database.");

if ( $attend == 1 )
	{

$sql1 = "INSERT INTO `Attendance` (id, ip1, date, timein) VALUES ('$id', '$message', '$t1', '$t2')" ;
echo $sql1;

$result1 = mysql_query($sql1)
or die (mysql_error());

echo "

<p>Thank you for signing in  "; echo $id;  echo "  !</p> ";



$sql2="select max(AID) from `Attendance`";

$result2 = mysql_query($sql2)
	or die ("Unable to get results");

$myrow2 = mysql_fetch_row($result2);
$aid = $myrow2[0];

}

// Upto here the code works - problems from below

if ( $attend == 0 )
{


$result3 = mysql_query ("UPDATE `Attendance` SET ip2='$message',timeout='$t1' WHERE AID=$aid AND id = '$id' " );


echo "

<p>Thank you for logging out ! Good Day and see you back tomorrow !</p>";
}


?>

RyanJones

First a note: Why not just use the table, set the office enter time as when they first login (Easily checked by trying to extracy the value from the table) and then for the second one just update it everytime they login until the end of the dat, that was there will be no problems with multiple logins and outs 😃

Back too your script, you failed too mention of there were nay MySQL error messages, where there?

Try changing this line:

$result3 = mysql_query ("UPDATE `Attendance` SET ip2='$message',timeout='$t1' WHERE AID=$aid AND id = '$id' " );

Too...

$result3 = mysql_query ("UPDATE `Attendance` SET `ip2` = '" . $message . "' , `timeout` = '" . $t1 . "' WHERE `AID` = '" . $aid . "' AND `id` = '" . $id . "'");

I don't think it will work but its the best Idea I have unless you can tell if you have any mysql error messages :-)

Cheers,

Ryan Jones

swatisonee

Hi Ryan Jones,

a. First I need the start of day time to remain without getting updated.
b. As far as logout your suggestion is great but the problem is the UPDATE . I think the query itself is not getting executed because even though the mysqlerror statement is there, when I do an echo $sql; the screen is blank - so that mean the date and time are not getting processed at all.

I guess there has to be someway the script knows that $attend= 0 means theat $aid should exist and therefore data in $aid record should be updateable ..

Any pointers ?

Thanks @

bpat1434

I think I see the problem:

You have this code:

$sql2="select max(AID) from `Attendance`"; 
$result2 = mysql_query($sql2) or die ("Unable to get results"); 
$myrow2 = mysql_fetch_row($result2); 
$aid = $myrow2[0];

nested within the if($attend==0) so it doesn't get called. Thus $aid will always be NULL (as in not set) because it's not there. This is also why you get the empty screen. It's a parse error that the server is actually not showing. Since $aid doesn't have a value if $attend==1, then it doesn't know what you want, and thus it wigs out and wont' work. You have a choice:
1.) Move that out of the if($attend==0) statement
2.) Move it into the if($attend==1) statement
3.) Remove it completely
4.) Add some debugging... (research [man]ini_set/man and ERROR_REPORTING

~Brett

swatisonee

OHHHH Thank you !~!

It works !

bpat1434

glad to hear it.

~Brett