SUPEE-9767 Patch / CE 1.9.3.3-1ページのチェックアウト-顧客登録の問題

新規の顧客登録チェックアウトが成功した後、 SUPEE-8788、SUPEE-9652、およびSUPEE-9767でパッチされ、新しい「チェックアウト時のフォームキー検証を有効にする」設定がオンになっているMagento 1.9.2.4のクリーンなバニラインストールでデフォルトのOne Page Checkoutの場合、新しい顧客は作成されず、顧客はログインしませんが、注文は順調に進みます。

[チェックアウト時のフォームキー検証を有効にする]設定をオフにすると、この機能が再び機能します。他の誰かがこの問題を抱えていましたか？どの配送/支払い方法が使用されるかは問題ではないようです。

それ以来、Magento 1.9.3.3の新しい、変更されていないインストールでこれを試しましたが、同じ問題があるようです。1ページのチェックアウトで新規顧客を登録する場合、「チェックアウト時のフォームキー検証を有効にする」設定がオンになっている限り、注文が正常に完了しても顧客は作成されません。

OK、私が思いついた本当のバグ修正です。

以下を置き換えて/skin/frontend/base/default/js/opcheckout.js、setMethod()メソッドを編集および編集します。

setMethod: function(){
    if ($('login:guest') && $('login:guest').checked) {
        this.method = 'guest';
        new Ajax.Request(
            this.saveMethodUrl,
            {method: 'post', onFailure: this.ajaxFailure.bind(this), parameters: {method:'guest'}}
        );
        Element.hide('register-customer-password');
        this.gotoSection('billing', true);
    }
    else if($('login:register') && ($('login:register').checked || $('login:register').type == 'hidden')) {
        this.method = 'register';
        new Ajax.Request(
            this.saveMethodUrl,
            {method: 'post', onFailure: this.ajaxFailure.bind(this), parameters: {method:'register'}}
        );
        Element.show('register-customer-password');
        this.gotoSection('billing', true);
    }
    else{
        alert(Translator.translate('Please choose to register or to checkout as a guest').stripTags());
        return false;
    }
    document.body.fire('login:setMethod', {method : this.method});
},

と：

setMethod: function(){
    var formKey = $('checkout-step-login').select('[name=form_key]')[0].value;
    if ($('login:guest') && $('login:guest').checked) {
        this.method = 'guest';
        new Ajax.Request(
            this.saveMethodUrl,
            {method: 'post', onFailure: this.ajaxFailure.bind(this), parameters: {method:'guest', form_key:formKey}}
        );
        Element.hide('register-customer-password');
        this.gotoSection('billing', true);
    }
    else if($('login:register') && ($('login:register').checked || $('login:register').type == 'hidden')) {
        this.method = 'register';
        new Ajax.Request(
            this.saveMethodUrl,
            {method: 'post', onFailure: this.ajaxFailure.bind(this), parameters: {method:'register', form_key:formKey}}
        );
        Element.show('register-customer-password');
        this.gotoSection('billing', true);
    }
    else{
        alert(Translator.translate('Please choose to register or to checkout as a guest').stripTags());
        return false;
    }
    document.body.fire('login:setMethod', {method : this.method});
},

パッチのv2を待っている間にそれを行います

登録を選択して続行すると、JSスクリプトはcheckout.setMethod()にあるを呼び出しますskin/frontend/base/default/js/opcheckout.js。そこからAJAX POSTリクエストを行うことがわかりますが、this.saveMethodUrl渡されるパラメータはのみですmethod。私たちが見ればMage_Checkout_OnepageController::saveMethodActionそのAJAX要求の対象である、我々はパッチが追加されたことを確認できます。

if ($this->isFormkeyValidationOnCheckoutEnabled() && !$this->_validateFormKey()) {
        return;
}

以来_validateFormKey探しform_keyリクエスト内のパラメータ、および以来、JSのsetMethodそれはAJAX要求をした場合、要求は、それが簡単に早く戻って何もしないです、これを送信しませんでした。setMethod関数に戻ると、戻り値で何も実行しようとしていないため、他に何も起こらず、JSが続行することがわかります。この時点でJSは設定されてthis.method = 'register'いますが、引用符は更新されていないためcheckout_method、デフォルトの「ゲスト」も更新されていません。

