Discussion :

[benjamin122]

Bonjour ,

Je suis en train de créer mon tout premier trigger sous MYSQL.
Bien que je n'ai pas d'erreur de syntaxe, j'aimerai avoir l'avis de personnes plus calés que moi .

Mon trigger est lancé après l'update dans la table product_to_techdata qui contient (id,id_techdata,prix,quantité) ==> il s'agit d'une table créée pour stocker les données d'un fournisseur
, et avec ce trigger, j'aimerai qu'il me compare le prix et le stock de la table product AVEC le prix et le stock de ma table product_to_techdata.Le but étant simple : Si le stock > 0 et que le prix est inférieur,
j'aimerai updater l'occurence dans ma table product(son prix, son fournisseur et son stock) .

Pour résumer , les infos contenues dans la table product_to_techdata sont ajoutés par un script php , et j'aimerai que le SGBD se charge d'écraser le prix s'il est moin cher ainsi que le stock s'il est >0 de la table product_to_techdata vers la table product

Voici mon trigger

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
 
CREATE TRIGGER update_techdata AFTER UPDATE ON product_to_techdata
FOR EACH ROW
BEGIN
DECLARE v_stock integer;
DECLARE v_price decimal(15,2);
 
select price,quantity into @v_price,@v_stock from product where product_id = NEW.product_id;
 
if(@v_price > NEW.price AND @v_stock > 0) THEN
 
UPDATE product SET price = @v_price , quantity = @v_stock WHERE product_id = NEW.product_id;
END IF;
 
END

Malheureusement , je n'ai pas d'idée pour tester les données recues ( je travaille sur phpmyadmin) et pour résoudre le problème
Ceux qui ont une idée est la bienvenue

Merci beaucoup!

Benjamin122

[CinePhil]

Je ne suis pas spécialiste des triggers (j'en ai écrit très peu à ce jour) mais le votre me semble bon, à première vue.

Malheureusement , je n'ai pas d'idée pour tester les données recues ( je travaille sur phpmyadmin) et pour résoudre le problème

Ben copiez votre base de production sur une base de test, et tentez de faire une mise à jour par une requête UPDATE et regardez si ça fonctionne.
Testez aussi les cas qui ne doivent pas mettre à jour le prix dans la table product.

[benjamin122]

Bonjour,

Merci pour la réponse. Bien entendu , si j'effectue l'update manuellement , ça fonctionne. Je crois que le problème vient du select.
J'aimerai vérifier avant le update ce que contiennent respectivement mes variables v_stock et v_price. Comment faire?
(Je travaillais essentiellement les triggers sur oracle-sqldevelopper et il suffit d'ajouter une sortie output et je pouvais tester ceux-ci)

Je remet une version de code amélioré au niveau de la syntaxe et des '`' , sans effet, ça ne marche toujours pas

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
 
CREATE TRIGGER update_techdata AFTER UPDATE ON product_to_techdata
FOR EACH ROW
BEGIN
DECLARE v_stock integer;
DECLARE v_price decimal(15,2);
 
select `price`,`quantity` into @v_price,@v_stock from `product` where `product_id` = NEW.product_id;
 
if(@v_price > NEW.price AND @v_stock > 0) THEN
 
UPDATE `product` SET `price` = @v_price , `quantity` = @v_stock WHERE `product_id` = NEW.product_id;
END IF;
 
END

Merci beaucoup

[aieeeuuuuu]

Bonjour,

Il me semble que sous MySQL, dans les triggers, les variables locales doivent être déclarées ET utilisées sans @.

Essayez de retirer les @ dans votre code.

Pour voir le contenu de vos variables, vous pouvez simplement les insérer dans une table créée pour l'occasion (avec par exemple une colonne datetime en plus pour vous y retrouver...)

[benjamin122]

Bonsoir,

Alors j'ai avancé dans mes recherches mais je n'ai toujours pas trouvé la solution . L'erreur semble étrange Ca doit être un problème interface chaise/clavier
Plus sérieusement , j'ai testé un insert comme aieeeuuuuu me l'a conseillé et étrangement , les données se stockent bien via le insert mais le update fonctionne a moitié ,
les 2 champs qui sont updatés avec les variables locales mysql restent inchangé , tandis que le date_modified est bien updaté.

Curieux de trouver le problème

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
DROP TRIGGER IF EXISTS `after_update_techdata`; TRIGGER `after_update_techdata` AFTER UPDATE ON `product_to_techdata` FOR EACH ROW BEGIN
DECLARE v_stock integer;
DECLARE v_price decimal(15,2);
 
select `price`,`quantity` into v_price,v_stock from `product` where `product_id` = NEW.product_id;
 
 
INSERT INTO error_logs VALUES (10,v_stock,v_price,'yo',NEW.price,NEW.product_id);
 
if(v_price > NEW.price AND v_stock > 0) THEN
 
UPDATE `product` SET price = v_price , quantity = v_stock , date_modified = NOW() WHERE `product_id` = NEW.product_id;
END IF;
 
END;

Benjamin122

[benjamin122]

Bonsoir,

Le problème vient d'une erreur de logique , ...

Lors du select , je récupère les deux variables et les ré-écrase ensuite , alors que les nouvelles valeurs à updater sont NEW.price et NEW.stock

Je mets mon code qui fonctionne

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
BEGIN
DECLARE v_stock integer;
DECLARE v_price decimal(15,2);
 
select `price`,`quantity` into v_price,v_stock from `product` where `product_id` = NEW.product_id;
 
if(v_price > NEW.price AND v_stock > 0) THEN
 
UPDATE `product` SET `price` =  NEW.price, `quantity` = NEW.quantity , date_modified = NOW() WHERE `product_id` = NEW.product_id;
END IF;
 
END

Un grand merci à tout ceux qui ont participé


Benjamin122

[aieeeuuuuu]

En effet, comme quoi, plus il y a de code, plus il y a de bugs.
Il me semble que vous pourriez simplement écrire votre déclencheur ainsi :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
BEGIN
 
UPDATE `product` 
    SET `price` =  NEW.price
    , `quantity` = NEW.quantity 
    , date_modified = NOW() 
    WHERE `product_id` = NEW.product_id;
    AND price > NEW.price 
    AND quantity > 0
;
 
END