Réalisation d'un Trigger

**ODYSSEUS13** · 14/01/2018, 18h07

Bonjour

l'application est la suivante:

je dois mettre à jour une date de prochaine visite à partir de la date de la dernière visite en fonction d'une valeur exprimée en mois periodicite

A l'origine ma requête manuelle était la suivante:

Code :

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

UPDATE `maintenance` SET `date_prochaine_visite` = DATE_ADD( `date_derniere_visite` , INTERVAL `periodicite` MONTH )

donc j'ajoute par exemple 12 (periodicite) mois à la date_derniere_visite

J'ai crée une nouvelle table car, sauf erreur mysql ne permet pas la mise à jour par trigger sur la même table.

j'ai donc deux tables: la table maintenance qui contient date_derniere_visite et la table cal_revision qui contient id_maintenance pour la jointure et date_revision pour le champ que je veux mettre à jour.

J'ai fait de multiples essais en parcourant toutes les erreurs possibles de my sql

voici ma dernière tentative

Si quelqu'un peut m'aider à corriger ce bout de code je l'en remercie (j'utilise l'interface de PHPMyadmin mise à jour de la table maintenance Before Update)

Code :

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

1
2
3
4
5
6
7
8
9
SELECT `date_derniere_visite.maintenance`, date_revision.cal_revision`
 
FROM `maintenance`,`cal_revision`
 
UPDATE `cal_revision`
 
SET `date_revision` = DATE_ADD( `date_derniere_visite` , INTERVAL `periodicite` MONTH )
 
WHERE `cal_revision.id_maintenance`=`maintenance.id`

**escartefigue** · 14/01/2018, 19h17

Envoyé par ODYSSEUS13

J'ai crée une nouvelle table car, sauf erreur mysql ne permet pas la mise à jour par trigger sur la même table.

Que voulez vous dire par là ? un trigger est associé à une action de mise à jour (update, delete ou insert) sur une table. Donc vous pouvez créer un trigger sur n'importe quelle table
Par contre, ce que ne sait pas faire MySQL, c'est un trigger qui se substitue à la mise à jour par le SGBD, pas de trigger "instead of" dans MySQL

**ODYSSEUS13** · 14/01/2018, 19h40

Bonjour Escartfigue et merci

Aucun soucis, si je peux mettre à jour le champ date_prochaine_visite de ma table principale je préfèrerais mais j'ai cru comprendre que le trigger ne mettait à jour que le champ d'une autre table

**isabelle.letrong** · 16/01/2018, 12h47

Bonjour ODYSSEUS13,

Non, aucun souci pour mettre à jour la table dès que la périodicité ou la date de dernière visite sont mises à jour par exemple (on compare dans le trigger BEFORE les valeurs anciennes et nouvelles des colonnes via NEW.NomColonne et OLD.NomColonne (ne pas oublier de se poser la question des NULL...)

ceci étant le code suivant me laisse perplexe :

Envoyé par ODYSSEUS13

Code :

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

1
2
3
SELECT `date_derniere_visite.maintenance`, date_revision.cal_revision`
 
FROM `maintenance`,`cal_revision`

Bien cordialement

**ODYSSEUS13** · 16/01/2018, 14h05

Bonjour Isabelle et merci.

Pour le code, quand on coule on complique ........

donc, pour revenir au fondamentaux, il faut que dans mon trigger j'arrive à faire fonctionner le code suivant dans une procédure AFTER UPDATE (via phpmyadmin qui génère automatiquement une partie du code CREATE END etc..)

Code :

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

SET `prochaine_visite` = DATE_ADD( `derniere_visite` , INTERVAL `periodicite` MONTH )

J'ai essayé d'entrer les variables NEW et OLD sans succès, mais peut être il y a t-il une erreur de syntaxe

Code :

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

SET `new.prochaine_visite` = DATE_ADD( `old.derniere_visite` , INTERVAL `old.periodicite` MONTH )

J'ai essayé d'autres variantes par exemple en mettant tout en new. ou en ne mettant que new.prochaine_visite ou en ne mettant pas old etc...

C'est mon premier trigger et il y a quelque chose qui m'échappe.

Merci si vous pouvez m'aider

**CinePhil** · 16/01/2018, 14h26

On peut avoir un exemple de requête de mise à jour et le code complet de votre trigger ?

**ODYSSEUS13** · 16/01/2018, 14h37

Bonjour Cinephil et merci de votre intervention

