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

[nickargall]

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 :

<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 ...

 

[f_trt]

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.

 

[nickargall]

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 !

 

[FlorentP]

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

 

[nickargall]

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 :

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();

    }

 }

}

}

 

[FlorentP]

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

 

[nickargall]

Ok merci j'ai plus le blocage du double click

(ps : qu'est ce que j'aime pas le javascript ! )

Mille mercis
++

 

[geekman01]

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