Skip to content
Permalink
Browse files

DRY out $.get and $.post. Fixes #7847. Thanks to cowboy for the patch.

  • Loading branch information...
csnover committed Dec 27, 2010
2 parents 52b1709 + 78a6f5b commit 4443371dbaf751ce0d0d92e40f924e41fbd7b54c
Showing with 7 additions and 22 deletions.
  1. +7 −22 src/ajax.js
@@ -113,9 +113,8 @@ jQuery.each( "ajaxStart ajaxStop ajaxComplete ajaxError ajaxSuccess ajaxSend".sp
};
});

jQuery.extend({

get: function( url, data, callback, type ) {
jQuery.each( [ "get", "post" ], function( i, method ) {
jQuery[ method ] = function( url, data, callback, type ) {
// shift arguments if data argument was omited
if ( jQuery.isFunction( data ) ) {
type = type || callback;
@@ -124,13 +123,16 @@ jQuery.extend({
}

return jQuery.ajax({
type: "GET",
type: method,
url: url,
data: data,
success: callback,
dataType: type
});
},
};
});

jQuery.extend({

getScript: function( url, callback ) {
return jQuery.get(url, null, callback, "script");
@@ -140,23 +142,6 @@ jQuery.extend({
return jQuery.get(url, data, callback, "json");
},

post: function( url, data, callback, type ) {
// shift arguments if data argument was omited
if ( jQuery.isFunction( data ) ) {
type = type || callback;
callback = data;
data = {};
}

return jQuery.ajax({
type: "POST",
url: url,
data: data,
success: callback,
dataType: type
});
},

ajaxSetup: function( settings ) {
jQuery.extend( jQuery.ajaxSettings, settings );
},

2 comments on commit 4443371

@fmeyer

This comment has been minimized.

Copy link

fmeyer replied Dec 27, 2010

really cool

@pokonski

This comment has been minimized.

Copy link

pokonski replied Dec 28, 2010

Clever!

Please sign in to comment.
You can’t perform that action at this time.