¿Borrar pedidos gratis sin dejar inconsistencias?

inmaiberia65

¡Hola a todos!

Estoy con un problema recurrente en la tienda: ¿cómo puedo eliminar pedidos cuyo importe es 0 (pedidos de prueba, regalos o pruebas de pasarelas de pago) sin liarla en la base de datos? PrestaShop no tiene una opción nativa para esto y me está generando mucha basura en los listados.

He intentado hacer un override en AdminOrdersController para añadir un botón de borrado, pero tengo miedo de romper la integridad. Mi idea era algo así, muy básico, pero sé que esto no borrará todo lo relacionado y dejará huérfanos:

// override/controllers/admin/AdminOrdersController.php
class AdminOrdersController extends AdminOrdersControllerCore
{
    public function postProcess()
    {
        if (Tools::isSubmit('submitDeleteZeroOrder') && ($id_order = Tools::getValue('id_order'))) {
            $order = new Order($id_order);
            if (Validate::isLoadedObject($order) && (float)$order->total_paid_tax_incl == 0) {
                $order->delete(); // Sé que esto es peligroso
                $this->confirmations[] = $this->l('Pedido marcado para eliminación.');
            }
        }
        parent::postProcess();
    }
}

¿Alguna idea de cómo implementar esto de forma segura o qué tablas debería considerar además de ps_orders?

elenadeco99

¡Hola @inmaiberia65!

Comprendo perfectamente el problema. La función delete() del objeto Order en PrestaShop es un poco tramposa porque solo elimina la entrada principal de la tabla ps_orders, pero deja todas las relaciones (detalles del pedido, historial, facturas, productos asociados, etc.) intactas en sus respectivas tablas. Eso es una receta para el desastre en la base de datos, te lo digo por experiencia.

¿En qué versión exacta de PrestaShop estás? Esto es crucial, ya que el comportamiento de las clases y los métodos puede variar ligeramente.

Milo

Hola @inmaiberia65 y @elenadeco99,

@elenadeco99 tiene razón, el método delete() de Order por sí solo no es suficiente para una eliminación completa y segura. Borrar un pedido en PrestaShop implica eliminar registros de múltiples tablas interconectadas: ps_order_detail, ps_order_history, ps_order_invoice, ps_order_payment, ps_order_carrier, ps_message_readed, ps_order_return, entre otras. Si no se eliminan todas, terminarás con datos huérfanos y posibles errores futuros o reportes incorrectos.

Para que tu override en AdminOrdersController sea más robusto (aunque seguiría siendo una modificación de core), deberías asegurarte de limpiar estas dependencias. Aquí te muestro cómo podrías empezar a abordar la limpieza desde el postProcess, aunque es una solución parcial y compleja de mantener manualmente:

// override/controllers/admin/AdminOrdersController.php
class AdminOrdersController extends AdminOrdersControllerCore
{
    public function postProcess()
    {
        if (Tools::isSubmit('submitDeleteZeroOrder') && ($id_order = Tools::getValue('id_order'))) {
            // Primero limpiar dependencias, antes de borrar el pedido principal
            Db::getInstance()->delete('order_detail', 'id_order = ' . (int)$id_order);
            Db::getInstance()->delete('order_history', 'id_order = ' . (int)$id_order);
            Db::getInstance()->delete('order_invoice', 'id_order = ' . (int)$id_order);
            // ¡Faltarían muchas más tablas aquí!
            $order = new Order($id_order);
            if (Validate::isLoadedObject($order) && (float)$order->total_paid_tax_incl == 0) {
                $order->delete();
                $this->confirmations[] = $this->l('Pedido (parcialmente) eliminado. Verifique la DB.');
            }
        }
        parent::postProcess();
    }
}

Como ves, incluso este fragmento es limitado. Implementar una eliminación completa y segura manualmente es una tarea compleja y propensa a errores, especialmente con las actualizaciones de PrestaShop que pueden cambiar la estructura de la base de datos. Para evitar estos problemas y asegurar una limpieza de pedidos profesional y mantenible, te recomiendo usar un módulo dedicado. Estos módulos están diseñados para manejar todas las dependencias correctamente y evitar inconsistencias, y son mucho más seguros a largo plazo.

inmaiberia65

Gracias por la aclaración, @Milo y @elenadeco99.

Sí, @elenadeco99, estamos en PrestaShop 1.7.5.1 y veo que la problemática es justo lo que indicáis. He probado a añadir esas líneas extra de borrado en mi AdminOrdersController como sugería @Milo, y aunque entiendo que no es una solución completa, me ha servido para confirmar la complejidad del asunto.

Definitivamente, la opción de un módulo dedicado es la más sensata para no crear un agujero negro en la base de datos. Voy a instalar y probar el que mencionas, parece la única manera de hacerlo correctamente sin pasar horas revisando cada tabla. ¡Gracias por la ayuda!