search

ssheduardo / sermepa

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

Quitar setIdioma #42

Closed xpiga closed 7 years ago

xpiga commented 7 years ago

Hola Eduardo!

Primero agradecerte tu enorme contribución con tus clases php para Redsys/Sermepa/CECA

Estoy un poco verde en como crear una pull request aquí, pero al ser un cambio mínimo te lo indico a modo de "propuesta".

Estaba intentando cambiar el idioma de la pasarela de pago del BBVA a inglés y lo estaba intentado hacer con setIdioma pues había visto el setter en Tpv.php. Me ha parecido extraño pues había visto que en esta nueva versión estaba todo en inglés y efectivamente luego he encontrado la función setLanguage. Quizás sea recomendable que no aparezca setIdioma tampoco en el constructor. Y puede que esté bien mostrar en el ejemplo avanzado e tu web el uso de este método.

¡Muchas gracias!

ssheduardo commented 7 years ago

Efectivamente, con el cambio de la clase y reestructuración se me olvido quitar esa propiedad. Hago merge de este pull