[Erp5-report] r27474 - in /erp5/trunk/products/CMFActivity: Activity/ dtml/ skins/activity/

nobody at svn.erp5.org nobody at svn.erp5.org
Tue Jun 9 15:14:37 CEST 2009


Author: jm
Date: Tue Jun  9 15:14:34 2009
New Revision: 27474

URL: http://svn.erp5.org?rev=27474&view=rev
Log:
* Remove links to cancel or invoke activities from manageActivities. It was too dangerous and requires too much work to fix.
* Fix display of args column. Use html_quote to display activity_kw, agrs & kw columns.
* Display all activities (processing SQLQueue activities were hidden).
* Add uid column.

Modified:
    erp5/trunk/products/CMFActivity/Activity/SQLBase.py
    erp5/trunk/products/CMFActivity/Activity/SQLDict.py
    erp5/trunk/products/CMFActivity/Activity/SQLQueue.py
    erp5/trunk/products/CMFActivity/dtml/manageActivities.dtml
    erp5/trunk/products/CMFActivity/skins/activity/SQLQueue_readMessageList.zsql

Modified: erp5/trunk/products/CMFActivity/Activity/SQLBase.py
URL: http://svn.erp5.org/erp5/trunk/products/CMFActivity/Activity/SQLBase.py?rev=27474&r1=27473&r2=27474&view=diff
==============================================================================
--- erp5/trunk/products/CMFActivity/Activity/SQLBase.py [utf8] (original)
+++ erp5/trunk/products/CMFActivity/Activity/SQLBase.py [utf8] Tue Jun  9 15:14:34 2009
@@ -44,6 +44,25 @@
     assert len(result) == 1
     assert len(result[0]) == 1
     return result[0][0]
+
+  def getMessageList(self, activity_tool, processing_node=None,
+                     include_processing=0, **kw):
+    # YO: reading all lines might cause a deadlock
+    readMessageList = getattr(activity_tool,
+                              self.__class__.__name__ + '_readMessageList',
+                              None)
+    if readMessageList is None:
+      return []
+    return [self.loadMessage(line.message,
+                             uid=line.uid,
+                             processing_node=line.processing_node,
+                             priority=line.priority,
+                             processing=line.processing)
+            for line in readMessageList(path=None,
+                                        method_id=None,
+                                        processing_node=processing_node,
+                                        to_date=None,
+                                        include_processing=include_processing)]
 
   def _getPriority(self, activity_tool, method, default):
     result = method()

Modified: erp5/trunk/products/CMFActivity/Activity/SQLDict.py
URL: http://svn.erp5.org/erp5/trunk/products/CMFActivity/Activity/SQLDict.py?rev=27474&r1=27473&r2=27474&view=diff
==============================================================================
--- erp5/trunk/products/CMFActivity/Activity/SQLDict.py [utf8] (original)
+++ erp5/trunk/products/CMFActivity/Activity/SQLDict.py [utf8] Tue Jun  9 15:14:34 2009
@@ -592,20 +592,7 @@
         if len(uid_list)>0:
           activity_tool.SQLDict_delMessage(uid = [x.uid for x in uid_list])
 
-  def getMessageList(self, activity_tool, processing_node=None, include_processing=0, **kw):
-    # YO: reading all lines might cause a deadlock
-    message_list = []
-    readMessageList = getattr(activity_tool, 'SQLDict_readMessageList', None)
-    if readMessageList is not None:
-      result = readMessageList(path=None, method_id=None, processing_node=None,
-                               to_date=None, include_processing=include_processing)
-      for line in result:
-        m = self.loadMessage(line.message, uid = line.uid)
-        m.processing_node = line.processing_node
-        m.priority = line.priority
-        m.processing = line.processing
-        message_list.append(m)
-    return message_list
+  getMessageList = SQLBase.getMessageList
 
   def dumpMessageList(self, activity_tool):
     # Dump all messages in the table.

Modified: erp5/trunk/products/CMFActivity/Activity/SQLQueue.py
URL: http://svn.erp5.org/erp5/trunk/products/CMFActivity/Activity/SQLQueue.py?rev=27474&r1=27473&r2=27474&view=diff
==============================================================================
--- erp5/trunk/products/CMFActivity/Activity/SQLQueue.py [utf8] (original)
+++ erp5/trunk/products/CMFActivity/Activity/SQLQueue.py [utf8] Tue Jun  9 15:14:34 2009
@@ -426,17 +426,7 @@
       if len(result):
         activity_tool.SQLQueue_delMessage(uid = [line.uid for line in result])
 
