Discussion :

[M4kn4sh]

Bonjour,

Je ne comprends ni pourquoi ni comment, mais j'ai une page phtml qui fait appel à une page php par le biais de $.ajax de jQuery et parfois, mon navigateur envoie 2 requêtes, ou plus, d'un coup. Si je re-clique, ça renvoie à chaque fois les anciennes requêtes j'imagine.
Pourtant, en cherchant sur internet, j'avais trouvé unbind, mais ça ne change rien... En ajoutant async : false, le bug se produit moins souvent.

Voici la requête :

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
$('#transfer').click(function() {
	$.ajax({
		url: '/index/update',
		async: false,
		data: {
			tid: eventId,
			wid: '<?php echo $this->wid; ?>',
			statut: 4
		},
		type: "POST",
		success: function(responseText) {
			// on change la couleur
			if(event.color == "green") {
				event.color = "darkblue";
			} else {
				event.color = "green";
			}
			$('#calendar').fullCalendar('updateEvent', event);
			$('#transfer').unbind('click');
		},
		error: function(jqXHR, textStatus, errorThrown) {
			alert('Impossible de marquer la tâche comme "transferée"');
			$('#transfer').unbind('click');
		}
	})
});

Je précise tout de même que je ne suis pas expert en jQuery...

Merci pour vos réponses.

[SpaceFrog]

Une des techniques les plus radicales pour prévenir de multiples click c'est de faire disparaitre le bouton le temps de l'opération

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
$('#transfer').click(function() {
	me=$(this);
	me.hide();
	$.ajax({
		url: '/index/update',
		async: false,
		data: {
			tid: eventId,
			wid: '<?php echo $this->wid; ?>',
			statut: 4
		},
		type: "POST",
		success: function(responseText) {
			// on change la couleur
			if(event.color == "green") {
				event.color = "darkblue";
			} else {
				event.color = "green";
			}
			$('#calendar').fullCalendar('updateEvent', event);
			me.unbind('click');
			me.show();	
		},
		error: function(jqXHR, textStatus, errorThrown) {
			alert('Impossible de marquer la tâche comme "transferée"');
			me.unbind('click');
			me.show();
		}
	})
});

[M4kn4sh]

Merci pour cette réponse, mais en fait, ce n'est pas un problème de "double clic" étant donné que je ne clique qu'une seule et unique fois.
De toute façon, avec l'unbind, ça empêche toute autre action.

[SpaceFrog]

il n'y a rien dans le bout de code montré qui declenche plusieurs envois...

Code :

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

$('#transfer').click

on peut vois le html du id transfer ?

[M4kn4sh]

Bah pareil, je ne comprends pas non plus...
Pour le code de transfer, c'est un simple menu qui s'affiche lorsque l'on clique sur une div, qui fait ensuite apparaître un menu avec une balise p contenant le transfer et quand on clique, c'est censé faire le traitement ajax...

C'est vraiment tout bête, rien de bien méchant, pour les autres ça fonctionne, mais pas pour transfer, c'est assez aléatoire en fait : tantôt ça marche, tantôt ça ne marche pas.

Code html :

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

1
2
3
4
5
6
7
<div id="menuArea" style="position:absolute;top:0px;left:0px;Z-index:10;visibility:collapse;">
	<ul id="menu" class="ulMenuArea">
	    <li id="1"><p id="consult">Consulter la fiche du client</p></li>
	    <li id="2"><p id="transfer">Transférer au groupe</p></li>
	    <li id="3"><p id="markAsFinished">Marquer comme terminé</p></li>
	</ul>
</div>

Le fait de mettre async: false permet de beaucoup moins le faire, et quand ça se produit, ça m'exécute d'abord une fois l'action et, quand elle est finie, ça relance une action... C'est à s'arracher les cheveux car pour moi le code est correct

[ABCIWEB]

Avec jquery il y a bien le .one qui permet de ne déclencher l'action qu'une seule fois. Mais il doit y avoir un pb ailleurs dans ton code et l'emploi de async: false est déconseillé la plupart du temps (sauf à en avoir réellement besoin mais pas pour régler ce genre de problème).
Faut te mettre en mode debug, cad revoir et tester ton code bloc par bloc pour voir quand ça cloche.