search

ssheduardo / sermepa

Clase para utilizar la pasarela de pagos redsys, sermepa.
MIT License
194 stars 59 forks source link

Actualizar urls para método InSite #95

Closed daparisi closed 1 year ago

daparisi commented 1 year ago

Hola, Seria posible actualizar a las nuevas urls para el método InSite, según esta en la documentación oficial. Parece que solo seria cambiar las urls en la función estática de getJsPath().

// Test

// Real

Saludos

ssheduardo commented 1 year ago

Si tienes a la mano el manual para echarle un vistazo.

daparisi commented 1 year ago

I tanto!!

https://pagosonline.redsys.es/conexion-insite.html En el apartado Creación del iFrame.

Incluso para que tenga compatibilidad con gente que use la versión 2, tal vez no estaría mal añadir un segundo parámetro a la función getJsPath() para poder indicar la versión del fichero a cargar.

Gracias.

ssheduardo commented 1 year ago

Si, esto estaba pensando. Pero estoy viendo cuando agregarlo que un compañero esta haciendo un PR para sacar la V2 de la librería con compatibilidad desde la 8.1, 8.2, 8.3. Pero con el código más alineado a dichas versiones, aunque actualmente esta librería es compatible desde PHP 7.4 creo hasta la 8.2 pero sin aplicar lo bueno que trae la 8.1 en adelante.

https://github.com/ssheduardo/sermepa/pull/94

rogervila commented 1 year ago

Podemos añadir esta option en la 2.1 una vez haya salido la 2.0.

ssheduardo commented 1 year ago

Lo malo que le dejaremos a los que aún usan 7.4 o 8.0 sin ese cambio.

rogervila commented 1 year ago

@ssheduardo entonces podrías incluir esta opción en la 1.5 y ya haré el merge en la 2.0. qué te parece?

ssheduardo commented 1 year ago

Si mejor, eso haré.

ssheduardo commented 1 year ago

Va ser la versión 1.4.6 dado que es una corrección menor. Ya esta subido.