Eye of the trigger

**Pymento** · 27/05/2011, 09h19

Bonjour j'ouvre mon nouveau problème dans un nouveau fil, l'ancien commençant à être un peu long.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
DELIMITER $$
CREATE TRIGGER delete_cascade_element
AFTER DELETE ON T1
FOR EACH ROW
BEGIN
DELETE FROM T2 WHERE T2_id=Old.T1_id;
END$$
DELIMITER ;

J'ai deux table T1 et T2.
Une entrée T1 a une colonne parent qui pointe sur une autre entrée T1.
T2 à pour identifiant le même id que l'entrée sur T1 à laquelle il correspond.

T1 a la propriété on cascade delete activé, et qui fonctionne.

Mon pb -> le trigger fonctionne partiellement, càd lors d'une suppression d'une entrée sur T1, la suppression se passe sur l'entrée correspondante sur T2.
En revanche si la suppression est soumise "à la cascade" ça ne marche pas.

Ou est mon erreur..?

Merci bien.

**ericd69** · 27/05/2011, 09h31

comme je l'ai dit dans l'autre post tu dois le paramètrer au niveau de la colonne concernée et en myisam je suis pas sur que ça marche... il me semble que tu dois être en innodb

**Pymento** · 27/05/2011, 09h43

T1 est en innoDB.
T2 en MyIsam.

Quand je fais un delete sur T1 et que l'entrée a un fils(qui se supprime bien), mysql ne considère pas ça comme une condition pour "rentrer" dans mon trigger ?

Je trouve ça illogique.
ça voudrait dire que un delete et un delete issu d'une cascade ne sont pas fait de la même manière.

**ericd69** · 27/05/2011, 09h58

ton trigger est local à une table

la cascade est un comportement d'intégrité référentielle

donc pas du tout les même choses même si ça touche à une action commune qui les déclenche

**Pymento** · 27/05/2011, 10h04

Donc la seule solution, serait de virer mes foreign key et mon cascade et de mettre un trigger à la place

on delete, delete fils ET delete sur T2.

ça semble propre mais niveau performance, j'en ai aucune idée.

**ericd69** · 27/05/2011, 10h53

lit ça : contrainte de cascade

tu peux les laisser, ça les auto indexe au moins...

mais sinon je pense que c'est la solution...

ou change le moteur de ta table myisam...

**Pymento** · 27/05/2011, 11h33

