search

jesusangel / wc-sermepa

Sermepa payment gateway plugin for WooCommerce
Other
28 stars 16 forks source link

Eliminar código legacy #22

Closed gonssal closed 8 years ago

gonssal commented 8 years ago

Hola. Teniendo en cuenta que WordPress actualmente requiere como mínimo PHP 5.2.4, ¿aceptarías un PR eliminando todo el código legacy de la carpeta libs?

Estas librerías dan avisos de Deprecated en PHP 7 ya que usan constructores con el mismo nombre que las clases.

Mandé un PR muy escueto hace una semana para ver si el proyecto estaba activo y de momento no ha habido respuesta, por eso pregunto por aquí antes de hacer un fork. Si aceptas PRs he hecho algún otro cambio que permitiría cerrar Issues abiertas.

jesusangel commented 8 years ago

Hola.

Vi tus cambios en el fichero readme, pero como no eran críticos lo dejé pasar y se me olvidó. Mándame los nuevos y los integraré.

Saludos cordiales. El 26/7/2016 12:21, "Marc" notifications@github.com escribió:

Hola. Teniendo en cuenta que WordPress actualmente requiere como mínimo PHP 5.2.4, ¿aceptarías un PR eliminando todo el código legacy de la carpeta libs?

Estas librerías dan avisos de Deprecated en PHP 7 ya que usan constructores con el mismo nombre que las clases.

Mandé un PR muy escueto hace una semana para ver si el proyecto estaba activo y de momento no ha habido respuesta, por eso pregunto por aquí antes de hacer un fork. Si aceptas PRs he hecho algún otro cambio que permitiría cerrar Issues abiertas.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jesusangel/wc-sermepa/issues/22, or mute the thread https://github.com/notifications/unsubscribe-auth/AAMroZKtwfKrkY99Y7l0FRYQnC-YD4uPks5qZd-kgaJpZM4JU_5V .