Discussion :

[stc074]

Bonjour, j'ai créer un trigger pour ma table users :

Code :

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

1
2
3
4
5
CREATE TRIGGER update_users
  AFTER UPDATE
  ON users
  FOR EACH ROW
  EXECUTE PROCEDURE update_users();

Code :

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

1
2
3
4
5
6
7
8
9
10
CREATE OR REPLACE FUNCTION update_users()
  RETURNS trigger AS
$BODY$
begin
delete from code_mdp where id_user=new.id;
 return null;
end
$BODY$
  LANGUAGE plpgsql VOLATILE
  COST 100;

Quand je fais un update de users en java les entrées correspondant à l'id que j'utilise ne s’effacent pas :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	protected void updatePassword() throws SQLException, NamingException, MyException {
		StrongPasswordEncryptor passwordEncryptor = new StrongPasswordEncryptor();
		String encryptedPassword = passwordEncryptor.encryptPassword(password);
		//
		String query = "UPDATE users SET mdp=? WHERE id=?";
		try (PreparedStatement prepare = getConn().prepareStatement(query)) {
			prepare.setString(1, encryptedPassword);
			prepare.setLong(2, id);
			if (prepare.executeUpdate() != 1) {
				throw new MyException("Erreur lors du changement de mot de passe.<br/>");
			}
		}
    }

Je n'ai pas de message d'erreur dans les logs donc difficile de voir ou est le problème.
Merci.

[estofilo]

Il n'y pas de problème apparent dans ce code mais le fait d'ajouter ça sous le DELETE
donnerait peut être un indice:

Code :

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

raise warning 'delete: %, id=%', FOUND, new.id;

FOUND est un booléen qui vaudra t si au moins une ligne est effaçée par le DELETE et f sinon.
un RAISE WARNING sort par défaut dans les logs du serveur, car log_min_messages=WARNING

[champomy62]

Bonjour,

Pour commencer, il est toujours bon de tester son trigger directement en SQL



Sinon:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
CREATE OR REPLACE FUNCTION update_users()
 RETURNS trigger AS $update_users$
BEGIN
IF (TG_OP = 'DELETE') THEN
        delete from code_mdp where id_user=OLD.id;
        return OLD;
END IF;
END;
$update_users$ LANGUAGE plpgsql;

Teste ce code ...

[stc074]

Bonjour, je crée mes trigger via la console SQL, ton code diffère, par contre c'est pas plutôt :

Code :

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

IF (TG_OP = 'UPDATE')

(Mon trigger doit se lancer après un update)

Le SQL du trigger semble bon (aucune erreur reportée) mais la fonction n'a pas l'air d'être appelée.
Merci.

[estofilo]

Au vu de la déclaration du trigger:

Code :

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

1
2
3
4
CREATE TRIGGER update_users
  AFTER UPDATE
  ON users
  etc...

Ajouter un test TG_OP = 'DELETE' n'a aucun intérêt car il sera toujours faux

Et un test TG_OP = 'UPDATE' n'a aucun intérêt non plus car il sera toujours vrai

[stc074]

Au vu de la déclaration du trigger:

Code :

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

1
2
3
4
CREATE TRIGGER update_users
  AFTER UPDATE
  ON users
  etc...

Ajouter un test TG_OP = 'DELETE' n'a aucun intérêt car il sera toujours faux

Et un test TG_OP = 'UPDATE' n'a aucun intérêt non plus car il sera toujours vrai

C'est bien ce qui me semblait aussi. :-))