SUPEE-9767 Patch / CE 1.9.3.3 - Pago de una página - Problema de registro del cliente

En una instalación limpia y vainilla de Magento 1.9.2.4, parcheada con SUPEE-8788, SUPEE-9652 y SUPEE-9767, y con la nueva configuración 'Habilitar validación de clave de formulario al finalizar la compra' activada, luego de un pago exitoso del registro del nuevo cliente en el pago predeterminado de una página, no se crea un nuevo cliente y el cliente no ha iniciado sesión, aunque el pedido se realiza correctamente.

Desactivar la opción 'Habilitar validación de clave de formulario al finalizar la compra' hace que esto funcione nuevamente. ¿Alguién más ha tenido este problema? No parece importar qué métodos de envío / pago se utilicen.

Desde entonces he intentado esto con una instalación nueva y sin modificaciones de Magento 1.9.3.3 y parece tener el mismo problema. Al registrar un nuevo cliente a través de la comprobación de una página, no se crea ningún cliente, incluso si el pedido se procesa correctamente, siempre que la configuración 'Habilitar validación de clave de formulario al finalizar la compra' esté activada.

Ok, aquí está la verdadera corrección de errores que se me ocurrió.

Edite /skin/frontend/base/default/js/opcheckout.jsy edite el setMethod()método reemplazando:

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

Con:

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

Eso lo hará mientras esperamos la v2 del parche

Cuando selecciona registrarse y continuar, el script JS llama checkout.setMethod(), que se encuentra en skin/frontend/base/default/js/opcheckout.js. Desde allí podemos ver que realiza una solicitud POST AJAX this.saveMethodUrl, pero el único parámetro que pasa es method. Si observamos Mage_Checkout_OnepageController::saveMethodAction, cuál es el objetivo de esa solicitud AJAX, podemos ver que el parche agregó:

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

Dado que _validateFormKeybusca un form_keyparámetro en la solicitud, y dado que la setMethodsolicitud JS no envió esto cuando realizó la solicitud AJAX, simplemente regresa temprano y no hace nada. Volviendo a la setMethodfunción y podemos ver que, dado que no intenta hacer nada con un valor de retorno, no sucede nada más y el JS continúa. En este punto, el JS se ha establecido, this.method = 'register'pero la cotización no se ha actualizado, por lo que checkout_methodes el "invitado" predeterminado.

Dado que JS conoce el registro seleccionado por el cliente, muestra los campos de contraseña, por lo que parece que se está registrando. Pero en lo que respecta al lado de PHP, es un pago de invitado, por lo que no crea el cliente cuando se completa el pago.

Editar: la solución más simple es comentar esas tres líneas de saveMethodAction. La solución más correcta / compleja es que setMethoddebería tomar la tecla form_key de la página y enviarla con la solicitud AJAX.

¡Los créditos completos van a Peter por la solución! Me gustaría señalar instrucciones paso a paso sobre qué cambiar.

Vaya a app / code / core / Mage / Checkout / controllers / OnepageController.php

Localizar:

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

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

Comente la línea con / * * / etiquetas.

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

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

Un buen punto para comenzar:

Parche de seguridad SUPEE-9767 - ¿Posibles problemas?

Necesita actualizar sus archivos de plantilla. Tenga en cuenta que solo quedan unas pocas horas desde el lanzamiento de este parche y por el momento tenemos que lidiar con lo que es público. Estoy bastante seguro de que en los próximos días las cosas se aclararán.

EDITAR: ¡Gracias por votar abajo! Lamento no poder dar una solución en 8 horas desde el lanzamiento de este parche.

Gracias por el parche @ Raphael en Digital Pianism.

Por conveniencia, creé un diff para que pueda aplicar rápidamente el parche.

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

La versión 2 del parche SUPEE-9767 se lanzó hoy, junto con Magento CE 1.9.3.4 . V2 corrige una serie de problemas, incluido este error de registro de pago.

Puede actualizar a la última versión (1.9.3.4) o revertir V1 y luego aplicar V2 del parche. Cualquiera de las opciones resolverá el problema.

El cambio oficial en V2 es efectivamente el mismo que Peter O'Callaghan describió, eliminando las tres líneas agregadas Mage_Checkout_OnepageController::saveMethodAction.