don't test self._reindex twice

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@18783 20353a03-c40f-0410-a6d1-a30d3c3de9de

product/ERP5Type/Accessor/Base.py

@@ -98,7 +98,7 @@ class Setter(Method):
                       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() ?
 

product/ERP5Type/Accessor/List.py

@@ -98,7 +98,7 @@ class DefaultSetter(Method):
                       DeprecationWarning)
         method = getattr(instance, '_' + self._id)
         method(*args, **kw)
-      if self._reindex: instance.reindexObject()
+        instance.reindexObject()
 
 class ListSetter(DefaultSetter):
 
@@ -122,8 +122,7 @@ class ListSetter(DefaultSetter):
                       DeprecationWarning)
         method = getattr(instance, '_' + self._id)
         method(*args, **kw)
-
-      if self._reindex: instance.reindexObject()
+        instance.reindexObject()
 
 Setter = ListSetter
 
@@ -196,7 +195,7 @@ class SetSetter(Method):
                       DeprecationWarning)
         method = getattr(instance, '_' + self._id)
         method(*args, **kw)
-      if self._reindex: instance.reindexObject()
+        instance.reindexObject()
 
 class DefaultGetter(Method):
     """