SUPEE-9767 Patch / 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 продовжується. На даний момент СР встановив, this.method = 'register'але пропозиція не оновлювалася, тому checkout_methodза замовчуванням "гість".

Оскільки JS знає обраний клієнтом реєстр, він відображає поля паролів, тож на його обличчі це виглядає так, що ви реєструєтесь. Але що стосується сторони PHP, це замовлення гостей, тому воно не створює клієнта, коли оформлення замовлення завершено.

Редагувати: найпростіший виправлення - прокоментувати ці три рядки з SaveMethodAction. Більш правильне / складне рішення полягає в тому, що setMethodслід захопити form_key зі сторінки та надіслати її разом із запитом AJAX.

Повні кредити йдуть на Пітера за рішенням! Я хотів би вказати покрокову інструкцію, що потрібно змінити.

Йти до додаток / код / ​​core / Mage / Checkout / контролери / 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.