Réalisation d'un Trigger

Version imprimable

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:

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:

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`

14/01/2018, 19h17
escartefigue

Citation:

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 :(
14/01/2018, 19h40
ODYSSEUS13

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
16/01/2018, 12h47
isabelle.letrong
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 :
Citation:
Envoyé par ODYSSEUS13

Code:

1 2 3 SELECT `date_derniere_visite.maintenance`, date_revision.cal_revision` FROM `maintenance`,`cal_revision`
Bien cordialement
16/01/2018, 14h05
ODYSSEUS13

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:

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:

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
16/01/2018, 14h26
CinePhil

On peut avoir un exemple de requête de mise à jour et le code complet de votre trigger ?
16/01/2018, 14h37
ODYSSEUS13

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:

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

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

Code:

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 ?

16/01/2018, 15h04
ODYSSEUS13

1 pièce(s) jointe(s)

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
16/01/2018, 15h32
escartefigue

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 :)
16/01/2018, 15h45
isabelle.letrong

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....

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:

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

16/01/2018, 16h45
isabelle.letrong
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' !

Citation:

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:

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:

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;

1 pièce(s) jointe(s)

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:

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

Citation:

Envoyé par ODYSSEUS13

Code:

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:

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

@CINEPHIL

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

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

16/01/2018, 17h41
ODYSSEUS13

@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
16/01/2018, 17h49
ODYSSEUS13

BINGO !

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
16/01/2018, 18h41
CinePhil

Attention toutefois à votre trigger si new.periodicite n'est pas valorisé dans la requête de mise à jour ! :aie:

Comme pour tout programme informatique, on doit maîtriser les données en entrée (type, valeurs acceptables...) sous peine de potentielle catastrophe. :aie:
16/01/2018, 18h55
isabelle.letrong

ODYSSEUS13, peux-tu marquer le pb comme résolu ?
16/01/2018, 19h21
ODYSSEUS13

Merci beaucoup Cinephil

Bonne soirée à Tous
16/01/2018, 20h02
ODYSSEUS13

@Isabelle

Je l'ai fait mais il est devenu: n'est plus résolu

je regarde à nouveau