View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004671 | OXID eShop (all versions) | 4.07. Source code, Test | public | 2012-10-29 15:20 | 2012-12-18 15:51 |
| Reporter | tjungcl | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 4.6.5 revision 49955 | ||||
| Target Version | 4.7.2 / 5.0.2 revision 53018 | Fixed in Version | 4.7.3 / 5.0.3 revision 54408 | ||
| Summary | 0004671: delivery address md5 function not extendable | ||||
| Description | the new feature that checks the delivery address for modifications is not extendable without overwriting the functions totally. So, if i want to add a new field to the md5 checksum, i have to overwrite two functions without calling parent::... You should move the checksum generation to the classes oxaddress and oxuser. The string, that is checksummed should be constructed in private functions that return only the concatinated field values, so that is what users can extend, when they want more fields in the checksum. oxorder.php and order.php then would call those md5s. Btw: the name of the function and hidden parameter is misleading. Its not the deliveryaddress checksum, it is the checksum of both billing and delivery address. | ||||
| Tags | No tags attached. | ||||
| Theme | Both | ||||
| Browser | All | ||||
| PHP Version | any | ||||
| Database Version | any | ||||
