Commit 0e571301 authored by Sebastien Robin's avatar Sebastien Robin

avoid commit of persistent mapping when not needed

parent 36ec80da
......@@ -86,7 +86,6 @@ class TaskDistributionTool(BaseTool):
-> (test_result_path, revision) or None if already completed
"""
LOG('createTestResult', 0, (name, revision, test_title, project_title))
self._p_changed = 1
portal = self.getPortalObject()
if test_title is None:
test_title = name
......@@ -143,6 +142,7 @@ class TaskDistributionTool(BaseTool):
elif reference:
last_revision = last_revision, reference
if len(line_dict) == 0 and len(test_name_list):
self._p_changed = 1
createTestResultLineList(test_result, test_name_list, line_dict)
return test_result_path, last_revision
if last_state == 'stopped':
......@@ -169,6 +169,7 @@ class TaskDistributionTool(BaseTool):
test_result.updateLocalRolesOnSecurityGroups() # XXX
test_result_path = test_result.getRelativeUrl()
self.test_result_dict[test_title] = test_result_path, line_dict
self._p_changed = 1
test_result.start()
createTestResultLineList(test_result, test_name_list, line_dict)
createNode(test_result, node_title)
......
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