Skip to content

Commit

Permalink
Revised conversion logic to handle converter equal to true properly.
Browse files Browse the repository at this point in the history
  • Loading branch information
jaubourg committed Dec 29, 2010
1 parent 54d965f commit 7490eb4
Showing 1 changed file with 19 additions and 14 deletions.
33 changes: 19 additions & 14 deletions src/ajax.js
Expand Up @@ -393,9 +393,9 @@ jQuery.extend({
current,
prev,
checker,
conv,
conv1,
conv2,
oneConv,
convertion,
dataTypes = s.dataTypes,
converters = s.converters,
Expand All @@ -419,23 +419,28 @@ jQuery.extend({

if ( prev !== "*" && current !== "*" && prev !== current ) {

oneConv = conv1 =
converters[ ( conversion = prev + " " + current ) ] ||
conv = converters[ ( conversion = prev + " " + current ) ] ||
converters[ "* " + current ];

if ( oneConv !== true ) {

if ( ! oneConv && prev !== "text" && current !== "text" ) {
conv1 = converters[ prev + " text" ] || converters[ "* text" ];
conv2 = converters[ "text " + current ];
}

if ( oneConv || conv1 && conv2 ) {
response = oneConv ? conv1( response ) : conv2( conv1( response ) );
} else {
throw "no " + conversion;
conv1 = conv2 = 0;

if ( ! conv && prev !== "text" && current !== "text" ) {
conv1 = converters[ prev + " text" ] || converters[ "* text" ];
conv2 = converters[ "text " + current ];
if ( conv1 === true ) {
conv = conv2;
} else if ( conv2 === true ) {
conv = conv1;
}
}

if ( ! ( conv || conv1 && conv2 ) ) {
throw conversion;
}

if ( conv !== true ) {
response = conv ? conv( response ) : conv2( conv1( response ) );
}
}
} else if ( s.dataFilter ) {

Expand Down

0 comments on commit 7490eb4

Please sign in to comment.