Discussion :

[Arnaud F.]

Bonjour,

je souhaiterais toujours exécuter une fonction avant de rentrer dans les callbacks "success" et "error" et de ceux de manière systématique et automatisée, or je n'ai trouvé aucun moyen de le faire.

Mon exemple:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
// Eviter ceci :
$.ajax({
    url: ".",
    success: function() { success_fn(); alert("success") },
    error: function() { error_fn(); alert("error") }
});
 
// Mais avoir ça :
$.ajax({
    url: ".",
    success: function() { alert("success") },
    error: function() { alert("error") }
});

J'ai pensé passé par :

1. dataFilter: problème, on ne dispose pas de la requête XHR ni du statut de cette dernière (échec, succès)
2. $.ajaxSuccess: Cette fonction s'exécute après success (et non avant)
3. $.ajaxError: Cette fonction s'exécute après error (et non avant)

Une idée?

[marcha]

Salut,

Je vois pas de solution propre à ton problème alors je te propose
une piste (pas sur que ça puisse fonctionner)

J'ai regardé un peu dans le code source de JQuery

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
handleSuccess: function( s, xhr, status, data ) {
	// If a local callback was specified, fire it and pass it the data
	if ( s.success ) {
		s.success.call( s.context, data, status, xhr );
	}
	// Fire the global callback
	if ( s.global ) {
		jQuery.triggerGlobal( s, "ajaxSuccess", [xhr, s] );
	}
},

On voit que ajaxSuccess est bien appelé après success.

Maintenant on pourrait imaginer que tu fasse ton appel ainsi:

Code :

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

1
2
3
4
5
$.ajax({
    url: ".",
    my_success: function() { alert("success") },
    my_error: function() { alert("error") }
});

si success et error sont pas défini JQuery va pas les appeler. Tu
register ajaxSuccess sur un élément quelconque

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
$('#test').ajaxSuccess(function(e, xhr, s) {
 
  // ton traitement généraliste ici
 
  if ( s.my_success ) {
        var data = ... // récupérer data depuis xhr ?
        var status = ... // récupérer status depuis xhr ?
	s.my_success.call( s.context, data, status, xhr );
  }
}

Dis nous si ça peut donner qqch :-)

[Arnaud F.]

Première option que j'avais envisagée, modifier jQuery de la sorte (même si c'est bof ) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
var aSuccess =  jQuery.Event("ajaxSuccess");
 
// Fire the global callback
if ( s.global ) {
	trigger( aSuccess, [xhr, s] );
}
 
// If a local callback was specified, fire it and pass it the data
if( (s.success && !s.global) || (s.success && s.global && aSuccess.result !== false)) {
	s.success.call( callbackContext, data, status, xhr );		    
}

Testé et approuvé, ça fonctionne bien


Ta méthode? Idem, ça fonctionne bien

Je trouve ça dommage que ça soit pas natif

[Arnaud F.]

Re,

pour que ça soit totalement transparent côté utilisateur, j'ai modifié ton code de la sorte :

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
 
$("body").ajaxSend(function(event, XMLHttpRequest, s) {
     s._success = s.success;
     s.success = $.noop;
     s._error = s.error;
     s.error = $.noop;
     s._complete = s.complete;
     s.complete = $.noop;
});
 
$("body").ajaxSuccess(function(event, XMLHttpRequest, s) {
    // Mon code generique.
    if(s._success) {
        s._success.call(s.context || s, s.data, s.status, XMLHttpRequest)
    }
});

encore