Discussion :

[WhoIsTiti]

Salut,

Localement, je suis avec une configuration MySQL v5.5.9 et phpMyAdmin 3.3.9.1.
J'ai programmé un simple trigger interdisant la suppression de compte clients dont les ID sont entre 1 et 10 :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
delimiter //
CREATE TRIGGER delete_forbidden BEFORE DELETE ON osc_customers
FOR EACH ROW 
BEGIN 
    DECLARE ERR_INITIAL CONDITION FOR SQLSTATE '12345';
    IF ((SELECT customers_id FROM osc_customers WHERE customers_id = OLD.customers_id) BETWEEN 1 AND 10 ) THEN
SIGNAL ERR_INITIAL
SET MESSAGE_TEXT = 'Permission denied : this customer can not be deleted'; 
    END IF; 
END//
delimiter ;

le trigger est bien enregistré et fonctionnel (la suppression d'un client entre 1 et 10 renvoit bien le message d'erreur)

Le soucis est que sur ma base de données distante, j'ai l'erreur suivante :

#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 'ERR_INITIAL SET MESSAGE_TEXT = 'Permission denied : this customer can not be del' at line 7

en plus la config n'est pas la même : MySQL v5.1.39-1 et phpMyAdmin 3.4.8

là, je sèche complet.
Où est mon erreur de syntaxe ?

Merci.

Cdt,

Laurent

[Grandal]

Bonjour

Il ne manque pas un ";" à la fin du Signal ??

Francis

[ericd69]

salut,

ça vient peut-être de ton numéro de code d'erreur:

Attention, avant de choisir un code d'erreur, il faut s'assurer qu'il n'est pas déjà pris par le SGBD.
MySQL utilise les codes allant de 1000 à 2999 (tous ne sont pas utilisés mais il faut s'attendre à ce qu'ils puissent être pris dans les versions ultérieurs). Nous ne savons pas précisément comment le SGBD va évoluer et si les codes que nous allons choisir seront utilisés par la suite ou resterons libres, il est alors préférable de choisir un code négatif qui ne sera pas utilisé par MySQL.

il se peut donc qu'entre les 2 versions ce code soit utilisé...

[WhoIsTiti]

Salut,

ça fait une journée que j'écume tous les sujets du net ...

d'ailleurs, lorsque j'exporte la table TRIGGERS de la base système Information_schema :

Code :

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

1
2
3
 
INSERT INTO `TRIGGERS` (`TRIGGER_CATALOG`, `TRIGGER_SCHEMA`, `TRIGGER_NAME`, `EVENT_MANIPULATION`, `EVENT_OBJECT_CATALOG`, `EVENT_OBJECT_SCHEMA`, `EVENT_OBJECT_TABLE`, `ACTION_ORDER`, `ACTION_CONDITION`, `ACTION_STATEMENT`, `ACTION_ORIENTATION`, `ACTION_TIMING`, `ACTION_REFERENCE_OLD_TABLE`, `ACTION_REFERENCE_NEW_TABLE`, `ACTION_REFERENCE_OLD_ROW`, `ACTION_REFERENCE_NEW_ROW`, `CREATED`, `SQL_MODE`, `DEFINER`, `CHARACTER_SET_CLIENT`, `COLLATION_CONNECTION`, `DATABASE_COLLATION`) VALUES
('def', 'nom_de_ma_base', 'delete_forbidden', 'DELETE', 'def', 'nom_de_ma_base', 'osc_customers', 0, NULL, 'BEGIN \r\n    DECLARE msg VARCHAR(255);\r\n    DECLARE ERR_INITIAL CONDITION FOR SQLSTATE ''12345'';\r\n    IF ((SELECT customers_id FROM osc_customers WHERE customers_id = OLD.customers_id) BETWEEN 1 AND 10 ) THEN\r\nSIGNAL ERR_INITIAL\r\nSET MESSAGE_TEXT = ''Permission denied : this customer can not be deleted''; \r\n    END IF; \r\nEND', 'ROW', 'BEFORE', NULL, NULL, 'OLD', 'NEW', NULL, '', 'root@localhost', 'utf8', 'utf8_general_ci', 'utf8_general_ci');

on retrouve bien le trigger.
Bref, le sql dynamique sur ce SGBD est vraiment merdique.

au pire des cas, je coderai une grosse bouse du style :

Code :

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

1
2
3
4
5
6
7
8
 
CREATE TRIGGER delete_forbidden BEFORE DELETE ON osc_customers
FOR EACH ROW 
BEGIN
IF OLD.customers_id < 10 THEN    
   CALL CustLimit();
END IF;
END

avec une procédure CustLimit() inexistante ...

[edit]
sauf erreur de ma part, la gestion des SIGNAL pour les triggers n'est pas implémentée dans la version MySQL 5.1, d' où l'erreur de syntaxe.
ça serait bien que cela soit confirmé par un cador du mysql.
[/edit]

[WhoIsTiti]

Donc je confirme bien qu'avec MySQL 5.1.x, les SIGNAL ne sont pas implémentés.
Chez mon hébergeur, c'est cette version en cours.
Alors j'ai changé mon fusil d'épaule, aujourd'hui je développe dans le cloud avec Pagodabox, qui eux, ont la dernière version MySQL 5.5.22.
Petite restriction quand même, la programmation de TRIGGER n'est pas possible (pour une histoire de réplication des DB hébergées dans le cloud)
Néanmoins, les procédures stockées sont possibles.
Du coup, mon code a légèrement changé :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
DROP PROCEDURE `DeleteCustomer`;
DELIMITER ;;
CREATE PROCEDURE `DeleteCustomer` (IN `custid` int)
BEGIN
DECLARE msg VARCHAR(255);
IF (SELECT protected FROM osc_customers WHERE customers_id = custid AND protected = 1) THEN 
set msg = CONCAT("This action is forbidden for the customer with the ID " , custid);
SIGNAL SQLSTATE '45000' SET MESSAGE_TEXT = msg;
ELSE
DELETE FROM osc_customers WHERE customers_id = custid; 
END IF;
END;;
DELIMITER ;

Donc, dans mon application PHP, j'ai viré les requêtes en les remplaçant avec des CALL xxx(IN yyy)
Je n'ai plus qu'à créer une class pdo_exception pour gérer les retours des messages dans l'interface.

Un de ses quatre, je présenterai l'ensemble du projet sur Developpez.com afin de recueillir l'avis de professionnels.

A+