Re: RETURN cannot have a parameter in function with OUT parameters

Bonjour à tous,

Toujours dans mon fonction "contact" de la dernière fois, PostgreSQL me
sort une erreur bizare avec une simple fonction plpgsql...

"5-clients-fonctions.sql:36: ERREUR: RETURN cannot have a parameter in
function with OUT parameters sur ou près de « v_id » at character 1486"

Je ne comprend vraiment pas qu'est-ce qu'il ce passe... Pouvez-vous
m'aider ? Ci-dessous la fonction concernée.

-----------------------
CREATE OR REPLACE FUNCTION clients.contact (p_nom text, p_email text,
inout TRIGGER integer) RETURNS integer AS $contact$
DECLARE
v_id integer DEFAULT 0;
BEGIN
IF (TRIGGER != 0) THEN
TRIGGER := 1;
END IF;
SELECT id INTO v_id FROM clients.contacts WHERE nom = p_nom AND
email = p_email LIMIT 1;
IF NOT FOUND THEN
-- Le couple nom/email n'as pas été trouvé. Cependant, est-ce
que l'adresse mail
-- existe déjà et est acceptée ?
PERFORM id FROM clients.contacts WHERE email = p_email AND actif
= 1;
IF FOUND THEN
TRIGGER := 0;
END IF;
INSERT INTO clients.contacts (nom, email, _trigger) VALUES
(p_nom, p_email, TRIGGER) RETURNING id INTO v_id;
IF (v_trigger = 0) THEN
-- Si on as choisi de pas faire éxécuter les triggers, mais
que l'on passe par ce trigger
-- c'est que le choix à été automatique dans 99% des cas.
-- Par conséquent, ça veux dire que l'adresse mail existe
déjà et est déjà activée.
-- On va donc pas chercher à envoyer un mail au contact pour
qu'il active son mail mais
-- pour valider l'ajout de l'adresse mail. Pour ça, on va
éxécuter un script PHP
-- similaire à celui de l'action du mail (même fichier PHP,
mais pas même action)
PERFORM clients."php-maj-contact"(v_id, 'nv', p_nom,
p_email);
-- nv = Nouveau
END IF;
END IF;
RETURN v_id;
END;
$contact$ language plpgsql;
-----------------------

Samuel ROZE a écrit :
> Bonjour à tous,
>
> Toujours dans mon fonction "contact" de la dernière fois, PostgreSQL me
> sort une erreur bizare avec une simple fonction plpgsql...
>
> "5-clients-fonctions.sql:36: ERREUR: RETURN cannot have a parameter in
> function with OUT parameters sur ou près de « v_id » at character 1486"
>
> Je ne comprend vraiment pas qu'est-ce qu'il ce passe... Pouvez-vous
> m'aider ? Ci-dessous la fonction concernée.
>
> -----------------------
> CREATE OR REPLACE FUNCTION clients.contact (p_nom text, p_email text,
> inout TRIGGER integer) RETURNS integer AS $contact$

Tu déclares ta fonction comme renvoyant un integer, non pas grâce à la
clause returns, mais avec le mode inout du paramètre trigger (fort
mauvais nom pour une variable car trigger est un mot clé réservé).

> [...]
> RETURN v_id;

Du coup, ici, tu ne dois pas avoir

RETURN une_variable;

mais

RETURN;

et la valeur actuelle de la variable TRIGGER sera renvoyée.

Bref, vu ton code (et surtout vu ce que tu veux renvoyer), je pense que
le mode inout est à supprimer.

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com

Le mode "inout" m'as été conseillé enfin du moins m'as été proposer dans
cette même liste de discussion...

C'est donc une mauvaise idée ? Je vais revenir à l'ancienne méthode :)

Merci.

