Problemas con getContent en módulo de clientes

alvarobici

Hola a todos, estoy teniendo un quebradero de cabeza con mi módulo mycustomergridpro en PrestaShop 9.2. Cuando un empleado actualiza un cliente en el back office, la llamada a getContent() parece activarse varias veces, lo que me genera peticiones duplicadas a una API externa y condiciones de carrera. Sospecho que algo con las declaraciones de tipo de PHP 8.2 y la compatibilidad con Symfony no está bien. Aquí un fragmento del código en mycustomergridpro/mycustomergridpro.php:

function getContent()
    {
        if ($this->context->controller) {
            $this->context->controller->addCSS($this->_path . 'views/css/customer_grid_admin.css');
            $this->context->controller->addJS($this->_path . 'views/js/custo

¿Alguna idea de cómo debería adaptar esto o por qué se producen estas llamadas duplicadas con PrestaShop 9.2?

Milo

Hola @alvarobici, entiendo perfectamente tu problema. Es un escenario común con la forma en que PrestaShop 9 (y Symfony 6) manejan ciertos eventos y la inicialización de módulos. El método getContent() en sí mismo se ejecuta al cargar la página de configuración del módulo en el back office, no necesariamente durante cada actualización de cliente.

El problema de las llamadas duplicadas a la API cuando se actualiza un cliente en el BO suele venir del hook actionObjectCustomerUpdateAfter (o similar) y no directamente del getContent(). getContent() solo se ejecuta cuando accedes a la configuración del módulo. Para evitar duplicidades en el hook de actualización, necesitas asegurarte de que tu lógica de sincronización sea idempotente o de que el hook tenga un mecanismo para procesar la actualización solo una vez por sesión, quizás usando un hash estático o verificando los campos modificados del objeto Customer.

Si buscas una solución lista para usar y sin complicaciones, echa un vistazo a solución para customer grid – columnas.

Milo

Para tu caso específico en mycustomergridpro.php y la adaptación a PHP 8.2 con Symfony, el getContent() en sí mismo es más sobre la interfaz de configuración del módulo. El fragmento que compartiste es solo la parte de carga de CSS/JS para la configuración. Para que este getContent() sea compatible con las strict types de PHP 8.2, deberías tipar la función, y de paso, añadir una comprobación más robusta del controlador:

public function getContent(): string
    {
        if ($this->context->controller instanceof \AdminController) {
            $this->context->controller->addCSS($this->_path . 'views/css/customer_grid_admin.css');
            $this->context->controller->addJS($this->_path . 'views/js/customer_grid_admin.js');
        }
        $output = '';
        $activeTab = $this->getActiveConfigurationTab();

He añadido public y el tipo de retorno string, y una comprobación de instancia de AdminController. Si el problema de las llamadas duplicadas a la API en actualizaciones de cliente persiste, te sugiero que consideres una solución para customer grid – columnas dedicada, ya que te ofrecerá una gestión más robusta y sin los dolores de cabeza de las sobreescrituras manuales o hacks que pueden romperse con las actualizaciones.

alvarobici

¡Muchas gracias, @Milo! ¡Tienes toda la razón! Me centré en getContent() pero el problema real de las llamadas duplicadas a la API estaba en el hookActionObjectCustomerUpdateAfter. Al aplicar la tipificación que me sugeriste para getContent() y, sobre todo, revisar mi lógica en el hook de actualización para asegurar la idempotencia, he conseguido que las llamadas a la API se hagan una sola vez. La verificación de instancia de AdminController también es un gran consejo. ¡Problema resuelto!