Issues with ABN Amro: Incorrect parameter entered / Test URL
Magical7 opened this issue · 21 comments
Hey,
I'm pretty new to using Lemonstand, iDeal and the ABN Amro payment system, so I am not fully sure about how to use this. There may be something wrong in my setup.
I have generated a certificate and private key as instructed in the readme, using the default password for testing purposes. I have no idea what to fill in for the acquirer certificate, so I left it to '/' for now.
Then I tried to render the payment form, which causes an Exception ('Unable to retreive information from the payment gateway.') and when I echo the response gotten, it shows a blank order reference, total charge and beneficiary, under which it says 'Incorrect parameter entered'.
Is this a problem caused by my settings? And if so, could you explain how I can fix this?
Secondly, while looking at the code for this module, I found this peculiar line in the $acquirer_urls array:
'abn' => array('internetkassa.abnamro.nl/ncol/prod/orderstandard.asp','internetkassa.abnamro.nl/ncol/prod/orderstandard.asp'),
I could be wrong, but shouldn't the second element be something different?
Thanks for taking the time to look into this!
Did you ever manage to get Ideal working on lemonstand?
I did eventually implement support for iDeal ABN Amro on my own, which will probably not work for other banks in its current state. It utilizes a separate payment method an has the user send a form with certain hidden fields to the ABN servers, where the rest of the transaction is performed. This is based on ABNs guide on implementing iDeal on websites.
Yeah I'm running in to multiple problems with this module.
Main problem I run into at the moment is when you are send back from the bank to the shop. Status request problems.
Doesn't look like that the developer is still working on this module..
I'm going to see if I can get ABN Amro support working properly shortly.
@Magical7 Do you have a testing ABN Amro account and would you be willing to help me test this? Thank you in advance 😄
Not anymore, I no longer work at the company where I worked with this payment module.
OK, thanks anyway! :)
Phil Schalm
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
On Monday, July 22, 2013 at 9:49 PM, Lex Utama wrote:
Not anymore, I no longer work at the company where I worked with this payment module.
—
Reply to this email directly or view it on GitHub (#3 (comment)).
@dutchwave Do you use ABN or another provider? I'm working on updating this for iDEAL v3 and would like to make sure that any other bugs are fixed while I'm at it. Thanks!
I'm using ING, what would you need?
I was hoping you'd be able to test ABN support for me, but that's alright. I did just push a new branch - if you wouldn't mind taking a try at the new code that'd be greatly appreciated. I think I've gotten the iDEAL 3.3.1 support fully implemented now. You'll need to re-generate your certificate and private key - the updated commands are in the readme.
Tried the new files thanks, with ING.
2 problems from the log:
In old version modus: Phpr_PhpException: Invalid argument supplied for foreach(). In /usr/home/domain/domains/domain.nl/public_html/modules/idealpaymentgateway/classes/idealpaymentgateway_helper.php, line 315
New version: Phpr_PhpException: Invalid argument supplied for foreach(). In /usr/home/domain/domains/domain.nl/public_html/modules/idealpaymentgateway/classes/idealpaymentgateway_helper.php, line 329
Tried it with new certificates a few times now, no luck.
Ok got it to work.
Used the following:
genrsa –aes128 –out priv.pem –passout pass:yourpasword 2048
req –x509 –sha256 –new –key priv.pem –passin pass:yourpasword –days 1825 –out cert.cer
Do wonder how important the input values are. For example before I didn't always enter organizational unit name, now I did..
Something else. How can you update an order status in the order route?
Order status: Ideal new - you don't get notifications (At the moment I have to set notifications on, even when no actual transfer happened. So when there are false and cancelled orders, you also get notified. )
Ideal paid: customer and sales receive a notification that the payment was successful. Although I set this transition in order status, orders don't get updated to this status.
Do I need to fill out something in the API code?
@dutchwave Did you ever get it working in old mode or did it only work in new mode? I want to make sure that during the transition it doesn't break for anyone.
Regarding the order status - You have the following two dropdowns - http://cl.ly/image/1M3F3A1X340o - which you can use to set the order states based on what happens when an order's processed by iDEAL. Normally I used the Paid state and a an additional state to handle the cancelled side of things. If someone makes an order but abandons paying (never even completes the payment processes) then it will stay in 'New' status. Does that cover everything you're thinking of?
Yes got it thank you! For some reason I thought you had to set the 'New' status. Then after success it would use a transitions to the next state. I was thinking way to difficult there :s
I actually didn't get it working in old modus. Hard to say where it went wrong as I used the files from Marijn. (the guy you emailed). Using these after upgrading nothing happened and I might have broken stuff. Overriding them with your files did work out. But by then I think I replaced a certificate already. Yes, one big mess ;)
Small suggestion:
'description' => 'Order_from_' . preg_replace('/\s+/', '_', Shop_CompanyInformation::get()->name),
Wouldn't it be more logical to have the ordernumber at the description?
I didn't, just in case you have multiple shops - this way you can easily distinguish between them. I think it should be working in the old mode now, but I have to re-generate old certificates to properly test (the tokenCode generated is too long with a 2048-bit key). As long as I get that working, do you see any issues with it going live?
Well I have only tested ING offcourse. But I can't really see why not to put it live either.
Maybe just need some extra readme information, explaining why there is an old version and maybe some other stuff. Could do it in Dutch aswell....
What do you think of the shopname and order number? It would be easier to find a payment belonging to an order and more clear for a customer I think.
shop name and order id - done!
I'll add information about the old version. I don't know dutch, so unfortunately I can't write anything for that. If you do, would you be willing to translate the README for the ideal331 branch?
Great work!
Yes that's what I mean, I can translate it for you. Also sentences and words in the files I can translate. I'm doing that anyways.
I'm closing this because I believe issues with ABN Amro have been sorted out, as well as the issue with the merchant return URL. These will be going live on the marketplace in the next few days.