Lien avec javascript onClick, marche qu'au deuxième click

WRInaute accro
Bonjour

J'ai sur un site des liens avec des appels de fonctions javascript qui ne sont lancées qu'au deuxième click, et ça a le don de m'énerver. :evil: :evil:

Le code du lien :

Code:
<a onClick='SubmitForm(2)' href="#" >LIEN</a>

ça me le fait sur tous les liens contenant cette fonction ou une autre. Et ça m'énerve :evil: . Je ne trouve rien sur le web, même pas un début d'explication ou de solution.)

Merci pour vos réponses ...
 
WRInaute impliqué
Jamais vu cela pour ma part, ton code est ok le mieux je pense est de partir d'une page vierge et de la compléter jusqu'à ce que tu trouves ce qui interfère.
 
WRInaute accro
AAaaargh
c'est une page d'un site dont je m'occupe, crois moi c'est difficilement réalisable dans le peu de temps qui m'est imparti !

:)
 
WRInaute discret
t'es marrant, là tu nous montre une boite fermée en disant qu'elle ne marche pas... donne le code de la fonction JS appelée et après éventuellement tu pourras avoir des réponses intéressantes, là ya rien a en tirer de ton code HTML :D
 
WRInaute accro
Ok si tu y tiens .. j'en suis pas l'auteur, donc je patauge un peu :)

Edit : je la remets, j'ai essayé de la nettooyer en l'indentant un peu mieux et en virant les commentaires qui me semblaient superflus mais bon ça a encore moins bien marché ensuite. je la remets brute de décoffrage, telle que je l'ai reçue :

