Skip to content

Commit b37780f

Browse files
authored
Update AuthorizeRequest.php
I've come across an issue with the SendEmail option in Omnipay\SagePay\Message\Form\AuthorizeRequest At https://github.com/thephpleague/omnipay-sagepay/blob/master/src/Message/Form/AuthorizeRequest.php#L127 the SendEmail data is set up, but it's then filtered out because of the mis-match in case sensitivity at https://github.com/thephpleague/omnipay-sagepay/blob/master/src/Message/Form/AuthorizeRequest.php#L29 So $data['SendEmail'] is set up properly but then filtered out because it's not in the $validFields array (the $validFields array contains SendEMail but not SendEmail). I'd propose updating $validFields to allow 'SendEmail' rather than 'SendEMail', but I wanted to check this with the creators of this useful project :)
1 parent 1ef9095 commit b37780f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Message/Form/AuthorizeRequest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ class AuthorizeRequest extends DirectAuthorizeRequest
2626
'CustomerName' => false,
2727
'CustomerEMail' => false,
2828
'VendorEMail' => false,
29-
'SendEMail' => false,
29+
'SendEmail' => false,
3030
'EmailMessage' => false,
3131
'BillingSurname' => true,
3232
'BillingFirstnames' => true,

0 commit comments

Comments
 (0)