View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004325 | OXID eShop (all versions) | 2.6. Administer orders | public | 2012-08-07 19:13 | 2012-12-10 13:17 |
| Reporter | d3 | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | won't fix | ||
| Product Version | 4.6.3 revision 47975 | ||||
| Summary | 0004325: wrong function change | ||||
| Description | myorder.php change the original method getOrderArticles() and create following problem: For example the function is used in first time maybe with storno = true, you can't change the articlelist later in the object to get all articles include stornoarticles. The original method in oxorder make it right: if ( $blExcludeCanceled ) { return $this->_getArticles( true ); } ..... | ||||
| Tags | PDF Invoice | ||||
| Theme | Both | ||||
| Browser | All | ||||
| PHP Version | any | ||||
| Database Version | any | ||||
|
|
General problem is: With "if ( $this->_oArticles == null) ..." you can't change the articlelist with another "storno status". possible solution: if ( $this->_oArticles == null || $this->blListStornoStatus != $blStorno) { .... $this->blListStornoStatus = $blStorno; } |
|
|
Hi, we checked this place and everything seems to work like expected. If you do want to change functionality, you can just rewrite that place and add needed functionality. If we missed something, please reopen this bug and give some more details on why this needs to be changed. |
