Commit 3cf4d7f7 authored by Ayush Tiwari's avatar Ayush Tiwari Committed by Romain Courteaud

erp5_standard_property: Add select_variable property for Standard Property object.

This change solves the error which one gets while trying to access properties
Form for an ERP5 object having one or more of its property of type 'selection'
or 'multiple selection'.
For example: If you try to access property form for any portal_type objects,
you'll get an error : "'select_variable' is not defined". This is because of the
absence of the property 'select_variable' for the property of type multiple selection,
which in case of portal_type object is 'Property Sheet List'.

This is helpful for ERP5 views/forms which displays selection or multiple selection
type object.
parent 7567d7d6
......@@ -133,6 +133,10 @@ class StandardProperty(IdAsReferenceMixin('_property'), XMLObject):
'translation_domain',
'string')
getSelectVariable = Base.Getter('getSelectVariable',
'select_variable',
'string')
@classmethod
def _asPropertyMap(cls, property_dict):
"""
......@@ -158,6 +162,13 @@ class StandardProperty(IdAsReferenceMixin('_property'), XMLObject):
property_dict['base_id'] = property_dict['id']
property_dict['id'] = property_dict['id'] + '_list'
# Maintain consistency while displaying properties form.
# Addition of select_variable property is required for 'selection'
# and 'multiple selection' property type as while rendering properties
# dtml file, it asks for 'select_variable' property
if property_dict['type'] in ['selection', 'multiple selection']:
property_dict['select_variable'] = property_dict.pop('select_variable')
return property_dict
@staticmethod
......@@ -609,7 +620,8 @@ class StandardProperty(IdAsReferenceMixin('_property'), XMLObject):
'read_permission': self.getReadPermission(),
'write_permission': self.getWritePermission(),
'translatable': self.getTranslatable(),
'translation_domain': self.getTranslationDomain()}
'translation_domain': self.getTranslationDomain(),
'select_variable': self.getSelectVariable()}
security.declareProtected(Permissions.ModifyPortalContent,
'applyOnAccessorHolder')
......
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