-  def getMessageList(self, activity_tool, processing_node=None,**kw):
-    message_list = []
-    readMessageList = getattr(activity_tool, 'SQLQueue_readMessageList', None)
-    if readMessageList is not None:
-      result = readMessageList(path=None, method_id=None, processing_node=None, to_date=None, include_processing=0)
-      for line in result:
-        m = self.loadMessage(line.message)
-        m.processing_node = line.processing_node
-        m.priority = line.priority
-        message_list.append(m)
-    return message_list
+  getMessageList = SQLBase.getMessageList
 
   def countMessage(self, activity_tool, tag=None, path=None,
                    method_id=None, message_uid=None, **kw):

Modified: erp5/trunk/products/CMFActivity/dtml/manageActivities.dtml
URL: http://svn.erp5.org/erp5/trunk/products/CMFActivity/dtml/manageActivities.dtml?rev=27474&r1=27473&r2=27474&view=diff
==============================================================================
--- erp5/trunk/products/CMFActivity/dtml/manageActivities.dtml [utf8] (original)
+++ erp5/trunk/products/CMFActivity/dtml/manageActivities.dtml [utf8] Tue Jun  9 15:14:34 2009
@@ -30,11 +30,9 @@
 <dtml-var manage_tabs>
 
 <h3>Queued Activities</h3>
-<p>Warning: Invoking or canceling activities will invoke/cancel all activities with both the same path and the same method_id.</p>
-<p>Warning: Invoking an activity will <b>not</b> run any dependency check.</p>
 <table cellspacing="0" cellpadding="2" border="1">
   <tr>
-    <th align="left" valign="top"></th>
+    <th align="left" valign="top">Uid</th>
     <th align="left" valign="top">Object</th>
     <th align="left" valign="top">Method Id</th>
     <th align="left" valign="top">Activity Kw</th>
@@ -47,16 +45,24 @@
   </tr>
   <dtml-in expr="getMessageList(include_processing=1)">
       <tr>
-        <td align="left" valign="top"><a href="manageCancel?object_path=<dtml-var "'/'.join(object_path)">&method_id=<dtml-var method_id>">Cancel</a> <a href="manageInvoke?object_path=<dtml-var "'/'.join(object_path)">&method_id=<dtml-var method_id>">Invoke</a></td>
+        <td align="left" valign="top"><dtml-var uid></td>
         <td align="left" valign="top"><a href="<dtml-var "'/'.join(object_path)">"><dtml-var "'/'.join(object_path)"></a></td>
         <td align="left" valign="top"><dtml-var method_id></td>
-        <td align="left" valign="top"><dtml-var "_.str(activity_kw)[1:-1]"></td>
-        <dtml-try>
-          <td align="left" valign="top"><dtml-var "', '.join(args)"></td>
-        <dtml-except>
-          <th align="left" valign="top">Not printable</th>
-        </dtml-try>
-        <td align="left" valign="top"><dtml-var "_.str(kw)[1:-1]"></td>
+        <td align="left" valign="top">
+          <dtml-if activity_kw><dtml-var expr="_.repr(activity_kw)[1:-1]" html_quote>
+          <dtml-else>&nbsp;
+          </dtml-if>
+        </td>
+        <td align="left" valign="top">
+          <dtml-if args><dtml-var expr="_.repr(_.list(args))[1:-1]" html_quote>
+          <dtml-else>&nbsp;
+          </dtml-if>
+        </td>
+        <td align="left" valign="top">
+          <dtml-if kw><dtml-var expr="_.repr(kw)[1:-1]" html_quote>
+          <dtml-else>&nbsp;
+          </dtml-if>
+        </td>
         <td align="left" valign="top"><dtml-var processing_node></td>
         <td align="left" valign="top"><dtml-var priority></td>
         <td align="left" valign="top">

Modified: erp5/trunk/products/CMFActivity/skins/activity/SQLQueue_readMessageList.zsql
URL: http://svn.erp5.org/erp5/trunk/products/CMFActivity/skins/activity/SQLQueue_readMessageList.zsql?rev=27474&r1=27473&r2=27474&view=diff
==============================================================================
--- erp5/trunk/products/CMFActivity/skins/activity/SQLQueue_readMessageList.zsql [utf8] (original)
+++ erp5/trunk/products/CMFActivity/skins/activity/SQLQueue_readMessageList.zsql [utf8] Tue Jun  9 15:14:34 2009
@@ -19,7 +19,7 @@
     message_queue
 WHERE
 1 = 1
-AND processing = 0
+<dtml-if expr="not include_processing"> AND processing = 0 </dtml-if>
 <dtml-if expr="processing_node is not None"> AND processing_node = <dtml-sqlvar processing_node type="int"> </dtml-if>
 <dtml-if expr="path is not None">AND path = <dtml-sqlvar path type="string"></dtml-if>
 <dtml-if expr="method_id is not None">AND method_id = <dtml-sqlvar method_id type="string"></dtml-if>




More information about the Erp5-report mailing list