Code: Select all
$_POST
Code: Select all
$this->request->post
In this file, replace all instances of:
with:
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Other issues have been discovered on the Fraud Labs Pro extension v3.x (and possibly on v2.x releases as well). Ensure the order_id VARCHAR length is set to: 16 rather than 11 from the fraud labs pro table on the database for those who uses transaction IDs rather than order IDs.
Second, in catalog/model/extension/fraud/fraudlabspro.php file,
find:
replace with:
Otherwise, you'll encounter issues with the transaction when an order gets rejected.
Second, in catalog/model/extension/fraud/fraudlabspro.php file,
find:
Code: Select all
fraudlabspro_reject_status_id
Code: Select all
fraud_fraudlabspro_reject_status_id
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Issue now addressed on GitHub: https://github.com/opencart/opencart/is ... -container
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Based on my understanding to the FraudLabs Pro's API, only the Transaction ID generated by FraudLabs Pro Screen Order API can be used for the FraudLabs Pro Feedback API. Thus, the order_id length in fraudlabspro table of database will not be affected.
That still doesn't cover the already addressed issue on GitHub regarding the order_id type field which should be an INT rather than VARCHAR since the order_id is being sanitized as an integer in the addOrderHistory in anyhow.
Dedication and passion goes to those who are able to push and merge a project.
Regards,
Straightlight
Programmer / Opencart Tester
Who is online
Users browsing this forum: No registered users and 68 guests
