View Issue Details

IDProjectCategoryView StatusLast Update
0005781OXID eShop (all versions)2. ----- eShop backend (admin) -----public2017-06-27 14:21
Reportermpienkny 
PrioritynormalSeverityfeatureReproducibilityalways
Status confirmedResolutionopen 
Product Version4.7.12 / 5.0.12 
Target VersionFixed in Version 
Summary0005781: oxNew should be used (e.g. admin/sysreq_main.php and admin/navigation.php)
DescriptionThere are some calls of "new oxSysRequirements()" instead of "oxNew('oxsysrequirements')" in these controllers, so we are unable so override the class !
Additional InformationWe 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)
TagsAdmin
ThemeAll
BrowserAll
PHP VersionNot defined
MySQL VersionNot defined

Activities

QA

2015-09-07 15:25

administrator   ~0011204

Last edited: 2015-09-07 17:21

View 2 revisions

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.

florian.auer

2015-09-07 15:46

developer   ~0011205

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!

FibreFoX

2015-09-07 16:06

reporter   ~0011207

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).

QA

2017-06-23 13:59

administrator   ~0012135

Changed status to acknowledged as the report itself is reproducible and a decision is needed how it will be handled in the future.