Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paymill\Models\Request\SofortChecksum customer_email parameter is missing #146

Open
Voltash opened this issue Apr 10, 2018 · 2 comments
Open

Comments

@Voltash
Copy link

Voltash commented Apr 10, 2018

Parameter customer_email is mandatory and needs to be added to Paymill\Models\Request\SofortChecksum

@thyseus
Copy link

thyseus commented Apr 18, 2018

Unfortunately I need to confirm this bug. I have the same Problem, Sofortüberweisung does not seem to work at the moment.

I had some temporary success by adding a hack:

$parameter['customer_email'] = $parameter['client_email'] ?? null;

to: vendor/paymill/paymill/lib/Paymill/Request.php#237

but it is not working persistently.

@Raven24
Copy link

Raven24 commented Jul 15, 2018

I also ran into this issue, my "fix" is to add this subclass in my code:

class MySofortChecksum extends \Paymill\Models\Request\SofortChecksum {
  public function parameterize($method) {
    $parameterArray = parent::parameterize($method);
    if('create' == $method) {
      if ($this->getClientEmail()) {
        $parameterArray['customer_email'] = $this->getClientEmail();
      }
    }
    return $parameterArray;
  }
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants