Hi Frank,

> i would suggest to label oxshopcontrol "can be overloaded but oxwidgetcontrol 
> extends it" like alist/details. 
Already updated, thanks for your suggestion!

> Also i suggest to refactor oxwidgetcontrol into oxshopcontrol so it can be 
> fully extended again. Are there any objections against this?
oxshopcontrol belongs to the core of the system and shouldn't be overwritten. 
Anyways, there's no business logic in it. 
On the other hand I see your point, of course. We are presently wringing our 
brains about solutions on how to make pieces like that more flexible. One can 
always have a quick and working (but dirty) solution or do it the very right 
way, you see? :-)

________________________________________
From: [email protected] 
[[email protected]] on behalf of Frank Zunderer IT 
Beratung [[email protected]]
Sent: Tuesday, October 13, 2015 12:43 PM
To: [email protected]
Subject: Re: [oxid-dev-general] New blog post on planet: Overloading    
non-overloadable classes - don't!

Hello all,

i would suggest to label oxshopcontrol "can be overloaded but oxwidgetcontrol 
extends it" like alist/details. Also i suggest to refactor oxwidgetcontrol into 
oxshopcontrol so it can be fully extended again. Are there any objections 
against this?

Regards,
Frank Zunderer



-----Ursprüngliche Nachricht-----
Von: [email protected] 
[mailto:[email protected]] Im Auftrag von Marco 
Steinhaeuser
Gesendet: Mittwoch, 26. August 2015 14:11
An: [email protected]
Betreff: [oxid-dev-general] New blog post on planet: Overloading 
non-overloadable classes - don't!

Hey folks,

did you use oxShopControl in modules yet? Well, it was moved to 
non-overloadable classes recently. Please read this blog post to find a 
workaround. All others: hands off these classes 
http://planet.oxidforge.org/2015/08/overloading-non-overloadable-classes-dont.html

Cheers
Marco
_______________________________________________
dev-general mailing list
[email protected]
http://dir.gmane.org/gmane.comp.php.oxid.general
_______________________________________________
dev-general mailing list
[email protected]
http://dir.gmane.org/gmane.comp.php.oxid.general
_______________________________________________
dev-general mailing list
[email protected]
http://dir.gmane.org/gmane.comp.php.oxid.general

Reply via email to