Voici le code complet généré par phpmyadmin

je ne met à jour qu'après l'instruction EACH ROW

Code :

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

CREATE DEFINER = `root`@`localhost` TRIGGER `revision update` AFTER UPDATE ON `maintenance` FOR EACH ROW  BEGIN SET `new.prochaine_visite` = DATE_ADD( `derniere_visite` , INTERVAL `periodicite` MONTH ) END;

Je n'ai pas fait pour le moment de code pour la mise à jour dans un formulaire, je vérifie seulement le fonctionnement avec phpmyadmin en affichant la table.

J'ai fait des variantes en ajoutant new. (ou old.) à toutes les colonnes.

Merci à vous

**CinePhil** · 16/01/2018, 14h50

Déjà, je pense qu'il manque un point-virgule :

Code :

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

1
2
3
4
CREATE DEFINER = `root`@`localhost` TRIGGER `revision update` AFTER UPDATE ON `maintenance` FOR EACH ROW  
BEGIN 
	SET `new.prochaine_visite` = DATE_ADD( `derniere_visite` , INTERVAL `periodicite` MONTH ) -- << manque ; ici
END;

Et la requête de mise à jour ressemble à quoi ?

**ODYSSEUS13** · 16/01/2018, 15h04

Merci Cinephil

j'ai ajouté le ; après month ) ;

Je suis toujours immédiatement en erreur lors de l'enregistrement

#1193 la variable new.derniere_visite est inconnue

Qu'entendez vous par requête de mise à jour ? j'ai peur que quelque chose m'échappe.

Pour le moment j'utilise seulement l'interface d'affichage de phpmyadmin pour entrer les valeurs dans la table.

Voir en PJ la capture d'écran

**escartefigue** · 16/01/2018, 15h32

Je suppose que dans la vraie vie, il y aura une application qui effectuera la mise à jour dans laquelle vous connaissez la date de dernière visite et l'intevalle
Du coup nul besoin de trigger pour calculer la date de prochaine visite, il suffit d'appliquer directement l'intervalle à la date de dernière visite

**isabelle.letrong** · 16/01/2018, 15h45

ODYSSEUS13,

Dans un AFTER update, on ne peut plus modifier une colonne ....
En effet la table est déjà mise à jour (after update...)

Donc toute modification de colonne dans la table cible du trigger ne peut être faite que dans un trigger BEFORE....

**ODYSSEUS13** · 16/01/2018, 16h29

Bonjour Isabelle, Escartefigue,

@Escartefigue

La base pouvant être consultée et modifiée à partir d'autres applications (excel, odbc, front-end ) sur lesquelles je ne peux intervenir, je dois régler le problème du trigger.

@Isabelle

Effectivement c'est logique; mais ce n'était pas suffisant.

Le trigger est rejeté à son enregistrement me déclarant la colonne new.prochaine visite comme erronée

Je pensais que new. était une variable temporaire système qui me permettrait de mettre à jour la table

Ce qui me surprend c'est que la requête qui contient une redondance avec UPDATE `maintenance` après FOR EACH ROW est enregistrée sans erreur;

Code :

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

1
2
 
CREATE DEFINER = `root`@`localhost` TRIGGER `revision update` BEFORE UPDATE ON `maintenance` FOR EACH ROW UPDATE `maintenance` SET `new.prochaine_visite` = DATE_ADD( `derniere_visite` , INTERVAL `periodicite` MONTH );

Nota pour CINEPHIL le ; après MONTH) est effacé, probablement parce qu'il est généré automatiquement

**isabelle.letrong** · 16/01/2018, 16h45

ODYSSEUS13

Disposer du DDL de la table maintenance aiderait : la colonne prochaine_visite est-t-elle bien de type DATE et fait bien partie de la table maintenance ?

Par ailleurs, un BEGIN après EACH ROW et un END terminal seraient les bienvenus, les cotes dans `new.prochaine_visite`, `derniere_visite` et `periodicite`sont inutiles,

Enfin, si `derniere_visite` et `periodicite` font bien partie de la table 'maintenance' alors il faut les faire précéder également de 'NEW' !

**CinePhil** · 16/01/2018, 17h14

Envoyé par ODYSSEUS13

Qu'entendez vous par requête de mise à jour ? j'ai peur que quelque chose m'échappe.

Pour le moment j'utilise seulement l'interface d'affichage de phpmyadmin pour entrer les valeurs dans la table.

