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

About the "titular" parameter #18

Open
devnix opened this issue Aug 3, 2020 · 2 comments
Open

About the "titular" parameter #18

devnix opened this issue Aug 3, 2020 · 2 comments

Comments

@devnix
Copy link
Contributor

devnix commented Aug 3, 2020

Hi! I was checking out your integration as I want to implement Omnipay instead of working directly with Redsys.

I've found that you are setting the Ds_Merchant_Titular parameter the same as Ds_Merchant_MerchantName in

public function setMerchantName($merchantName)
{
$this->setParameter('merchantName', $merchantName);
$this->setParameter('titular', $merchantName); //is this right??
}

If I'm not wrong, the merchant name would be the name of your business and the "titular" parameter would be the cardholder, so it must be the complete name of the client, and not the business name at all!

Would you want a PR for this?

@jsampedro77
Copy link
Owner

Hi @devnix
You are right, that parameter value is wrong. I am sorry I have not time at all to fix this, but I can merge a PR if you want to create one.

thanks,

@devnix
Copy link
Contributor Author

devnix commented Aug 10, 2020

Can be closed when a new tag is released?

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

2 participants