Commit b32e984b authored by Yoshinori Okuji's avatar Yoshinori Okuji

Add a comment on some possible performance improvements.

git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@37668 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent 4f19fe48
...@@ -792,6 +792,15 @@ class Delivery(XMLObject, ImmobilisationDelivery, CompositionMixin): ...@@ -792,6 +792,15 @@ class Delivery(XMLObject, ImmobilisationDelivery, CompositionMixin):
# once expanded, the applied_rule must be reindexed # once expanded, the applied_rule must be reindexed
# because some simulation_movement may change even # because some simulation_movement may change even
# if there are not edited (acquisition) # if there are not edited (acquisition)
#
# XXX yo thinks that this is excessive. First of all, we may
# need to reindex simulation movements but not applied rules
# here. So we should skip reindexing applied rules.
# In addition, the policy is "copy everything required to
# simulation movements", so acquisitions should not matter to
# indexing. The only exception is the simulation state.
# I think, if each simulation movement remembers the previous
# state, we can avoid unnecessary reindexing.
my_applied_rule.recursiveReindexObject(activate_kw=activate_kw) my_applied_rule.recursiveReindexObject(activate_kw=activate_kw)
else: else:
LOG("ERP5", PROBLEM, LOG("ERP5", PROBLEM,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment