[Erp5-report] r18783 - in /erp5/trunk/products/ERP5Type/Accessor: Base.py List.py

[nobody at svn.erp5.org]

Author: jerome
Date: Thu Jan 17 18:38:58 2008
New Revision: 18783

URL: http://svn.erp5.org?rev=18783&view=rev
Log:
don't test self._reindex twice

Modified:
    erp5/trunk/products/ERP5Type/Accessor/Base.py
    erp5/trunk/products/ERP5Type/Accessor/List.py

Modified: erp5/trunk/products/ERP5Type/Accessor/Base.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Accessor/Base.py?rev=18783&r1=18782&r2=18783&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Accessor/Base.py (original)
+++ erp5/trunk/products/ERP5Type/Accessor/Base.py Thu Jan 17 18:38:58 2008
@@ -98,7 +98,7 @@
                       DeprecationWarning)
         method = getattr(instance, '_' + self._id)
         method(*args, **kw)
-      if self._reindex: instance.reindexObject() # XXX Should the Setter check
+        instance.reindexObject() # XXX Should the Setter check
                                                  # if the property value has changed
                                                  # to decide reindexing, like edit() ?
 

Modified: erp5/trunk/products/ERP5Type/Accessor/List.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5Type/Accessor/List.py?rev=18783&r1=18782&r2=18783&view=diff
==============================================================================
--- erp5/trunk/products/ERP5Type/Accessor/List.py (original)
+++ erp5/trunk/products/ERP5Type/Accessor/List.py Thu Jan 17 18:38:58 2008
@@ -98,7 +98,7 @@
                       DeprecationWarning)
         method = getattr(instance, '_' + self._id)
         method(*args, **kw)
-      if self._reindex: instance.reindexObject()
+        instance.reindexObject()
 
 class ListSetter(DefaultSetter):
 
@@ -122,8 +122,7 @@
                       DeprecationWarning)
         method = getattr(instance, '_' + self._id)
         method(*args, **kw)
-
-      if self._reindex: instance.reindexObject()
+        instance.reindexObject()
 
 Setter = ListSetter
 
@@ -196,7 +195,7 @@
                       DeprecationWarning)
         method = getattr(instance, '_' + self._id)
         method(*args, **kw)
-      if self._reindex: instance.reindexObject()
+        instance.reindexObject()
 
 class DefaultGetter(Method):
     """