Патч SUPEE-9767 / CE 1.9.3.3 - оформление заказа на одной странице - проблема с регистрацией клиента

На чистой, ванильной установке Magento 1.9.2.4, исправленной с помощью SUPEE-8788, SUPEE-9652 и SUPEE-9767, и с включенной новой настройкой «Включить проверку ключа формы при проверке» , после успешной регистрации новой регистрации клиента на одностраничный заказ по умолчанию, новый клиент не создается, и клиент не вошел в систему, хотя заказ проходит нормально.

Отключение параметра «Включить проверку ключа формы при извлечении» делает эту работу снова. У кого-нибудь еще была эта проблема? Кажется, не имеет значения, какие способы доставки / оплаты используются.

С тех пор я попробовал это с новой, неизмененной установкой Magento 1.9.3.3, и, похоже, та же проблема. При регистрации нового клиента с помощью одностраничной проверки ни один клиент не создается, даже если заказ проходит через штраф, если включен параметр «Включить проверку ключа формы при оформлении заказа».

Хорошо, вот настоящее исправление ошибки, которое я придумала.

Отредактируйте /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параметр в запросе, и поскольку setMethodзапрос JS не отправил его, когда он сделал запрос AJAX, он просто возвращается рано и ничего не делает. Вернемся к setMethodфункции, и мы видим, что, поскольку она ничего не пытается сделать с возвращаемым значением, ничего не происходит, и JS продолжается. На данный момент JS настроен, this.method = 'register'но цитата не была обновлена, так checkout_methodчто по умолчанию это «гость».

Поскольку JS знает выбранный пользователем регистр, он отображает поля пароля, поэтому на первый взгляд он выглядит так, как будто вы регистрируетесь. Но что касается стороны PHP, это проверка гостя, поэтому он не создает клиента, когда проверка завершена.

Редактировать: самое простое исправление - закомментировать эти три строки из saveMethodAction. Более правильное / сложное решение заключается в том, что setMethodнужно взять ключ form_key со страницы и отправить его с запросом AJAX.

Полные кредиты идут в Питер для решения! Я хотел бы указать пошаговую инструкцию, что изменить.

Перейдите в приложение / code / core / Mage / Checkout / controllers / 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.

Для удобства я создал diff, чтобы вы могли быстро применить патч.

 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);

Версия 2 патча SUPEE-9767 была выпущена ранее сегодня вместе с Magento CE 1.9.3.4 . V2 исправляет ряд проблем, в том числе эту ошибку регистрации оформления заказа.

Вы можете обновить до последней версии (1.9.3.4) или вернуть V1, а затем применить V2 патча. Любой вариант решит проблему.

Официальные изменения в V2 практически такие же, как описал Питер О'Каллаган, удалив три добавленные строки Mage_Checkout_OnepageController::saveMethodAction.