View Issue Details

IDProjectCategoryView StatusLast Update
0003576OXID eShop (all versions)4.05. Performancepublic2012-12-10 13:29
Reporterwanis 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version 
Target VersionFixed in Version4.5.8 revision 42471 
Summary0003576: oxorder::getOrderArticles() doesnt stores list in object, and recalculates every time it is called
Descriptionarticle list is loaded in $oArticles varoiable, but it is not assigned to $this->_oArticles
Additional Informationto fix it, before return assign to $this->_oArticles so it will look like this:
    public function getOrderArticles( $blExcludeCanceled = false )
    {
        if ( $this->_oArticles === null ) {
...
            $oArticles = oxNew( 'oxlist' );
...
            $this->_oArticles = $oArticles;
            return $oArticles;
        }
        return $this->_oArticles;
TagsPerformance
ThemeBoth
BrowserAll
PHP Versionany
Database Versionany

Activities

Linas Kukulskis

2012-02-13 15:37

reporter   ~0005724

added assign

Linas Kukulskis

2012-02-14 14:43

reporter   ~0005732

added caching for all articles, but if needed exclude goes to db