Discussion :

[flagodzki]

Bonjour,

j'ai un problème avec la syntaxe de mes trigger.

Mon trigger est accepté comme ceci par le moteur mysql

Code :

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

1
2
3
4
5
6
CREATE TRIGGER `trigger_update_searchText` AFTER UPDATE ON `annonce`
 FOR EACH ROW 
 
 UPDATE search_text SET data=CONCAT_WS(' ',NEW.Sujet,NEW.SujetNL,NEW.Contenu,NEW.ContenuNL),Actif=NEW.Actif,DateUpdated=NOW()
 
WHERE model = 'annonce' and Ref_Model = NEW.Ref

et quand je l'introduit comme ceci il n'en veut pas, quelqu'un peut m'aider

Code :

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

1
2
3
4
5
6
CREATE TRIGGER `trigger_update_searchText` AFTER UPDATE ON `annonce`
 FOR EACH ROW BEGIN
 IF NEW.hit = OLD.hit THEN
	UPDATE search_text SET data=CONCAT_WS(' ',NEW.Sujet,NEW.SujetNL,NEW.Contenu,NEW.ContenuNL),Actif=NEW.Actif,DateUpdated=NOW() WHERE model = 'annonce' and Ref_Model = NEW.Ref;
 END IF;
 END;

Et j'ai essayé pas mal de combinaison, pas moyen...

Merci d'avance

[ericd69]

salut,

sans en dire plus on peut pas t'aider...

message erreurs?

comportement attendu et obtenu?

[flagodzki]

Bonjour,

j'ai sans cesse ce genre de message :

#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

et pourtant je vérifie bien ma syntaxe, j'ai été voir sur plusieurs sites. Il doit y avoir autres choses...

Ici ma requete réduite, même problème

CREATE TRIGGER `trigger_update_searchText` AFTER UPDATE ON `annonce`
FOR EACH ROW
BEGIN
IF NEW.hit != OLD.hit THEN
UPDATE search_text SET DateUpdated=NOW() WHERE Ref_Model = NEW.Ref;
END IF;
END;

[Fred_34]

Est-ce mieux comme ça ?

Code :

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

1
2
3
4
5
6
7
8
9
10
DELIMITER $$
CREATE TRIGGER `trigger_update_searchText` AFTER UPDATE ON `annonce`
FOR EACH ROW
BEGIN
IF NEW.hit != OLD.hit THEN 
UPDATE search_text SET DateUpdated=NOW() WHERE Ref_Model = NEW.Ref;
END IF;
END$$
 
DELIMITER ;

[flagodzki]

parfait, mais peux-tu être plus explicite quand à l'utilisation ds DELIMITER ?

merci bcp en tout cas

[ericd69]

Code sql :

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

1
2
3
4
5
6
7
8
9
10
delimiter $$
CREATE TRIGGER `trigger_update_searchText` AFTER UPDATE ON `annonce`
FOR EACH ROW
BEGIN
IF NEW.hit <> OLD.hit THEN
  UPDATE search_text
  SET DateUpdated=NOW()
  WHERE Ref_Model = NEW.Ref;
END IF;
END$$

delimiter change le délimiteur de fin de ligne pour éviter que l'interpréteur s'arrête sur les lignes dans un bloc begin...end lors de sa création

attention c'est pas != pour différent en sql mais <>

[Fred_34]

parfait, mais peux-tu être plus explicite quand à l'utilisation ds DELIMITER ?

merci bcp en tout cas

MyQL interprète les ; en tant que fin d'une commande SQL.

Donc ta requète était vu comme 3 instructions :

Code :

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

1
2
3
4
CREATE TRIGGER `trigger_update_searchText` AFTER UPDATE ON `annonce`
 FOR EACH ROW BEGIN
 IF NEW.hit = OLD.hit THEN
	UPDATE search_text SET DATA=CONCAT_WS(' ',NEW.Sujet,NEW.SujetNL,NEW.Contenu,NEW.ContenuNL),Actif=NEW.Actif,DateUpdated=NOW() WHERE model = 'annonce' AND Ref_Model = NEW.Ref;

Code :

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

 END IF;

Code :

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

 END;

Il faut donc changer le DELIMITER afin que le ; ne soit pas interprété. (J'ai mis $$ mais tu peux mettre //, toto, tutu... Le but est de mettre quelque chose que tu ne risques pas d'avoir dans ta fonction)
Et n'oublie pas de le repositionner à ; à la fin !

Edit: Pas vu la réponse d'Eric...

[ericd69]

normal, j'avais édité la réponse, j'avais pas vu la tienne non plus

bref par contre attention aux confusion c/c++ et sql