View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0004385 | OXID eShop (all versions) | 4.07. Source code, Test | public | 2012-08-14 15:42 | 2023-11-22 16:15 |
Reporter | leofonic | Assigned To | |||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 4.6.3 revision 47975 | ||||
Fixed in Version | 6.3.1 | ||||
Summary | 0004385: oxDeliveryList::getDeliveryList is not usable in modules | ||||
Description | I wanted to use oxDeliveryList::getDeliveryList in a module, but it's not possible, because: 1. $this->_aDeliveries is not resetted, so if you call the method multiple times, _aDeliveries add up. As it is only used in this method, i think it is not neccesary to use a property here. 2. Session Var 'sShipSet' is set inside the method, so it is not possible to call the method with different parameters than current selection, as this will mess up selected shipset. | ||||
Tags | Delivery | ||||
Theme | Not defined | ||||
Browser | All | ||||
PHP Version | Not defined | ||||
Database Version | Not defined | ||||
|
Hi, one option would be to set the setCollectFittingDeliveriesSets to true. This would give all fitting deliveries, not just first one found, but would also not add up to currently added ones and would not change session. |
|
If setCollectFittingDeliveriesSets is set to true, the method does not return fitting deliveries but only fitting deliverysets. |
|
I was working on a module to split orders and got stuck with this issue, too. Fixed it with writing a simple module that sets $this->_aDeliveries = array(); and calls the parent method. Thanks. |
|
fixed in 6.3.1 |