View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006568 | OXID eShop (all versions) | 1.03. Basket, checkout process | public | 2017-01-05 10:41 | 2017-01-16 11:38 |
| Reporter | JCT | Assigned To | |||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 4.10.2 / 5.3.2 | ||||
| Fixed in Version | 4.10.3 / 5.3.3 | ||||
| Summary | 0006568: [Order Confirmation] Form field 'ord_agb' is not checked to be '1' | ||||
| Description | The parameter 'ord_agb' is totally ignored by the checkout process. It is possible to place a order without passing the parameter 'ord_agb' , 'ord_agb' could also be '0'. In the shop configuration 'blConfirmAGB' is checked (user has to accept the terms and conditions). | ||||
| Tags | No tags attached. | ||||
| Theme | All | ||||
| Browser | All | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
|
|
The mistake is in the 'fcPayOneOrderView.php' which is shipped with in OXID EE (maybe also PE) on line 347: if(parent::_validateTermsAndConditions() === true); { Should be: if(parent::_validateTermsAndConditions() === true) { The semicolon ignores the condition so that the 'if-block' (which actually does not belong to the if) is executed always! |
|
|
This one is already fixed in the PayOne repository: https://github.com/PAYONE-GmbH/oxid-5/commit/630c1338d7354025a85879284c8bd7d59ecb734a I reckon it will be merged with the next patch release. |
|
|
This has been fixed by Payone in their module. The new Payone module version 2.1.5 will be shipped with the upcoming OXID eShop 5.3.3 patch release. |
|
|
Fixed in Payone module v2.1.5, will be released within the OXID eShop 5.3.3 patch release. |
