Closed (fixed)
Project:
Ogone
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
10 Oct 2013 at 14:57 UTC
Updated:
3 Feb 2014 at 08:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
segi commentedI attached the patch.
Comment #2
xanoGood catch! The Ogone parameter cookbook says the value must be alphanumeric with a maximum length of 15 characters. A colon is not alphanumeric, so what we can do is use an
xas separator?Also, what is this value used for within Ogone? Is it even required by the API?
Comment #3
xanoBump.
Comment #4
segi commentedSorry for late, I was a little busy.
I checked order without ITEM ID and it seems to me it works, but I think this is not an unnecessary property, it is proper for identify the product. I modified the patch based on your suggestion and toke a X instead of underscore.
So, feel free, you decided the ITEM ID is necessary or not, but at the moment it doesn't work.
Comment #5
segi commentedComment #6
segi commentedComment #9
segi commentedSorry, I fixed the last patch.
Comment #10
segi commentedComment #11
xanoI checked the manual again, and 15 characters should be enough to contain the payment ID and the line item's number. Assuming there will be no more than 99 line items for a payment, this allows your solution to process a trillion payments through the same Ogone account, before things might go wrong.
Committed and pushed. Thanks!