Stored procedure cancella tutto!

[sawada91]

DROP PROCEDURE IF EXISTS Changerole; delimiter $ CREATE PROCEDURE Changerole (OldRole CHAR(8), Newrole CHAR(8), IdUser INT) BEGIN IF (OldRole="admin" AND NewRole="mod") THEN DELETE FROM administrators WHERE iduser=IdUser; INSERT INTO moderators (iduser) VALUES (IdUser); END IF; IF (OldRole="admin" AND NewRole="user") THEN DELETE FROM administrators WHERE iduser=IdUser; INSERT INTO normalusers (iduser) VALUES (IdUser); END IF; IF (OldRole="mod" AND NewRole="admin") THEN DELETE FROM moderators WHERE iduser=IdUser; INSERT INTO administrators (iduser) VALUES (IdUser); END IF; IF (OldRole="mod" AND NewRole="user") THEN DELETE FROM moderators WHERE iduser=IdUser; INSERT INTO normalusers (iduser) VALUES (IdUser); END IF; IF (OldRole="user" AND NewRole="admin") THEN DELETE FROM normalusers WHERE iduser=IdUser; INSERT INTO administrators (iduser) VALUES (IdUser); END IF; IF (OldRole="user" AND NewRole="mod") THEN DELETE FROM normalusers WHERE iduser=IdUser; INSERT INTO moderators (iduser) VALUES (IdUser); END IF; END; $ delimiter ;

Praticamente questa procedura mi cancella da una certa tabella e mette la stessa riga in un altra. Il problema è che l'insert funziona correttamente, ma quando fa il delete cancella tutta la tabella invece che solo quella riga. Perché?

[philbert]

Ciao,

THEN DELETE FROM moderators WHERE iduser=IdUser;

pessima idea quella di chiamare il parametro locale della procedura come il nome del campo!!!


mysql non è case sensitive a priori ma dipende dal sistema operativo che c'è sotto...

...quindi molto probabilmente quando scrivi

THEN DELETE FROM moderators WHERE iduser=IdUser;

in realtà stai scrivendo qualcosa del tipo

THEN DELETE FROM moderators WHERE 23=23;

il che equivale a cancellare tutta la tabella perché la where è sempre verificata (23 è sempre uguale a 23)

Fai così:, cambia nome al parametro idUser e chiamalo ad esempio p_IdUser

PROCEDURE Changerole (OldRole CHAR(8), Newrole CHAR(8), p_IdUser INT)

poi sostituisci nelle delete di conseguenza:

THEN DELETE FROM moderators WHERE iduser=p_IdUser;

vedi se va!

[sawada91]

È vero, ora va. Non mi ricordavo che non fosse case-sensitive. Grazie mille