6

このコード スニペットは、1.7.2 で成功/エラー コールバックと promise スタイル コールバックの両方で機能しました。1.8.2 では、成功/エラーのコールバックは引き続き機能しますが、promise は機能しません。私の推測では、return dfd.promise(jqXHR);ラインが問題であるということですが、確かではありません。

$.ajaxPrefilter(function (options, originalOptions, jqXHR) {

    // Don't infinitely recurse
    originalOptions._retry = isNaN(originalOptions._retry)
        ? Common.auth.maxExpiredAuthorizationRetries
        : originalOptions._retry - 1;

    // set up to date authorization header with every request
    jqXHR.setRequestHeader("Authorization", Common.auth.getAuthorizationHeader());

    // save the original error callback for later
    if (originalOptions.error)
        originalOptions._error = originalOptions.error;

    // overwrite *current request* error callback
    options.error = $.noop();

    // setup our own deferred object to also support promises that are only invoked
    // once all of the retry attempts have been exhausted
    var dfd = $.Deferred();
    jqXHR.done(dfd.resolve);

    // if the request fails, do something else yet still resolve
    jqXHR.fail(function () {
        var args = Array.prototype.slice.call(arguments);

        if (jqXHR.status === 401 && originalOptions._retry > 0) {

            // refresh the oauth credentials for the next attempt(s)
            // (will be stored and returned by Common.auth.getAuthorizationHeader())
            Common.auth.handleUnauthorized();

            // retry with our modified
            $.ajax(originalOptions).then(dfd.resolve, dfd.reject);

        } else {
            // add our _error callback to our promise object
            if (originalOptions._error)
                dfd.fail(originalOptions._error);
            dfd.rejectWith(jqXHR, args);
        }
    });

    // NOW override the jqXHR's promise functions with our deferred
    return dfd.promise(jqXHR);
});

更新:失敗した ajax リクエストは次のとおりです。

$.ajax({
        url: someFunctionToGetUrl(),
        // works
        //success: callback,
        //error: ajaxErrorHandler
    }).then(
        [callback],
        [errorback, ajaxErrorHandler]
    );
};
4

1 に答える 1

7

編集:これはドキュメントのバグですが、動作は設計によるものです。deferred.thenのように動作deferred.pipeし、配列を渡すことを許可しないように API が変更されましたが、ドキュメントはそれを反映するように更新されていません。

関連するバグ:

以下の元の回答の最後に記載されている回避策は引き続き適用されます。

元の答え:

私にはjQueryのバグのように見えます。最初の引数として単一の関数参照を渡すと機能しますが、関数の配列を渡すと機能しません。

http://jsfiddle.net/tunDH/

ただし、ドキュメントには、関数の配列で問題ないと書かれています。

doneCallbacks   Deferred が解決されたときに呼び出される関数または関数の配列。

そして、あなたは正しいです。jQuery 1.7 で動作します: http://jsfiddle.net/tunDH/1/

回避策は、配列内ではなく、単一の関数内にすべての関数呼び出しをラップすることです。

$.ajax({ 
    url: someFunctionToGetUrl(), 
    // works 
    //success: callback, 
    //error: ajaxErrorHandler 
}).then( 
    function(){
        callback1.apply(this, arguments);
        callback2.apply(this, arguments);
    }, 
    [errorback, ajaxErrorHandler] 
);

http://jsfiddle.net/tunDH/2/

おそらくエラー コールバックでも同じことを行う必要がありますが、私はそれをテストしていません。

于 2012-10-19T19:40:06.227 に答える