Gestionar y eliminar combinaciones por atributo

samueldev

¡Hola a todos!

Estoy intentando añadir una funcionalidad en el back office para poder eliminar combinaciones de productos por atributo. Es decir, si tengo un producto con Tallas (S, M, L) y Colores (Rojo, Azul), quiero poder borrar todas las combinaciones que sean, por ejemplo, de Color Azul, independientemente de la Talla.

He intentado crear un override en AdminProductsController.php y añadir algo en postProcess, pero no consigo que funcione de forma fiable. O no se activa mi lógica, o cuando parece que sí, no borra las combinaciones correctamente y luego el producto queda raro en el frontal.

Aquí tenéis un fragmento de lo que estoy intentando:

class AdminProductsController extends AdminProductsControllerCore
{
    public function postProcess()
    {
        if (Tools::isSubmit('submitDeleteCombinationsByAttribute')) {
            $id_product = (int)Tools::getValue('id_product');
            $id_attribute_to_delete = (int)Tools::getValue('id_attribute_to_delete');
            // Esto no borra bien o no actualiza todo lo necesario
            if ($id_product && $id_attribute_to_delete) {
                // Logic to find product_attribute_ids based on id_attribute_to_delete

¿Alguien ha implementado algo parecido o sabe cómo hacerlo de forma robusta sin romper la base de datos o que se desincronicen los stocks?

marcos94

¡Uf, clásico problema! @samueldev, esa lógica de borrar directamente de la base de datos es súper peligrosa. PrestaShop tiene muchas tablas relacionadas con las combinaciones (stocks, precios específicos, imágenes...). Si no usas los métodos de su ObjectModel, te vas a dejar cosas sin borrar o desincronizadas, lo que lleva a errores extraños.

¿En qué versión exacta de PrestaShop estás? Porque en PS 1.7, especialmente si usas PHP 7.2 o superior, hay que tener mucho cuidado con el postProcess y cómo se envían los datos desde el formulario.

natalia_lab

Coincido con @marcos94, esa aproximación es muy frágil. PrestaShop, incluso en 1.7, ya tiene una capa de abstracción importante. Intentar borrar directamente de product_attribute sin pasar por Product::deleteProductAttributes o similar es una receta para el desastre. Es fácil que te queden registros huérfanos en product_attribute_combination, stock_available, image_shop, etc.

Tendrías que iterar sobre todas las combinaciones del producto, identificar las que contengan el atributo a borrar, y luego usar los métodos adecuados de la clase Product o Combination para eliminarlas de una en una, lo cual puede ser lento para productos con muchas combinaciones. Además, hay que asegurarse de que la reindexación y la regeneración de la caché se hagan correctamente.

Milo

Hola @samueldev, entiendo perfectamente tu frustración. El problema con tu fragmento de código es que estás intentando un borrado de bajo nivel que no respeta la lógica de negocio de PrestaShop. No basta con eliminar de product_attribute; hay que manejar todas las dependencias (stock, precios, imágenes, ganchos de módulos, etc.).

La forma correcta de hacerlo sería utilizando los métodos de la clase Product o Combination de PrestaShop, que se encargan de borrar todas las entradas relacionadas y de disparar los ganchos necesarios. Aquí te dejo un ejemplo de cómo podrías enfocarlo en un override de AdminProductsController para PS 1.7.7.7:

class AdminProductsController extends AdminProductsControllerCore
{
    public function postProcess()
    {
        if (Tools::isSubmit('submitDeleteCombinationsByAttribute')) {
            $id_product = (int)Tools::getValue('id_product');
            $id_attribute_to_delete = (int)Tools::getValue('id_attribute_to_delete');
            if ($id_product && $id_attribute_to_delete) {
                $product = new Product($id_product);
                $product->deleteProductAttributes(array($id_attribute_to_delete));

Esta línea clave deleteProductAttributes(array($id_attribute_to_delete)) dentro de la clase Product es la que te permite eliminar combinaciones que contengan ese atributo. Asegúrate de que $id_attribute_to_delete es el ID del atributo específico que quieres eliminar. Esto activará los ganchos y se encargará de las dependencias.

Sin embargo, crear y mantener estos overrides para cada funcionalidad específica puede volverse complejo y propenso a errores en cada actualización de PrestaShop. Para una solución más robusta y fácil de mantener en tiendas en producción, te recomiendo considerar un módulo dedicado. Existe una buena herramienta para la automatización de eliminar combinaciones por atributo que simplifica este proceso y se integra sin overrides manuales.

samueldev

¡Muchísimas gracias, @Milo! Esa es la clave que me faltaba. Había intentado con deleteProductAttributes pero no lo estaba pasando como un array y por eso no me funcionaba. Lo he probado con tu código y ahora sí que funciona como un tiro, eliminando todo correctamente y sin dejar rastros en el Back Office.

Me apunto lo del módulo también, porque el override siempre me da un poco de miedo con las actualizaciones. Pero para salir del paso, esto me ha salvado. ¡Gracias de nuevo!