From ec4589b214b36a0dab83f58f1d2a65d0da068e2a Mon Sep 17 00:00:00 2001
From: Jean-Paul Smets <jp@nexedi.com>
Date: Wed, 5 May 2010 08:57:45 +0000
Subject: [PATCH] Provide default value for private variables. Fixed missing
 self.

git-svn-id: https://svn.erp5.org/repos/public/erp5/sandbox/amount_generator@34998 20353a03-c40f-0410-a6d1-a30d3c3de9de
---
 product/ERP5/mixin/movement_generator.py | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/product/ERP5/mixin/movement_generator.py b/product/ERP5/mixin/movement_generator.py
index 0ed1a5780c..72dda39a98 100644
--- a/product/ERP5/mixin/movement_generator.py
+++ b/product/ERP5/mixin/movement_generator.py
@@ -38,6 +38,11 @@ class MovementGeneratorMixin:
     Deliveries ? Movements ? Items ?
     Does it depend by default on IAmountGeneratorMixin
   """
+  # Default values
+  _applied_rule = None
+  _rule = None
+  _trade_phase_list = None
+
   def __init__(self, applied_rule=None, rule=None, trade_phase_list=None):
     self._trade_phase_list = trade_phase_list # XXX
     self._applied_rule = applied_rule
@@ -47,7 +52,7 @@ class MovementGeneratorMixin:
       self._rule = rule # for rule specific stuff
 
   # Implementation of IMovementGenerator
-  def getGeneratedMovementList(movement_list=None, rounding=False):
+  def getGeneratedMovementList(self, movement_list=None, rounding=False):
     """
     Returns an IMovementList generated by a model applied to the context
 
-- 
2.30.9