The right fix for checking processing state in flush.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@22773 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/CMFActivity/Activity/SQLQueue.py

@@ -376,11 +376,8 @@ class SQLQueue(RAMQueue, SQLBase):
       for m in activity_tool.getRegisteredMessageList(self):
         if object_path == m.object_path and (method_id is None or method_id == m.method_id):
           if invoke:
-            # First Validate (only if message is marked as new)
-            if line.processing_node == -1:
-              validate_value = m.validate(self, activity_tool)
-            else:
-              validate_value = VALID
+            # First Validate
+            validate_value = m.validate(self, activity_tool)
             if validate_value is VALID:
               activity_tool.invoke(m) # Try to invoke the message - what happens if invoke calls flushActivity ??
               if m.is_executed != MESSAGE_EXECUTED:                                                 # Make sure message could be invoked
@@ -402,8 +399,11 @@ class SQLQueue(RAMQueue, SQLBase):
         method_id = line.method_id
         m = self.loadMessage(line.message, uid = line.uid)
         if invoke:
-          # First Validate
-          validate_value = m.validate(self, activity_tool)
+          # First Validate (only if message is marked as new)
+          if line.processing_node == -1:
+            validate_value = m.validate(self, activity_tool)
+          else:
+            validate_value = VALID
           if validate_value is VALID:
             activity_tool.invoke(m) # Try to invoke the message - what happens if invoke calls flushActivity ??
             if m.is_executed != MESSAGE_EXECUTED:                                                 # Make sure message could be invoked