Code:
function SubmitForm(strType){



	// if(Signs[document.resForm.destiagent.value] != Signs[document.resForm.sourceagent.value])

	if(document.resForm.destiagent.value != document.resForm.sourceagent.value)

	{

		// alert('Vous ne pouvez pas choisir des agences de départ et d\'arrivée de 2 enseignes différentes');

		//alert('Vous ne pouvez pas choisir une agence de retour différente de celle de départ');



		document.resForm.destiagent.value = document.resForm.sourceagent.value;

	}

	else

	{



 frmObj = document.resForm;

 var elemSelected;

 elemSelected = false;





if (frmObj.catid.selectedIndex>=0) elemSelected=true;



var txtstday = frmObj.stday[frmObj.stday.selectedIndex].value;

var txtstmonth = frmObj.stmonth[frmObj.stmonth.selectedIndex].value;

var txtstyear = frmObj.styear[frmObj.styear.selectedIndex].value;



var txtendday = frmObj.endday[frmObj.endday.selectedIndex].value;

var txtendmonth = frmObj.endmonth[frmObj.endmonth.selectedIndex].value;

var txtendyear = frmObj.endyear[frmObj.endyear.selectedIndex].value;

var today =new Date();

<?php
	/* Pour St-Martin, 1 jour entre la reservation et la prise de vehicule */
	if ($intDestination == $CONFIG['dest_st_martin']) {
		echo 'var minDate = new Date(today.getTime() + 1*24*60*60*1000);';
	}
	else {
		echo 'var minDate = new Date(today.getTime() + 2*24*60*60*1000);';
	}
?>

var stdate =new Date(txtstyear,txtstmonth-1,txtstday);

var enddate =new Date(txtendyear,txtendmonth-1,txtendday);

var txtstarthour = frmObj.sthour[frmObj.sthour.selectedIndex].value;
var txtstartmins = frmObj.stminute[frmObj.stminute.selectedIndex].value;
var txtendhour = frmObj.endhour[frmObj.endhour.selectedIndex].value;
var txtendmins = frmObj.endminute[frmObj.endminute.selectedIndex].value;
var resstartdate =new Date(txtstyear,txtstmonth-1,txtstday,txtstarthour,txtstartmins);
var resenddate =new Date(txtendyear,txtendmonth-1,txtendday,txtendhour,txtendmins);


 var frmObj = document.resForm;
if(!isValidDate(txtstday,txtstmonth,txtstyear)){

     alert("<?php print($LABLES["p6_lbl34"]);?>");

     frmObj.stday.focus();

}else if(!isValidDate(txtendday,txtendmonth,txtendyear)){

     alert("<?php print($LABLES["p6_lbl34"]);?>");

     frmObj.endday.focus();

}else if(stdate > enddate){

     alert("La date de retour doit être postérieure à la date de location");

     frmObj.endday.focus();

}else if(stdate < today){

     alert("Vous ne pouvez pas louer de véhicule pour aujourd'hui, ni même pour des dates antérieures.");

     frmObj.stday.focus();

}else if(stdate < minDate){

	alert("<?php
				if ($intDestination == $CONFIG['dest_st_martin']) {
				  	print("Il est impossible de réserver une voiture moins d\'un jour à l\'avance");
				}
				else {
                    print("Il est impossible de réserver une voiture moins de 2 jours à l\'avance");
				}
            ?>");

	frmObj.stday.focus();

}else if(enddate < today){

     alert("<?php print($LABLES["p5_lbl15"]);?>");

     frmObj.endday.focus();
}<?php // La reservation peut durer moins de trois jours
/*
 // Si le client est pas un pro, la reservation peut durer moins de 3 jours

			  	if ($_SESSION['usertype'] != $CONFIG["Type_AGENT"]) { ?>

	else if(daysElapsed(resenddate,resstartdate) < 3){

	alert("La durée de la de la réservation ne peut pas être inférieure à 3 jours.");
     frmObj.endday.focus();

}<?php}  */?>else if(!IsValidTime(frmObj.sthour.value,frmObj.stminute.value)) {

			frmObj.sthour.focus();

}else if(!IsValidTime(frmObj.endhour.value,frmObj.endminute.value)){

		frmObj.endhour.focus();

}else if(!elemSelected){

    alert("<?php print($LABLES["p2_lbl1"]);?>");

}else{



     if (strType == 1)

	 {

		frmObj.target='_self';

		frmObj.action='Detail.php';

		frmObj.submit();

	}

	else if(strType == 2)

	{

//



window.open("","printdetails","toolbar=no,status=no,scrollbars=yes,location=no,menubar=no,directories=no,resizable=yes,width=550,height=250,left=0,top=0")

		frmObj.action='printResDetail.php';

		frmObj.target='printdetails'

		frmObj.submit();



    }

	else if (strType == 3)

	{

		frmObj.target='_self';

		frmObj.action='infoline.php?a=<? echo $intAmount; ?>&d=<? echo $strStDate;?>&e=<? echo $strEndDate;?>&dest=<? echo $intDestination;?>&p=<? echo $provenance; ?>';

		frmObj.submit();

	}
	
	else

	{

		frmObj.target='_self';

		frmObj.action='ReservationForm.php#supp';

		frmObj.submit();

    }

 }

}

}
 
WRInaute discret
Bah c'est simple, le premier clic va rentrer dans ce if :

if(document.resForm.destiagent.value != document.resForm.sourceagent.value)
{
document.resForm.destiagent.value = document.resForm.sourceagent.value;
}

Et lors du deuxieme clic, le test du if sera false, donc ça partira dans le else, et ça te fera le submit().

Si tu veux fixer ça a l'arrache, modifie :


if(document.resForm.destiagent.value != document.resForm.sourceagent.value) {
code1
} else {
code2
}

Par :

code1
code2
 
Nouveau WRInaute
Bonjour je sais ca fait longtemps que le sujet est clos mais jai le meme probleme en javascript, c'est a dire que je suis oblige de cliquer 2 fois pour executer la fonction en javascript je vous la met:
<script type="text/javascript">
var scrolling = function(){
var speed = 1000;
jQuery('a[href^="#"]').bind('click',function(){
var id = jQuery(this).attr('href');
if(id == '#')
goTo('body');
else
goTo(id);
return(false);
});
function goTo(ancre){jQuery('html,body').animate({scrollTop:jQuery(ancre).offset().top},speed,'swing',function(){
if(ancre != 'body')
window.location.hash = ancre;
else
window.location.hash = '#';
jQuery(ancre).attr('tabindex','-1');
jQuery(ancre).focus();
jQuery(ancre).removeAttr('tabindex');
});
}
};
</script>



Merci de m'aider car ca m enerve
 
Discussions similaires
Haut