JSは顧客が選択した登録を知っているため、パスワードフィールドが表示されるため、その表面には登録しているように見えます。しかし、PHP側に関する限り、これはゲストチェックアウトであるため、チェックアウトの完了時に顧客を作成しません。

編集：最も簡単な修正は、saveMethodActionからこれら3行をコメントアウトすることです。より正確で複雑なソリューションはsetMethod、ページからform_keyを取得し、AJAXリクエストで送信することです。

完全なクレジットは、解決策のためにピーターに行きます！何を変えるべきか、ステップバイステップの指示を指摘したいと思います。

移動し 、アプリ/コード/コア/メイジ/チェックアウト/コントローラ/ OnepageController.php

見つける：

 public function saveMethodAction()
{
    if ($this->_expireAjax()) {
        return;
    }

    if ($this->isFormkeyValidationOnCheckoutEnabled() && !$this->_validateFormKey()) {
        return;
    }

/ * * /タグで行をコメント化します。

 public function saveMethodAction()
{
    if ($this->_expireAjax()) {
        return;
    }

    /*if ($this->isFormkeyValidationOnCheckoutEnabled() && !$this->_validateFormKey()) {
        return;
    }*/

開始する良い点：

セキュリティパッチSUPEE-9767-考えられる問題？

テンプレートファイルを更新する必要があります。このパッチのリリースからわずか数時間であり、現時点では公開されているものに対処する必要があることに注意してください。私は次の日に物事が明らかになると確信しています。

編集：ダウン投票していただきありがとうございます！このパッチをリリースしてから8時間以内に解決策を提供できないのが残念です。

Digital Pianismの@ Raphaelのパッチに感謝します。

便宜上、差分を作成して、パッチをすばやく適用できるようにしました。

 skin/frontend/base/default/js/opcheckout.js | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/skin/frontend/base/default/js/opcheckout.js b/skin/frontend/base/default/js/opcheckout.js
index 8935793af..9ccbe61a9 100644
--- a/skin/frontend/base/default/js/opcheckout.js
+++ b/skin/frontend/base/default/js/opcheckout.js
@@ -165,20 +165,21 @@ Checkout.prototype = {
     },

     setMethod: function(){
+        var formKey = $('checkout-step-login').select('[name=form_key]')[0].value;
         if ($('login:guest') && $('login:guest').checked) {
             this.method = 'guest';
-            var request = new Ajax.Request(
+            new Ajax.Request(
                 this.saveMethodUrl,
-                {method: 'post', onFailure: this.ajaxFailure.bind(this), parameters: {method:'guest'}}
+                {method: 'post', onFailure: this.ajaxFailure.bind(this), parameters: {method:'guest', form_key:formKey}}
             );
             Element.hide('register-customer-password');
             this.gotoSection('billing', true);
         }
         else if($('login:register') && ($('login:register').checked || $('login:register').type == 'hidden')) {
             this.method = 'register';
-            var request = new Ajax.Request(
+            new Ajax.Request(
                 this.saveMethodUrl,
-                {method: 'post', onFailure: this.ajaxFailure.bind(this), parameters: {method:'register'}}
+                {method: 'post', onFailure: this.ajaxFailure.bind(this), parameters: {method:'register', form_key:formKey}}
             );
             Element.show('register-customer-password');
             this.gotoSection('billing', true);

SUPEE-9767パッチのバージョン2は、Magento CE 1.9.3.4とともに本日より早くリリースされました。V2は、このチェックアウト登録バグを含む多くの問題を修正します。

最新バージョン（1.9.3.4）にアップグレードするか、V1を元に戻してからV2のパッチを適用できます。どちらのオプションでも問題は解決します。

V2の公式の変更は、Peter O'Callaghanが説明したものと事実上同じで、に追加された3行を削除しMage_Checkout_OnepageController::saveMethodActionます。