select e insert con trigger mysql

[danirota]

ciao, sto usando un db mysql e dove fare un trigger che mi selezioni le tuple con data < di quella odierna e che mi copi queste tuple in una nuova tabella, e poi le cancelli da quella in questione.
Il codice che ho fatto è così

delimiter //
CREATE TRIGGER vecchie_prenotazione
       BEFORE UPDATE ON prenotazioni
         FOR EACH ROW
         BEGIN
         
         IF OLD.data_partenza < CURDATE() THEN INSERT INTO vecchie_prenotazioni VALUES ('', 'OLD.id_camera', 'OLD.tipo', 'OLD.data_arrivo', 'OLD.data_partenza', 'OLD.confermata', 'OLD.nome', 'OLD.telefono', 'OLD.email', 'OLD.richieste', 'OLD.data_prenotazione');
         
         
         END IF;
         
         IF OLD.data_partenza < CURDATE() THEN DELETE FROM prenotazioni WHERE id_prenotazione = OLD.id_prenotazione;
         END IF;
         END; //
delimiter ;

però non funziona, non fa nè l'inserimento nella nuova tabella, nè la cancellazione...

dove sbaglio? come posso vedere se il trigger funziona?


GRAZIE !!!

[and77]

Puoi inviare le tabelle su cui lavori e il comando update che digiti?Saluti.

[danirota]

le tabelle sono identiche, semplicemente su una ci 'lavoro' mentre l'altra la uso come archivio

questa è la struttura

vecchie_prenotazioni` (
  `id_prenotazione` int(11) NOT NULL AUTO_INCREMENT,
  `id_camera` int(11) NOT NULL,
  `tipo` varchar(20) NOT NULL,
  `data_arrivo` date NOT NULL,
  `data_partenza` date NOT NULL,
  `confermata` tinyint(1) NOT NULL,
  `nome` varchar(150) NOT NULL,
  `telefono` varchar(50) NOT NULL,
  `email` varchar(100) NOT NULL,
  `richieste` text NOT NULL,
  `data_prenotazione` datetime NOT NULL,
  PRIMARY KEY (`id_prenotazione`)

e la funziona update è

UPDATE prenotazioni SET confermata = 2 WHERE id_prenotazione = ".$id_prenotazione;

[and77]

Ciao,secondo me il problema sta nel fatto che vecchie_prenotazioni abbia id_prenotazione come AUTO_INCREMENT.Se non ho capito male vecchie_prenotazioni funge da archivio.Perciò prova a rimuoverlo.Di conseguenza la insert dovrebbe essere questa:
INSERT INTO vecchie_prenotazioni VALUES ('old.id_prenotazione', 'OLD.id_camera', 'OLD.tipo', 'OLD.data_arrivo', 'OLD.data_partenza', 'OLD.confermata', 'OLD.nome', 'OLD.telefono', 'OLD.email', 'OLD.richieste', 'OLD.data_prenotazione');

Poi ,anche se è solo una sfumatura,fai un unico if.....then....Spero di esserti stato di aiuto facci sapere.Saluti.

[optime]

ma la insert non dovrebbe essere fatta così

INSERT INTO
(colonna1, colonna2, ..., colonnaN)
VALUES
(valore1, valore2, ..., valoreN)

???

[piero.mac]

Non devi usare empty ma NULL per l'auto increment

VALUES (NULL, 'OLD.id_camera', ......

@optime
non serve l'elenco dei campi se nel value metti il valore di tutti i campi e nello stesso ordine in cui sono descritti in tabella.

[optime]

a parte l'omissione dei nomi delle colonne, a me pare che lo statement sia stato scritto mettendo i nomi delle colonne tra ' ', quindi IMHO sono dei literal - ma non conosco così bene mySQL

[piero.mac]

Originariamente inviato da optime
a parte l'omissione dei nomi delle colonne, a me pare che lo statement sia stato scritto mettendo i nomi delle colonne tra ' ', quindi IMHO sono dei literal - ma non conosco così bene mySQL

se e' per lo statement per copiare da una tabella all'altra dovrebbe usare insert ... select e non 'OLD.id_camera', 'OLD.tipo', etc. che verrebbero interpretate come semplici stringhe.

ma il problema e' che dice che le tabelle dovrebbero essere identiche e quindi ci si aspetterebbe avessero anche l'identico id del record e non ognuna un id per proprio conto.

Ma vedo anche altro. Diciamo che a me piace che i problemi vengano risolti uno alla volta. Almeno si possono capire meglio.

[danirota]

ho modificato il trigger così

delimiter //
CREATE TRIGGER vecchie_prenotazioni
       BEFORE UPDATE ON prenotazioni
         FOR EACH ROW
         BEGIN
         
         IF OLD.data_partenza < CURDATE() THEN INSERT INTO vecchie_prenotazioni VALUES (NULL, 'OLD.id_camera', 'OLD.tipo', 'OLD.data_arrivo', 'OLD.data_partenza', 'OLD.confermata', 'OLD.nome', 'OLD.telefono', 'OLD.email', 'OLD.richieste', 'OLD.data_prenotazione');
       
         DELETE FROM prenotazioni WHERE id_prenotazione = OLD.id_prenotazione;
         
         END IF;
         
         END; //
delimiter ;

ma non funziona.... non inserisce nè cancella....

[optime]

'OLD.id_camera' scritto così, tra apici, NON è il valore di id_camera della tabella OLD, ma bensì proprio 'OLD.id_camera', letterale.