View Issue Details

IDProjectCategoryView StatusLast Update
0006120OXID eShop (all versions)4.05. Performancepublic2016-11-03 12:55
Reporterkeywan.ghadami 
PrioritylowSeveritytweakReproducibilityalways
Status acknowledgedResolutionopen 
Product Version4.9.3 / 5.2.3 
Target VersionFixed in Version 
Summary0006120: product list not cached when count is 0
Descriptionapplication/controllers/alist.php:825 (EE v 5.2.2)

the if statement in:

if (count($aArticleList)) {
     $this->_aArticleList = $aArticleList;
}

Does block the alist controller from caching the result if the result is an empty list. This causes the subjacent layers (db or in my case factfinder) to be fetched more then once.

That is only the case if the result list is empty, which is a very rare case in productive environment but very common in testing environments.

As there might be a reason for this if statement i am interested in the reason.
TagsNo tags attached.
ThemeAll
BrowserAll
PHP Versionany
MySQL Versionany

Activities

keywan.ghadami

2015-06-01 11:37

developer  

performance_alist.zip (2,258 bytes)

keywan.ghadami

2015-06-01 11:39

developer   ~0011000

Last edited: 2015-06-01 12:15

View 2 revisions

this finding pops up in other projects that are using Factfinder with "Category navigation" enabled, so i added my path-module to this issue.

keywan.ghadami

2016-11-03 12:55

developer   ~0011857

any news on if my patch could do any harm?
btw i need it again in an other project so we should raise priority, seams to work in production without any negative effects so far.