search

TransbankDevelopers / transbank-plugin-woocommerce-webpay

OBSOLETO: Código fuente de Plugin WooCommerce para Transbank Webpay
BSD 3-Clause "New" or "Revised" License
19 stars 17 forks source link

Carga de estilos en opciones de administrador #150

Closed 72OTA closed 3 years ago

72OTA commented 4 years ago

Realizando una configuración, note que los estilos llamados desde las opciones de administración, no son encontrados. De igual forma la página muestra una visualización correcta.

  1. /woocommerce-transbank-1/libwebpay/css/bootstrap-switch.css net::ERR_ABORTED 404 (Not Found)
  2. /woocommerce-transbank-1/libwebpay/css/tbk.css net::ERR_ABORTED 404 (Not Found)

Al entrar a la página de administración llamada admin-options.php me di cuenta que estaban utilizando: plugin_dir_url( __FILE__ ). El cual se utiliza para obtener el path donde se contiene el archivo que se le pasa como parámetro, por ende jamas encontrara el estilo dentro de la sub-carpeta, ya que estos se encuentran en la carpeta principal del plugin.

Para mitigar el problema simplemente se creo una constante dentro de webpay.php la cual obtiene el path base del plugin, y luego se hizo el llamado respectivo para que los estilos cargaran.

También es a considerar, que para obtener mejores tiempos en la carga de las opciones de administración las librerías jquery y bootstrap debieran llamarse directamente desde las carpetas internas del plugin, para no generar request extras al momento de cargar.

gdespirito commented 4 years ago

Hola @72OTA ! Gracias por reportar el problema y mandar un PR! 🎉 Revisando, me parece que ese estilo debió quedar ahí de hace mucho tiempo y si lo aprobamos cambiaría el estilo gráfico que tiene la administración. Creo que la solución es eliminar ese estilo que nunca se cargó correctamente y arreglar los estilos de raíz.

72OTA commented 4 years ago

Hola @gonzunigad, acorde a lo que comentaste, se procedió eliminando los estilos no utilizados existentes en la carpeta respectiva y se modifico el path de los script y css que si se utilizaban para las opciones de administración, en este caso. También se crearon los archivos para que existieran dentro de la carpeta base del plugin en las carpetas css y js respectivas. Y por ultimo en el enqueue de script también se reemplazo la ruta a la cual iba a buscar el archivo para que lo hiciera por defecto mediante la constante.

Espero sirva como solución acorde a tu comentario.