¿Control de acceso a productos por grupo de clientes?

miguelhub

¡Hola a todos!

Estoy intentando implementar un sistema para restringir el acceso a ciertos productos basándome en los grupos de clientes. La idea es que solo los clientes de un grupo específico puedan ver y comprar determinados productos. He creado una tabla personalizada ps_product_group_access para almacenar las relaciones, pero mi lógica en el ProductController falla para clientes invitados o cuando un producto no tiene restricciones explícitas.

Aquí mi intento en el override del controlador:

// Fichero: override/controllers/front/ProductController.php
class ProductController extends ProductControllerCore
{
    public function initContent()
    {
        parent::initContent();
        $product = $this->product;
        if (Validate::isLoadedObject($product)) {
            $customerGroups = Customer::getGroupsStatic((int)$this->context->customer->id);
            $allowedProductGroups = Db::getInstance()->executeS('SELECT id_group FROM `'._DB_PREFIX_.'product_group_access` WHERE id_product = '.(int)$product->id);

El problema es que si el cliente no está logueado (id_customer = 0), Customer::getGroupsStatic devuelve el grupo de visitantes. Si ese grupo no está explícitamente en mi tabla ps_product_group_access para un producto con restricciones, los invitados son redirigidos incluso si deberían ver otros productos. Además, si un producto NO tiene entradas en ps_product_group_access, también quiero que sea visible para todos. ¿Alguna idea de cómo mejorar esta lógica para PrestaShop 8.0.3?

mariaiberia59

Hola @miguelhub, veo tu problema. Tal como lo tienes, Customer::getGroupsStatic para un cliente no logueado (id_customer = 0) te dará el PS_UNIDENTIFIED_GROUP por defecto, que es el grupo de visitantes. Si ese grupo no está en tu product_group_access, cualquier invitado será redirigido, incluso a productos que no deberían tener restricciones.

Necesitas manejar el caso de los invitados de forma explícita y también la situación donde el producto no tiene entradas en tu tabla personalizada. ¿En qué versión exacta de PrestaShop estás trabajando? Mencionas 8.0.3, ¿verdad? Eso es importante por si hay alguna función de contexto diferente.

sara_madrid

Estoy de acuerdo con @mariaiberia59. Tu lógica actual tiene dos puntos débiles: los invitados y los productos sin restricciones. Si allowedProductGroups está vacío, significa que el producto no tiene restricciones específicas, por lo que debería ser visible para todos. Ahora mismo, si tu if (!empty($allowedProductGroups)) es falso, el código simplemente pasa, pero no hay una condición else que lo marque como accesible si no hay reglas.

Normalmente, para una funcionalidad tan específica, la gente acaba creando un módulo con sus propios hooks y overrides controlados. Hacerlo directamente en el ProductController es un buen inicio para probar, pero luego hay que pulirlo bastante.

Milo

Hola @miguelhub, @mariaiberia59 y @sara_madrid han identificado los puntos clave. El ProductController es un lugar adecuado para esta comprobación de acceso en la vista del producto, pero la lógica para los grupos de clientes y para los productos sin restricciones debe ser más robusta.

El problema con Customer::getGroupsStatic((int)$this->context->customer->id) es que si el cliente no está logueado, id_customer es 0, y te devolverá el grupo de visitantes. Debes asegurarte de incluir este grupo en tu comprobación si quieres que los invitados puedan ver algo. Además, si allowedProductGroups está vacío, el producto no debería tener restricciones, no al revés.

Aquí te propongo una modificación a tu código que aborda estos puntos:

// Fichero: override/controllers/front/ProductController.php
class ProductController extends ProductControllerCore
{
    public function initContent()
    {
        parent::initContent();
        $product = $this->product;
        if (Validate::isLoadedObject($product)) {
            $customerGroups = $this->context->customer->isLogged() ? Customer::getGroupsStatic((int)$this->context->customer->id) : [(int)Configuration::get('PS_UNIDENTIFIED_GROUP')];
            $allowedProductGroups = Db::getInstance()->executeS('SELECT id_group FROM `'._DB_PREFIX_.'product_group_access` WHERE id_product = '.(int)$product->id);

Con este cambio, primero determinamos los grupos del cliente de manera segura (incluyendo el grupo de visitantes si no está logueado). Luego, si hay restricciones (!empty($allowedProductGroups)), comprobamos la intersección de grupos. Si no hay restricciones definidas para el producto, simplemente se salta el if y el producto es visible.

Ten en cuenta que hacer esto mediante overrides directos puede ser frágil con futuras actualizaciones de PrestaShop. Para una solución más mantenible y con una gestión más cómoda desde el Back Office, te sugiero considerar un módulo para acceso a productos que ya implementa esta lógica de forma robusta y con interfaz de usuario.

miguelhub

¡Muchísimas gracias, @Milo! La verdad es que la lógica del isLogged() y cómo manejar el PS_UNIDENTIFIED_GROUP era justo lo que me faltaba. Con el código que proporcionaste, ahora los productos sin restricciones se muestran correctamente a todos, y los productos restringidos funcionan bien tanto para usuarios logueados como para invitados según las reglas de mi tabla personalizada.

Es cierto que los overrides pueden ser un dolor de cabeza en las actualizaciones, así que miraré el módulo que mencionas para ver si puedo simplificarme la vida a largo plazo. De momento, con tu solución, he podido avanzar un montón. ¡Gracias de nuevo a ti y a @mariaiberia59 y @sara_madrid por vuestras aportaciones!