From 458b5366b2c1aaf403303fa3d9672723f4d42464 Mon Sep 17 00:00:00 2001
From: Julien Muchembled <jm@nexedi.com>
Date: Fri, 29 Jan 2010 13:58:07 +0000
Subject: [PATCH] typos + code factoring

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@32088 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/CMFActivity/Activity/SQLDict.py      | 17 +++++----
 product/CMFActivity/Activity/SQLQueue.py     |  4 +-
 product/CMFActivity/tests/testCMFActivity.py | 39 +++++++-------------
 3 files changed, 25 insertions(+), 35 deletions(-)

diff --git a/product/CMFActivity/Activity/SQLDict.py b/product/CMFActivity/Activity/SQLDict.py
index ce87a2ba6b..293ab78b90 100644
--- a/product/CMFActivity/Activity/SQLDict.py
+++ b/product/CMFActivity/Activity/SQLDict.py
@@ -260,8 +260,7 @@ class SQLDict(RAMDict, SQLBase):
     now_date = self.getNow(activity_tool)
     message_list = []
     def append(line, message):
-      uid = line.uid
-      message_list.append((uid, message, line.priority))
+      message_list.append((line.uid, message, line.priority))
     count = 0
     group_method_id = None
     try:
@@ -269,11 +268,13 @@ class SQLDict(RAMDict, SQLBase):
       uid_to_duplicate_uid_list_dict = {}
       if len(result) > 0:
         line = result[0]
-        m = self.loadMessage(line.message, uid=line.uid)
+        uid = line.uid
+        m = self.loadMessage(line.message, uid=uid)
         append(line, m)
         group_method_id = line.group_method_id
-        activity_tool.SQLDict_processMessage(uid=[line.uid])
-        uid_to_duplicate_uid_list_dict.setdefault(line.uid, []).extend(getDuplicateMessageUidList(line))
+        activity_tool.SQLDict_processMessage(uid=[uid])
+        uid_to_duplicate_uid_list_dict.setdefault(uid, []) \
+          .extend(getDuplicateMessageUidList(line))
         if group_method_id not in (None, '', '\0'):
           # Count the number of objects to prevent too many objects.
           count += len(m.getObjectList(activity_tool))
@@ -448,13 +449,13 @@ class SQLDict(RAMDict, SQLBase):
                                     'uid': message_uid_priority_list[0][0]})
       setActivityRuntimeValue('processing_node', processing_node)
       # Commit right before executing messages.
-      # As MySQL transaction do no start exactly at the same time as ZODB
+      # As MySQL transaction does not start exactly at the same time as ZODB
       # transactions but a bit later, messages available might be called
       # on objects which are not available - or available in an old
       # version - to ZODB connector.
-      # So all connectors must be commited now that we have selected
+      # So all connectors must be committed now that we have selected
       # everything needed from MySQL to get a fresh view of ZODB objects.
-      get_transaction().commit() 
+      get_transaction().commit()
       # Try to invoke
       try:
         method(*args)
diff --git a/product/CMFActivity/Activity/SQLQueue.py b/product/CMFActivity/Activity/SQLQueue.py
index 730007c407..a46a24a9f6 100644
--- a/product/CMFActivity/Activity/SQLQueue.py
+++ b/product/CMFActivity/Activity/SQLQueue.py
@@ -294,11 +294,11 @@ class SQLQueue(RAMQueue, SQLBase):
       processing_stop_time = time() + 30 # Stop processing after more than 10 seconds were spent
       processed_message_uid_list = []
       # Commit right before executing messages.
-      # As MySQL transaction do no start exactly at the same time as ZODB
+      # As MySQL transaction does not start exactly at the same time as ZODB
       # transactions but a bit later, messages available might be called
       # on objects which are not available - or available in an old
       # version - to ZODB connector.
-      # So all connectors must be commited now that we have selected
+      # So all connectors must be committed now that we have selected
       # everything needed from MySQL to get a fresh view of ZODB objects.
       get_transaction().commit()
       for value in message_uid_priority_list:
diff --git a/product/CMFActivity/tests/testCMFActivity.py b/product/CMFActivity/tests/testCMFActivity.py
index 73eabab2cc..6bf134bc01 100644
--- a/product/CMFActivity/tests/testCMFActivity.py
+++ b/product/CMFActivity/tests/testCMFActivity.py
@@ -56,6 +56,20 @@ try:
 except ImportError:
   pass
 
+class CommitFailed(Exception):
+  pass
+
+def registerFailingTransactionManager(*args, **kw):
+  from Shared.DC.ZRDB.TM import TM
+  class dummy_tm(TM):
+    def tpc_vote(self, *ignored):
+      raise CommitFailed
+    def _finish(self):
+      pass
+    def _abort(self):
+      pass
+  dummy_tm()._register()
+
 class TestCMFActivity(ERP5TypeTestCase):
 
   run_all_test = 1
@@ -2525,18 +2539,6 @@ class TestCMFActivity(ERP5TypeTestCase):
     self.tic()
     activity_tool = self.getActivityTool()
     from Shared.DC.ZRDB.TM import TM
-    class dummy_tm(TM):
-      def tpc_vote(self, *ignored):
-        raise Exception, 'vote always raises'
-
-      def _finish(self):
-        pass
-
-      def _abort(self):
-        pass
-    dummy_tm_instance = dummy_tm()
-    def registerFailingTransactionManager(self, *args, **kw):
-      dummy_tm_instance._register()
     try:
       Organisation.registerFailingTransactionManager = registerFailingTransactionManager
       obj = self.getPortal().organisation_module.newContent(portal_type='Organisation')
@@ -2581,19 +2583,6 @@ class TestCMFActivity(ERP5TypeTestCase):
     get_transaction().commit()
     self.tic()
     activity_tool = self.getActivityTool()
-    from Shared.DC.ZRDB.TM import TM
-    class dummy_tm(TM):
-      def tpc_vote(self, *ignored):
-        raise Exception, 'vote always raises'
-
-      def _finish(self):
-        pass
-
-      def _abort(self):
-        pass
-    dummy_tm_instance = dummy_tm()
-    def registerFailingTransactionManager(self, *args, **kw):
-      dummy_tm_instance._register()
     try:
       Organisation.registerFailingTransactionManager = registerFailingTransactionManager
       obj = self.getPortal().organisation_module.newContent(portal_type='Organisation')
-- 
2.30.9