Le mardi 02 septembre 2008 à 13:52 +0200, Guillaume Lelarge a écrit :
> Samuel ROZE a écrit :
> > Bonjour à tous,
> >
> > Toujours dans mon fonction "contact" de la dernière fois, PostgreSQL me
> > sort une erreur bizare avec une simple fonction plpgsql...
> >
> > "5-clients-fonctions.sql:36: ERREUR: RETURN cannot have a parameter in
> > function with OUT parameters sur ou près de « v_id » at character 1486"
> >
> > Je ne comprend vraiment pas qu'est-ce qu'il ce passe... Pouvez-vous
> > m'aider ? Ci-dessous la fonction concernée.
> >
> > -----------------------
> > CREATE OR REPLACE FUNCTION clients.contact (p_nom text, p_email text,
> > inout TRIGGER integer) RETURNS integer AS $contact$
>
> Tu déclares ta fonction comme renvoyant un integer, non pas grâce à la
> clause returns, mais avec le mode inout du paramètre trigger (fort
> mauvais nom pour une variable car trigger est un mot clé réservé).
>
> > [...]
> > RETURN v_id;
>
> Du coup, ici, tu ne dois pas avoir
>
> RETURN une_variable;
>
> mais
>
> RETURN;
>
> et la valeur actuelle de la variable TRIGGER sera renvoyée.
>
> Bref, vu ton code (et surtout vu ce que tu veux renvoyer), je pense que
> le mode inout est à supprimer.
>
>

Samuel ROZE a écrit :
> Le mode "inout" m'as été conseillé enfin du moins m'as été proposer dans
> cette même liste de discussion...
>
> C'est donc une mauvaise idée ? Je vais revenir à l'ancienne méthode :)
>

C'est pas forcément une mauvaise idée. Là, dans ce cas, je ne vois pas
l'intérêt.

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com

Bonjour

Samuel ROZE a écrit :
> Le mode "inout" m'as été conseillé enfin du moins m'as été proposer dans
> cette même liste de discussion...
>
> C'est donc une mauvaise idée ? Je vais revenir à l'ancienne méthode :)
>
> Merci.
>

Voila comment je combine RETURN et INOUT, INOUT je m'en sert pour
alimenter mes variables, quand a RETURN, sa valeur je m'en sert pour
information si le traitement de la fonction s'est bien passé.

Donc dans mon cas 2 utilisations différentes,

Cordialement,

Christophe Chauvet

Christophe Chauvet a écrit :
> [...]
> Samuel ROZE a écrit :
>> Le mode "inout" m'as été conseillé enfin du moins m'as été proposer dans
>> cette même liste de discussion...
>>
>> C'est donc une mauvaise idée ? Je vais revenir à l'ancienne méthode :)
>> Merci.
>>
>
> Voila comment je combine RETURN et INOUT, INOUT je m'en sert pour
> alimenter mes variables, quand a RETURN, sa valeur je m'en sert pour
> information si le traitement de la fonction s'est bien passé.
>
> Donc dans mon cas 2 utilisations différentes,
>

Quelle version de PostgreSQL ?
Tu peux nous montrer ta fonction ?

Parce que, moi, en 8.3.3, j'ai ceci:

glmf=# create or replace function f (inout p1 integer) returns integer
glmf-# language plpgsql
glmf-# as $$
glmf$# begin
glmf$# return 2;
glmf$# end;
glmf$# $$;
ERREUR: RETURN cannot have a parameter in function with OUT parameters
sur ou près de « 2 »
LIGNE 5 : return 2;

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com

Guillaume Lelarge a écrit :
> Quelle version de PostgreSQL ?
> Tu peux nous montrer ta fonction ?
>
Autant pour moi, c'est sous Oracle que c'est possible :( ,
effectivement PostgreSQL ne gère pas cette possibilité.

Christophe.
P.S. j'avais oublié de mettre la liste en copie :)

Christophe Chauvet a écrit :
> Guillaume Lelarge a écrit :
>> Quelle version de PostgreSQL ?
>> Tu peux nous montrer ta fonction ?
>>
> Autant pour moi, c'est sous Oracle que c'est possible :( ,
> effectivement PostgreSQL ne gère pas cette possibilité.
>

Pour une fois que c'est à mon avantage de ne pas connaître Oracle :)

--
Guillaume.
http://www.postgresqlfr.org
http://dalibo.com