> CREATE TABLE `SessionsTable` (
>   `SID` varchar(32) NOT NULL default '',

This doesn't need to be a varchar.  The sid will always be 32 chars, so
make it a char(32)

>   `expiration` int(11) NOT NULL default '0',

I would suggest using a "timestamp" type here so MySQL will handle
updating it for you automatically.

> function mysql_session_open($session_path, $session_name) {
>
>   mysql_pconnect("localhost", "root", "")
>          or die("Can't connect to MySQL server! ");
>
>   mysql_select_db("globalDB")
>          or die("Can't select MySQL sessions database");
>
> } // end mysql_session_open()

You need to return true; at the end of this.

> function mysql_session_close() {
>
>   return 1;

No, use return true;

> function mysql_session_select($SID) {
>
>   GLOBAL $sess_db;
>   GLOBAL $sess_table;
>
>   $query = "SELECT value FROM $sess_table
>       WHERE SID = '$SID' AND
>       expiration > ". time();
>
>   $result = mysql_query($query);
>
> } // end mysql_session_select()

Uh, you need to return the actual value here or an empty string on an
error.

> function mysql_session_write($SID,$value) {
>
>   GLOBAL $sess_db;
>   GLOBAL $sess_table;
>   GLOBAL $lifetime;
>
>   $expiration = time() + $lifetime;
>
>   $query = "INSERT INTO $sess_table
>       VALUES('$SID', '$expiration', '$value')";
>
>   $result = mysql_query($query);
>
>   if (! $result) :
>
>    $query = "UPDATE $sess_table SET
>        expiration = '$expiration',
>        value = '$value' WHERE
>        SID = '$SID' AND expiration >". time();
>    $result = mysql_query($query);
>
>   endif;
>
> } // end mysql_session_write()

Again, you *must* return true; on a sucessful write.

> function mysql_session_destroy($sessionID) {
>
>   GLOBAL $sess_table;
>
>   $query = "DELETE FROM $sess_table
>       WHERE SID = '$sessionID'";
>   $result = mysql_query($query);
>
> } // end mysql_session_destroy()

return true;

-Rasmus


-- 
PHP Database Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to