View Issue Details

IDProjectCategoryView StatusLast Update
0006120OXID eShop (all versions)4.05. Performancepublic2024-01-31 14:19
Reporterkeywan.ghadami Assigned To 
PrioritylowSeveritytweakReproducibilityalways
Status confirmedResolutionopen 
Product Version4.9.3 / 5.2.3 
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.
Attached Files
performance_alist.zip (2,258 bytes)
ThemeAll
BrowserAll
PHP VersionNot defined
Database VersionNot defined

Activities

keywan.ghadami

2015-06-01 11:39

reporter   ~0011000

Last edited: 2015-06-01 12:15

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

reporter   ~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.