Skip to content

Commit de58db0

Browse files
committed
Backing out fix for #5803 from 3b50eac.
1 parent 9b97599 commit de58db0

File tree

2 files changed

+0
-21
lines changed

2 files changed

+0
-21
lines changed

src/ajax.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -208,12 +208,6 @@ jQuery.extend({
208208
s.data = jQuery.param( s.data, s.traditional );
209209
}
210210

211-
// If the jsonpCallback has been set, we can assume that dataType is jsonp
212-
// Ticket #5803
213-
if ( s.jsonpCallback ) {
214-
s.dataType = "jsonp";
215-
}
216-
217211
// Handle JSONP Parameter Callbacks
218212
if ( s.dataType === "jsonp" ) {
219213
if ( type === "GET" ) {

test/unit/ajax.js

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -825,21 +825,6 @@ test("jQuery.ajax() - JSONP, Local", function() {
825825
plus();
826826
}
827827
});
828-
829-
// Supports Ticket #5803
830-
jQuery.ajax({
831-
url: "data/jsonp.php",
832-
jsonpCallback: "jsonpResults",
833-
success: function(data){
834-
ok( data.data, "JSON results returned without dataType:jsonp when jsonpCallback is defined" );
835-
plus();
836-
},
837-
error: function(data){
838-
ok( false, "Ajax error JSON (GET, custom callback name)" );
839-
plus();
840-
}
841-
});
842-
843828
});
844829

845830
test("JSONP - Custom JSONP Callback", function() {

0 commit comments

Comments
 (0)