Actualizar tipos de cambio MNB

martabici

¡Hola a todos!

Estoy intentando integrar los tipos de cambio del Banco Nacional Húngaro (MNB) en mi tienda PrestaShop, ya que tenemos muchos clientes de Hungría y necesito que las tasas estén siempre al día. El problema es que PrestaShop no tiene una integración nativa para el MNB, y los métodos de actualización manual son un dolor de cabeza y siempre acaban rompiéndose con alguna pequeña actualización.

He intentado hacer un override de la clase Currency para que, al refrescar las tasas, también consulte el XML del MNB para el HUF. Aquí está mi intento, pero no parece funcionar correctamente o las tasas se calculan mal. Necesito que el tipo de cambio del HUF se base en el MNB, no en el default de PrestaShop.

// En override/classes/Currency.php (o similar)
class Currency extends CurrencyCore
{
    public function refreshCurrency()
    {
        $id_default_currency = (int)Configuration::get('PS_CURRENCY_DEFAULT');
        $default_currency_obj = new Currency($id_default_currency);
        $mnbXmlUrl = 'http://www.mnb.hu/arfolyamok.xml';
        $xml = @simplexml_load_file($mnbXmlUrl);
        if ($xml && isset($xml->Day->Rate)) {

El problema es que las tasas no se actualizan como espero, o me da valores erróneos. ¿Alguien ha lidiado con esto antes o ve algún fallo en mi lógica?

manuelhub

Coincido con @lorenamadrid. La versión es clave. Además, en PrestaShop 1.7.2.4 (si es esa tu versión), los overrides directos de Currency pueden ser un arma de doble filo. Es fácil que se pierdan con actualizaciones o causen conflictos si otros módulos intentan modificar la misma clase.

El conversion_rate en PrestaShop es siempre 'cantidad de la moneda por defecto por 1 unidad de esta moneda'. Es decir, si el EUR es tu moneda por defecto, y quieres actualizar el HUF, necesitas saber cuántos EUR vale 1 HUF. Si el MNB te da 1 EUR = 380 HUF, entonces 1 HUF = 1/380 EUR. Tu cálculo parece ir en esa dirección, pero la robustez de la obtención del hufRateAgainstDefault es lo que me preocupa. ¿Qué pasa si el XML no tiene el tipo de cambio para tu moneda por defecto en un día concreto?

Quizás un módulo específico o un script externo que actualice los valores directamente en la base de datos a través de una tarea programada (cron job) sería más estable que un override de una función tan central.

Milo

Hola @martabici, este es un escenario común con PrestaShop 1.7.x, especialmente en versiones como la 1.7.2.4, donde la gestión de divisas no es tan flexible para fuentes externas como el MNB. El problema principal de tu enfoque es doble: primero, refreshCurrency no es el lugar ideal para un proveedor de tasas externo, ya que se espera que procese todos los tipos de cambio de la fuente por defecto (usualmente el Banco Central Europeo). Segundo, la lógica de cálculo del conversion_rate para monedas no base, aunque tu intento es cercano, debe ser muy precisa para que no haya desviaciones.

Para PrestaShop, el conversion_rate de una moneda extranjera (ej. HUF) es la cantidad de tu moneda por defecto que equivale a 1 unidad de esa moneda extranjera. Si el MNB te da que 1 EUR equivale a 380 HUF (asumiendo EUR es tu moneda por defecto), entonces para el HUF, su conversion_rate debe ser 1/380. Tu cálculo 1 / $hufRateAgainstDefault es correcto bajo esa suposición.

Sin embargo, la forma más limpia y mantenible de gestionar esto, especialmente en entornos de producción con PrestaShop 1.7, no es con overrides manuales que pueden romperse o generar conflictos. Lo ideal es un módulo dedicado que gestione la obtención del XML del MNB y actualice las tasas de forma segura, quizás a través de un controlador Admin customizado o un cron job.

Aquí te muestro una versión más robusta de la lógica de actualización para el HUF, que podrías integrar en un módulo propio o un script externo ejecutado por cron:

// Lógica para un módulo o script personalizado
// Asumiendo que $xml ya ha sido cargado desde http://www.mnb.hu/arfolyamok.xml
$id_default_currency = (int)Configuration::get('PS_CURRENCY_DEFAULT');
$default_currency_iso = (new Currency($id_default_currency))->iso_code;
$huf_currency_data = Currency::getCurrencyByISO('HUF');
if ($huf_currency_data && (int)$huf_currency_data['id_currency'] !== $id_default_currency) {
    $huf_currency_obj = new Currency((int)$huf_currency_data['id_currency']);
    $rate_of_default_currency_in_huf = 0; // Ejemplo: 380.00 si 1 EUR = 380 HUF
    foreach ($xml->Day->Rate as $rateNode) {
        if ((string)$rateNode['currency'] === $default_currency_iso) {

Esta lógica asegura que el cálculo del conversion_rate del HUF sea correcto en relación a tu moneda por defecto. Para evitar los problemas de mantenimiento de código personalizado y asegurarte de que las actualizaciones sean seguras y fiables, te recomiendo usar un módulo ya desarrollado para esta función. Puedes considerar algo como Banco Nacional de Hungría (MNB) - tipos de cambio, que ya incorpora esta lógica y maneja las peculiaridades del MNB de forma profesional.

martabici

¡Muchas gracias, @Milo! La verdad es que tenía esa duda sobre si refreshCurrency era el lugar adecuado o si la lógica del conversion_rate estaba bien. Tu explicación sobre cómo debe ser el cálculo y la sugerencia del módulo son oro. Estoy en PrestaShop 1.7.2.4, como bien comentaron @lorenamadrid y @manuelhub, así que los overrides son siempre un riesgo.

He implementado la lógica que has compartido en un pequeño script temporal que ejecuto manualmente para probar, y el cálculo de la tasa para el HUF ahora es perfecto. Definitivamente voy a mirar el módulo dedicado, parece la solución más limpia y robusta para evitar futuros dolores de cabeza. ¡Gracias de nuevo!

lorenamadrid

Hola @martabici, entiendo perfectamente tu frustración con las tasas de cambio. Siempre es un problema cuando la moneda que necesitas no está en los proveedores por defecto.

Una pregunta rápida, ¿qué versión exacta de PrestaShop estás usando? Saber esto puede ayudar mucho, porque la forma de gestionar las divisas y los overrides puede variar un poco entre versiones. En las 1.7.x, especialmente las más antiguas, el refreshCurrency puede ser un poco delicado de tocar.

Sobre tu código, el simplexml_load_file sin un try-catch puede fallar silenciosamente si hay problemas de red o si el XML del MNB cambia de formato. Y la lógica para 1 / $hufRateAgainstDefault es crucial. Si el MNB te da 'X HUF por 1 EUR', y tu moneda por defecto es EUR, entonces para HUF necesitas '1 EUR por X HUF', que es 1/X. Parece que vas bien, pero hay que asegurarse de que hufRateAgainstDefault siempre sea el valor correcto de tu moneda por defecto en HUF.