Skip to content

Commit

Permalink
Backing out fix for #5803 from 3b50eac.
Browse files Browse the repository at this point in the history
  • Loading branch information
jeresig committed Oct 26, 2010
1 parent 9b97599 commit de58db0
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 21 deletions.
6 changes: 0 additions & 6 deletions src/ajax.js
Expand Up @@ -208,12 +208,6 @@ jQuery.extend({
s.data = jQuery.param( s.data, s.traditional );
}

// If the jsonpCallback has been set, we can assume that dataType is jsonp
// Ticket #5803
if ( s.jsonpCallback ) {
s.dataType = "jsonp";
}

// Handle JSONP Parameter Callbacks
if ( s.dataType === "jsonp" ) {
if ( type === "GET" ) {
Expand Down
15 changes: 0 additions & 15 deletions test/unit/ajax.js
Expand Up @@ -825,21 +825,6 @@ test("jQuery.ajax() - JSONP, Local", function() {
plus();
}
});

// Supports Ticket #5803
jQuery.ajax({
url: "data/jsonp.php",
jsonpCallback: "jsonpResults",
success: function(data){
ok( data.data, "JSON results returned without dataType:jsonp when jsonpCallback is defined" );
plus();
},
error: function(data){
ok( false, "Ajax error JSON (GET, custom callback name)" );
plus();
}
});

});

test("JSONP - Custom JSONP Callback", function() {
Expand Down

0 comments on commit de58db0

Please sign in to comment.