View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005781 | OXID eShop (all versions) | 2. ----- eShop backend (admin) ----- | public | 2014-06-02 15:14 | 2023-02-13 09:39 |
| Reporter | mpienkny | Assigned To | |||
| Priority | normal | Severity | feature | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 4.7.12 / 5.0.12 | ||||
| Fixed in Version | 6.4.0 | ||||
| Summary | 0005781: oxNew should be used (e.g. admin/sysreq_main.php and admin/navigation.php) | ||||
| Description | There are some calls of "new oxSysRequirements()" instead of "oxNew('oxsysrequirements')" in these controllers, so we are unable so override the class ! | ||||
| Additional Information | We need to override these model (and many more), because our development shops are behind a proxy server and so loading of shop admin always ends in a coffee break (until a timeout is reached, because the connection to any external url (e.g. oxid web services for version information) is not possible) | ||||
| Tags | Admin | ||||
| Theme | All | ||||
| Browser | All | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
|
|
Due to software architecture limitations not all classes can be extended with a module. Oxsysrequirements is already documented as being not overloadable: http://wiki.oxidforge.org/Tutorials/List_of_not_overloadable_classes#core.2F Therefore this issue is classified as a feature request. |
|
|
Reminder sent to: mpienkny Can you check if this helps you out here: http://planet.oxidforge.org/2015/08/overloading-non-overloadable-classes-dont.html But again: This is NOT RECOMMENDED! |
|
|
Why not adding a hook or dummy-class, which CAN be overridden/extended? As far as I understood @mpienkny, this is a feature-request for being able to do some own checks (not changing existing ones). |
|
|
Changed status to acknowledged as the report itself is reproducible and a decision is needed how it will be handled in the future. |
|
|
Fixed in CE component version 6.4.0 |