Discussion :

[aurelie83]

J'ai un pb de mise à jour.
Je connais mon erreur mais je ne sais pas du tt comment la resoudre...

Voila la requete:

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
23
24
25
26
27
28
29
update FRH set (fait_id,frh.absence_id,
  frh.temps_id,
  frh.temps_id_fin,
  frh.salarie_id,
  nb_jour_ouvre )= 
 
(select DISTINCT frh.fait_id,
  frh.absence_id,
  frh.temps_id,
  frh.temps_id_fin,
  frh.salarie_id,
(CASE 
WHEN sum(drh_temps.ferie) = to_date(frh.temps_id_fin)-to_date(frh.temps_id)+1 AND To_CHAR(frh.temps_id,'D')='6' THEN 0
WHEN sum(drh_temps.ferie) = to_date(frh.temps_id_fin)-to_date(frh.temps_id)+1 AND To_CHAR(frh.temps_id,'D')='7' THEN 0
WHEN sum(drh_temps.ferie) is null THEN (to_date(frh.temps_id_fin)-to_date(frh.temps_id)) + 1
WHEN sum(drh_temps.ferie)>=1 THEN to_date(frh.temps_id_fin)-to_date(frh.temps_id)-sum(drh_temps.ferie)+1
ELSE 0
END) as NB_JOUR_OUVRE
from frh,drh_temps
where drh_temps.temps_id between frh.temps_id and frh.temps_id_fin
And absence_id is not null
and fait_id=(select distinct frh.fait_id from frh where absence_id is not null 
and nb_jour_ouvre is null)
group by frh.fait_id,frh.absence_id,
  frh.temps_id,
  frh.temps_id_fin,
  frh.salarie_id)
 
WHERE fait_id=(select distinct frh.fait_id from frh where absence_id is not null and nb_jour_ouvre is null )

L'erreur est :
ORA-01427: single-row subquery returns more than one row

Ce qui est parfaitement logique.
En fait, j'ai besoin absolument de mettre a jour NB_JOUR_OUVRE. Mais pour faire l'update et pour que la sous-requete fonctionne, il me faut lire ts les champs....

Vous avez une solution?

Merci d'avance!

[chrifo]

A l'instinct : il ne manquerai pas un niveau de sous requete ?

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
23
24
25
26
27
28
29
30
update FRH set (fait_id,frh.absence_id,
  frh.temps_id,
  frh.temps_id_fin,
  frh.salarie_id,
  nb_jour_ouvre )= 
(select * from
(select DISTINCT frh.fait_id,
  frh.absence_id,
  frh.temps_id,
  frh.temps_id_fin,
  frh.salarie_id,
(CASE 
WHEN sum(drh_temps.ferie) = to_date(frh.temps_id_fin)-to_date(frh.temps_id)+1 AND To_CHAR(frh.temps_id,'D')='6' THEN 0
WHEN sum(drh_temps.ferie) = to_date(frh.temps_id_fin)-to_date(frh.temps_id)+1 AND To_CHAR(frh.temps_id,'D')='7' THEN 0
WHEN sum(drh_temps.ferie) is null THEN (to_date(frh.temps_id_fin)-to_date(frh.temps_id)) + 1
WHEN sum(drh_temps.ferie)>=1 THEN to_date(frh.temps_id_fin)-to_date(frh.temps_id)-sum(drh_temps.ferie)+1
ELSE 0
END) as NB_JOUR_OUVRE
from frh,drh_temps
where drh_temps.temps_id between frh.temps_id and frh.temps_id_fin
And absence_id is not null
and fait_id=(select distinct frh.fait_id from frh where absence_id is not null 
and nb_jour_ouvre is null)
group by frh.fait_id,frh.absence_id,
  frh.temps_id,
  frh.temps_id_fin,
  frh.salarie_id) FRH_MAJ
where FRH.fait_id = FRH_MAJ.fait_id)
WHERE fait_id=(select distinct frh.fait_id from frh where absence_id is not null and nb_jour_ouvre is null )

[McM]

Tu sais quelle sous requete ramène plus d'une ligne ?
Moi je pense à celui ci :

Code :

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

1
2
3
4
WHERE fait_id=(SELECT DISTINCT frh.fait_id 
FROM frh 
WHERE absence_id IS NOT NULL 
AND nb_jour_ouvre IS NULL )

Sinon évite les DISTINCT et les GROUP BY combinés.. ça sert à rien.

En fait ta requete est fausse, faut tout relier à frh de l'UPDATE.

[McM]

Ce serait pas plutôt un code comme celui ci ?

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
UPDATE FRH frh
SET nb_jour_ouvre = 
(SELECT (CASE 
	WHEN SUM(ferie) = frh.temps_id_fin-frh.temps_id+1 AND TO_CHAR(frh.temps_id,'D')='6' THEN 0
	WHEN SUM(ferie) = frh.temps_id_fin-frh.temps_id+1 AND TO_CHAR(frh.temps_id,'D')='7' THEN 0
	WHEN SUM(ferie) IS NULL THEN frh.temps_id_fin - frh.temps_id + 1
	WHEN SUM(ferie)>=1 THEN frh.temps_id_fin-frh.temps_id-SUM(drh_temps.ferie)+1
	ELSE 0
	END) AS NB_JOUR_OUVRE
FROM DRH_TEMPS
WHERE drh_temps.temps_id BETWEEN frh.temps_id AND frh.temps_id_fin
)
WHERE fait_id IN (	SELECT fait_id
		FROM FRH 
		WHERE absence_id IS NOT NULL 
		AND nb_jour_ouvre IS NULL )

PS : Pas de TO_DATE sur une DATE !

[aurelie83]

Merci beaucoup. c'est exactement ce que je voulais.

[McM]

Au fait, fait_id : C'est la PK de FRH ?

Si oui, faut remplacer

Code :

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

1
2
3
4
WHERE fait_id IN (	SELECT fait_id
		FROM FRH 
		WHERE absence_id IS NOT NULL 
		AND nb_jour_ouvre IS NULL )

par

Code :

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

1
2
WHERE absence_id IS NOT NULL 
AND nb_jour_ouvre IS NULL

[aurelie83]

oui j'avais fait la correction

MErci bcp!