Subida de imágenes en pedidos: ¿cómo implementarla?

joaquin_bici

¡Hola a todos!

Estoy intentando añadir una funcionalidad para que se puedan subir imágenes a los pedidos directamente desde el Back Office. La idea es que nuestros clientes puedan adjuntar justificantes o fotos de productos defectuosos tras realizar un pedido, y que se queden guardadas con él. He estado probando a modificar el AdminOrdersController para añadir un campo de subida, pero me encuentro con problemas para asociar el archivo subido al pedido y que persista después de guardar.

He intentado algo así en mi override/controllers/admin/AdminOrdersController.php:

class AdminOrdersController extends AdminOrdersControllerCore
{
    public function postProcess()
    {
        if (Tools::isSubmit('submitAddorder') || Tools::isSubmit('submitEditorder')) {
            if (isset($_FILES['order_image']) && $_FILES['order_image']['error'] == UPLOAD_ERR_OK) {
                $tmpName = $_FILES['order_image']['tmp_name'];
                $fileName = $_FILES['order_image']['name'];
                $ext = pathinfo($fileName, PATHINFO_EXTENSION);
                $newFileName = 'order_' . (int)Tools::getValue('id_order') . '_' . time() . '.' . $ext;

El archivo se sube bien a _PS_UPLOAD_DIR_ pero no sé cómo hacer para que el nombre del archivo se guarde en la base de datos asociado al pedido. Me gustaría que esto fuera robusto y no se rompiera si actualizamos PrestaShop en el futuro, ya que en PHP 8.2/8.3 a veces estos overrides tan directos dan problemas. ¿Alguna idea?

valeria9483

Hola @joaquin_bici,

Entiendo perfectamente el problema. Eso de asociar datos extra a objetos core como Order es un clásico en PrestaShop 1.7.x. Sin un campo definido previamente en la clase Order (y su tabla asociada en la base de datos), no vas a poder asignar $this->object->image_name porque simplemente no existe en su definición.

¿En qué versión exacta de PrestaShop estás trabajando? Si es 1.7, la forma más limpia suele ser extender la clase Order para añadir ese campo y luego ya manipularlo desde el controlador. Tu código para mover el archivo está bien, el problema es la persistencia.

teresa_dev51

Coincido con @valeria9483. El postProcess() del AdminOrdersController es el lugar adecuado para interceptar los datos y subir el archivo, pero para que el dato del nombre de la imagen se guarde, necesitas que el objeto Order sepa que tiene un nuevo campo llamado image_name.

También te sugiero crear una subcarpeta específica dentro de _PS_UPLOAD_DIR_ para las imágenes de los pedidos, así no se mezclan con otros uploads y es más fácil de gestionar y limpiar si fuera necesario. Asegúrate de que los permisos de esa carpeta sean correctos (normalmente 775).

Milo

Hola @joaquin_bici y a todos,

@valeria9483 y @teresa_dev51 están en lo cierto. Para que PrestaShop persista un nuevo campo asociado a un objeto core como Order, primero debes extender la clase Order y definir ese campo. Esto se hace creando un archivo override/classes/Order.php y añadiendo la definición de tu nuevo campo image_name.

Una vez hecho eso, tu postProcess() en AdminOrdersController puede asignar el nombre del archivo directamente a $this->object->image_name. También es vital asegurarse de que la carpeta de destino exista y tenga los permisos adecuados, como bien ha apuntado @teresa_dev51.

Aquí tienes el fragmento de código corregido para tu postProcess(), asumiendo que ya has extendido la clase Order en override/classes/Order.php para añadir el campo image_name:

// Dentro de tu postProcess() en override/controllers/admin/AdminOrdersController.php
// ...
if ($this->object instanceof Order && isset($_FILES['order_image']) && $_FILES['order_image']['error'] == UPLOAD_ERR_OK) {
    $uploadDir = _PS_UPLOAD_DIR_ . 'order_images/';
    if (!is_dir($uploadDir)) {
        mkdir($uploadDir, 0775, true);
    }
    $newFileName = 'order_' . (int)$this->object->id . '_' . time() . '.' . pathinfo($_FILES['order_image']['name'], PATHINFO_EXTENSION);
    if (move_uploaded_file($_FILES['order_image']['tmp_name'], $uploadDir . $newFileName)) {
        $this->object->image_name = $newFileName; // Ahora este campo sí existe

Recuerda borrar la caché de overrides (var/cache/prod/class_index.php o similar) después de crear o modificar el archivo override/classes/Order.php.

Aunque este override funciona en PrestaShop 1.7.4.2, como mencionas, este tipo de modificaciones manuales pueden ser frágiles en futuras actualizaciones o migraciones a versiones superiores (como PrestaShop 9 con PHP 8.2+), donde la estructura de clases y Symfony puede cambiar. Para una solución más robusta y mantenible a largo plazo, especialmente en entornos de producción, te recomendaría considerar un módulo para cargador de imágenes dedicado, que ya gestiona toda la lógica de persistencia, validación y compatibilidad entre versiones de forma profesional.

joaquin_bici

¡Muchas gracias, @Milo! ¡Y también a @valeria9483 y @teresa_dev51 por la ayuda!

Tu explicación sobre la extensión de la clase Order y el ajuste en postProcess() ha sido clave. He implementado el override en override/classes/Order.php para añadir el campo image_name y he aplicado el fragmento de código que me has dado en AdminOrdersController. Después de limpiar la caché, ¡funciona perfectamente!

Definitivamente, el punto de la mantenibilidad es importante, y con las futuras actualizaciones en mente, miraré el módulo que has recomendado para tener una solución más sólida. ¡Problema resuelto!