View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005477 | OXID eShop (all versions) | 4. ------ eShop Core ------- | public | 2013-10-18 10:35 | 2023-11-22 15:00 |
| Reporter | d3 | Assigned To | |||
| Priority | low | Severity | minor | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 4.7.0 / 5.0.0 revision 51243 | ||||
| Summary | 0005477: oxSuperCfg has to be refactored | ||||
| Description | Since 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); } | ||||
| Tags | No tags attached. | ||||
| Theme | Not defined | ||||
| Browser | All | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
