View Issue Details

IDProjectCategoryView StatusLast Update
0005477OXID eShop (all versions)4. ------ eShop Core -------public2015-12-01 10:37
Reporterd3 
PrioritylowSeverityminorReproducibilityalways
Status acknowledgedResolutionopen 
Product Version4.7.0 / 5.0.0 revision 51243 
Target VersionFixed in Version 
Summary0005477: oxSuperCfg has to be refactored
DescriptionSince the oxid team introduced the oxRegistry class (+1),
there is no need for static properties in oxSuperCfg.

You can use oxSuperCfg::setConfig() to set any! variable as config, which is returned in oxSuperCfg::getConfig().

For example:
$controllerObject = oxNew('order');
$controllerObject->setConfig('foobar');

Please refactor this elementary class!

set type hints:
public function setConfig(oxConfig $oConfig ) {...}

only use oxRegistry:
public function getConfig() {
 // unit test part...

 return oxRegistry::getConfig();
}
public function setConfig(oxConfig $oConfig) {
 // unit test part...

 return oxRegistry::setConfig('oxConfig', $oConfig);
}
TagsNo tags attached.
ThemeNot defined
BrowserAll
PHP VersionNot defined
MySQL VersionNot defined

Activities

There are no notes attached to this issue.