[Erp5-report] r35588 nicolas.dumazet - /erp5/trunk/products/ERP5Type/CopySupport.py

[nobody at svn.erp5.org]

Author: nicolas.dumazet
Date: Tue May 25 04:11:23 2010
New Revision: 35588

URL: http://svn.erp5.org?rev=35588&view=rev
Log:
small boolean cleanup

Modified:
    erp5/trunk/products/ERP5Type/CopySupport.py

Modified: erp5/trunk/products/ERP5Type/CopySupport.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/CopySupport.py?rev=35588&r1=35587&r2=35588&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/CopySupport.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/CopySupport.py [utf8] Tue May 25 04:11:23 2010
@@ -112,7 +112,7 @@
           self._updateInternalRelatedContent(object=subobject,
                                              path_item_list=path_item_list,
                                              new_id=new_id)
-      changed = 0
+      changed = False
       category_list = object.getCategoryList()
       path_len = len(path_item_list)
       for position in xrange(len(category_list)):
@@ -120,8 +120,8 @@
           if category_name[1:path_len + 1] == path_item_list: # XXX Should be possible to do this in a cleaner way
               category_name[path_len] = new_id
               category_list[position] = '/'.join(category_name)
-              changed = 1
-      if changed != 0:
+              changed = True
+      if changed:
           object.setCategoryList(category_list)
 
   def _recursiveSetActivityAfterTag(self, obj, activate_kw=None):