Error 'Unauthorized' en controlador simulate.php

marinaauto

Hola a todos,

Estoy teniendo un problema con un módulo en mi tienda PrestaShop 1.7.6.9. Cuando intento acceder al controlador simulate.php de un módulo personalizado (en paymethodrules/controllers/front/simulate.php), siempre recibo un error "No autorizado". Lo extraño es que estoy logueado en el front office como cliente.

He revisado el código y parece que la validación está comprobando si soy un empleado del back office. Aquí está el fragmento de código que lo provoca:

function postProcess()
    {
        if (!$this->context->employee || !(int) $this->context->employee->id) {
            $this->ajaxDie(json_encode(['success' => false, 'error' => 'Unauthorized']));
        }
        $idCart = (int) Tools::getValue('

¿Alguien sabe por qué este controlador frontal estaría verificando $this->context->employee y cómo puedo adaptar esto para permitir el acceso a clientes del front office? Necesito que funcione correctamente para que el módulo pueda simular los métodos de pago.

manuel24

Entiendo tu problema, @marinaauto. Esa línea de código es el culpable, sin duda. El $this->context->employee es para verificar si un usuario logueado es un administrador o empleado de la tienda, no un cliente. Si este simulate.php es un controlador frontal (/controllers/front/), lo lógico es que lo esté llamando el navegador de un cliente, no de un empleado.

¿Estás seguro de que este controlador debe ser accesible por clientes? A veces, los desarrolladores usan un controlador frontal para acciones internas que solo deberían ser llamadas por el sistema o por un empleado si el módulo tiene una interfaz de administración avanzada. Pero si es para simular opciones de pago para el cliente, definitivamente está mal enfocado.

Milo

Hola @marinaauto y @manuel24,

Tenéis toda la razón. El problema es que el controlador simulate.php se encuentra en la carpeta front, lo que indica que está destinado a ser utilizado por el front office y, por lo tanto, por los clientes. Sin embargo, la lógica de autenticación está verificando $this->context->employee, que solo existe cuando un usuario ha iniciado sesión como empleado en el back office de PrestaShop.

Para que este controlador sea accesible para los clientes logueados en el front office, debes cambiar la condición de employee a customer. Así, verificará si hay un cliente activo.

Aquí te muestro cómo debería quedar el fragmento corregido:

function postProcess()
    {
        if (!$this->context->customer || !(int) $this->context->customer->id) {
            $this->ajaxDie(json_encode(['success' => false, 'error' => 'Unauthorized']));
        }
        $idCart = (int) Tools::getValue('

Con este cambio, el controlador ahora verificará si un cliente está logueado y tiene un ID válido, permitiendo el acceso esperado para las simulaciones de pago.

marinaauto

¡Mil gracias, @Milo!

Acabo de aplicar tu sugerencia en el archivo paymethodrules/controllers/front/simulate.php y funciona perfectamente. Ahora el controlador es accesible cuando estoy logueado como cliente y el módulo puede hacer su trabajo de simulación sin el error "Unauthorized".

Era exactamente lo que necesitaba. ¡Gracias por la ayuda tan rápida y precisa!