SUPEE-9767 Patch / CE 1.9.3.3 - Zamówienie jednej strony - Problem z rejestracją klienta

W czystej, waniliowej instalacji Magento 1.9.2.4, z poprawkami SUPEE-8788, SUPEE-9652 i SUPEE-9767 oraz z włączonym nowym ustawieniem „Włącz sprawdzanie poprawności klucza formularza przy kasie ”, po pomyślnym zakończeniu rejestracji nowego klienta na domyślna kasa z jedną stroną, żaden nowy klient nie jest tworzony i klient nie jest zalogowany, chociaż zamówienie jest prawidłowe.

Wyłączenie ustawienia „Włącz sprawdzanie poprawności klucza formularza przy kasie” powoduje, że działa to ponownie. Czy ktoś jeszcze miał ten problem? Wydaje się, że nie ma znaczenia, jakie metody wysyłki / płatności są używane.

Od tego czasu próbowałem tego ze świeżą, niezmienioną instalacją Magento 1.9.3.3 i wydaje się, że ma ten sam problem. Podczas rejestrowania nowego klienta przy kasie jednostronicowej żaden klient nie jest tworzony, nawet jeśli zamówienie przechodzi prawidłowo, o ile włączone jest ustawienie „Włącz sprawdzanie klucza formularza przy kasie”.

Ok, oto prawdziwa poprawka, którą wymyśliłem.

Edytuj /skin/frontend/base/default/js/opcheckout.jsi edytuj setMethod()metodę, zastępując:

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

Z:

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

To zrobi, gdy będziemy czekać na wersję v2 łatki

Po wybraniu opcji Zarejestruj się i kontynuuj, wywołania skryptu JS checkout.setMethod(), które znajdują się w skin/frontend/base/default/js/opcheckout.js. Stamtąd widzimy, że wysyła żądanie AJAX POST this.saveMethodUrl, ale jedyny przekazywany parametr to method. Jeśli spojrzymy na Mage_Checkout_OnepageController::saveMethodAction, który jest celem tego żądania AJAX, możemy zobaczyć, że poprawka została dodana:

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

Ponieważ _validateFormKeyszuka form_keyparametru w żądaniu, a ponieważ setMethodżądanie JS nie wysłało tego, gdy wykonało żądanie AJAX, po prostu wraca wcześnie i nic nie robi. Wróć do setMethodfunkcji i widzimy, że ponieważ nie próbuje nic zrobić z wartością zwracaną, nic innego się nie dzieje i JS kontynuuje działanie. W tym momencie JS ustawił, this.method = 'register'ale cytat nie został zaktualizowany, więccheckout_method jak domyślny „gość”.

Ponieważ JS zna rejestr wybrany przez klienta, wyświetla pola hasła, więc na pierwszy rzut oka wygląda na to, że się rejestrujesz. Ale jeśli chodzi o stronę PHP, jest to kasa gościa, więc nie tworzy klienta po jej zakończeniu.

Edycja: najprostszym rozwiązaniem jest skomentowanie tych trzech wierszy z saveMethodAction. Bardziej poprawnym / złożonym rozwiązaniem jest setMethodpobranie klucza form_ ze strony i wysłanie go wraz z żądaniem AJAX.

Pełne podziękowania dla Peter za rozwiązanie! Chciałbym wskazać instrukcje krok po kroku, co zmienić.

Przejdź do app / code / core / Mage / Checkout / controllers / OnepageController.php

Znajdź:

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

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

Skomentuj wiersz za pomocą tagów / * * /.

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

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

Warto zacząć:

Poprawka bezpieczeństwa SUPEE-9767 - Możliwe problemy?

Musisz zaktualizować pliki szablonów. Pamiętaj, że jest tylko kilka godzin od wydania tej łatki i na razie musimy poradzić sobie z tym, co jest publiczne. Jestem całkiem pewien, że w ciągu najbliższych dni wszystko się wyjaśni.

EDYCJA: Dziękujemy za głosowanie w dół! Przykro mi, że nie mogę rozwiązać problemu w ciągu 8 godzin od opublikowania tej poprawki.

Dzięki za łatkę @ Raphael w Digital Pianism.

Dla wygody stworzyłem diff, abyś mógł szybko zastosować łatkę.

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

Wersja 2 poprawki SUPEE-9767 została wydana wcześniej, wraz z Magento CE 1.9.3.4 . Wersja 2 rozwiązuje wiele problemów, w tym błąd rejestracji w kasie.

Możesz zaktualizować do najnowszej wersji (1.9.3.4) lub przywrócić V1, a następnie zastosować V2 poprawki. Każda z opcji rozwiąże problem.

Oficjalna zmiana w V2 jest faktycznie taka sama, jak opisał Peter O'Callaghan, usuwając trzy dodane linie Mage_Checkout_OnepageController::saveMethodAction.