Skip to content
Permalink
Browse files

Revert "Fix #11586. Ajax DELETE ain't got no body, so data goes to it…

…s URL."

This reverts commit c01e022.

We actually changed DELETE to accept a body on purpose, see http://bugs.jquery.com/ticket/7285 .
  • Loading branch information
dmethvin committed Aug 24, 2012
1 parent a5be986 commit 05337e78fa68aac3a3d703d7cc59f145f13ea779
Showing with 1 addition and 16 deletions.
  1. +1 −1 src/ajax.js
  2. +0 −15 test/unit/ajax.js
@@ -7,7 +7,7 @@ var // Document location
rheaders = /^(.*?):[ \t]*([^\r\n]*)\r?$/mg, // IE leaves an \r character at EOL
// #7653, #8125, #8152: local protocol detection
rlocalProtocol = /^(?:about|app|app\-storage|.+\-extension|file|res|widget):$/,
rnoContent = /^(?:GET|HEAD|DELETE)$/,
rnoContent = /^(?:GET|HEAD)$/,
rprotocol = /^\/\//,
rquery = /\?/,
rscript = /<script\b[^<]*(?:(?!<\/script>)<[^<]*)*<\/script>/gi,
@@ -840,21 +840,6 @@ test("jQuery.ajax - HEAD requests", function() {

});


test("jQuery.ajax - DELETE requests", function() {
expect(1);

jQuery.ajax({
url: "devo.html",
type: "DELETE",
data: { "whip": "it", "qty": "good" },
beforeSend: function( _ , s ) {
equal( s.url, "devo.html?whip=it&qty=good", "Delete requests encode data in URL" );
return false;
}
});
});

test("jQuery.ajax - beforeSend", function() {
expect(1);
stop();

0 comments on commit 05337e7

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