search

ssheduardo / sermepa

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

Orden en argumentos predeterminados #66

Closed hmorv closed 3 years ago

hmorv commented 5 years ago

https://www.php.net/manual/es/functions.arguments.php "Obsérvese que cuando se emplean argumentos predeterminados, cualquiera de ellos debería estar a la derecha de los argumentos no predeterminados; si no, las cosas no funcionarán como se esperaba."

También actualizo README con el ejemplo.

Saludos!

ssheduardo commented 5 years ago

Lo que te comente si eso podría ser:

public function check($key = '', $postData=[])
hmorv commented 5 years ago

Ah disculpa, lo cerré y no vi tu comentario, puedo modificarlo en este mismo pull request o cierro y añado otro con el cambio?

ssheduardo commented 5 years ago

Si quieres cambialo, de paso ya que estamos, agregamos en este if, si no es array que lance error:

if (!isset($postData)  || !is_array($postData)) {
            throw new TpvException("Add data return of bank");
}