Ficheros CSS/JS no cargan en configuración de módulo

marinaauto

Hola, comunidad. Tengo un problema persistente con la página de configuración de un módulo en PrestaShop 8.0.5. Estoy usando PHP 7.4.

El CSS (admin_conorderfee.css) y el JS (adminConditionBuilder.js) que se añaden a través de addCSS y addJS dentro del método run() de mi clase ConorderfeeAdmin simplemente no cargan en el navegador. Esto deja la página de administración del módulo con un diseño desorganizado y algunos elementos no funcionan como deberían.

He revisado las rutas mil veces, los archivos existen y tienen permisos correctos. Aquí está el fragmento de código donde intento añadir los assets:

function run()
    {
        if (isset($this->context->controller) && method_exists($this->context->controller, 'addCSS')) {
            $this->context->controller->addCSS($this->module->getPathUri() . 'views/css/admin_conorderfee.css');

¿Alguien sabe por qué podrían no estar cargando estos ficheros si el método addCSS se está llamando?

valeriamadrid

Hola @marinaauto,

Entiendo tu frustración. Esto es bastante común en PrestaShop si la forma de añadir los assets no se alinea con el ciclo de vida del controlador de administración. ¿Tu clase ConorderfeeAdmin extiende de ModuleAdminController o es una clase helper que se usa dentro del getContent() de tu módulo principal?

La verdad es que addCSS y addJS son métodos del AdminController que está renderizando la página, y a veces, si se llaman desde un contexto diferente o en un momento no óptimo, no tienen efecto. ¿Has comprobado la consola del navegador para ver si hay algún error 404 o de otro tipo al cargar esos archivos?

Milo

Hola @marinaauto y @valeriamadrid,

@marinaauto, el problema que describes es un clásico de PrestaShop 8.0.5 y versiones anteriores cuando se intenta añadir assets (CSS/JS) desde una clase auxiliar o en un punto incorrecto del ciclo de vida de la petición.

Aunque el código que muestras llama correctamente a addCSS y addJS en el $this->context->controller, el método run() de ConorderfeeAdmin (que asumo es una clase helper y no un AdminController propiamente dicho) no es el lugar adecuado para estas llamadas. El controlador principal que renderiza la página (normalmente un AdminModulesController para la configuración de módulos, o tu propio ModuleAdminController si lo tienes) ya ha procesado o está a punto de procesar sus propios assets a través del método setMedia().

Al llamar a addCSS/addJS desde ConorderfeeAdmin::run(), es muy probable que los assets se añadan a una cola que ya ha sido 'sellada' o que será sobrescrita por el controlador principal. La solución más robusta y recomendada es mover esas llamadas al método getContent() de tu archivo principal del módulo (conorderfee.php).

Para conorderfee/classes/ConorderfeeAdmin.php, la modificación sería simplemente eliminar esas líneas, ya que no son efectivas allí:

function run()
    {
        // Las llamadas a addCSS y addJS se eliminan de aquí
        // if (isset($this->context->controller) && method_exists($this->context->controller, 'addCSS')) {
        //     $this->context->controller->addCSS($this->module->getPathUri() . 'views/css/admin_conorderfee.css');
        //     $this->context->controller->addJS($this->module->getPathUri() . 'views/js/adminConditionBuilder.js');
        // }
        $activeTab = Tools::getValue('conorderfee_tab', 'dashboard');
        if ($activeTab === 'policies' && isset($this->context->controller)) {

Luego, en el método getContent() de tu módulo principal (conorderfee.php), deberías añadir estas líneas, asegurándote de que $this->context->controller esté disponible y sea el controlador correcto:

public function getContent()
{
    // ... lógica del formulario o procesamiento ...
    if (isset($this->context->controller) && method_exists($this->context->controller, 'addCSS')) {
        $this->context->controller->addCSS($this->getPathUri() . 'views/css/admin_conorderfee.css');
        $this->context->controller->addJS($this->getPathUri() . 'views/js/adminConditionBuilder.js');
    }
    // ... aquí llamas a ConorderfeeAdmin::run() para obtener el contenido HTML ...
    $conorderfeeAdmin = new ConorderfeeAdmin($this);
    // ... y retornas el HTML renderizado

De esta forma, los assets se registran en el contexto del controlador principal en el momento adecuado.

marinaauto

¡Mil gracias, @Milo!

Tu explicación tiene todo el sentido del mundo. He quitado las llamadas a addCSS y addJS de ConorderfeeAdmin::run() y las he puesto directamente en el getContent() de mi módulo principal, tal como sugeriste. ¡Y funciona a la perfección! La página de configuración ahora carga con todos los estilos y scripts. Estaba dándole vueltas a por qué no cargaban si el código se ejecutaba. Era el contexto. ¡Un alivio!