View Issue Details

IDProjectCategoryView StatusLast Update
0006505OXID eShop (all versions)1.01. Products (product, categories, manufacturer, promotions etc.)public2016-09-20 14:21
Reportergerldental 
PrioritynormalSeverityminorReproducibilityalways
Status acknowledgedResolutionopen 
Product Version4.9.9 / 5.2.9 
Target VersionFixed in Version 
Summary0006505: oxcmp_categories::_loadManufacturerTree - performance
Descriptionin 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 ReproduceMay 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);
}
TagsNo tags attached.
ThemeNot defined
BrowserNot defined
PHP VersionNot defined
MySQL VersionNot defined

Activities

There are no notes attached to this issue.