Commit 3d209aa1 authored by Arnaud Fontaine's avatar Arnaud Fontaine

Make sure that 'erp5' version is always defined when setting version priority list on ERP5Site.

parent db4b73b1
...@@ -455,15 +455,23 @@ class ERP5Site(FolderMixIn, CMFSite, CacheCookieMixin): ...@@ -455,15 +455,23 @@ class ERP5Site(FolderMixIn, CMFSite, CacheCookieMixin):
security.declareProtected(Permissions.ModifyPortalContent, security.declareProtected(Permissions.ModifyPortalContent,
'setVersionPriorityList' ) 'setVersionPriorityList' )
def setVersionPriorityList(self, value): def setVersionPriorityList(self, version_priority_tuple):
""" """
XXX-arnau: must be written through an interaction workflow when ERP5Site XXX-arnau: must be written through an interaction workflow when ERP5Site
will become a real ERP5 object... will become a real ERP5 object...
""" """
if not isinstance(value, tuple): if not isinstance(version_priority_tuple, tuple):
value = tuple(value) version_priority_tuple = tuple(version_priority_tuple)
self._version_priority_list = value # erp5 version must always be present, thus add it at the end if it's not
# already there
for version_priority in version_priority_tuple:
if version_priority.split('|')[0].strip() == 'erp5':
break
else:
version_priority_tuple = version_priority_tuple + ('erp5 | 0.0',)
self._version_priority_list = version_priority_tuple
# Reset cached value of getVersionPriorityNameList() if present # Reset cached value of getVersionPriorityNameList() if present
try: try:
......
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