#2447: WidgetsList should not remove declared widgets from class scope
-------------------------+--------------------------------------------------
Reporter: xaka | Owner: chrisz
Type: enhancement | Status: assigned
Priority: normal | Milestone: 1.x
Component: TurboGears | Version: 1.1 HEAD
Severity: normal | Resolution:
Keywords: |
-------------------------+--------------------------------------------------
Comment (by xaka):
In TG's code you have few places where you subclassing from base types and
define yourself attributes without fear to override base attributes. Why?
Because you know what are you doing and for what. TG users too know their
intention and what they do.
If i make the class (WidgetsList is the class, yea?) and declare some
class-scope attributes, perhaps i'll use them in dot-notation
(MyClass.myattr). Also i know that WidgetsList is based on list and i will
not use "bad" names for attributes.
About what i'm? :) As Python's way says: "Keep it simple and elegant".
Python's core guys don't worry about inheriting from base classes and
possible overriding, they are don't say you "Hey, man, what are you doing?
You overrided our attribute!"
TG too must not worry about such things. If i defined something it should
remains here for future use. If i override something and error occured -
that's my bad, not TG.
That's all about overriding, subclassing and defining. IMHO.
What about "...widgets appended or extend later to the widget list..."?
Keep it simle. User just added item to list, nothing more. In that case he
just used it as list...list with additional attirbutes.
--
Ticket URL: <http://trac.turbogears.org/ticket/2447#comment:3>
TurboGears <http://www.turbogears.org/>
TurboGears front-to-back web development
--
You received this message because you are subscribed to the Google
Groups "TurboGears Tickets" group.
This group is read-only. No posting by normal members allowed.
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/turbogears-tickets?hl=en?hl=en
