Discussion :

[palassou]

Bonjour,

J'ai créé une procédure afin de lister chaque jour, les saisies d'activités qui n'ont pas été effectué pour la date du jour.
Quand j'exécute ma fonction, je n'ai pas de message d'erreur mais il ne me retourne aucun enregistrement : "La requête a été exécutée avec succès en 15 ms, mais ne renvoie aucun résultat."
Quand j'exécute ma requête à part, je récupère bien des enregistrements :

Code :

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

1
2
3
4
5
6
7
8
9
 
id integer
_________
2
3
4
6
12
14

Je pense que je n'utilise pas correctement FETCH.
Merci d'avance pour votre aide.

Voici le code de ma fonction :

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
16
17
18
19
20
21
 
CREATE OR REPLACE FUNCTION fonction_nonsaisie() RETURNS INTEGER AS
'
DECLARE
    ref CURSOR for select personnes.id from personnes where (datedeb<=CURRENT_DATE and datefin>=CURRENT_DATE and actif=true) or (datedeb is null and datefin is null and actif=true) EXCEPT select saisies.personne_id from saisies where date_saisie=CURRENT_DATE group by saisies.personne_id having count(saisies.personne_id)=2;
    pers_id int;
 
BEGIN
 
OPEN ref;
 
FETCH ref INTO pers_id;
LOOP
  insert into nonsaisies (personne_id, date_nonsaisie) values (pers_id,CURRENT_DATE);
end loop;
close ref;
 
RETURN 1;
END;
'
LANGUAGE 'plpgsql';

[estofilo]

Dans le code montré, il n'y a pas de condition de sortie à l'intérieur du LOOP donc ça devrait faire une boucle infinie.

[palassou]

Merci.
J'ai modifié le code comme ci-dessous mais j'ai toujours la même erreur.

Code :

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

1
2
3
4
5
6
 
FETCH ref INTO pers_id;
LOOP 
  insert into nonsaisies (personne_id, date_nonsaisie) values (pers_id, CURRENT_DATE);
  exit when ref%notfound ;
end loop;

[estofilo]

Faire plutôt:

Code :

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

1
2
3
4
5
6
 
LOOP 
  FETCH ref INTO pers_id;
  exit when ref%notfound ;
  INSERT INTO nonsaisies (personne_id, date_nonsaisie) VALUES (pers_id, CURRENT_DATE);
end loop;

[palassou]

En modifiant c'est toujours pareil. J'ai essayé en modifiant l'ordre de valorisatiob du curseur, j'obtiens le même résultat.

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
16
17
18
19
20
21
22
CREATE OR REPLACE FUNCTION fonction_nonsaisie() RETURNS INTEGER AS
'
DECLARE
    ref refcursor;
    pers_id int;
 
BEGIN
 
OPEN ref FOR select personnes.id from personnes where (datedeb<=CURRENT_DATE and datefin>=CURRENT_DATE and actif=true) or (datedeb is null and datefin is null and actif=true) EXCEPT select saisies.personne_id from saisies where date_saisie=CURRENT_DATE group by saisies.personne_id having count(saisies.personne_id)=2;
 
LOOP 
  FETCH ref INTO pers_id;
  exit when ref%notfound ;
  INSERT INTO nonsaisies (personne_id, date_nonsaisie) VALUES (pers_id, CURRENT_DATE);
end loop;
 
close ref;
 
RETURN 1;
END;
'
LANGUAGE 'plpgsql';

[estofilo]

Le même résultat, c'est-à-dire? Parce que le résultat de la fonction à proprement parler est toujours 1, puisqu'elle fait RETURN 1.
C'est le contenu de la table nonsaisies au retour de la fonction qui n'est pas bon? Dans ce cas quel est le contenu et qu'est-ce qu'il devrait être à la place?

[palassou]

Cela venait du "exit when".
Bon j'ai finalement changer de boucle est là j'obtiens mes résultats. Merci pour votre aide.

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 OR REPLACE FUNCTION fonction_nonsaisie() RETURNS INTEGER AS
'
DECLARE
    mviews RECORD;
 
BEGIN
 
FOR mviews IN select personnes.id from personnes where (datedeb<=CURRENT_DATE and datefin>=CURRENT_DATE and actif=true) or (datedeb is null and datefin is null and actif=true) EXCEPT select saisies.personne_id from saisies where date_saisie=CURRENT_DATE group by saisies.personne_id having count(saisies.personne_id)=2 LOOP
  INSERT INTO nonsaisies (personne_id, date_nonsaisie) VALUES (mviews.id, CURRENT_DATE);
end loop;
 
RETURN 1;
END;
'
LANGUAGE 'plpgsql';

[estofilo]

OK. A noter que la boucle n'est même pas nécessaire, il suffit de mettre le select derrière l'insert. Et dans ce cas la fonction elle-même n'est plus nécessaire non plus.

Code :

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

1
2
3
4
5
6
7
 
INSERT INTO nonsaisies (personne_id, date_nonsaisie) 
select personnes.id, CURRENT_DATE
 from personnes where (datedeb<=CURRENT_DATE and datefin>=CURRENT_DATE and actif=true) or (datedeb is null and datefin is null and actif=true)
EXCEPT
 select saisies.personne_id from saisies where date_saisie=CURRENT_DATE
group by saisies.personne_id having count(saisies.personne_id)=2