Commit 2f869800 authored by Romain Courteaud's avatar Romain Courteaud

Use accessors in variation property API.

Add some parameters to variation category API.


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@4089 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent 1b75bd2d
...@@ -205,18 +205,18 @@ class Amount(Base, Variated): ...@@ -205,18 +205,18 @@ class Amount(Base, Variated):
security.declareProtected(Permissions.AccessContentsInformation, \ security.declareProtected(Permissions.AccessContentsInformation, \
'getVariationRangeCategoryList') 'getVariationRangeCategoryList')
def getVariationRangeCategoryList(self, base_category_list=(), base=1): def getVariationRangeCategoryList(self, base_category_list=(), base=1, **kw):
""" """
Returns possible variation category values for the Returns possible variation category values for the
order line according to the default resource. order line according to the default resource.
""" """
return [x[1] for x in self.getVariationRangeCategoryItemList( return [x[1] for x in self.getVariationRangeCategoryItemList(
base_category_list=base_category_list, base_category_list=base_category_list,
base=base)] base=base, **kw)]
security.declareProtected(Permissions.AccessContentsInformation, security.declareProtected(Permissions.AccessContentsInformation,
'getVariationRangeBaseCategoryList') 'getVariationRangeBaseCategoryList')
def getVariationRangeBaseCategoryList(self): def getVariationRangeBaseCategoryList(self, omit_option_base_category=0):
""" """
Returns possible variations base categories for this amount ie. Returns possible variations base categories for this amount ie.
the variation base category of the resource (not the the variation base category of the resource (not the
...@@ -229,7 +229,8 @@ class Amount(Base, Variated): ...@@ -229,7 +229,8 @@ class Amount(Base, Variated):
""" """
resource = self.getDefaultResourceValue() resource = self.getDefaultResourceValue()
if resource is not None: if resource is not None:
result = resource.getVariationBaseCategoryList() result = resource.getVariationBaseCategoryList(
omit_option_base_category=omit_option_base_category)
else: else:
result = self.portal_categories.getBaseCategoryList() result = self.portal_categories.getBaseCategoryList()
return result return result
...@@ -252,9 +253,8 @@ class Amount(Base, Variated): ...@@ -252,9 +253,8 @@ class Amount(Base, Variated):
if resource is not None: if resource is not None:
variation_list = resource.getVariationPropertyList() variation_list = resource.getVariationPropertyList()
for variation_property in variation_list: for variation_property in variation_list:
# XXX Accessor not used
property_dict[variation_property] = \ property_dict[variation_property] = \
getattr(self, variation_property, None) self.getProperty(variation_property)
return property_dict return property_dict
security.declareProtected(Permissions.ModifyPortalContent, security.declareProtected(Permissions.ModifyPortalContent,
...@@ -277,8 +277,7 @@ class Amount(Base, Variated): ...@@ -277,8 +277,7 @@ class Amount(Base, Variated):
raise KeyError, "Can not set the property variation '%s'" % \ raise KeyError, "Can not set the property variation '%s'" % \
property_id property_id
else: else:
# XXX Accessor not used self.setProperty(property_id, property_value)
setattr(self, property_id, property_value)
security.declareProtected(Permissions.AccessContentsInformation, security.declareProtected(Permissions.AccessContentsInformation,
'getQuantityUnitRangeItemList') 'getQuantityUnitRangeItemList')
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment