deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Made some tweaks to the different AJAX callbacks.

Browse source
This commit is contained in:
John Resig 2006-06-22 22:25:03 +00:00
parent 0fd3648d45
commit 2d9979c73f
1 changed files with 13 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
ajax/ajax.js
View file

@ -83,7 +83,7 @@ if ( jQuery.browser == "msie" )
// Attach a bunch of functions for handling common AJAX events
(function(){
var e = "ajaxStart.ajaxComplete.ajaxError.ajaxSuccess".split(',');
var e = "ajaxStart,ajaxStop,ajaxComplete,ajaxError,ajaxSuccess".split(',');
for ( var i = 0; i < e.length; i++ ){ (function(){
var o = e[i];
@ -105,6 +105,10 @@ jQuery.ajax = function( type, url, data, ret ) {
url = type.url;
type = type.type;
}
// Watch for a new set of requests
if ( ! jQuery.ajax.active++ )
jQuery.event.trigger( "ajaxStart" );
// Create the request object
var xml = new XMLHttpRequest();
@ -125,23 +129,8 @@ jQuery.ajax = function( type, url, data, ret ) {
// Wait for a response to come back
xml.onreadystatechange = function(){
// Socket is openend
if ( xml.readyState == 1 ) {
// Increase counter
jQuery.ajax.active++;
// Show 'loader'
jQuery.event.trigger( "ajaxStart" );
}
// Socket is closed and data is available
// The transfer is complete and the data is available
if ( xml.readyState == 4 ) {
// Hide loader if needed
if ( ! --jQuery.ajax.active ) {
jQuery.event.trigger( "ajaxComplete" );
jQuery.ajax.active = 0
}
// Make sure that the request was successful
if ( jQuery.httpSuccess( xml ) ) {
@ -159,6 +148,13 @@ jQuery.ajax = function( type, url, data, ret ) {
// Fire the global callback
jQuery.event.trigger( "ajaxError" );
}
// The request was completed
jQuery.event.trigger( "ajaxComplete" );
// Handle the global AJAX counter
if ( ! --jQuery.ajax.active )
jQuery.event.trigger( "ajaxStop" );
// Process result
if ( ret ) ret(xml);