Commit 2e22ce4e authored by Jérome Perrin's avatar Jérome Perrin

support a list as local_sort_id


git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@17919 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent 23d1b47b
...@@ -250,7 +250,9 @@ class Category(Folder): ...@@ -250,7 +250,9 @@ class Category(Folder):
local_sort_id - When using the default preorder traversal, sort local_sort_id - When using the default preorder traversal, sort
objects of the same depth by comparing their objects of the same depth by comparing their
'local_sort_id' property. 'local_sort_id' property. local_sort_id can be a
list, in this case properties are compared in the
same order than this list.
Renderer parameters are also supported here. Renderer parameters are also supported here.
""" """
...@@ -263,6 +265,15 @@ class Category(Folder): ...@@ -263,6 +265,15 @@ class Category(Folder):
child_value_list = self.objectValues(self.allowed_types) child_value_list = self.objectValues(self.allowed_types)
if local_sort_id: if local_sort_id:
if isinstance(local_sort_id, (tuple, list)):
def sort_method(a, b):
for sort_id in local_sort_id:
diff = cmp(a.getProperty(sort_id, 0), b.getProperty(sort_id, 0))
if diff != 0:
return diff
return 0
local_sort_method = sort_method
else:
local_sort_method = lambda a, b: cmp(a.getProperty(local_sort_id, 0), local_sort_method = lambda a, b: cmp(a.getProperty(local_sort_id, 0),
b.getProperty(local_sort_id, 0)) b.getProperty(local_sort_id, 0))
if local_sort_method: if local_sort_method:
...@@ -272,8 +283,8 @@ class Category(Folder): ...@@ -272,8 +283,8 @@ class Category(Folder):
if recursive: if recursive:
for c in child_value_list: for c in child_value_list:
# Do not global pass sort parameters intentionally, because sorting # Do not pass sort_on / sort_order parameters intentionally, because
# needs to be done only at the end of recursive calls. # sorting needs to be done only at the end of recursive calls.
value_list.extend(c.getCategoryChildValueList(recursive=1, value_list.extend(c.getCategoryChildValueList(recursive=1,
is_self_excluded=0, is_self_excluded=0,
include_if_child=include_if_child, include_if_child=include_if_child,
...@@ -758,7 +769,9 @@ class BaseCategory(Category): ...@@ -758,7 +769,9 @@ class BaseCategory(Category):
local_sort_id - When using the default preorder traversal, sort local_sort_id - When using the default preorder traversal, sort
objects of the same depth by comparing their objects of the same depth by comparing their
'local_sort_id' property. 'local_sort_id' property. local_sort_id can be a
list, in this case properties are compared in the
same order than this list.
Renderer parameters are also supported here. Renderer parameters are also supported here.
...@@ -770,6 +783,15 @@ class BaseCategory(Category): ...@@ -770,6 +783,15 @@ class BaseCategory(Category):
child_value_list = self.objectValues(self.allowed_types) child_value_list = self.objectValues(self.allowed_types)
if local_sort_id: if local_sort_id:
if isinstance(local_sort_id, (tuple, list)):
def sort_method(a, b):
for sort_id in local_sort_id:
diff = cmp(a.getProperty(sort_id, 0), b.getProperty(sort_id, 0))
if diff != 0:
return diff
return 0
local_sort_method = sort_method
else:
local_sort_method = lambda a, b: cmp(a.getProperty(local_sort_id, 0), local_sort_method = lambda a, b: cmp(a.getProperty(local_sort_id, 0),
b.getProperty(local_sort_id, 0)) b.getProperty(local_sort_id, 0))
if local_sort_method: if local_sort_method:
......
...@@ -899,6 +899,21 @@ class TestCMFCategory(ERP5TypeTestCase): ...@@ -899,6 +899,21 @@ class TestCMFCategory(ERP5TypeTestCase):
self.assertEquals(list(c1.getCategoryChildValueList( self.assertEquals(list(c1.getCategoryChildValueList(
local_sort_id='int_index')), [c12, c11, c111]) local_sort_id='int_index')), [c12, c11, c111])
# local_sort_id can be a list, in this case document are sorted with the
# first sort_id in the list, then the second, and so on.
# When we use those category properties:
# int_index title
# c1 C
# c2 B
# c3 1 A
c1.setIntIndex(None)
c2.setIntIndex(None)
# and we sort on int_index then title, we should have this order:
self.assertEquals(list(bc.getCategoryChildValueList(
local_sort_id=['int_index', 'title'])),
[c2, c1, c12, c11, c111, c3])
def test_25_getCategoryChildItemList_base_parameter(self): def test_25_getCategoryChildItemList_base_parameter(self):
pc = self.getCategoriesTool() pc = self.getCategoriesTool()
......
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