erreur SQL lié a un appel de fonction?

Version imprimable

Bonjour,

J'espère etre sur le bon forum...J'essaye d'executer une requete SQL afin d'exporter les données de ma base sous format excel. J'utilise pour cela du javascript. Mon code javascript construit la requête selon un appel de fonction dans la balise

Code:

<form>

Lorsque je clique sur le submit exporter j'ai un message d'erreur me disant qu'il y a une erreur de syntaxe dans mon SQL. mais c'est en faite parce que mon champs texte ne prends pas la requete comme texte... et je ne comprends pas pourquoi.

voici mon code :

Code:

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
31
32
33
34
35
36
 
<script language="javascript">
function verifier_saisie_date_debut_et_fin(num_semaine_deb,num_semaine_fin,annee,num_crc,code_canal,date_deb,date_fin) 
{	
var execute = true;
alert(num_semaine_deb+","+num_semaine_fin+","+annee+","+num_crc+","+code_canal+","+date_deb+","+date_fin);)
if(execute)
{
 
if (date_fin=="")
{	 
 
if((code_canal ==5) || (code_canal ==9)  || (code_canal ==1))
{
requete = "SELECT of.libelle_offre,ob.objectifs,v.nbvente FROM offres of LEFT OUTER JOIN ( SELECT ob.code_offre, sum(ob.obj) AS objectifs FROM obectifs ob WHERE ob.num_semaine = "+num_semaine_deb+" AND ob.annee = "+annee+"  AND ob.num_agent IN (SELECT num_agent from agents,equipes,crc where agents.num_equipe = equipes.code_equipe AND crc.num_crc = equipes.crc AND equipes.crc = '"+num_crc+"') GROUP BY ob.code_offre) ob ON ob.code_offre = of.code_offre";
 
requete = requete+" LEFT OUTER JOIN (SELECT v.code_offre, count(v.num_vente) AS nbvente FROM ventes v WHERE v.num_agent IN (SELECT num_agent from agents,equipes,crc where agents.num_equipe = equipes.code_equipe AND crc.num_crc = equipes.crc AND equipes.crc = '"+num_crc+"')AND v.date_vente = '"+date_deb+"' GROUP BY v.code_offre) v ON v.code_offre = of.code_offre   WHERE of.code_canal = 'crc_qe' order by ordre";
}
 
this.document.res_obj.requete_XLS.value=requete;
return true;
}
 
}	
else
{
i=num_semaine_fin-num_semaine_deb;
 
}
else 
{ 
return false;
}
}
 
</script>

le code html

Code:

1
2
3
4
5
6
 
<form name="res_obj" action="javascript:exporter_au_format_XLS(this.document.res_obj.requete_XLS.value)" method="POST" onSubmit="return verifier_saisie_date_debut_et_fin(22,0,2010,'CRC_ANGOU',5,'2010-06-01','')">
 
<input type="text" name="requete_XLS" value="Exporter"/>
<input type="submit" value="Exporter" class="btntxt"/>
</form>

Je ne comprends pas trop ce qu'il se passe. Merci de votre aide

15/06/2010, 15h09
SpaceFrog

hormis le fait que passer le string de requete est une hérésie, mais je pense qu'il doit s'agir d'une application en local ...
je pense que tu devrais tester le contenu de la chaine à reception
ton form doit url_encoder les données et si tu les files brut de pomme à SQL je comprends qu'il rechigne car ton string de requete doit etre truffé de %0A etc ...
il faut selon moi urldecoder ou unescaper en fonction du langage de traitement
15/06/2010, 15h14
RomainVALERI
Commence peut-être par faire référence à tes éléments d'une manière plus efficace :
Code:

1 2 this.document.res_obj.requete_XLS.value=requete; this.document.getElementById("requete_XLS").value = requete;
avec une propriété id="requete_XLS" pour l'input servant à l'exportation.
15/06/2010, 15h20
ballantine's
Citation:
Envoyé par RomainVALERI

Commence peut-être par faire référence à tes éléments d'une manière plus efficace :

Code:

