Discussion :

[Jéjé34]

Salut,

Merci c'est corrigé sur mes fichiers!

Du coup, grâce au lien que tu m'as donné, j'ai crée 2 triggers comme ceci :

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
24
25
26
27
28
29
30
31
32
33
34
DELIMITER //
CREATE TRIGGER `siteSondages`.`tre_idProposition`
  BEFORE INSERT ON Proposition 
  FOR EACH ROW
BEGIN
  DECLARE num INTEGER;
  IF (NEW.idProposition IS NULL OR NEW.idProposition = 0) THEN
    SET num =  
    (
      SELECT COALESCE(MAX(idProposition), 0) + 1
      FROM Proposition
      WHERE idSondage = NEW.idSondage
    );
    SET NEW.idProposition= num;
  END IF;
END//
 
DELIMITER //
CREATE TRIGGER `siteSondages`.`tre_idCommentaire`
  BEFORE INSERT ON Commentaire
  FOR EACH ROW
BEGIN
  DECLARE num INTEGER;
  IF (NEW.idCommentaire IS NULL OR NEW.idCommentaire = 0) THEN
    SET num =  
    (
      SELECT COALESCE(MAX(idCommentaire), 0) + 1
      FROM Commentaire
      WHERE idAuteur = NEW.idAuteur
	  AND idSondage = NEW.idSondage
    );
    SET NEW.idCommentaire = num;
  END IF;
END//

Je les ai écrit a la fin de mon fichier SQL, mais lors de l'importation de ce dernier,
MySQL me renvoie l'erreur suivante sur la ligne "FROM Proposition" du premier trigger.

#1235 - This version of MySQL doesn't yet support 'multiple triggers with the same action time and event for one table'

Malgré quelques recherches, j'ai toujours pas trouvé pourquoi.

Désolé de t’embêter une fois de plus.

[fsmrel]

Hello jégé,


Et avec un DROP IF EXISTS TRIGGER avant chaque CREATE TRIGGER ?

[Jéjé34]

Je les ai rajoutés mais rien ne change.

Je joins ci-dessus le fichier, si tu as le temps d'y jeter un oeil.

[fsmrel]

J'ai fait un test. Pour ne plus avoir de code d'erreur, étonnament, dans le trigger, il faut mettre le nom de la table en lettres capitales :

BEFORE INSERT ON PROPOSITION

En plus, ce qui paraît logique, il faut faire suivre le DROP TRIGGER d'un COMMIT.


Ça marche aussi chez vous ?

[Jéjé34]

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
24
25
26
27
28
29
30
31
32
33
34
35
36
DROP TRIGGER IF EXISTS `siteSondages`.`tre_idProposition`;
DELIMITER //
CREATE TRIGGER `siteSondages`.`tre_idProposition`
  BEFORE INSERT ON PROPOSITION
  FOR EACH ROW
BEGIN
  DECLARE num INTEGER;
  IF (NEW.idProposition IS NULL OR NEW.idProposition = 0) THEN
    SET num =  
    (
      SELECT COALESCE(MAX(idProposition), 0) + 1
      FROM PROPOSITION
      WHERE idSondage = NEW.idSondage
    );
    SET NEW.idProposition= num;
  END IF;
END//
 
DROP TRIGGER IF EXISTS `siteSondages`.`tre_idCommentaire`;*/
DELIMITER //
CREATE TRIGGER `siteSondages`.`tre_idCommentaire`
  BEFORE INSERT ON COMMENTAIRE
  FOR EACH ROW
BEGIN
  DECLARE num INTEGER;
  IF (NEW.idCommentaire IS NULL OR NEW.idCommentaire = 0) THEN
    SET num =  
    (
      SELECT COALESCE(MAX(idCommentaire), 0) + 1
      FROM COMMENTAIRE
      WHERE idAuteur = NEW.idAuteur
	  AND idSondage = NEW.idSondage
    );
    SET NEW.idCommentaire = num;
  END IF;
END//

Merci.
Mais malheureusement, toujours la même erreur, malgré les majuscules.
Bien embêtant tout ça.

