[Zope-Checkins] CVS: Zope/lib/python/OFS - IOrderSupport.py:1.1.2.1 OrderSupport.py:1.1.2.1 OrderedFolder.py:1.1.2.1

[Florent Guillaume]

In article <200304251307.h3PD7M402006@cvs.baymountain.com> you write:
> class OrderSupport:
>     """ Ordered container mixin class - This is an extension to the regular
>     ObjectManager. It saves the objects in order and lets you change the order
>     of the contained objects. This is particular helpful, if the order does
>     not depend on object attributes, but is totally user-specific.
>     """
> 
>     __implements__ = IOrderedContainer
>     security = ClassSecurityInfo()
> 
>     has_order_support = 1
>     _key = 'position'
>     _reverse = 0

Please, use attributes with names a bit less generic than "_key" and
"_reverse" for a mixin class. For instance "_order_support_key" and
"_order_support_reverse".

Florent


-- 
Florent Guillaume, Nuxeo (Paris, France)
+33 1 40 33 79 87  http://nuxeo.com  mailto:fg@nuxeo.com