1 2 this.document.res_obj.requete_XLS.value=requete; this.document.getElementById("requete_XLS").value = requete;

avec une propriété id="requete_XLS" pour l'input servant à l'exportation.
cela ne change rien, j'ai toujours la même erreur.Dans ma barre d'adresse j'ai toujours req= Exporter au lieu de ma requete SQL...

Merci.
15/06/2010, 15h25
RomainVALERI

Je ne connais pas le contexte d'execution et j'ai essayé de coller à la syntaxe que tu utilisais : d'où le "this" que j'ai laissé avant le "document.getElementById". Mais dans le contexte d'une page web il faudrait le retirer.
15/06/2010, 15h34
ballantine's

J'ai déjà utilisé ce script pour faire d'autres exports qui fonctionnent très bien. Je comprends pas trop
15/06/2010, 16h06
E.Bzz

Bonjour,

Citation:

Envoyé par ballantine's

j'ai un message d'erreur me disant qu'il y a une erreur de syntaxe dans mon SQL.

Le terme est mal choisi : il s'agit visiblement plutôt d'une faute d'orthographe

Citation:

FROM obectifs

;)

A+
15/06/2010, 16h08
ballantine's

Non non ce n'est pas une faute d'orthographe...la table est mal nommée mais elle n'a pas encore été refaite c'est bien obectifs
15/06/2010, 16h11
E.Bzz

:aie:

Ok.

Ca dépend sûrement des SGBD, mais l'espace manquant ici ne pose pas problème ?

Code:

AND equipes.crc = '"+num_crc+"')AND v.date_vente

Sinon, un copié/collé de cette requête, lancé en ligne de commande SQL fonctionne ?

A+
15/06/2010, 16h18
ballantine's

L'espace ne change rien :(

Lorsque je l'execute directement sur MySQL la requête fonctionne sans problème
15/06/2010, 16h34
devyan

Citation:

Envoyé par ballantine's

L'espace ne change rien :(

Lorsque je l'execute directement sur MySQL la requête fonctionne sans problème

Donc tu as fais un alert(requete) que tu as collé dans ton éditeur de requête (phpmyadmin ou autre) et ça fonctionne ...
15/06/2010, 16h37
ballantine's

Citation:

Envoyé par devyan

Donc tu as fais un alert(requete) que tu as collé dans ton éditeur de requête (phpmyadmin ou autre) et ça fonctionne ...

pas tout à fait...le alert que je tente de faire ne s'affiche pas. mais lorsque je copie colle ma requete avec les valeurs que j'envoie dans ma fonction ça marche. sauf que c'est moi qui mets les valeurs et la est peut-être le problème...les valeurs
15/06/2010, 16h39
E.Bzz

Affecte le contenu de "requete" à un textarea, par exemple : tu auras le texte exacte de la requête lancée ...
Ceci dit, si le alert() ne s'affiche pas, c'est que tu as une erreur Javascript (peut être sans aucun lien) ...

A+
15/06/2010, 16h40
E.Bzz

Au fait : elle est déclarée comment et où, cette variable "requete" ?

A+
15/06/2010, 16h41
RomainVALERI

Citation:

Envoyé par ballantine's

sauf que c'est moi qui mets les valeurs et la est peut-être le problème...les valeurs

C'est pour ça que je trouvais l'intervention de SpaceFrog assez judicieuse ^^
Tu as vérifié l'encodage de la page / du serveur ? ...quitte à vérifier très précisément l'entête HTTP correspondant à ton submit avec LiveHTTPHeaders ou un équivalent ?
15/06/2010, 16h44
devyan

Citation:

Envoyé par ballantine's

pas tout à fait...le alert que je tente de faire ne s'affiche pas. mais lorsque je copie colle ma requete avec les valeurs que j'envoie dans ma fonction ça marche. sauf que c'est moi qui mets les valeurs et la est peut-être le problème...les valeurs

Si le alert() ne s'affiche pas c'est qu'il y a déjà un problème avant, genre dans le remplissage de "requete"
15/06/2010, 16h50
E.Bzz

