• Resolved andreu

    (@andreu)


    Hola!

    La URL de la imagen del logo de Redsys no funciona con WPML activado porque se le a?ade el idioma a la URL. Me sale este error:

    Failed to load resource: the server responded with a status of 404 (Not Found) https://domain.com/es/wp-content/plugins/woocommerce-sermepa-payment-gateway/assets/images/icons/redsys.png

    Lo he arreglado cambiando home_url() por site_url() en la línea 101 de wc_redsys_payment_gateway.php

    Gracias por tu trabajo con el plugin y el update a la nueva normativa!

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author jesusangel.delpozo

    (@jesusangeldelpozo)

    Hola.

    Es mejor usar home_url(): https://w3guy.com/subtle-difference-home_url-site_url/

    Puesto que las modificaciones del código fuente se perderán en caso de actualizar el plugin, en este caso es mejor que uses el mecanismo del filtro para especificar la ubicación del logotipo.

    Por ejemplo, puedes a?adir lo siguiente al archivo functions.php:

    function mitema_wc_redsys_icon( $icon ) {

    if( file_exists( get_stylesheet_directory() . ‘/img/credit-card-logos-100×35.jpg’ ) )
    $icon = get_stylesheet_directory_uri() . ‘/img/credit-card-logos-100×35.jpg’;

    return $icon;

    }
    add_filter( ‘wc_redsys_icon’, ‘mitema_wc_redsys_icon’, 99, 1 );

    Eso buscará el icono en la carpeta con las hojas de estilo de tu tema, bajo img/.

    Puedes cambiar la ruta del icono y poner otra si quieres.

    Saludos.

    Thread Starter andreu

    (@andreu)

    Ok, gracias!

    De todas formas, para a?adir soporte al plugin para WPML quizá se podría hacer algo como:

    if ( function_exists('icl_get_home_url') ) {
    $home_url = icl_get_home_url();
    } else {
    $home_url = home_url();
    }

    ?No crees que sería mejor?

    Un saludo!

    Plugin Author jesusangel.delpozo

    (@jesusangeldelpozo)

    Sí. Correcto.

    Me lo apunto.

    Muchas gracias.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘Error image URL with WPML’ is closed to new replies.