Discussion :

[pseudomh]

Bonjour,

j'essaie d'utiliser des conditions dans une procédure stocké mais ça ne marche pas :
CREATE OR REPLACE FUNCTION list(varchar,varchar) RETURNS setof org AS
$body$
IF $2 <> 'ADM'
select * from organigramme where entorg like (select entorg from agent where matricule like $1) || '%';
else
select * from organigramme;
end
$body$
LANGUAGE 'sql' VOLATILE CALLED ON NULL INPUT SECURITY INVOKER;

quelqu'un aurait-il une idée ?

merci

[piff62]

La syntaxe a utiliser

Code :

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

1
2
3
4
5
6
 
IF condition THEN
    ...
ELSE
    ...
END IF;

[pseudomh]

j'avais déjà essayé mais ça ne marche pas.

CREATE OR REPLACE FUNCTION list(varchar,varchar) RETURNS setof org AS
$body$
IF $2 <> 'ADM' then
select * from org where entorg like (select entorg from agent where matricule like $1) || '%';
else
select * from org;
end if;
$body$
LANGUAGE 'sql' VOLATILE CALLED ON NULL INPUT SECURITY INVOKER;

=> il me met error at or near if

[hpalpha]

il manque BEGIN .... END

Code :

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

1
2
3
4
5
6
7
8
9
10
11
CREATE OR REPLACE FUNCTION list(varchar,varchar) RETURNS setof org AS
$body$
BEGIN
IF $2 <> 'ADM' then
select * from org where entorg like (select entorg from agent where matricule like $1) || '%';
else
select * from org;
end if;
END
$body$
LANGUAGE 'sql' VOLATILE CALLED ON NULL INPUT SECURITY INVOKER;

[pseudomh]

...ça ne change rien, même message

[hpalpha]

j'avais pas remarqué , mais tu prend comme langage 'sql' , je ne pense pas que begin, if, else, .... existent, c'est le plpgsql qui le permet

[Quentin]

Forcément, un bloc "IF THEN ELSE" dans une procédure codée en sql, il va avoir du mal à comprendre

Code :

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

LANGUAGE 'plpgsql'

EDIT : arf, trop lent

[pseudomh]

oui mais là il me met "SELECT query has no destination for result data"

[Quentin]

Effectivement, c'est normal... Si on reste en plpgsql, il faut boucler sur ton select avec un for et faire une série de 'return next'...

Mais bon, il doit y avoir moyen de rester avec du SQL pur. Essaie comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
CREATE OR REPLACE FUNCTION list(varchar,varchar) RETURNS setof org AS
$body$
 
select * from org
where (case when $2 <> 'ADM' 
      then (entorg like (select entorg from agent where matricule like $1) || '%')
      else true end);
$body$
LANGUAGE 'sql' VOLATILE CALLED ON NULL INPUT SECURITY INVOKER;

[pseudomh]

ça marche impec !!!

merci merci merci !!