Je ne peux pas changer le MyIsam, j'en ai besoin pour faire de l'indexage fullText (chose qu'innoDB ne fait pas)

Je n'arrive pas, ni en conservant le cascading ni en l'enlevant, à mettre en place un trigger qui me supprime mon entrée fille.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
DELIMITER $$
CREATE TRIGGER delete_cascade_element
BEFORE DELETE ON elements
FOR EACH ROW
BEGIN
DELETE FROM elements WHERE Old.id=parent;
END$$
DELIMITER ;

J'ai essayé avec AFTER à la place de BEFORE, mais sans succès.

**ericd69** · 27/05/2011, 15h18

la solution est de passer par une procédure stockée alors...

**Pymento** · 27/05/2011, 15h54

J'obtient ça comme message d'erreur

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
 
#1442 - Can't update table 'table' in stored function/trigger because it is already used by statement which invoked this stored function/trigger.

Donc à ce que j'ai pu lire, mysql bloque la table et donc ne permet pas de faire un trigger d'une table sur elle même... j'espère avoir lu quelque chose de faux..
Alors deux solution s'offrent à moi (voire 3 si je continue à chercher):
la première changer de sgbd...
Ou alors faire une procédure stockée récursive.
Du genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
delimiter $$

CREATE PROCEDURE cascade_delete (current_id)
DECLARE ids int[];
BEGIN

 ids = SELECT id FROM elements WHERE parent=current_id;
foreach ids as value
   begin
      cascade_delete(value)
   end
DELETE FROM elements WHERE id = current_id
END
$$

Mais bon la partie en gras je ne sais pas dutout comment la mettre en place, et si c'est faisable.

**ericd69** · 27/05/2011, 16h11

pas du tout comme ça

les tableaux n'existent pas en mysql...

tu dois faire au moins un set @@max_sp_recursion_depth=255; pour permettre le maximum de niveau de récursion (moins si tu sais la profondeur maximum)...

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
delimiter $$
 
CREATE PROCEDURE cascade_delete (current_id)
begin
	declare i int(4);
	declare done bool default 0;
	declare lit cursor for select id from elements where parent=current_id;
	declare continue handler for SQLSTATE '02000' set done=1;
	set @@max_sp_recursion_depth=255;
	repeat
		fetch lit into i;
		if not done then
			call cascade_delete(i);
		end if;
	until done end repeat;
	delete from elements where id=current_id;
end$$
 
delimiter;

**Pymento** · 30/05/2011, 14h40

Merci pour cette procédure.
Même si y'a 2-3 lignes que je ne comprend pas encore, je viens de l'essayer.
Et ça me crache une erreur:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
 
MySQL a répondu: 
 
#1064 - Erreur de syntaxe près de ')
BEGIN
    declare i int(4);
    declare done bool DEFAULT 0;
    declare lit cursor' à la ligne 1

Et ça me met la croix rouge à hauteur du i dans le rappel de la procédure.

**ericd69** · 30/05/2011, 14h50

autan pour moi, j'avais pompé le début de ton code sans le changer :p

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

CREATE PROCEDURE cascade_delete (in current_id int(4))

**Pymento** · 30/05/2011, 15h21

Exact, en revanche, ça serait trop simple de fonctionner directement !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
 Cannot load from mysql.proc. The table is probably corrupted

a chaque fois. (avec un DROP PROCEDURE cascade_delete aussi)

j'ai regardé un peu sur le net et ils disent de faire un mysql_upgrade, et sur phpmyadmin... je sais pas ou le faire ou même si c'est la bonne solution

**ericd69** · 30/05/2011, 15h24

tu as quelle version de mysql?

**Pymento** · 30/05/2011, 15h26

Apache/2.2.17 (Win32) PHP/5.3.6
Version du client MySQL: mysqlnd 5.0.8-dev - 20102224 - $Revision: 308673 $

**Pymento** · 30/05/2011, 15h57

Enfin.. non j'ai trouvé ça plutot, c'est écrit en plus gros:

MYSQL 5.5.10

**ericd69** · 30/05/2011, 16h47

ça les prend en charge...

bon le truc complet qui doit marcher avec : db le nom de la base qui contiendra la procédure stockée

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
use db;
drop procedure if exists cascade_delete;
delimiter $$
 
CREATE PROCEDURE cascade_delete(in current_id int(4))
begin
	declare i int(4);
	declare done bool DEFAULT 0;
	declare lit cursor FOR SELECT id FROM elements WHERE parent=current_id;
	declare continue handler FOR SQLSTATE '02000' SET done=1;
	SET @@max_sp_recursion_depth=255;
	repeat
		fetch lit INTO i;
		IF NOT done then
			call cascade_delete(i);
		end IF;
	until done end repeat;
	DELETE FROM elements WHERE id=current_id;
end$$
 
delimiter;

**Pymento** · 30/05/2011, 17h03

J'ai fais un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

mysql_upgrade

avec l'invite de commande, et ça m'a donc effectivement laisser enregistrer ma procédure.

Par contre quand je l'exécute ça me dit:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
 
#1326 - Cursor is not open
CALL cascade_delete( 5 )

Je pense que ça a un rapport avec cette ligne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

declare continue handler FOR SQLSTATE '02000' SET done=1;

A ce que j'ai pu lire, y'a une histoire de fermeture sans ouverture préalable du curseur

**ericd69** · 30/05/2011, 17h27

oui j'ai oublié de mettre l'ouverture et la fermeture, dernière rectification

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
USE db;
DROP procedure IF EXISTS cascade_delete;
delimiter $$
 
CREATE PROCEDURE cascade_delete(IN current_id int(4))
begin
	declare i int(4);
	declare done bool DEFAULT 0;
	declare lit cursor FOR SELECT id FROM elements WHERE parent=current_id;
	declare continue handler FOR SQLSTATE '02000' SET done=1;
	SET @@max_sp_recursion_depth=255;
	open lit;
	repeat
		fetch lit INTO i;
		IF NOT done then
			call cascade_delete(i);
		end IF;
	until done end repeat;
	close lit;
	DELETE FROM elements WHERE id=current_id;
end$$
 
delimiter;