Commit 97239903 authored by Jean-Paul Smets's avatar Jean-Paul Smets

Do not provide context to movement generator. Current mixin is questionable...

Do not provide context to movement generator. Current  mixin is questionable since it is not self contained. Added comment for this. Either make it inherit from dependent mixin, or design better.

git-svn-id: https://svn.erp5.org/repos/public/erp5/sandbox/amount_generator@34999 20353a03-c40f-0410-a6d1-a30d3c3de9de
parent ec4589b2
......@@ -67,8 +67,7 @@ class MovementCollectionUpdaterMixin:
# We suppose here that we have an IMovementCollection in hand
decision_movement_list = context.getMovementList()
prevision_movement_list = movement_generator.getGeneratedMovementList(
self._getMovementGeneratorContext(context),
movement_list=self._getMovementGeneratorMovementList(context), rounding=rounding)
movement_list=self._getMovementGeneratorMovementList(context), rounding=rounding) # XXX-JPS This mixin is not self-contained
# Get divergence testers
tester_list = self._getMatchingTesterList()
......
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