Integrar tasas NBU

rociohub

Hola a todos,

Estoy intentando integrar las tasas de cambio del Banco Nacional de Ucrania (NBU) en mi tienda PrestaShop. Actualmente, no veo una opción nativa para esto y los cambios manuales son un dolor de cabeza y se rompen con cada actualización.

He intentado crear una pequeña modificación en AdminCurrenciesController para que al actualizar las divisas, también consulte una API externa del NBU. Pero no funciona bien, los valores no se guardan correctamente o la llamada a la API es inestable.

Mi intento básico fue algo así:

// En un override de AdminCurrenciesController.php
public function postProcess()
{
    if (Tools::isSubmit('submitRefreshCurrency')) {
        // Intentando añadir lógica para NBU
        $nbu_rates = $this->fetchNbuRates(); // Método custom
        if ($nbu_rates) {
            // Lógica para actualizar EUR a UAH o USD a UAH
            // Esto es lo que no consigo hacer bien y es muy frágil.
            // Por ejemplo, si USD es la divisa base y quiero UAH:

El problema es que esta lógica es muy susceptible a errores y me gustaría algo más robusto, que no se rompa al actualizar PrestaShop. ¿Alguna idea de cómo manejar esto mejor?

joaquindeco

Hola @rociohub,

Entiendo perfectamente el problema. Tocar los controladores directamente es peligroso y PrestaShop suele sobrescribirlos en las actualizaciones mayores, especialmente cuando son parches de seguridad o de compatibilidad.

¿En qué versión de PrestaShop estás trabajando? Eso es clave, porque el comportamiento de la clase Currency y cómo PrestaShop maneja los hooks de actualización de divisas pueden variar bastante. Por ejemplo, en las últimas versiones la inyección de dependencias es la forma recomendada.

Tu enfoque de postProcess() es delicado. Estás asumiendo que submitRefreshCurrency siempre se invoca y que puedes inyectar tu lógica sin colisiones. Además, la llamada a fetchNbuRates() debería ser asíncrona o ejecutarse en un cron para no ralentizar el backoffice cuando un administrador intenta refrescar las divisas. Intentar actualizar conversion_rate directamente así puede no ser la forma más limpia ni la más segura.

guillermo_lab

@joaquindeco tiene razón, modificar controladores directamente es una receta para el desastre en PrestaShop. Además, la gestión de divisas en PrestaShop está pensada para tasas fijas o las que se obtienen de servicios muy generales como el del Banco Central Europeo por defecto.

Para el NBU, que es una fuente externa específica, necesitas un mecanismo que:

Consulte la API del NBU regularmente (un cron job es lo ideal para esto, fuera de las peticiones de los administradores).
Parseé la respuesta de la API y actualice solo las divisas relevantes en PrestaShop (en tu caso, la UAH).
Lo haga sin tocar archivos core. Es decir, con un módulo bien estructurado que use los servicios o los hooks adecuados.

No hay un hook específico tipo actionAfterCurrencyRefreshFromExternalAPI que puedas usar fácilmente para esto en el estándar de PrestaShop 8. Podrías intentar sobrescribir el método refreshCurrency de la clase Currency pero eso también es una sobrescritura de core, y sigue siendo frágil.

Milo

Hola @rociohub, @joaquindeco y @guillermo_lab,

Confirmando lo que comentáis, intentar modificar el AdminCurrenciesController o la clase Currency directamente para integrar fuentes de datos externas como el NBU es un enfoque propenso a errores y muy poco mantenible en PrestaShop 8.0.3 (o cualquier versión). PrestaShop no tiene una API o hook dedicado para integrar fuentes de tasas de cambio tan específicas de forma nativa.

El problema con tu código, @rociohub, es que estás insertando una lógica de negocio compleja (la llamada a la API y el parseo) dentro de un método de controlador que tiene una responsabilidad muy específica y que puede ser sobreescrito. Para gestionar esto de forma robusta, necesitarías un módulo que:

Defina un servicio (si estás en PS 1.7+ con Symfony) para la comunicación con la API del NBU.
Utilice un cron job para ejecutar este servicio periódicamente.
Actualice las tasas de cambio de las divisas específicas (UAH) a través de la clase Currency de PrestaShop de manera controlada.

Una implementación más limpia, dentro de un módulo, podría registrar una tarea cron que ejecute algo así:

// Dentro de un método de un servicio o clase de tu módulo
public function updateNbuCurrencyRates()
{
    // Suponemos que ya tienes un método para obtener las tasas de la API
    $nbuRates = $this->nbuApiService->getLatestRates();
    if ($nbuRates && isset($nbuRates['USD'])) { // Ejemplo: si la API devuelve tasas relativas al USD
        $currencyUAHId = (int)Currency::getIdByIsoCode('UAH');
        if ($currencyUAHId) {
            $currency = new Currency($currencyUAHId);
            // Asegúrate de que la tasa de conversión sea correcta respecto a tu divisa por defecto

Este enfoque mantiene la lógica de negocio separada y programada. Para tiendas que necesitan una integración profunda y automatizada con el Banco Nacional de Ucrania sin preocuparse por roturas en futuras actualizaciones, existe una herramienta de nbu cambio de divisas que hace exactamente esto. Facilita la gestión y automatización de las tasas de cambio del NBU, lo que te ahorra mucho tiempo y posibles problemas técnicos.

rociohub

¡Muchas gracias por la aclaración, @Milo!

El enfoque de un módulo con un servicio y un cron job tiene mucho más sentido. Es verdad que mi idea de sobreescribir el controlador era una solución temporal y muy frágil, como bien señalasteis @joaquindeco y @guillermo_lab.

He estado mirando el módulo que has recomendado y es exactamente lo que necesito para automatizar la actualización de las tasas del NBU sin tener que preocuparme por las actualizaciones del core de PrestaShop. ¡Lo voy a instalar y configurar hoy mismo!

Me alegro de haber preguntado antes de seguir adelante con un override que me habría dado más de un dolor de cabeza. ¡Gracias a todos por vuestra ayuda!