Ajax: $.post and $.get can now take an options object

Fixes gh-1986
Closes gh-1995

diff --git a/src/ajax.js b/src/ajax.js
index dafbc6082b386469a1c474c093f8b07f6537615a..a9faee78d212489724450893c931e1915cea7051 100644 (file)
--- a/src/ajax.js
+++ b/src/ajax.js
@@ -796,13 +796,14 @@ jQuery.each( [ "get", "post" ], function( i, method ) {
                        data = undefined;
                }
 
-               return jQuery.ajax({
+               // The url can be an options object (which then must have .url)
+               return jQuery.ajax( jQuery.extend({
                        url: url,
                        type: method,
                        dataType: type,
                        data: data,
                        success: callback
-               });
+               }, jQuery.isPlainObject( url ) && url ) );
        };
 });
 

diff --git a/test/unit/ajax.js b/test/unit/ajax.js
index b41654db91c160e4e87afc571cffd3ed9c5f9c57..29a4dfb46e35c73b37eebe71a2e2c80b37bee791 100644 (file)
--- a/test/unit/ajax.js
+++ b/test/unit/ajax.js
@@ -2034,6 +2034,26 @@ module( "ajax", {
                });
        });
 
+       asyncTest( "jQuery[get|post]( options ) - simple with xml", 2, function() {
+               jQuery.when.apply( jQuery,
+                       jQuery.map( [ "get", "post" ] , function( method ) {
+                               return jQuery[ method ]({
+                                       url: url( "data/name.php" ),
+                                       data: {
+                                               "xml": "5-2"
+                                       },
+                                       success: function( xml ) {
+                                               jQuery( "math", xml ).each(function() {
+                                                       strictEqual( jQuery( "result", this ).text(), "3", "Check for XML" );
+                                               });
+                                       }
+                               });
+                       })
+               ).always(function() {
+                       start();
+               });
+       });
+
 //----------- jQuery.active
 
        test( "jQuery.active", 1, function() {