View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006120 | OXID eShop (all versions) | 4.05. Performance | public | 2015-04-14 15:03 | 2022-04-04 15:38 |
| Reporter | keywan.ghadami | ||||
| Priority | low | Severity | tweak | Reproducibility | always |
| Status | acknowledged | Resolution | open | ||
| Product Version | 4.9.3 / 5.2.3 | ||||
| Target Version | Fixed in Version | ||||
| Summary | 0006120: product list not cached when count is 0 | ||||
| Description | application/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. | ||||
| Tags | No tags attached. | ||||
| Theme | All | ||||
| Browser | All | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
|
|
performance_alist.zip (2,258 bytes) |
|
|
this finding pops up in other projects that are using Factfinder with "Category navigation" enabled, so i added my path-module to this issue. |
|
|
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. |
