TransbankDevelopers / transbank-plugin-magento2-webpay-rest

Plugin Magento 2 using Transbank REST API
BSD 3-Clause "New" or "Revised" License
4 stars 10 forks source link

Cambio de Title de metodo de pago por Store. #53

Closed slimzc closed 4 months ago

slimzc commented 1 year ago

Describe el bug

Cuando intentas modificar el titulo del metodo de pago por el scope Store desde el admin, no se puede ver y desde el Website Scope ocurre un error en el observer SaveConfigObserver.php

Para reproducir

  1. Configura una tienda Magento 2.4.4 y ve al administrador (aunque esto deberia fallar en cualquier versión)
  2. Entra el Stores -> Setting -> Configuration -> Sales -> Payment Method
  3. Cambia el Scope al Store por defecto.
  4. No aparece la configuración de Webpay

Ahora para ver el error del observer

  1. Cambia el Scope al website por defecto
  2. Intenta cambiar el titulo del metodo de pago
  3. Un error no te deja guardar la configuración

Comportamiento observado

No puedes guardar la configuración por Store del Title aunque creo que fue configurado para eso.

En el scope de Website el observer espera tener definido parametros en el request que no necesariamente van a ser configurados por el usuario, ocurriendo un error en la linea 31 y posteriormente en la 38 del SaveConfigObserver.php

Comportamiento esperado

Se espera que en system.xml el grupo sea visible en el scope de store para que pueda configurarse el titulo en caso de tener una tienda multi store o multi idioma.

Además la logica del observer mencionado debería tener fallback para obtener el valor ya sea porque fue configurado en el website o en el scope por defecto.

Versiones (por favor agrega aquí la siguiente información):

slimzc commented 1 year ago

Hola Chicos, creé un pull request con los cambios que yo implementé para mi tienda. Comparto y dejo a su criterio.

https://github.com/TransbankDevelopers/transbank-plugin-magento2-webpay-rest/pull/56

Tener en cuenta que no revisé compatibilidad con versiones inferiores de PHP, lo cual podria ser facilmente refactorizado.

Me comentan :)