En plus, ce qui paraît logique, il faut faire suivre le DROP TRIGGER d'un COMMIT.

C'est à dire ?

[fsmrel]

Un COMMIT en gros, entérine la mise à jour physique des données des tables (suite à INSERT, UPDATE, DELETE), y-compris celles du catalogue : au moins, une fois le commit effectif, vous êtes certain que le DROP est effectif lui aussi et que le trigger est effacé du catalogue. A défaut, il y est toujours inscrit et une instruction CREATE trigger connaîtra des déboires (trigger déjà existant), les opérations n'étant alors effectives qu'en fin de transaction (cela dit, les SGBD n'ont pas tous forcément le même comportement) .

Code :

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

1
2
3
4
5
6
7
 DROP TRIGGER IF EXISTS `siteSondages`.`tre_idProposition`; 

 COMMIT ;

DELIMITER //

CREATE TRIGGER `siteSondages`.`tre_idProposition`

[fsmrel]

La fin du fichier que l'ai soumis sas erreur (je ne lésine pas sur les commits...) :

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
CREATE TABLE IF NOT EXISTS COMMENTAIRE (
  idAuteur INT UNSIGNED NOT NULL,
  idSondage INT UNSIGNED NOT NULL,
  idCommentaire INT UNSIGNED NOT NULL,
  contenu TEXT NOT NULL,
  dateEnvoi DATETIME NOT NULL,
  PRIMARY KEY (idAuteur, idSondage, idCommentaire),
  INDEX fk_COMMENTAIRE_UTILISATEUR1_idx (idAuteur ASC),
  INDEX fk_COMMENTAIRE_SONDAGE1_idx (idSondage ASC),
  CONSTRAINT fk_COMMENTAIRE_UTILISATEUR1
    FOREIGN KEY (idAuteur)
    REFERENCES UTILISATEUR (idUtilisateur)
    ON DELETE CASCADE
    ON UPDATE NO ACTION,
  CONSTRAINT fk_COMMENTAIRE_SONDAGE1
    FOREIGN KEY (idSondage)
    REFERENCES SONDAGE (idSondage)
    ON DELETE CASCADE
    ON UPDATE NO ACTION)
ENGINE = InnoDB;

COMMIT ;

 DROP TRIGGER IF EXISTS tre_idProposition; 
 DROP TRIGGER IF EXISTS tre_idCommentaire;

COMMIT ;

DELIMITER //
CREATE TRIGGER tre_idProposition
  BEFORE INSERT ON PROPOSITION 
  FOR EACH ROW
BEGIN
  DECLARE num INTEGER;
  IF (NEW.idProposition IS NULL OR NEW.idProposition = 0) THEN
    SET num = 
    (
      SELECT COALESCE(MAX(idProposition), 0) + 1
      FROM PROPOSITION
       WHERE idSondage = NEW.idSondage
    );
    SET NEW.idProposition= num;
  END IF;
END//

DELIMITER ;


DELIMITER //

CREATE TRIGGER tre_idCommentaire
  BEFORE INSERT ON COMMENTAIRE
  FOR EACH ROW
BEGIN
  DECLARE num INTEGER;
  IF (NEW.idCommentaire IS NULL OR NEW.idCommentaire = 0) THEN
    SET num =  
    (
      SELECT COALESCE(MAX(idCommentaire), 0) + 1
      FROM COMMENTAIRE
      WHERE idAuteur = NEW.idAuteur
	  AND idSondage = NEW.idSondage
    );
    SET NEW.idCommentaire = num;
  END IF;
END//
-- */

DELIMITER ;


SET SQL_MODE=@OLD_SQL_MODE;
SET FOREIGN_KEY_CHECKS=@OLD_FOREIGN_KEY_CHECKS;
SET UNIQUE_CHECKS=@OLD_UNIQUE_CHECKS;

[Jéjé34]

Merci, ça fonctionne.

A moi de jouer maintenant.
Un grand merci pour tout fsmrel!

[fsmrel]

Bon vent Jégé34, ce fut un plaisir !

François