Commit 4e444654 authored by Sebastien Robin's avatar Sebastien Robin

CopySupport: _duplicate should not call manage_afterPaste

Since last factorisation of code, _duplicate started to call manager_afterPaste
while this was not done before. Thus it prevented the possibility to duplicate
objects with preserved workflow history.

Add test to check _duplicate
parent 05fbafcf
...@@ -461,7 +461,7 @@ class CopyContainer: ...@@ -461,7 +461,7 @@ class CopyContainer:
lambda self, ob: True, lambda self, ob: True,
lambda self, ob: ob._getCopy(self), lambda self, ob: ob._getCopy(self),
1, # Take ownership. 1, # Take ownership.
True, not(duplicate),
), ),
( # 1: Move ( # 1: Move
'cb_isMoveable', 'cb_isMoveable',
......
...@@ -133,6 +133,24 @@ class TestCopySupport(ERP5TypeTestCase): ...@@ -133,6 +133,24 @@ class TestCopySupport(ERP5TypeTestCase):
['portal_catalog/uncatalogObjectList', uid_list, 'SQLQueue', False]) ['portal_catalog/uncatalogObjectList', uid_list, 'SQLQueue', False])
self.assertEqual(len(search_catalog(uid=uid_list)), 0) self.assertEqual(len(search_catalog(uid=uid_list)), 0)
def test_04_duplicate(self):
"""
duplicate is a low level copy method that should preserve workflow histories
"""
person = self.portal.person_module.newContent(portal_type='Person')
person.edit(first_name="John")
person.edit(last_name="Doe")
original_edit_workflow_history = person.workflow_history["edit_workflow"]
cp, = self.portal.person_module._duplicate(
self.portal.person_module.manage_copyObjects(ids=person.getId()))
new_person = self.portal.person_module[cp['new_id']]
self.assertNotEqual(new_person.getId(), person.getId())
new_edit_workflow_history = new_person.workflow_history["edit_workflow"]
self.assertEqual(3, len(original_edit_workflow_history))
self.assertEqual(3, len(new_edit_workflow_history))
new_person.edit(first_name="Henry")
self.assertEqual(3, len(original_edit_workflow_history))
self.assertEqual(4, len(new_edit_workflow_history))
def test_suite(): def test_suite():
suite = unittest.TestSuite() suite = unittest.TestSuite()
......
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