[Erp5-report] r36512 jm - /erp5/trunk/products/ERP5/Tool/TemplateTool.py

[nobody at svn.erp5.org]

Author: jm
Date: Tue Jun 22 15:52:47 2010
New Revision: 36512

URL: http://svn.erp5.org?rev=36512&view=rev
Log:
Add a comment about a limitation of [36156]

Modified:
    erp5/trunk/products/ERP5/Tool/TemplateTool.py

Modified: erp5/trunk/products/ERP5/Tool/TemplateTool.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/Tool/TemplateTool.py?rev=36512&r1=36511&r2=36512&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/Tool/TemplateTool.py [utf8] (original)
+++ erp5/trunk/products/ERP5/Tool/TemplateTool.py [utf8] Tue Jun 22 15:52:47 2010
@@ -1114,8 +1114,12 @@ class TemplateTool (BaseTool):
         state = listbox_line.object_state
         removed = state.startswith('Removed')
         if removed:
-          moved = installed_dict.get(listbox_line.object_id, '')
-          log('%s: %s%s' % (state, item, moved and ' (moved to %s)' % moved))
+          # The following condition could not be used to automatically decide
+          # if an item must be kept or not. For example, this would not work
+          # for items installed by PortalTypeWorkflowChainTemplateItem.
+          maybe_moved = installed_dict.get(listbox_line.object_id, '')
+          log('%s: %s%s' % (state, item,
+            maybe_moved and ' (moved to %s ?)' % maybe_moved))
         else:
           installed_dict[item] = bt_title
         # if a bt5 item is removed we may still want to keep it