View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006505 | OXID eShop (all versions) | 1.01. Products (product, categories, manufacturer, promotions etc.) | public | 2016-09-14 15:01 | 2024-03-27 13:30 |
| Reporter | gerldental | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | confirmed | Resolution | open | ||
| Product Version | 4.9.9 / 5.2.9 | ||||
| Summary | 0006505: oxcmp_categories::_loadManufacturerTree - performance | ||||
| Description | in oxcmp_categories.php line 74 if ($myConfig->getConfigParam('bl_perfLoadManufacturerTree')) { // building Manufacturer tree $sActManufacturer = oxRegistry::getConfig()->getRequestParameter('mnid'); $this->_loadManufacturerTree($sActManufacturer); } Question: does it sense _always_ load manufacturer tree? It could be time expensive by many manufacturers... | ||||
| Steps To Reproduce | May be load manufacturer list only if REQUEST-variable mnid exists? like this: $sActManufacturer = oxRegistry::getConfig()->getRequestParameter('mnid'); if ($myConfig->getConfigParam('bl_perfLoadManufacturerTree') && $sActManufacturer) { $this->_loadManufacturerTree($sActManufacturer); } | ||||
| Tags | No tags attached. | ||||
| Theme | Not defined | ||||
| Browser | Not defined | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||