[Erp5-report] r43959 arnaud.fontaine - /erp5/trunk/products/ERP5Type/Core/CategoryProperty.py

[nobody at svn.erp5.org]

Author: arnaud.fontaine
Date: Fri Mar  4 04:55:39 2011
New Revision: 43959

URL: http://svn.erp5.org?rev=43959&view=rev
Log:
Only add the category ID if it is not already in _categories, which
may happen when getting the categories with acquisition as done in
test_getAcquiredCategoryList for example. This fixes testERP5Category.

Modified:
    erp5/trunk/products/ERP5Type/Core/CategoryProperty.py

Modified: erp5/trunk/products/ERP5Type/Core/CategoryProperty.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Core/CategoryProperty.py?rev=43959&r1=43958&r2=43959&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Core/CategoryProperty.py [utf8] (original)
+++ erp5/trunk/products/ERP5Type/Core/CategoryProperty.py [utf8] Fri Mar  4 04:55:39 2011
@@ -229,7 +229,10 @@ class CategoryProperty(IdAsReferenceMixi
         accessor = Alias.Reindex(accessor_name[1:], accessor_name)
         accessor_holder.registerAccessor(accessor, write_permission)
 
-    accessor_holder._categories.append(category_id)
+    # Only add the category ID if it is not already in _categories,
+    # which may happen when getting the categories with acquisition
+    if category_id not in accessor_holder._categories:
+      accessor_holder._categories.append(category_id)
 
   security.declareProtected(Permissions.AccessContentsInformation,
                             'applyOnAccessorHolder')