"unterminated string literal" oui mais je ne vois rien

**lodan** · 16/02/2008, 12h57

Bonjour,

Damned, je suis fait...j'utilise pourtant souvent ce genre d'astuce, mais là je ne vois rien.

Ma console firebug me met : "unterminated string literal"

sur le code suivant :

Code :

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

1
2
3
onclick="document.forms['recherche'].elements['date_debut'].value = prompt('Saisissez votre date de début :', 'JJ/MM/AAAA')
document.forms['recherche'].elements['date_fin'].value = prompt('Saisissez votre date de fin :', 'JJ/MM/AAAA')
javascript:window.location = 'liste_sql.php?send_sql_id=22&send_date_debut='+document.getElementById('date_debut').value+'&send_date_fin='+document.getElementById('date_fin').value+';"

Je ne vois pas mon erreur de syntaxe.

Dites docteur c'est grave tellement c'est évident ?

Merci d'avance

**sachav** · 16/02/2008, 13h06

Ouhla...
A votre place je ne ferai pas ça...
Je créerai plutôt une fonction qui regroupe tout ça...

**sachav** · 16/02/2008, 13h10

Ah oui voila...
D'où sort le +' à la fin ?

**Auteur** · 16/02/2008, 13h15

d'accord avec sachav, le code est un peu indigeste. Les instructions ne sont même pas séparées par des points-virgules.

Il faut créer une fonction qui sera appelée lors de l'événement.

Quant aux signes + je me demande si lodan ne voulait pas en fait concaténer des châines de caractères.

**le_chomeur** · 16/02/2008, 13h34

de plus ton érreur proviens de la a vu de nez :
+';" ( tout a la fin de ta chaine ) enlève le : +'

**lodan** · 16/02/2008, 18h10

Je me suis senti très coupable de rendre malade d'une indigestion certains d'entre vous. Aussi ai-je allégé le code comme ceci :

Code :

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

1
2
3
4
5
6
function demandPeriode(sql_id)
{
	var periode_debut = prompt("Saisissez votre date de début :", "JJ/MM/AAAA");
	var periode_fin = prompt("Saisissez votre date de fin :", "JJ/MM/AAAA");
javascript:window.location = "liste_sql.php?send_sql_id="+sql_id+"&send_date_debut="+periode_debut+"&send_date_fin="+periode_fin;
}

Cela fonctionne.

Merci beaucoup.

**FremyCompany** · 16/02/2008, 19h02

Attention
- le javascript: est tout à fait inutile et est compris comme un numéro de ligne par le compilateur (goto javascript enverrait là)
- prompt n'est plus vraiment supporté par IE 7 (à cause de la sécurité, tout appel à prompt est interdit sauf si l'utilisteur les actives)

**Auteur** · 16/02/2008, 19h26

Fremy : par quoi remplacerais-tu le prompt s'il n'est plus supporté par IE 7 ?

**FremyCompany** · 16/02/2008, 19h46

Excellente question.
J'ai déjà envisagé showModalDialog mais je n'ai jamais mis en oeuvre.

L'autre option c'est l'asynchrone avec une DIV qui s'ouvre sur la page.