Función de protección de carro sin sobreescribir core

natalia8994

Hola a todos,

Estoy intentando implementar una funcionalidad de 'protección de carro' para evitar que los carros se borren o caduquen demasiado rápido, o para guardar el estado del carro de un cliente si se cierra la sesión por accidente. He probado a modificar la clase Cart directamente, pero me preocupa que se rompa con cada actualización o que no sea la forma correcta de hacerlo en PrestaShop.

Mi idea era extender el tiempo de vida de la sesión del carro o guardar el carro de forma persistente. Aquí un intento que hice en override/classes/Cart.php:

class Cart extends CartCore
{
    public function __construct($id = null, $id_lang = null)
    {
        parent::__construct($id, $id_lang);
        // Intentando extender la validez del carro
        if ($this->id)
        {
            $this->date_upd = date('Y-m-d H:i:s');
            $this->save(); // Esto parece guardar el carro pero no la sesión

Esto no me funciona como espero, el carro sigue desapareciendo o no se mantiene la información de la sesión. ¿Alguna idea de cómo abordar esto de forma más limpia o si hay algún hook que se me escape?

raulonline

Hola @natalia8994,

Entiendo el problema. Es una fastidio cuando los clientes pierden el carrito. El tema de guardar el carro y la sesión son dos cosas distintas en PrestaShop. Tu override en el constructor de Cart solo actualiza la fecha de modificación del objeto Cart en la base de datos, pero no afecta la sesión PHP real ni la forma en que PrestaShop decide qué carrito está activo para un usuario en una sesión específica.

Para poder ayudarte mejor, ¿en qué versión exacta de PrestaShop estás trabajando? El comportamiento de las sesiones y los carritos ha cambiado un poco entre las versiones de 1.7, especialmente con PHP 7.0/7.1.

pedroauto96

Sí, @raulonline tiene razón. Si estás en una versión antigua de 1.7 (como las primeras de 1.7.x), la gestión de sesiones y la validación del carro pueden ser un poco escurridizas. Yo tuve un problema similar con la caducidad del token de seguridad del formulario de carrito, y no era tan sencillo como actualizar una fecha.

Ese save() dentro del constructor me parece un poco peligroso, podría generar bucles o sobrecargas si no se maneja con cuidado. Para 'proteger' el carro de forma más efectiva, podrías necesitar intervenir en el FrontController o en el AuthController para manejar la reasignación de carritos o la extensión de sesiones cuando un usuario regresa.

Milo

Hola @natalia8994,

Gracias por proporcionar el fragmento de código. @raulonline y @pedroauto96 están en lo cierto. La modificación del constructor de Cart no es el lugar ideal para implementar una lógica de 'protección de carro' persistente, ya que solo manipula el objeto Cart en la base de datos y no la sesión del usuario o la lógica de carga del carrito de PrestaShop.

En PrestaShop 1.7.2.4, la lógica para cargar y guardar el carrito se gestiona en varios puntos, principalmente en el FrontController y también en la clase Context. Para asegurar que un carrito sea persistente más allá de la sesión actual o de un cierre de sesión, necesitas asociar el carrito a un cliente de forma más robusta y gestionarlo en el momento de la autenticación o al iniciar la sesión.

Una forma más limpia de abordar esto sería utilizar un hook, como actionAuthentication o actionFrontControllerSetMedia, para verificar y cargar un carrito persistente asociado a un cliente. Aquí te muestro un ejemplo de cómo podrías hacer esto en un módulo o en un override de FrontController (aunque un módulo es siempre preferible):

class FrontController extends FrontControllerCore
{
    public function initCtxt()
    {
        parent::initCtxt();
        if ($this->context->customer->isLogged() && !isset($this->context->cart->id))
        {
            $id_cart_from_db = (int)Db::getInstance()->getValue('SELECT id_cart FROM '._DB_PREFIX_.'cart WHERE id_customer = '.(int)$this->context->customer->id.' AND id_shop = '.(int)$this->context->shop->id.' ORDER BY date_upd DESC');
            if ($id_cart_from_db)
            {

Este código, colocado en un override de FrontController o en un hook adecuado dentro de un módulo, intentaría cargar el último carrito activo de un cliente si no tiene uno asociado en la sesión actual. Es una solución más dirigida a mantener el carrito asociado al cliente.

Sin embargo, implementar una funcionalidad completa de 'protección de carro' que maneje múltiples escenarios (carritos abandonados, recuperación de sesión, caducidad extendida, etc.) es complejo y requiere tocar varias partes del core. Para una solución robusta y que no tengas que mantener manualmente con cada actualización de PrestaShop, te recomendaría considerar una solución para protección de carro dedicada. Estos módulos están diseñados para gestionar la persistencia del carrito de forma segura y eficiente sin riesgo de romper tu tienda.

natalia8994

¡Muchísimas gracias, @Milo! La explicación de la diferencia entre el objeto Cart y la sesión ha sido clave, y el código que propones en el initCtxt del FrontController me ha ayudado a entender mejor cómo funciona la carga del carrito.

He probado tu sugerencia y parece que funciona mucho mejor para el objetivo de persistencia. Sin embargo, como bien señalas, para una solución más completa y a largo plazo, el módulo dedicado parece la opción más sensata para evitar futuras complicaciones. Estoy valorando adquirirlo. ¡Gracias de nuevo por la ayuda tan detallada!