dubbio su query da guida sessiono alternative

[felpone]

Salve,sto utilizzando la guida di questo sito "sessioni alternative" ed in particolare la classe php
db_session_class,ma c'è una query sql di cui non riesco a capire il significato e che mi da errore in compilazione.

function my_session_start() {
      $cookie_expire = ($this->session_time > 0) ? (time() + $this->session_time) : 0;
      if(!isset($_COOKIE['my_session_id'])) {
         setcookie('my_session_id', $this->my_session_id, $cookie_expire);
         mysql_query("
    INSERT INTO my_session
    VALUES
    ('". $this->my_session_id ."',"/*quì*/, ". time() .")"); 

In particolare all'interno del costrutto VALUES non mi è chiaro il secondo valore che si inserisce,ossia,un doppio apice

,", 

,è un errore?Ho commentato con un "quì" nel codice per una facile individuazione.

[alcio74]

non so come sia strutturata la tabella my_session, ma di primo acchitto mi sembra un errore.
Sembra ci sia un apice che deve contenere una stringa ma non è chiuso.
Quindi o manca un apice perché il secondo parametro deve essere una stringa vuota, o ce ne è uno di troppo!

Hai provato a mandare in esecuzione il codice?

[satifal]

Ho buttato una veloce occhiata all'articolo da te citato.
Probabilmente c'è un errore nel codice ed invece delle virgolette co vogliono due apici singoli:

mysql_query("INSERT INTO my_session VALUES('" . $this->my_session_id . "', '', " . time() . ")"); 

L'idea è quella di creare un record col session id e il time e lasciare vuoto (per il momento) il campo contenente le variabili.
Purtroppo la tabella my_session è stata creata in questo modo:

CREATE TABLE my_session (
sessid varchar(32) NOT NULL default ”,
session_vars text NOT NULL,
session_date int(10) unsigned NOT NULL default ’0′,
UNIQUE KEY sessid (sessid)
) TYPE=MyISAM;

per cui la insert precedente darebbe errore in quanto il campo session_vars è settato NOT NULL.
Quindi dovresti modificare anche la struttura della tabella permettendo di inserire in questo campo valori vuoti:

CREATE TABLE my_session (
sessid varchar(32) NOT NULL default ”,
session_vars text NULL,
session_date int(10) unsigned NOT NULL default ’0′,
UNIQUE KEY sessid (sessid)
) TYPE=MyISAM;