Voir en PJ la capture d'écran

En principe, quand vous exécutez la mise à our à l'aide de l'interface de phpMyAdmin, la requête composée par phpMyAdmin et exécutée par MySQL/MariaDB est affichée. Une requête de ce genre :

Code :

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

UPDATE la_table SET la_colonne = 'une valeur'

.

C'est cette requête qu'on aimerait voir.

Ce que je crois comprendre, c'est que vous donnez la valeur de la dernière visite et vous souhaitez que la date de la prochaine se mette à jour automatiquement en fonction de la périodicité.

Pas sûr de la syntaxe (ça fait un bail que je n'ai pus développé de procédure MySQL), mais voilà le genre de truc qu'il faut faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
CREATE TRIGGER revision_update BEFORE UPDATE ON maintenance FOR EACH ROW 
BEGIN
	IF(new.periodicite <> old.periodicite) THEN 
	{
		@periodicite = new.periodicite;
	}
	ELSE
	{
		@periodicite = old.periodicite;
	}
 
	new.prochaine_visite = DATE_ADD( derniere_visite , INTERVAL @periodicite MONTH );
END;

**ODYSSEUS13** · 16/01/2018, 17h18

Merci Isabelle

En PJ le descriptif de la table

Voici en l'état ma dernière requête qui me renvoi l'erreur #1442

Code :

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

1
2
3
4
5
6
7
8
9
10
 
CREATE DEFINER = `root`@`localhost` 
TRIGGER `update_maintenance` 
BEFORE UPDATE ON `maintenance` 
FOR EACH ROW 
BEGIN 
UPDATE maintenance 
SET prochaine_visite = NEW.derniere_visite + INTERVAL NEW.periodicite MONTH WHERE id = NEW.id;
 
END

**isabelle.letrong** · 16/01/2018, 17h25

Envoyé par ODYSSEUS13

Code :

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

1
2
3
4
5
6
7
8
9
10
 
CREATE DEFINER = `root`@`localhost` 
TRIGGER `update_maintenance` 
BEFORE UPDATE ON `maintenance` 
FOR EACH ROW 
BEGIN 
UPDATE maintenance 
SET prochaine_visite = NEW.derniere_visite + INTERVAL NEW.periodicite MONTH WHERE id = NEW.id;
 
END

Pas d'update puisqu'on est dans le trigger d'update !

Tout simplement :

Code :

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

1
2
3
4
5
6
7
8
 
CREATE DEFINER = `root`@`localhost` 
TRIGGER `update_maintenance` 
BEFORE UPDATE ON `maintenance` 
FOR EACH ROW 
BEGIN 
SET NEW.prochaine_visite = NEW.derniere_visite + INTERVAL NEW.periodicite MONTH;
END

**ODYSSEUS13** · 16/01/2018, 17h34

@CINEPHIL

J'ai modifié le code car le changement porte sur derniere_visite ce qui donne

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
BEGIN
	IF(new.derniere_visite <> old.derniere_visite) THEN 
	{
		@derniere_visite = new.derniere_visite;
	}
	ELSE
	{
		@derniere_visite = old.derniere_visite;
	}
 
	prochaine_visite = DATE_ADD( derniere_visite , INTERVAL @periodicite MONTH );
END;

Mais que ce soit votre code ou le mien je suis en erreur à sa création

Certes je découvre les triggers sous mysql mais c'est assez complexe dans la syntaxe

**ODYSSEUS13** · 16/01/2018, 17h41

@ISABELLE

votre code me semble clair et évident

toutefois à la création il m'est retourné l'erreur

#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 6

**ODYSSEUS13** · 16/01/2018, 17h49

C'est Isabelle qui a gagné le Kilo de sucre !

Toutefois je ne suis pas passé par les commandes SQL mais par l'interface de mise à jour des triggers qui me renvoyait l'erreur précédente

en entrant la commande

BEGIN
SET NEW.prochaine_visite = NEW.derniere_visite + INTERVAL NEW.periodicite MONTH;
END

Merci à tous de votre aide, gentillesse et patience.

Je m'étais acheté le SQL pour les nuls et la partie trigger se résume à un paragraphe de 10 lignes !

Donc l'aide d'un forum est précieuse sur un sujet sur lequel je piétinais depuis 2 jours

Merci à tous et grosses bises à Isabelle

Sur ce je vais aller acheter mon pain