Un élément supplémentaire : le "this" ne semble rien avoir à faire ici :?

Code:

this.document.res_obj.requete_XLS.value=requete;

De plus, tu ne testes pas que la variable est alimentée avant de l'affecter au .value

Alors qu'il suffirait que la condition du if() précédent ne soit pas respectée ... :aie:

Conclusion : la même que les précédentes :fleche: montre-nous le contenu de "requete" lors de l'exécution (y/c pour les problèmes d'encodage, fort possibles aussi) ;)

A+
15/06/2010, 17h07
ballantine's

Citation:

Envoyé par E.Bzz

Affecte le contenu de "requete" à un textarea, par exemple : tu auras le texte exacte de la requête lancée ...
Ceci dit, si le alert() ne s'affiche pas, c'est que tu as une erreur Javascript (peut être sans aucun lien) ...

A+

je l'affecte à un input text, mais le texte n'est pas modifié il reste à la valeur par défaut 'Exporter'.
15/06/2010, 17h10
devyan

Citation:

Envoyé par ballantine's

je l'affecte à un input text, mais le texte n'est pas modifié il reste à la valeur par défaut 'Exporter'.

Regarde à partir de quel moment un alert('ici'); ne fonctionne plus dans ton code et tu auras la ligne qui pose problème juste au dessus ;)

C'est une méthode antédiluvienne mais elle fonctionne toujours :D

J'ai supprimé un if qui ne servait a rien et cela fonctionne ^^

Voici le code javascript une fois modifié :

Code:

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
31
32
 
<script language="javascript">
function verifier_saisie_date_debut_et_fin(num_semaine_deb,num_semaine_fin,annee,num_crc,code_canal,date_deb,date_fin) 
{	
var execute = true;
alert(num_semaine_deb+","+num_semaine_fin+","+annee+","+num_crc+","+code_canal+","+date_deb+","+date_fin);
var requete="";
 
 
if (date_fin=="")
{	 
 
 
	if((code_canal ==5) || (code_canal ==9)  || (code_canal ==1))
	{
	requete = "SELECT of.libelle_offre,ob.obj,v.nbvente FROM offres of LEFT OUTER JOIN ( SELECT ob.code_offre, obj FROM obectifs ob WHERE ob.num_semaine = "+num_semaine_deb+" AND ob.annee = "+annee+"  AND ob.num_agent IN (SELECT num_agent from agents,equipes,crc where agents.num_equipe = equipes.code_equipe AND crc.num_crc = equipes.crc AND equipes.crc = '"+num_crc+"') GROUP BY ob.code_offre) ob ON ob.code_offre = of.code_offre";
	requete = requete+" LEFT OUTER JOIN (SELECT v.code_offre, count(v.num_vente) AS nbvente FROM ventes v WHERE v.num_agent IN (SELECT num_agent from agents,equipes,crc where agents.num_equipe = equipes.code_equipe AND crc.num_crc = equipes.crc AND equipes.crc = '"+num_crc+"') AND v.date_vente = '"+date_deb+"' GROUP BY v.code_offre) v ON v.code_offre = of.code_offre   WHERE of.code_canal = 'crc_qe' order by ordre";
	}
	if(requete!="")
	{
	this.document.res_obj.requete_XLS.value=requete;
	return true;
	}
	else
	{
	return false;
	}
 
	}	
}
 
</script>

Merci de votre aide !!!

15/06/2010, 17h12
E.Bzz

Citation:

Envoyé par ballantine's

je l'affecte à un input text, mais le texte n'est pas modifié

Donc tu ne l'affectes pas ;)

Créé un textarea juste pour ce test. Tu le supprimeras ensuite ...

PS : et active dans ton nav la "notification des erreurs de scripts" (ou équivalent suivant les nav), car tu as visiblement une erreur JS qui ne t'est pas signalée ...

A+
15/06/2010, 17h13
E.Bzz

Citation:

Envoyé par ballantine's

J'ai supprimé un if qui ne servait a rien

Entre autre ... ;)

:ccool: