Merge lp:~numerigraphe/openobject-addons/trunk-unmutable-defaults into lp:openobject-addons

[Numérigraphe]

This branch should fix the last few methods that still have mutable defaults.
Lionel Sausin

[Anto (abourguignon)]

Hello,

Thanks for this proposal. Would you mind resolve the conflicts with trunk before we merge those together ?

Regards,
Anto.

[Anto (abourguignon)]

Not necessary anymore, I did it in this branch: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-unmutable-defaults-abo

[Numérigraphe (numerigraphe)]

OK I'm marking this as merged then. Thanks!

[Anto (abourguignon)]

Thanks to you, keep up the good work !

=== modified file 'account/account.py'
--- account/account.py	2012-02-28 14:08:16 +0000
+++ account/account.py	2012-03-05 18:43:21 +0000
@@ -591,12 +591,15 @@
             res.append((record['id'], name))
         return res
 
-    def copy(self, cr, uid, id, default={}, context=None, done_list=[], local=False):
+    def copy(self, cr, uid, id, None, context=None, done_list=None, local=False):
+        if default is None:
+            default = {}
+        else:
+            default = default.copy()
+        if done_list is None:
+            done_list = []
         account = self.browse(cr, uid, id, context=context)
         new_child_ids = []
-        if not default:
-            default = {}
-        default = default.copy()
         default['code'] = (account['code'] or '') + '(copy)'
         if not local:
             done_list = []
@@ -765,11 +768,14 @@
         (_check_currency, 'Configuration error! The currency chosen should be shared by the default accounts too.', ['currency','default_debit_account_id','default_credit_account_id']),
     ]
 
-    def copy(self, cr, uid, id, default={}, context=None, done_list=[], local=False):
+    def copy(self, cr, uid, id, default=None, context=None, done_list=None, local=False):
+        if default is None:
+            default = {}
+        else:
+            default = default.copy()
+        if done_list is None:
+            done_list = []
         journal = self.browse(cr, uid, id, context=context)
-        if not default:
-            default = {}
-        default = default.copy()
         default['code'] = (journal['code'] or '') + '(copy)'
         default['name'] = (journal['name'] or '') + '(copy)'
         default['sequence_id'] = False
@@ -1161,7 +1167,7 @@
         'end_journal_period_id':fields.many2one('account.journal.period','End of Year Entries Journal', readonly=True),
     }
 
-    def copy(self, cr, uid, id, default={}, context=None):
+    def copy(self, cr, uid, id, default=None, context=None):
         default.update({
             'period_ids': [],
             'end_journal_period_id': False
@@ -1420,9 +1426,15 @@
             result = super(account_move, self).create(cr, uid, vals, context)
         return result
 
-    def copy(self, cr, uid, id, default={}, context=None):
+    def copy(self, cr, uid, id, default=None, context=None):
+        if context is None:
+            default = {}
+        else:
+            default = default.copy()
         if context is None:
             context = {}
+        else:
+            context = context.copy()
         default.update({
             'state':'draft',
             'name':'/',
@@ -2239,7 +2251,9 @@
     _defaults = {
         'legend': lambda self, cr, uid, context:_('You can specify year, month and date in the name of the model using the following labels:\n\n%(year)s: To Specify Year \n%(month)s: To Specify Month \n%(date)s: Current Date\n\ne.g. My model on %(date)s'),
     }
-    def generate(self, cr, uid, ids, datas={}, context=None):
+    def generate(self, cr, uid, ids, datas=None, context=None):
+        if datas is None:
+            datas = {}
         move_ids = []
         entry = {}
         account_move_obj = self.pool.get('account.move')
@@ -3193,7 +3207,7 @@
                     property_obj.create(cr, uid, vals, context=context)
         return True
 
-    def _install_template(self, cr, uid, template_id, company_id, code_digits=None, obj_wizard=None, acc_ref={}, taxes_ref={}, tax_code_ref={}, context=None):
+    def _install_template(self, cr, uid, template_id, company_id, code_digits=None, obj_wizard=None, acc_ref=None, taxes_ref=None, tax_code_ref=None, context=None):
         '''
         This function recursively loads the template objects and create the real objects from them.
 
@@ -3211,6 +3225,12 @@
             * a last identical containing the mapping of tax code templates and tax codes
         :rtype: tuple(dict, dict, dict)
         '''
+        if acc_ref is None:
+            acc_ref = {}
+        if taxes_ref is None:
+            taxes_ref = {}
+        if tax_code_ref is None:
+            tax_code_ref = {}
         template = self.pool.get('account.chart.template').browse(cr, uid, template_id, context=context)
         if template.parent_id:
             tmp1, tmp2, tmp3 = self._install_template(cr, uid, template.parent_id.id, company_id, code_digits=code_digits, acc_ref=acc_ref, taxes_ref=taxes_ref, tax_code_ref=tax_code_ref, context=context)
@@ -3223,7 +3243,7 @@
         tax_code_ref.update(tmp3)
         return acc_ref, taxes_ref, tax_code_ref
 
-    def _load_template(self, cr, uid, template_id, company_id, code_digits=None, obj_wizard=None, account_ref={}, taxes_ref={}, tax_code_ref={}, context=None):
+    def _load_template(self, cr, uid, template_id, company_id, code_digits=None, obj_wizard=None, account_ref=None, taxes_ref=None, tax_code_ref=None, context=None):
         '''
         This function generates all the objects from the templates
 
@@ -3241,6 +3261,12 @@
             * a last identical containing the mapping of tax code templates and tax codes
         :rtype: tuple(dict, dict, dict)
         '''
+        if account_ref is None:
+            account_ref = {}
+        if taxes_ref is None:
+            taxes_ref = {}
+        if tax_code_ref is None:
+            tax_code_ref = {}
         template = self.pool.get('account.chart.template').browse(cr, uid, template_id, context=context)
         obj_tax_code_template = self.pool.get('account.tax.code.template')
         obj_acc_tax = self.pool.get('account.tax')
 
=== modified file 'account/account_bank.py'
--- account/account_bank.py	2012-02-01 12:50:42 +0000
+++ account/account_bank.py	2012-03-05 18:43:21 +0000
@@ -29,12 +29,12 @@
         'currency_id': fields.related('journal_id', 'currency', type="many2one", relation='res.currency', readonly=True,
             string="Currency", help="Currency of the related account journal."),
     }
-    def create(self, cr, uid, data, context={}):
+    def create(self, cr, uid, data, context=None):
         result = super(bank, self).create(cr, uid, data, context=context)
         self.post_write(cr, uid, [result], context=context)
         return result
 
-    def write(self, cr, uid, ids, data, context={}):
+    def write(self, cr, uid, ids, data, context=None):
         result = super(bank, self).write(cr, uid, ids, data, context=context)
         self.post_write(cr, uid, ids, context=context)
         return result
@@ -43,7 +43,7 @@
         "Return the name to use when creating a bank journal"
         return (bank.bank_name or '') + ' ' + bank.acc_number
 
-    def post_write(self, cr, uid, ids, context={}):
+    def post_write(self, cr, uid, ids, context=None):
         if isinstance(ids, (int, long)):
           ids = [ids]
 
 
=== modified file 'account/account_move_line.py'
--- account/account_move_line.py	2012-02-13 18:07:41 +0000
+++ account/account_move_line.py	2012-03-05 18:43:21 +0000
@@ -1097,7 +1097,7 @@
                                 'has been confirmed!') % res[2])
         return res
 
-    def _remove_move_reconcile(self, cr, uid, move_ids=[], context=None):
+    def _remove_move_reconcile(self, cr, uid, move_ids=None, context=None):
         # Function remove move rencocile ids related with moves
         obj_move_line = self.pool.get('account.move.line')
         obj_move_rec = self.pool.get('account.move.reconcile')
 
=== modified file 'account/project/wizard/account_analytic_chart.py'
--- account/project/wizard/account_analytic_chart.py	2011-01-14 00:11:01 +0000
+++ account/project/wizard/account_analytic_chart.py	2012-03-05 18:43:21 +0000
@@ -47,4 +47,4 @@
         return result
 
 account_analytic_chart()
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/project/wizard/account_analytic_inverted_balance_report.py'
--- account/project/wizard/account_analytic_inverted_balance_report.py	2011-04-29 08:49:48 +0000
+++ account/project/wizard/account_analytic_inverted_balance_report.py	2012-03-05 18:43:21 +0000
@@ -52,4 +52,4 @@
             }
 
 account_analytic_inverted_balance()
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/project/wizard/project_account_analytic_line.py'
--- account/project/wizard/project_account_analytic_line.py	2011-05-02 12:16:02 +0000
+++ account/project/wizard/project_account_analytic_line.py	2012-03-05 18:43:21 +0000
@@ -55,4 +55,4 @@
 
 project_account_analytic_line()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/report/account_balance.py'
--- account/report/account_balance.py	2011-06-08 10:04:27 +0000
+++ account/report/account_balance.py	2012-03-05 18:43:21 +0000
@@ -68,7 +68,7 @@
             return self.pool.get('account.account').browse(self.cr, self.uid, data['form']['id']).company_id.name
         return super(account_balance ,self)._get_account(data)
 
-    def lines(self, form, ids=[], done=None):#, level=1):
+    def lines(self, form, ids=None, done=None):
         def _process_child(accounts, disp_acc, parent):
                 account_rec = [acct for acct in accounts if acct['id']==parent][0]
                 currency_obj = self.pool.get('res.currency')
 
=== modified file 'account/report/account_central_journal.py'
--- account/report/account_central_journal.py	2011-01-14 00:11:01 +0000
+++ account/report/account_central_journal.py	2012-03-05 18:43:21 +0000
@@ -105,4 +105,4 @@
 
 report_sxw.report_sxw('report.account.central.journal', 'account.journal.period', 'addons/account/report/account_central_journal.rml', parser=journal_print, header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/report/account_general_journal.py'
--- account/report/account_general_journal.py	2011-01-14 00:11:01 +0000
+++ account/report/account_general_journal.py	2012-03-05 18:43:21 +0000
@@ -158,4 +158,4 @@
 
 report_sxw.report_sxw('report.account.general.journal', 'account.journal.period', 'addons/account/report/general_journal.rml', parser=journal_print, header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/report/account_report.py'
--- account/report/account_report.py	2011-12-21 22:15:04 +0000
+++ account/report/account_report.py	2012-03-05 18:43:21 +0000
@@ -283,4 +283,4 @@
             )""")
 report_account_sales()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/report/account_tax_report.py'
--- account/report/account_tax_report.py	2012-02-03 16:26:19 +0000
+++ account/report/account_tax_report.py	2012-03-05 18:43:21 +0000
@@ -160,7 +160,7 @@
             i+=1
         return res
 
-    def _get_codes(self, based_on, company_id, parent=False, level=0, period_list=[], context=None):
+    def _get_codes(self, based_on, company_id, parent=False, level=0, period_list=None, context=None):
         obj_tc = self.pool.get('account.tax.code')
         ids = obj_tc.search(self.cr, self.uid, [('parent_id','=',parent),('company_id','=',company_id)], order='sequence', context=context)
 
@@ -171,7 +171,11 @@
             res += self._get_codes(based_on, company_id, code.id, level+1, context=context)
         return res
 
-    def _add_codes(self, based_on, account_list=[], period_list=[], context=None):
+    def _add_codes(self, based_on, account_list=None, period_list=None, context=None):
+        if account_list is None:
+            account_list = []
+        if period_list is None:
+            period_list = []
         res = []
         obj_tc = self.pool.get('account.tax.code')
         for account in account_list:
 
=== modified file 'account/res_currency.py'
--- account/res_currency.py	2011-01-14 00:11:01 +0000
+++ account/res_currency.py	2012-03-05 18:43:21 +0000
@@ -44,4 +44,4 @@
 
 res_currency_account()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_invoice_state.py'
--- account/wizard/account_invoice_state.py	2011-01-14 00:11:01 +0000
+++ account/wizard/account_invoice_state.py	2012-03-05 18:43:21 +0000
@@ -71,4 +71,4 @@
 
 account_invoice_cancel()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_journal_select.py'
--- account/wizard/account_journal_select.py	2011-01-14 00:11:01 +0000
+++ account/wizard/account_journal_select.py	2012-03-05 18:43:21 +0000
@@ -47,4 +47,4 @@
 
 account_journal_select()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_move_bank_reconcile.py'
--- account/wizard/account_move_bank_reconcile.py	2011-02-15 12:13:39 +0000
+++ account/wizard/account_move_bank_reconcile.py	2012-03-05 18:43:21 +0000
@@ -61,4 +61,4 @@
 
 account_move_bank_reconcile()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_move_journal.py'
--- account/wizard/account_move_journal.py	2011-11-14 11:34:05 +0000
+++ account/wizard/account_move_journal.py	2012-03-05 18:43:21 +0000
@@ -38,7 +38,7 @@
     _defaults = {
         'target_move': 'all'
     }
-    def _get_period(self, cr, uid, context={}):
+    def _get_period(self, cr, uid, context=None):
         """
         Return  default account period value
         """
 
=== modified file 'account/wizard/account_move_line_reconcile_select.py'
--- account/wizard/account_move_line_reconcile_select.py	2011-02-15 12:13:39 +0000
+++ account/wizard/account_move_line_reconcile_select.py	2012-03-05 18:43:21 +0000
@@ -52,4 +52,4 @@
 
 account_move_line_reconcile_select()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_move_line_select.py'
--- account/wizard/account_move_line_select.py	2011-01-14 00:11:01 +0000
+++ account/wizard/account_move_line_select.py	2012-03-05 18:43:21 +0000
@@ -69,4 +69,4 @@
 
 account_move_line_select()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_move_line_unreconcile_select.py'
--- account/wizard/account_move_line_unreconcile_select.py	2011-02-15 12:13:39 +0000
+++ account/wizard/account_move_line_unreconcile_select.py	2012-03-05 18:43:21 +0000
@@ -41,4 +41,4 @@
 
 account_move_line_unreconcile_select()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_period_close.py'
--- account/wizard/account_period_close.py	2011-12-13 04:41:48 +0000
+++ account/wizard/account_period_close.py	2012-03-05 18:43:21 +0000
@@ -60,4 +60,4 @@
 
 account_period_close()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_reconcile.py'
--- account/wizard/account_reconcile.py	2011-11-03 09:56:12 +0000
+++ account/wizard/account_reconcile.py	2012-03-05 18:43:21 +0000
@@ -173,4 +173,4 @@
 
 account_move_line_reconcile_writeoff()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_reconcile_partner_process.py'
--- account/wizard/account_reconcile_partner_process.py	2011-01-14 00:11:01 +0000
+++ account/wizard/account_reconcile_partner_process.py	2012-03-05 18:43:21 +0000
@@ -100,4 +100,4 @@
 
 account_partner_reconcile_process()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_state_open.py'
--- account/wizard/account_state_open.py	2011-01-27 06:32:26 +0000
+++ account/wizard/account_state_open.py	2012-03-05 18:43:21 +0000
@@ -41,4 +41,4 @@
 
 account_state_open()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_subscription_generate.py'
--- account/wizard/account_subscription_generate.py	2011-04-29 08:49:48 +0000
+++ account/wizard/account_subscription_generate.py	2012-03-05 18:43:21 +0000
@@ -50,4 +50,4 @@
 
 account_subscription_generate()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account/wizard/account_unreconcile.py'
--- account/wizard/account_unreconcile.py	2011-01-14 00:11:01 +0000
+++ account/wizard/account_unreconcile.py	2012-03-05 18:43:21 +0000
@@ -50,4 +50,4 @@
 
 account_unreconcile_reconcile()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_analytic_plans/report/crossovered_analytic.py'
--- account_analytic_plans/report/crossovered_analytic.py	2011-01-14 00:11:01 +0000
+++ account_analytic_plans/report/crossovered_analytic.py	2012-03-05 18:43:21 +0000
@@ -34,7 +34,7 @@
         })
         self.base_amount = 0.00
 
-    def find_children(self,ref_ids):
+    def find_children(self, ref_ids):
         to_return_ids = []
         final_list = []
         parent_list = []
@@ -53,7 +53,7 @@
         final_list.extend(set_list)
         return final_list #to_return_ids[0]
 
-    def set_account(self,cats):
+    def set_account(self, cats):
         lst = []
         category = self.pool.get('account.analytic.account').read(self.cr, self.uid, cats)
         for cat in category:
@@ -62,7 +62,7 @@
                 lst.extend(self.set_account(cat['child_ids']))
         return lst
 
-    def _ref_lines(self,form):
+    def _ref_lines(self, form):
         result = []
         res = {}
         acc_pool = self.pool.get('account.analytic.account')
@@ -113,7 +113,9 @@
         result.append(res)
         return result
 
-    def _lines(self, form, ids={}):
+    def _lines(self, form, ids=None):
+        if ids is None:
+            ids = {}
         if not ids:
             ids = self.ids
 
 
=== modified file 'account_analytic_plans/wizard/analytic_plan_create_model.py'
--- account_analytic_plans/wizard/analytic_plan_create_model.py	2011-01-14 00:11:01 +0000
+++ account_analytic_plans/wizard/analytic_plan_create_model.py	2012-03-05 18:43:21 +0000
@@ -57,4 +57,4 @@
 
 analytic_plan_create_model()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_anglo_saxon/__init__.py'
--- account_anglo_saxon/__init__.py	2011-01-14 00:11:01 +0000
+++ account_anglo_saxon/__init__.py	2012-03-05 18:43:21 +0000
@@ -23,4 +23,4 @@
 import purchase
 import invoice
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== removed file 'account_anglo_saxon/sale.py'
--- account_anglo_saxon/sale.py	2011-01-14 00:11:01 +0000
+++ account_anglo_saxon/sale.py	1970-01-01 00:00:00 +0000
@@ -1,43 +0,0 @@
-# -*- encoding: utf-8 -*-
-##############################################################################
-#    
-#    OpenERP, Open Source Management Solution
-#    Copyright (C) 2004-2009 Tiny SPRL (<http://tiny.be>).
-#
-#    This program is free software: you can redistribute it and/or modify
-#    it under the terms of the GNU Affero General Public License as
-#    published by the Free Software Foundation, either version 3 of the
-#    License, or (at your option) any later version.
-#
-#    This program is distributed in the hope that it will be useful,
-#    but WITHOUT ANY WARRANTY; without even the implied warranty of
-#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-#    GNU Affero General Public License for more details.
-#
-#    You should have received a copy of the GNU Affero General Public License
-#    along with this program.  If not, see <http://www.gnu.org/licenses/>.     
-#
-##############################################################################
-
-from osv import fields, osv
-
-#class sale_order_line(osv.osv):
-#    _name = 'sale.order.line'
-#    _description = 'Sale Order line'
-#    _inherit = 'sale.order.line'
-#
-#    def invoice_line_create(self, cr, uid, ids, context={}):
-#        line_ids = super('sale_order_line',self).invoice_line_create(cr, uid, ids, context)
-#        invoice_line_obj = self.pool.get('account.invoice.line')
-#        for line in invoice_line_obj.browse(cr, uid, line_ids):
-#            if line.product_id:
-#                    a =  line.product_id.product_tmpl_id.property_stock_account_output and line.product_id.product_tmpl_id.property_stock_account_output.id
-#                    if not a:
-#                        a = line.product_id.categ_id.property_stock_account_output_categ and line.product_id.categ_id.property_stock_account_output_categ.id
-#                    if a:
-#                        a = self.pool.get('account.fiscal.position').map_account(cr, uid, fpos, a)
-#                        invoice_line_obj.write(cr, uid, line.id, {'account_id':a})
-#
-#sale_order_line()
-
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_asset/account_asset.py'
--- account_asset/account_asset.py	2012-01-31 13:36:57 +0000
+++ account_asset/account_asset.py	2012-03-05 18:43:21 +0000
@@ -75,7 +75,7 @@
     _name = 'account.asset.asset'
     _description = 'Asset'
 
-    def _get_period(self, cr, uid, context={}):
+    def _get_period(self, cr, uid, context=None):
         periods = self.pool.get('account.period').find(cr, uid)
         if periods:
             return periods[0]
@@ -176,7 +176,9 @@
                 year = depreciation_date.year
         return True
 
-    def validate(self, cr, uid, ids, context={}):
+    def validate(self, cr, uid, ids, context=None):
+        if context is None:
+            context = {}
         return self.write(cr, uid, ids, {
             'state':'open'
         }, context)
@@ -304,7 +306,7 @@
         default.update({'depreciation_line_ids': [], 'state': 'draft'})
         return super(account_asset_asset, self).copy(cr, uid, id, default, context=context)
 
-    def _compute_entries(self, cr, uid, ids, period_id, context={}):
+    def _compute_entries(self, cr, uid, ids, period_id, context=None):
         result = []
         period_obj = self.pool.get('account.period')
         depreciation_obj = self.pool.get('account.asset.depreciation.line')
 
=== modified file 'account_budget/report/analytic_account_budget_report.py'
--- account_budget/report/analytic_account_budget_report.py	2011-01-14 00:11:01 +0000
+++ account_budget/report/analytic_account_budget_report.py	2012-03-05 18:43:21 +0000
@@ -35,7 +35,9 @@
         })
         self.context = context
 
-    def funct(self, object, form, ids={}, done=None, level=1):
+    def funct(self, object, form, ids=None, done=None, level=1):
+        if ids is None:
+            ids = {}
         if not ids:
             ids = self.ids
         if not done:
@@ -153,7 +155,7 @@
                 tot['perc'] = float(tot['prac'] / tot['theo']) * 100
         return result
 
-    def funct_total(self,form):
+    def funct_total(self, form):
         result = []
         res = {}
         res = {
@@ -167,4 +169,4 @@
 
 report_sxw.report_sxw('report.account.analytic.account.budget', 'account.analytic.account', 'addons/account_budget/report/analytic_account_budget_report.rml',parser=analytic_account_budget_report,header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_budget/report/budget_report.py'
--- account_budget/report/budget_report.py	2011-04-15 06:09:42 +0000
+++ account_budget/report/budget_report.py	2012-03-05 18:43:21 +0000
@@ -34,7 +34,9 @@
         })
         self.context = context
 
-    def funct(self, object, form, ids={}, done=None, level=1):
+    def funct(self, object, form, ids=None, done=None, level=1):
+        if ids is None:
+            ids = {}
         if not ids:
             ids = self.ids
         if not done:
 
=== modified file 'account_budget/report/crossovered_budget_report.py'
--- account_budget/report/crossovered_budget_report.py	2011-04-15 06:09:42 +0000
+++ account_budget/report/crossovered_budget_report.py	2012-03-05 18:43:21 +0000
@@ -37,7 +37,9 @@
         })
         self.context = context
 
-    def funct(self, object, form, ids={}, done=None, level=1):
+    def funct(self, object, form, ids=None, done=None, level=1):
+        if ids is None:
+            ids = {}
         if not ids:
             ids = self.ids
         if not done:
 
=== modified file 'account_budget/wizard/account_budget_analytic.py'
--- account_budget/wizard/account_budget_analytic.py	2011-04-29 08:49:48 +0000
+++ account_budget/wizard/account_budget_analytic.py	2012-03-05 18:43:21 +0000
@@ -52,4 +52,4 @@
 
 account_budget_analytic()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_budget/wizard/account_budget_crossovered_report.py'
--- account_budget/wizard/account_budget_crossovered_report.py	2011-04-29 08:49:48 +0000
+++ account_budget/wizard/account_budget_crossovered_report.py	2012-03-05 18:43:21 +0000
@@ -53,4 +53,4 @@
 
 account_budget_crossvered_report()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_budget/wizard/account_budget_report.py'
--- account_budget/wizard/account_budget_report.py	2011-04-29 08:49:48 +0000
+++ account_budget/wizard/account_budget_report.py	2012-03-05 18:43:21 +0000
@@ -54,4 +54,4 @@
 
 account_budget_report()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_coda/account_coda.py'
--- account_coda/account_coda.py	2012-02-13 18:07:41 +0000
+++ account_coda/account_coda.py	2012-03-05 18:43:21 +0000
@@ -217,7 +217,9 @@
     _name = 'coda.bank.statement' 
     _description = 'CODA Bank Statement'    
     
-    def _default_journal_id(self, cr, uid, context={}):
+    def _default_journal_id(self, cr, uid, context=None):
+        if context is None:
+            context = {}
         if context.get('journal_id', False):
             return context['journal_id']
         return False
@@ -233,7 +235,7 @@
             res[r] = round(res[r], 2)
         return res
 
-    def _get_period(self, cr, uid, context={}):
+    def _get_period(self, cr, uid, context=None):
         periods = self.pool.get('account.period').find(cr, uid)
         if periods:
             return periods[0]
 
=== modified file 'account_followup/__init__.py'
--- account_followup/__init__.py	2011-01-14 00:11:01 +0000
+++ account_followup/__init__.py	2012-03-05 18:43:21 +0000
@@ -23,4 +23,4 @@
 import wizard
 import report
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_followup/wizard/__init__.py'
--- account_followup/wizard/__init__.py	2011-01-14 00:11:01 +0000
+++ account_followup/wizard/__init__.py	2012-03-05 18:43:21 +0000
@@ -21,4 +21,4 @@
 
 import account_followup_print
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_followup/wizard/account_followup_print.py'
--- account_followup/wizard/account_followup_print.py	2012-02-13 16:03:19 +0000
+++ account_followup/wizard/account_followup_print.py	2012-03-05 18:43:21 +0000
@@ -209,7 +209,7 @@
                 to_update[str(id)]= {'level': fups[followup_line_id][1], 'partner_id': stat_line_id}
         return {'partner_ids': partner_list, 'to_update': to_update}
 
-    def do_mail(self ,cr, uid, ids, context=None):
+    def do_mail(self, cr, uid, ids, context=None):
         mod_obj = self.pool.get('ir.model.data')
         move_obj = self.pool.get('account.move.line')
         user_obj = self.pool.get('res.users')
 
=== modified file 'account_invoice_layout/wizard/account_invoice_special_message.py'
--- account_invoice_layout/wizard/account_invoice_special_message.py	2011-04-29 08:49:48 +0000
+++ account_invoice_layout/wizard/account_invoice_special_message.py	2012-03-05 18:43:21 +0000
@@ -47,4 +47,4 @@
 
 account_invoice_special_msg()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_payment/__init__.py'
--- account_payment/__init__.py	2011-01-14 00:11:01 +0000
+++ account_payment/__init__.py	2012-03-05 18:43:21 +0000
@@ -29,4 +29,4 @@
 import account_invoice
 import report
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_payment/account_invoice.py'
--- account_payment/account_invoice.py	2011-07-01 23:41:24 +0000
+++ account_payment/account_invoice.py	2012-03-05 18:43:21 +0000
@@ -50,4 +50,4 @@
 
 Invoice()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_payment/account_move_line.py'
--- account_payment/account_move_line.py	2011-07-01 23:41:24 +0000
+++ account_payment/account_move_line.py	2012-03-05 18:43:21 +0000
@@ -26,7 +26,7 @@
 class account_move_line(osv.osv):
     _inherit = "account.move.line"
 
-    def amount_to_pay(self, cr, uid, ids, name, arg={}, context=None):
+    def amount_to_pay(self, cr, uid, ids, name, arg=None, context=None):
         """ Return the amount still to pay regarding all the payemnt orders
         (excepting cancelled orders)"""
         if not ids:
@@ -117,4 +117,4 @@
 
 account_move_line()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_payment/account_payment.py'
--- account_payment/account_payment.py	2012-01-31 13:36:57 +0000
+++ account_payment/account_payment.py	2012-03-05 18:43:21 +0000
@@ -139,7 +139,9 @@
         wf_service.trg_validate(uid, 'payment.order', ids[0], 'done', cr)
         return True
 
-    def copy(self, cr, uid, id, default={}, context=None):
+    def copy(self, cr, uid, id, default=None, context=None):
+        if default is None:
+            default = {}
         default.update({
             'state': 'draft',
             'line_ids': [],
 
=== modified file 'account_payment/report/__init__.py'
--- account_payment/report/__init__.py	2011-01-14 00:11:01 +0000
+++ account_payment/report/__init__.py	2012-03-05 18:43:21 +0000
@@ -20,4 +20,4 @@
 ##############################################################################
 
 import payment_order
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_payment/report/payment_order.py'
--- account_payment/report/payment_order.py	2011-01-14 00:11:01 +0000
+++ account_payment/report/payment_order.py	2012-03-05 18:43:21 +0000
@@ -77,7 +77,7 @@
         user = pool.get('res.users').browse(self.cr, self.uid, self.uid)
         return user.company_id and user.company_id.currency_id and user.company_id.currency_id.symbol or False
 
-    def _get_account_name(self,bank_id):
+    def _get_account_name(self, bank_id):
         if bank_id:
             pool = pooler.get_pool(self.cr.dbname)
             value_name = pool.get('res.partner.bank').name_get(self.cr, self.uid, [bank_id])
@@ -87,4 +87,4 @@
 
 report_sxw.report_sxw('report.payment.order', 'payment.order', 'addons/account_payment/report/payment_order.rml', parser=payment_order, header="external")
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_payment/wizard/__init__.py'
--- account_payment/wizard/__init__.py	2011-01-14 00:11:01 +0000
+++ account_payment/wizard/__init__.py	2012-03-05 18:43:21 +0000
@@ -23,4 +23,4 @@
 import account_payment_populate_statement
 import account_payment_pay
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_payment/wizard/account_payment_order.py'
--- account_payment/wizard/account_payment_order.py	2011-10-16 01:28:00 +0000
+++ account_payment/wizard/account_payment_order.py	2012-03-05 18:43:21 +0000
@@ -119,4 +119,4 @@
 
 payment_order_create()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_payment/wizard/account_payment_pay.py'
--- account_payment/wizard/account_payment_pay.py	2012-01-31 13:36:57 +0000
+++ account_payment/wizard/account_payment_pay.py	2012-03-05 18:43:21 +0000
@@ -56,4 +56,4 @@
 
 account_payment_make_payment()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_payment/wizard/account_payment_populate_statement.py'
--- account_payment/wizard/account_payment_populate_statement.py	2012-01-31 13:36:57 +0000
+++ account_payment/wizard/account_payment_populate_statement.py	2012-03-05 18:43:21 +0000
@@ -120,4 +120,4 @@
 
 account_payment_populate_statement()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_sequence/__init__.py'
--- account_sequence/__init__.py	2011-01-14 00:11:01 +0000
+++ account_sequence/__init__.py	2012-03-05 18:43:21 +0000
@@ -22,4 +22,4 @@
 import account_sequence
 import account_sequence_installer
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_voucher/__init__.py'
--- account_voucher/__init__.py	2011-10-16 01:28:00 +0000
+++ account_voucher/__init__.py	2012-03-05 18:43:21 +0000
@@ -24,4 +24,4 @@
 import report
 import wizard
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_voucher/account_voucher.py'
--- account_voucher/account_voucher.py	2012-02-24 10:23:05 +0000
+++ account_voucher/account_voucher.py	2012-03-05 18:43:21 +0000
@@ -830,7 +830,7 @@
             res['account_id'] = account_id
         return {'value':res}
 
-    def _sel_context(self, cr, uid, voucher_id,context=None):
+    def _sel_context(self, cr, uid, voucher_id, context=None):
         """
         Select the context to use accordingly if it needs to be multicurrency or not.
 
@@ -1251,7 +1251,9 @@
                     move_line_pool.reconcile_partial(cr, uid, rec_ids, writeoff_acc_id=voucher.writeoff_acc_id.id, writeoff_period_id=voucher.period_id.id, writeoff_journal_id=voucher.journal_id.id)
         return True
 
-    def copy(self, cr, uid, id, default={}, context=None):
+    def copy(self, cr, uid, id, default=None, context=None):
+        if default is None:
+            default = {}
         default.update({
             'state': 'draft',
             'number': False,
 
=== modified file 'account_voucher/report/__init__.py'
--- account_voucher/report/__init__.py	2011-01-14 00:11:01 +0000
+++ account_voucher/report/__init__.py	2012-03-05 18:43:21 +0000
@@ -23,4 +23,4 @@
 import account_voucher_print
 import account_voucher_sales_receipt
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_voucher/report/account_voucher.py'
--- account_voucher/report/account_voucher.py	2011-01-14 00:11:01 +0000
+++ account_voucher/report/account_voucher.py	2012-03-05 18:43:21 +0000
@@ -72,4 +72,4 @@
     parser=report_voucher,header="external"
 )
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_voucher/report/account_voucher_print.py'
--- account_voucher/report/account_voucher_print.py	2011-01-14 00:11:01 +0000
+++ account_voucher/report/account_voucher_print.py	2012-03-05 18:43:21 +0000
@@ -93,4 +93,4 @@
     parser=report_voucher_print,header="external"
 )
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_voucher/wizard/__init__.py'
--- account_voucher/wizard/__init__.py	2011-01-14 00:11:01 +0000
+++ account_voucher/wizard/__init__.py	2012-03-05 18:43:21 +0000
@@ -22,4 +22,4 @@
 import account_voucher_unreconcile
 import account_statement_from_invoice
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'account_voucher/wizard/account_voucher_unreconcile.py'
--- account_voucher/wizard/account_voucher_unreconcile.py	2011-01-14 00:11:01 +0000
+++ account_voucher/wizard/account_voucher_unreconcile.py	2012-03-05 18:43:21 +0000
@@ -59,4 +59,4 @@
 
 account_voucher_unreconcile()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'analytic/analytic.py'
--- analytic/analytic.py	2012-02-16 18:01:11 +0000
+++ analytic/analytic.py	2012-03-05 18:43:21 +0000
@@ -219,7 +219,7 @@
         default['line_ids'] = []
         return super(account_analytic_account, self).copy(cr, uid, id, default, context=context)
 
-    def on_change_partner_id(self, cr, uid, id, partner_id, context={}):
+    def on_change_partner_id(self, cr, uid, id, partner_id, context=None):
         if not partner_id:
             return {'value': {'contact_id': False}}
         addr = self.pool.get('res.partner').address_get(cr, uid, [partner_id], ['invoice'])
 
=== modified file 'anonymization/anonymization.py'
--- anonymization/anonymization.py	2011-12-19 16:54:40 +0000
+++ anonymization/anonymization.py	2012-03-05 18:43:21 +0000
@@ -350,7 +350,7 @@
         })
         raise osv.except_osv(error_type, error_msg)
 
-    def anonymize_database(self,cr, uid, ids, context=None):
+    def anonymize_database(self, cr, uid, ids, context=None):
         """Sets the 'anonymized' state to defined fields"""
 
         # create a new history record:
@@ -485,7 +485,7 @@
                 'target':'new',
         }
 
-    def reverse_anonymize_database(self,cr, uid, ids, context=None):
+    def reverse_anonymize_database(self, cr, uid, ids, context=None):
         """Set the 'clear' state to defined fields"""
 
         ir_model_fields_anonymization_model = self.pool.get('ir.model.fields.anonymization')
 
=== modified file 'auction/auction.py'
--- auction/auction.py	2011-12-09 06:03:08 +0000
+++ auction/auction.py	2012-03-05 18:43:21 +0000
@@ -533,7 +533,7 @@
         return self._sum_taxes_by_type_and_id(costs)
 
     # sum remise limite net and ristourne
-    def compute_seller_costs_summed(self, cr, uid, ids): #ach_pay_id
+    def compute_seller_costs_summed(self, cr, uid, ids):
 
         """This Fuction  sum Net remittance limit and refund"""
 
 
=== modified file 'auction/barcode/code39.py'
--- auction/barcode/code39.py	2011-01-14 00:11:01 +0000
+++ auction/barcode/code39.py	2012-03-05 18:43:21 +0000
@@ -114,7 +114,7 @@
 
         
 class _Code39Base(Barcode):
-    def __init__(self, value = "", **args):
+    def __init__(self, value="", **args):
         self.xdim = inch * 0.0075
         self.lquiet = None
         self.rquiet = None
 
=== modified file 'auction/barcode/common.py'
--- auction/barcode/common.py	2011-01-14 00:11:01 +0000
+++ auction/barcode/common.py	2012-03-05 18:43:21 +0000
@@ -39,7 +39,7 @@
     """Abstract Base for barcodes. Includes implementations of
     some methods suitable for the more primitive barcode types"""
     
-    def __init__(self, value = ''):
+    def __init__(self, value=''):
         self.value = value
         
         if not hasattr(self, 'gap'):
 
=== modified file 'auction/report/auction_invoice.py'
--- auction/report/auction_invoice.py	2011-01-14 00:11:01 +0000
+++ auction/report/auction_invoice.py	2012-03-05 18:43:21 +0000
@@ -26,7 +26,7 @@
     def __init__(self, name):
         report_int.__init__(self, name)
 
-    def create(self,cr, uid, ids, datas, context):
+    def create(self, cr, uid, ids, datas, context):
         lots = self.pool.get('auction.lots').read(cr, uid, ids, ['ach_inv_id'], context=context)
 
         invoices = {}
 
=== modified file 'auction/report/auction_total_rml.py'
--- auction/report/auction_total_rml.py	2011-04-29 08:49:48 +0000
+++ auction/report/auction_total_rml.py	2012-03-05 18:43:21 +0000
@@ -65,7 +65,7 @@
         return auct_dat
 
 
-    def sum_taxes(self,auction_id):
+    def sum_taxes(self, auction_id):
             self.cr.execute("select count(1) from auction_lots where id IN %s and auction_id=%s group by auction_id ", (tuple(self.total_obj),auction_id,))
             res = self.cr.fetchone()
             return res[0]
@@ -105,17 +105,17 @@
         res = self.cr.fetchone()
         return str(res[0]) or 0.0
 
-    def sum_credit(self,auction_id):
+    def sum_credit(self, auction_id):
         self.cr.execute("select sum(buyer_price) from auction_lots where id IN %s and auction_id=%s", (tuple(self.total_obj),auction_id,))
         res = self.cr.fetchone()
         return str(res[0])
 
-    def sum_debit_buyer(self,auction_id):
+    def sum_debit_buyer(self, auction_id):
         self.cr.execute("select sum(buyer_price) from auction_lots where id IN %s  and auction_id=%s", (tuple(self.total_obj),auction_id,))
         res = self.cr.fetchone()
         return str(res[0] or 0)
 
-    def sum_debit(self,object_id):
+    def sum_debit(self, object_id):
         self.cr.execute("select sum(seller_price) from auction_lots where auction_id=%s", (object_id,))
         res = self.cr.fetchone()
         return str(res[0] or 0)
 
=== modified file 'auction/report/buyer_form_report.py'
--- auction/report/buyer_form_report.py	2011-04-29 08:49:48 +0000
+++ auction/report/buyer_form_report.py	2012-03-05 18:43:21 +0000
@@ -58,7 +58,7 @@
                 lots.append(object)
         return ret_dict.values()
 
-    def grand_buyer_total(self,o):
+    def grand_buyer_total(self, o):
         grand_total = 0
         for oo in o:
             grand_total =grand_total + oo['obj_price'] +self.sum_taxes(oo)
 
=== modified file 'auction/report/buyer_list.py'
--- auction/report/buyer_list.py	2011-04-29 08:49:48 +0000
+++ auction/report/buyer_list.py	2012-03-05 18:43:21 +0000
@@ -62,7 +62,7 @@
             auct_dat.append(auc_dates_fields)
         return auct_dat
 
-    def lines_lots_auct_lot(self,obj):
+    def lines_lots_auct_lot(self, obj):
 
         auc_date_ids = self.pool.get('auction.dates').search(self.cr, self.uid, ([('name','like',obj['name'])]))
 
 
=== modified file 'auction/report/huissier.py'
--- auction/report/huissier.py	2011-04-29 08:49:48 +0000
+++ auction/report/huissier.py	2012-03-05 18:43:21 +0000
@@ -30,7 +30,7 @@
     def __init__(self, name, table, tmpl, xsl):
         report_rml.__init__(self, name, table, tmpl, xsl)
 
-    def create_xml(self,cr, uid, ids, datas, context=None):
+    def create_xml(self, cr, uid, ids, datas, context=None):
         pool= pooler.get_pool(cr.dbname)
         lots = pool.get('auction.lots').browse(cr, uid, ids, context=context)
         auction = lots[0].auction_id
 
=== modified file 'auction/report/photo_shadow.py'
--- auction/report/photo_shadow.py	2011-11-14 06:51:05 +0000
+++ auction/report/photo_shadow.py	2012-03-05 18:43:21 +0000
@@ -19,7 +19,7 @@
 #
 ##############################################################################
 
-def convert_catalog(from_file, to_file, size=220) :
+def convert_catalog(from_file, to_file, size=220):
     return __convert(from_file, to_file, size)
 
 def convert(from_file, to_file):
 
=== modified file 'auction/report/seller_form_report.py'
--- auction/report/seller_form_report.py	2011-04-29 08:49:48 +0000
+++ auction/report/seller_form_report.py	2012-03-05 18:43:21 +0000
@@ -58,7 +58,7 @@
                 lots = partner.get('lots')
                 lots.append(object)
         return ret_dict.values()
-    def grand_seller_total(self,o):
+    def grand_seller_total(self, o):
         grand_total = 0
         for oo in o:
             grand_total =grand_total + oo['obj_price']+ self.sum_taxes(oo)
 
=== modified file 'audittrail/audittrail.py'
--- audittrail/audittrail.py	2011-11-21 16:27:12 +0000
+++ audittrail/audittrail.py	2012-03-05 18:43:21 +0000
@@ -201,7 +201,7 @@
             res = value
         return res
 
-    def create_log_line(self, cr, uid, log_id, model, lines=[]):
+    def create_log_line(self, cr, uid, log_id, model, lines=None):
         """
         Creates lines for changed fields with its old and new values
 
@@ -210,6 +210,8 @@
         @param model: Object which values are being changed
         @param lines: List of values for line is to be created
         """
+        if lines is None:
+            lines = []
         pool = pooler.get_pool(cr.dbname)
         obj_pool = pool.get(model.model)
         model_pool = pool.get('ir.model')
@@ -348,7 +350,7 @@
             data[(model.id, resource_id)] = {'text':values_text, 'value': values}
         return data
 
-    def prepare_audittrail_log_line(self, cr, uid, pool, model, resource_id, method, old_values, new_values, field_list=[]):
+    def prepare_audittrail_log_line(self, cr, uid, pool, model, resource_id, method, old_values, new_values, field_list=None):
         """
         This function compares the old data (i.e before the method was executed) and the new data 
         (after the method was executed) and returns a structure with all the needed information to
@@ -378,6 +380,8 @@
         record (res.partner, for example), we may have to log a change done in a x2many field (on 
         res.partner.address, for example)
         """
+        if field_list is None:
+            field_list = []
         key = (model.id, resource_id)
         lines = {
             key: []
@@ -416,7 +420,7 @@
                 lines[key].append(data)
         return lines
 
-    def process_data(self, cr, uid, pool, res_ids, model, method, old_values={}, new_values={}, field_list=[]):
+    def process_data(self, cr, uid, pool, res_ids, model, method, old_values=None, new_values=None, field_list=None):
         """
         This function processes and iterates recursively to log the difference between the old
         data (i.e before the method was executed) and the new data and creates audittrail log
 
=== modified file 'base_contact/base_contact.py'
--- base_contact/base_contact.py	2012-02-15 13:22:13 +0000
+++ base_contact/base_contact.py	2012-03-05 18:43:21 +0000
@@ -175,7 +175,7 @@
         ids = self.pool.get('res.partner.location').search(cr, uid, [('partner_id','=',context['default_partner_id'])], context=context)
         return ids and ids[0] or False
 
-    def onchange_location_id(self,cr, uid, ids, location_id=False, context={}):
+    def onchange_location_id(self, cr, uid, ids, location_id=False, context=None):
         if not location_id:
             return {}
         location = self.pool.get('res.partner.location').browse(cr, uid, location_id, context=context)
@@ -209,7 +209,7 @@
         'name' : fields.related('contact_id', 'name', type='char', size=64, string="Contact Name", store=True),
         'title' : fields.related('contact_id', 'title', type='many2one', relation='res.partner.title', string="Title", store=True),
     }
-    def create(self, cr, uid, data, context={}):
+    def create(self, cr, uid, data, context=None):
         if not data.get('location_id', False):
             loc_id = self.pool.get('res.partner.location').create(cr, uid, {
                 'street': data.get('street',''),
@@ -241,7 +241,7 @@
         'location_id': _default_location_id
     }
 
-    def default_get(self, cr, uid, fields=[], context=None):
+    def default_get(self, cr, uid, fields=None, context=None):
         if context is None:
             context = {}
         if 'default_type' in context:
 
=== modified file 'base_crypt/crypt.py'
--- base_crypt/crypt.py	2011-10-18 09:38:13 +0000
+++ base_crypt/crypt.py	2012-03-05 18:43:21 +0000
@@ -45,7 +45,9 @@
 
 magic_md5 = '$1$'
 
-def gen_salt( length=8, symbols=ascii_letters + digits ):
+def gen_salt( length=8, symbols=None):
+    if symbols is None:
+        symbols = ascii_letters + digits
     seed()
     return ''.join( sample( symbols, length ) )
 
 
=== modified file 'base_module_doc_rst/report/ir_module_reference_print_graph.py'
--- base_module_doc_rst/report/ir_module_reference_print_graph.py	2011-01-14 00:11:01 +0000
+++ base_module_doc_rst/report/ir_module_reference_print_graph.py	2012-03-05 18:43:21 +0000
@@ -79,4 +79,4 @@
         'addons/base_module_doc_rst/report/ir_module_reference_graph.rml',
         parser=ir_module_reference_print_graph, header=False)
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_quality/__init__.py'
--- base_module_quality/__init__.py	2011-01-14 00:11:01 +0000
+++ base_module_quality/__init__.py	2012-03-05 18:43:21 +0000
@@ -22,4 +22,4 @@
 import base_module_quality
 import wizard
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_quality/base_module_quality.py'
--- base_module_quality/base_module_quality.py	2011-10-27 21:11:24 +0000
+++ base_module_quality/base_module_quality.py	2012-03-05 18:43:21 +0000
@@ -115,7 +115,9 @@
         self.log.debug('get_objects() obj_list: %s', ','.join(obj_list))
         return obj_list
 
-    def get_model_ids(self, cr, uid, models=[]):
+    def get_model_ids(self, cr, uid, models=None):
+        if models is None:
+            models = []
         # This function returns all ids of the given objects..
         if not models:
             return []
@@ -133,7 +135,12 @@
             result_ids[obj] = ids
         return result_ids
 
-    def format_table(self, header=[], data_list={}): #This function can work forwidget="text_wiki"
+    def format_table(self, header=None, data_list=None):
+        #This function can work forwidget="text_wiki"
+        if header is None:
+            header = []
+        if data_list is None:
+            data_list = {}
         detail = ""
         detail += (header[0]) % tuple(header[1])
         frow = '\n|-'
@@ -144,7 +151,12 @@
         detail = detail + '\n|}'
         return detail
 
-    def format_html_table(self, header=[], data_list=[]): #This function can work for widget="html_tag"
+    def format_html_table(self, header=None, data_list=None):
+        #This function can work for widget="html_tag"
+        if header is None:
+            header = []
+        if data_list is None:
+            data_list = []
         # function create html table....
         detail = ""
         detail += (header[0]) % tuple(header[1])
 
=== modified file 'base_module_quality/method_test/method_test.py'
--- base_module_quality/method_test/method_test.py	2011-01-14 00:11:01 +0000
+++ base_module_quality/method_test/method_test.py	2012-03-05 18:43:21 +0000
@@ -85,4 +85,4 @@
             detail += self.format_table(header, dict_method)
         return detail
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_quality/object_test/object_test.py'
--- base_module_quality/object_test/object_test.py	2011-01-14 00:11:01 +0000
+++ base_module_quality/object_test/object_test.py	2012-03-05 18:43:21 +0000
@@ -208,4 +208,4 @@
             return res
         return ""
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_quality/pep8_test/pep8_test.py'
--- base_module_quality/pep8_test/pep8_test.py	2011-01-14 00:11:01 +0000
+++ base_module_quality/pep8_test/pep8_test.py	2012-03-05 18:43:21 +0000
@@ -278,4 +278,4 @@
             return res
         return ""
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_quality/structure_test/structure_test.py'
--- base_module_quality/structure_test/structure_test.py	2011-01-14 00:11:01 +0000
+++ base_module_quality/structure_test/structure_test.py	2012-03-05 18:43:21 +0000
@@ -176,4 +176,4 @@
             return res
         return ""
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_quality/unit_test/unit_test.py'
--- base_module_quality/unit_test/unit_test.py	2011-01-14 00:11:01 +0000
+++ base_module_quality/unit_test/unit_test.py	2012-03-05 18:43:21 +0000
@@ -111,4 +111,4 @@
             return detail + html +'</tr></table></body></html>'
         return ''
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_quality/wizard/module_quality_check.py'
--- base_module_quality/wizard/module_quality_check.py	2011-10-27 21:11:24 +0000
+++ base_module_quality/wizard/module_quality_check.py	2012-03-05 18:43:21 +0000
@@ -49,4 +49,4 @@
         
 quality_check()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_quality/wizard/quality_save_report.py'
--- base_module_quality/wizard/quality_save_report.py	2011-10-27 21:11:24 +0000
+++ base_module_quality/wizard/quality_save_report.py	2012-03-05 18:43:21 +0000
@@ -49,4 +49,4 @@
 
 quality_save_report()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_quality/workflow_test/workflow_test.py'
--- base_module_quality/workflow_test/workflow_test.py	2011-01-14 00:11:01 +0000
+++ base_module_quality/workflow_test/workflow_test.py	2012-03-05 18:43:21 +0000
@@ -155,4 +155,4 @@
                 count = self.count_button(node, count)
         return count
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_module_record/wizard/base_module_record_objects.py'
--- base_module_record/wizard/base_module_record_objects.py	2012-02-09 09:12:03 +0000
+++ base_module_record/wizard/base_module_record_objects.py	2012-03-05 18:43:21 +0000
@@ -127,7 +127,7 @@
     _name = 'base.module.record.objects'
     _description = "Base Module Record Objects"
                 
-    def inter_call(self,cr,uid,data,context=None):
+    def inter_call(self, cr, uid, data, context=None):
         res=base_module_save._create_module(self, cr, uid, data, context)
         mod_obj = self.pool.get('ir.model.data')
         model_data_ids = mod_obj.search(cr, uid,[('model', '=', 'ir.ui.view'), ('name', '=', 'module_create_form_view')], context=context)
 
=== modified file 'base_module_record/wizard/base_module_save.py'
--- base_module_record/wizard/base_module_save.py	2011-09-08 07:06:49 +0000
+++ base_module_record/wizard/base_module_save.py	2012-03-05 18:43:21 +0000
@@ -167,4 +167,4 @@
         
 base_module_save()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_report_designer/openerp_sxw2rml/openerp_sxw2rml.py'
--- base_report_designer/openerp_sxw2rml/openerp_sxw2rml.py	2011-01-14 00:11:01 +0000
+++ base_report_designer/openerp_sxw2rml/openerp_sxw2rml.py	2012-03-05 18:43:21 +0000
@@ -42,11 +42,11 @@
 
 class DomApiGeneral:
     """General DOM API utilities."""
-    def __init__(self,content_string="",file=""):
+    def __init__(self, content_string="", file=""):
         self.content_string = content_string
         self.re_digits = re.compile(r"(.*?\d)(pt|cm|mm|inch|in)")
 
-    def _unitTuple(self,string):
+    def _unitTuple(self, string):
         """Split values and units to a tuple."""
         temp = self.re_digits.findall(string)
         if not temp:
@@ -54,13 +54,15 @@
         else:
             return (temp[0])
 
-    def stringPercentToFloat(self,string):
+    def stringPercentToFloat(self, string):
         temp = string.replace("""%""","")
         return float(temp)/100
 
-    def findChildrenByName(self,parent,name,attr_dict={}):
+    def findChildrenByName(self, parent, name, attr_dict=None):
         """Helper functions. Does not work recursively.
         Optional: also test for certain attribute/value pairs."""
+        if attr_dict is None:
+            attr_dict = {}
         children = []
         for c in parent.childNodes:
             if c.nodeType == c.ELEMENT_NODE and c.nodeName == name:
@@ -70,7 +72,7 @@
         else:
             return self._selectForAttributes(nodelist=children,attr_dict=attr_dict)
 
-    def _selectForAttributes(self,nodelist,attr_dict):
+    def _selectForAttributes(self, nodelist, attr_dict):
         "Helper function."""
         selected_nodes = []
         for n in nodelist:
@@ -83,7 +85,7 @@
                 selected_nodes.append(n)
         return selected_nodes
 
-    def _stringToTuple(self,s):
+    def _stringToTuple(self, s):
         """Helper function."""
         try:
             temp = string.split(s,",")
@@ -91,13 +93,13 @@
         except:
             return None
 
-    def _tupleToString(self,t):
+    def _tupleToString(self, t):
         try:
             return self.openOfficeStringUtf8("%s,%s" % (t[0],t[1]))
         except:
             return None
 
-    def _lengthToFloat(self,value):
+    def _lengthToFloat(self, value):
         v = value
         if not self.re_digits.search(v):
             return v
@@ -113,7 +115,7 @@
         except:
             return v
 
-    def openOfficeStringUtf8(self,string):
+    def openOfficeStringUtf8(self, string):
         if type(string) == unicode:
             return string.encode("utf-8")
         tempstring = unicode(string,"cp1252").encode("utf-8")
@@ -121,7 +123,7 @@
 
 class DomApi(DomApiGeneral):
     """This class provides a DOM-API for XML-Files from an SXW-Archive."""
-    def __init__(self,xml_content,xml_styles):
+    def __init__(self, xml_content, xml_styles):
         DomApiGeneral.__init__(self)
         self.content_dom = xml.dom.minidom.parseString(xml_content)
         self.styles_dom = xml.dom.minidom.parseString(xml_styles)
@@ -145,7 +147,7 @@
         for s in self.style_dict.keys():
             self.style_properties_dict[s] = self.getStylePropertiesDict(s)
 
-    def updateWithPercents(self,dict,updatedict):
+    def updateWithPercents(self, dict, updatedict):
         """Sometimes you find values like "115%" in the style hierarchy."""
         if not updatedict:
             # no style hierarchies for this style? =>
@@ -244,7 +246,7 @@
     def toxml(self):
         return self.content_dom.toxml(encoding="utf-8")
 
-    def getStylePropertiesDict(self,style_name):
+    def getStylePropertiesDict(self, style_name):
         res = {}
 
         if self.style_dict[style_name].hasAttribute("style:parent-style-name"):
@@ -265,7 +267,7 @@
         self.save_pict = save_pict
         self.images = {}
 
-    def oo_read(self,fname):
+    def oo_read(self, fname):
         z = zipfile.ZipFile(fname,"r")
         content = z.read('content.xml')
         style = z.read('styles.xml')
@@ -281,7 +283,7 @@
         z.close()
         return content,style
 
-    def oo_replace(self,content):
+    def oo_replace(self, content):
         regex = [
             (r"<para[^>]*/>", ""),
             (r"<para(.*)>(.*?)<text:line-break[^>]*/>", "<para$1>$2</para><para$1>"),
@@ -290,7 +292,7 @@
             content = re.sub(key, val, content)
         return content
 
-    def unpackNormalize(self,sourcefile):
+    def unpackNormalize(self, sourcefile):
         c,s = self.oo_read(sourcefile)
         c = self.oo_replace(c)
         dom = DomApi(c,s)
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/About.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/About.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/About.py	2012-03-05 18:43:21 +0000
@@ -52,7 +52,7 @@
     from lib.gui import *
 
 class About(unohelper.Base, XJobExecutor):
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/AddAttachment.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/AddAttachment.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/AddAttachment.py	2012-03-05 18:43:21 +0000
@@ -64,7 +64,7 @@
         'PDF' : 'pdf',
         'OpenOffice': 'sxw',
     }
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
@@ -132,7 +132,7 @@
         self.win.addButton('btnCancel', -2 - 27 , -5 , 30 , 15, 'Cancel' ,actionListenerProc = self.btnCancel_clicked )
         self.win.doModalDialog("lstResourceType", self.Kind.keys()[0])
 
-    def btnSearch_clicked( self, oActionEvent ):
+    def btnSearch_clicked(self, oActionEvent):
         modelSelectedItem = self.win.getListBoxSelectedItem("lstmodel")
         if modelSelectedItem == "":
             return
@@ -151,7 +151,7 @@
         for result in self.aSearchResult:
             self.lstResource.addItem(result[1],result[0])
 
-    def _send_attachment( self, name, data, res_model, res_id ):
+    def _send_attachment(self, name, data, res_model, res_id):
         desktop = getDesktop()
         oDoc2 = desktop.getCurrentComponent()
         docinfo = oDoc2.getDocumentInfo()
@@ -166,7 +166,7 @@
 
         return self.sock.execute( database, uid, self.password, 'ir.attachment', 'create', params )
 
-    def send_attachment( self, model, resource_id ):
+    def send_attachment(self, model, resource_id):
         desktop = getDesktop()
         oDoc2 = desktop.getCurrentComponent()
         docinfo = oDoc2.getDocumentInfo()
@@ -187,7 +187,7 @@
         data = read_data_from_file( get_absolute_file_path( url ) )
         return self._send_attachment( os.path.basename( url ), data, model, resource_id )
 
-    def btnOkWithoutInformation_clicked( self, oActionEvent ):
+    def btnOkWithoutInformation_clicked(self, oActionEvent):
         desktop = getDesktop()
         oDoc2 = desktop.getCurrentComponent()
         docinfo = oDoc2.getDocumentInfo()
@@ -199,7 +199,7 @@
         res = self.send_attachment( docinfo.getUserFieldValue(3), docinfo.getUserFieldValue(2) )
         self.win.endExecute()
 
-    def btnOkWithInformation_clicked(self,oActionEvent):
+    def btnOkWithInformation_clicked(self, oActionEvent):
         if self.win.getListBoxSelectedItem("lstResourceType") == "":
             ErrorDialog( "Please select resource type", "", "Selection ERROR" )
             return
@@ -221,7 +221,7 @@
         res = self.send_attachment( self.dModel[self.win.getListBoxSelectedItem('lstmodel')], resourceid )
         self.win.endExecute()
 
-    def btnCancel_clicked( self, oActionEvent ):
+    def btnCancel_clicked(self, oActionEvent):
         self.win.endExecute()
 
     def doc2pdf(self, strFile):
@@ -262,7 +262,7 @@
             # Can be None if len(strFilterSubName) <= 0
             return filename
 
-    def _MakePropertyValue(self, cName = "", uValue = u"" ):
+    def _MakePropertyValue(self, cName="", uValue=u"" ):
        oPropertyValue = createUnoStruct( "com.sun.star.beans.PropertyValue" )
        if cName:
           oPropertyValue.Name = cName
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/Change.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/Change.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Change.py	2012-03-05 18:43:21 +0000
@@ -59,7 +59,7 @@
     database="test"
 
 class Change( unohelper.Base, XJobExecutor ):
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
@@ -107,7 +107,7 @@
             self.lstProtocol.addItem(i,self.lstProtocol.getItemCount() )
         self.win.doModalDialog( "lstProtocol",  protocol)
 
-    def btnNext_clicked(self,oActionEvent):
+    def btnNext_clicked(self, oActionEvent):
         global url
         aVal=''
         #aVal= Fetature used 
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/ConvertBracesToField.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/ConvertBracesToField.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/ConvertBracesToField.py	2012-03-05 18:43:21 +0000
@@ -63,7 +63,7 @@
 
 class ConvertBracesToField( unohelper.Base, XJobExecutor ):
 
-    def __init__(self,ctx):
+    def __init__(self, ctx):
 
         self.ctx     = ctx
         self.module  = "openerp_report"
@@ -198,7 +198,7 @@
             info = reduce(lambda x, y: x+y, traceback.format_exception(sys.exc_type, sys.exc_value, sys.exc_traceback))
             self.logobj.log_write('ConvertBraceToField', LOG_ERROR, info)
 
-    def getRes(self,sock,sObject,sVar):
+    def getRes(self, sock, sObject, sVar):
         desktop=getDesktop()
         doc =desktop.getCurrentComponent()
         docinfo=doc.getDocumentInfo()
@@ -215,7 +215,7 @@
                 sObject = self.getRes(sock,res[myval]['relation'], sVar[sVar.find("/")+1:])
         return sObject
 
-    def getBraces(self,aReportSyntex=[]):
+    def getBraces(self, aReportSyntex=None):
         desktop=getDesktop()
         doc = desktop.getCurrentComponent()
         aSearchString=[]
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/ConvertFieldsToBraces.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/ConvertFieldsToBraces.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/ConvertFieldsToBraces.py	2012-03-05 18:43:21 +0000
@@ -57,7 +57,7 @@
     uid = 3
 
 class ConvertFieldsToBraces( unohelper.Base, XJobExecutor ):
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/ExportToRML.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/ExportToRML.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/ExportToRML.py	2012-03-05 18:43:21 +0000
@@ -67,7 +67,7 @@
 
 
 class ExportToRML( unohelper.Base, XJobExecutor ):
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/Expression.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/Expression.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Expression.py	2012-03-05 18:43:21 +0000
@@ -57,7 +57,7 @@
     uid = 3
 
 class Expression(unohelper.Base, XJobExecutor ):
-    def __init__(self,sExpression="",sName="", bFromModify=False):
+    def __init__(self, sExpression="", sName="", bFromModify=False):
         LoginTest()
         if not loginstatus and __name__=="package":
             exit(1)
@@ -75,7 +75,7 @@
         self.win.doModalDialog("",None)
 
 
-    def btnOk_clicked( self, oActionEvent ):
+    def btnOk_clicked(self, oActionEvent):
         desktop=getDesktop()
         doc = desktop.getCurrentComponent()
         text = doc.Text
@@ -105,7 +105,7 @@
             else:
                 ErrorDialog("Please Fill appropriate data in Name field or \nExpression field")
 
-    def btnCancel_clicked( self, oActionEvent ):
+    def btnCancel_clicked(self, oActionEvent):
         self.win.endExecute()
 
 if __name__<>"package" and __name__=="__main__":
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/Fields.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/Fields.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Fields.py	2012-03-05 18:43:21 +0000
@@ -64,7 +64,7 @@
 
 
 class Fields(unohelper.Base, XJobExecutor ):
-    def __init__(self,sVariable="",sFields="",sDisplayName="",bFromModify=False):
+    def __init__(self, sVariable="", sFields="", sDisplayName="", bFromModify=False):
         LoginTest()
         if not loginstatus and __name__=="package":
             exit(1)
@@ -177,7 +177,7 @@
             ErrorDialog("Please insert user define field Field-1 or Field-4","Just go to File->Properties->User Define \nField-1 Eg. http://localhost:8069 \nOR \nField-4 Eg. account.invoice")
             self.win.endExecute()
 
-    def lstbox_selected(self,oItemEvent):
+    def lstbox_selected(self, oItemEvent):
         try:
 
             desktop=getDesktop()
@@ -200,7 +200,7 @@
         if self.bModify:
             self.win.setEditText("txtUName",self.sGDisplayName)
 
-    def getRes(self,sock ,sObject,sVar):
+    def getRes(self, sock, sObject, sVar):
         desktop=getDesktop()
         doc =desktop.getCurrentComponent()
         docinfo=doc.getDocumentInfo()
@@ -219,7 +219,7 @@
             else:
                 return sObject
 
-    def cmbVariable_selected(self,oItemEvent):
+    def cmbVariable_selected(self, oItemEvent):
         if self.count > 0 :
             try:
                 desktop=getDesktop()
@@ -246,7 +246,7 @@
                 info = reduce(lambda x, y: x+y, traceback.format_exception(sys.exc_type, sys.exc_value, sys.exc_traceback))
                 self.logobj.log_write('Fields', LOG_ERROR, info)
 
-    def btnOk_clicked( self, oActionEvent ):
+    def btnOk_clicked(self, oActionEvent):
         desktop=getDesktop()
         doc = desktop.getCurrentComponent()
         cursor = doc.getCurrentController().getViewCursor()
@@ -281,7 +281,7 @@
                 else:
                     ErrorDialog("Please Fill appropriate data in Name field \nor select perticular value from the list of fields")
 
-    def btnCancel_clicked( self, oActionEvent ):
+    def btnCancel_clicked(self, oActionEvent):
         self.win.endExecute()
 
 if __name__<>"package" and __name__=="__main__":
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/ModifyExistingReport.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/ModifyExistingReport.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/ModifyExistingReport.py	2012-03-05 18:43:21 +0000
@@ -66,7 +66,7 @@
 
 #
 class ModifyExistingReport(unohelper.Base, XJobExecutor):
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
@@ -178,10 +178,10 @@
 
         self.win.endExecute()
 
-    def btnCancel_clicked( self, oActionEvent ):
+    def btnCancel_clicked(self, oActionEvent):
         self.win.endExecute()
 
-    def btnDelete_clicked( self, oActionEvent ):
+    def btnDelete_clicked(self, oActionEvent):
          desktop=getDesktop()
          doc = desktop.getCurrentComponent()
          docinfo=doc.getDocumentInfo()
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/NewReport.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/NewReport.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/NewReport.py	2012-03-05 18:43:21 +0000
@@ -66,7 +66,7 @@
 #
 #
 class NewReport(unohelper.Base, XJobExecutor):
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
@@ -99,7 +99,7 @@
         self.win.addButton('btnCancel',-2 - 70 - 5 ,-5, 35,15,'Cancel' ,actionListenerProc = self.btnCancel_clicked )
         self.win.doModalDialog("",None)
 
-    def btnOk_clicked(self,oActionEvent):
+    def btnOk_clicked(self, oActionEvent):
         desktop=getDesktop()
         doc = desktop.getCurrentComponent()
         docinfo=doc.getDocumentInfo()
@@ -107,7 +107,7 @@
         self.logobj.log_write('Module Name',LOG_INFO, ':Module use in creating a report %s  using database %s' % (self.aModuleName[self.lstModule.getSelectedItemPos()], database))
         self.win.endExecute()
 
-    def btnCancel_clicked( self, oActionEvent ):
+    def btnCancel_clicked(self, oActionEvent):
         self.win.endExecute()
 
 if __name__<>"package" and __name__=="__main__":
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/Repeatln.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/Repeatln.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Repeatln.py	2012-03-05 18:43:21 +0000
@@ -64,7 +64,7 @@
 
 #class RepeatIn:
 class RepeatIn( unohelper.Base, XJobExecutor ):
-    def __init__(self,sObject="",sVariable="",sFields="",sDisplayName="",bFromModify=False):
+    def __init__(self, sObject="", sVariable="", sFields="", sDisplayName="", bFromModify=False):
         # Interface Design
         LoginTest()
         self.logobj=Logger()
@@ -204,7 +204,7 @@
             ErrorDialog("Please Select Appropriate module" ,"Create new report from: \nOpenERP -> Open a New Report")
             self.win.endExecute()
 
-    def lstbox_selected(self,oItemEvent):
+    def lstbox_selected(self, oItemEvent):
         sItem=self.win.getListBoxSelectedItem("lstFields")
         sMain=self.aListRepeatIn[self.win.getListBoxSelectedItemPos("lstFields")]
 
@@ -215,7 +215,7 @@
 	    self.win.setEditText("txtName",sMain[sMain.rfind("/")+1:])
 	    self.win.setEditText("txtUName","|-."+sItem[sItem.rfind("/")+1:]+".-|")
 
-    def cmbVariable_selected(self,oItemEvent):
+    def cmbVariable_selected(self, oItemEvent):
 
         if self.count > 0 :
 
@@ -290,7 +290,7 @@
         else:
             ErrorDialog("Please Fill appropriate data in Object Field or Name field \nor select perticular value from the list of fields")
 
-    def btnCancel_clicked( self, oActionEvent ):
+    def btnCancel_clicked(self, oActionEvent):
         self.win.endExecute()
 
 if __name__<>"package" and __name__=="__main__":
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/SendToServer.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/SendToServer.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/SendToServer.py	2012-03-05 18:43:21 +0000
@@ -74,7 +74,7 @@
         'HTML' : 'html'
     }
 
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
@@ -138,10 +138,10 @@
 
         self.win.doModalDialog("lstResourceType", self.Kind.keys()[0])
 
-    def lstbox_selected(self,oItemEvent):
+    def lstbox_selected(self, oItemEvent):
         pass
 
-    def btnCancel_clicked( self, oActionEvent ):
+    def btnCancel_clicked(self, oActionEvent):
         self.win.endExecute()
 
     def btnOk_clicked(self, oActionEvent):
@@ -223,7 +223,7 @@
         id=self.sock.execute(database, uid, self.password, 'ir.actions.report.xml' ,'create', params)
         return id
 
-    def getInverseFieldsRecord(self,nVal):
+    def getInverseFieldsRecord(self, nVal):
         desktop=getDesktop()
         doc = desktop.getCurrentComponent()
         count=0
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/ServerParameter.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/ServerParameter.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/ServerParameter.py	2012-03-05 18:43:21 +0000
@@ -59,7 +59,7 @@
     database="test"
 
 class ServerParameter( unohelper.Base, XJobExecutor ):
-    def __init__(self, aVal= None, sURL=""):
+    def __init__(self, aVal=None, sURL=""):
         self.module  = "openerp_report"
         self.version = "0.1"
         desktop=getDesktop()
@@ -116,7 +116,7 @@
 
         #self.win.doModalDialog("lstDatabase",docinfo.getUserFieldValue(2))
 
-    def btnOk_clicked(self,oActionEvent):
+    def btnOk_clicked(self, oActionEvent):
 
         sLogin=self.win.getEditText("txtLoginName")
         sPassword=self.win.getEditText("txtPassword")
@@ -158,10 +158,10 @@
             self.win.endExecute()
 
       
-    def btnCancel_clicked( self, oActionEvent ):
+    def btnCancel_clicked(self, oActionEvent):
         self.win.endExecute()
 
-    def btnPrevious_clicked(self,oActionEvent):
+    def btnPrevious_clicked(self, oActionEvent):
         self.win.endExecute()
         Change(None)
         self.win.endExecute()
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/Translation.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/Translation.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Translation.py	2012-03-05 18:43:21 +0000
@@ -61,7 +61,7 @@
 
 
 class AddLang(unohelper.Base, XJobExecutor ):
-    def __init__(self,sVariable="",sFields="",sDisplayName="",bFromModify=False):
+    def __init__(self, sVariable="", sFields="", sDisplayName="", bFromModify=False):
         LoginTest()
         if not loginstatus and __name__=="package":
             exit(1)
@@ -157,7 +157,7 @@
             ErrorDialog("Please insert user define field Field-1 or Field-4","Just go to File->Properties->User Define \nField-1 Eg. http://localhost:8069 \nOR \nField-4 Eg. account.invoice")
             self.win.endExecute()
 
-    def lstbox_selected(self,oItemEvent):
+    def lstbox_selected(self, oItemEvent):
         try:
 
             desktop=getDesktop()
@@ -183,7 +183,7 @@
         if self.bModify:
             self.win.setEditText("txtUName",self.sGDisplayName)
 
-    def getRes(self,sock ,sObject,sVar):
+    def getRes(self, sock, sObject, sVar):
         desktop=getDesktop()
         doc =desktop.getCurrentComponent()
         docinfo=doc.getDocumentInfo()
@@ -203,7 +203,7 @@
                 return sObject
 
 
-    def cmbVariable_selected(self,oItemEvent):
+    def cmbVariable_selected(self, oItemEvent):
         if self.count > 0 :
             try:
                 desktop=getDesktop()
@@ -229,7 +229,7 @@
             except:
                 import traceback;traceback.print_exc()
 
-    def btnOk_clicked( self, oActionEvent ):
+    def btnOk_clicked(self, oActionEvent):
         self.bOkay = True
         desktop=getDesktop()
         doc = desktop.getCurrentComponent()
@@ -263,7 +263,7 @@
         else:
             ErrorDialog("Please Fill appropriate data in Name field \nor select perticular value from the list of fields")
 
-    def btnCancel_clicked( self, oActionEvent ):
+    def btnCancel_clicked(self, oActionEvent):
         self.win.endExecute()
 
 
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/lib/error.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/lib/error.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/lib/error.py	2012-03-05 18:43:21 +0000
@@ -48,7 +48,7 @@
 if __name__<>"package":
     from gui import *
 class ErrorDialog:
-    def __init__(self,sErrorMsg, sErrorHelpMsg="",sTitle="Error Message"):
+    def __init__(self, sErrorMsg, sErrorHelpMsg="", sTitle="Error Message"):
         self.win = DBModalDialog(50, 50, 150, 90, sTitle)
         self.win.addFixedText("lblErrMsg", 5, 5, 190, 25, sErrorMsg)
         self.win.addFixedText("lblErrHelpMsg", 5, 30, 190, 25, sErrorHelpMsg)
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/lib/functions.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/lib/functions.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/lib/functions.py	2012-03-05 18:43:21 +0000
@@ -59,7 +59,13 @@
     database="test"
     uid = 1
 
-def genTree(object,aList,insField,host,level=3, ending=[], ending_excl=[], recur=[], root='', actualroot=""):
+def genTree(object, aList, insField, host, level=3, ending=None, ending_excl=None, recur=None, root='', actualroot=""):
+    if ending is None:
+        ending = []
+    if ending_excl is None:
+        ending_excl = []
+    if recur is None:
+        recur = []
     try:
         global url
         sock=RPCSession(url)
@@ -79,7 +85,7 @@
         info = reduce(lambda x, y: x+y, traceback.format_exception(sys.exc_type, sys.exc_value, sys.exc_traceback))
         obj.log_write('Function', LOG_ERROR, info)
 
-def VariableScope(oTcur,insVariable,aObjectList,aComponentAdd,aItemList,sTableName=""):
+def VariableScope(oTcur, insVariable, aObjectList, aComponentAdd, aItemList, sTableName=""):
     if sTableName.find(".") != -1:
 	for i in range(len(aItemList)):
             if aComponentAdd[i]==sTableName:
@@ -96,7 +102,7 @@
 		    if aObjectList[j][:aObjectList[j].find("(")] == sLVal and sLVal!="":
                         insVariable.append(aObjectList[j])
 
-def getList(aObjectList,host,count):
+def getList(aObjectList, host, count):
     desktop=getDesktop()
     doc =desktop.getCurrentComponent()
     docinfo=doc.getDocumentInfo()
@@ -128,7 +134,7 @@
     else:
         aObjectList.append("List of " + docinfo.getUserFieldValue(3))
 
-def getRelation(sRelName, sItem, sObjName, aObjectList, host ):
+def getRelation(sRelName, sItem, sObjName, aObjectList, host):
         global url
         sock=RPCSession(url)
         global passwd
@@ -143,7 +149,7 @@
 		getRelation(res[k]['relation'], sItem[sItem.find(".")+1:], sObjName,aObjectList,host)
 
 
-def getPath(sPath,sMain):
+def getPath(sPath, sMain):
     desktop=getDesktop()
     doc =desktop.getCurrentComponent()
     oParEnum = doc.getTextFields().createEnumeration()
@@ -161,7 +167,7 @@
                         getPath(sPath, sMain)
     return sPath
 
-def EnumDocument(aItemList,aComponentAdd):
+def EnumDocument(aItemList, aComponentAdd):
     desktop = getDesktop()
     parent=""
     bFlag = False
@@ -183,7 +189,7 @@
             aItemList.append( templist )
         aComponentAdd.append( parent )
 
-def getChildTable(oPar,aItemList,aComponentAdd,sTableName=""):
+def getChildTable(oPar, aItemList, aComponentAdd, sTableName=""):
     sNames = oPar.getCellNames()
     bEmptyTableFlag=True
     for val in sNames:
@@ -229,7 +235,7 @@
                 aComponentAdd.append(sTableName+"."+oPar.Name)
     return 0
 
-def getRecersiveSection(oCurrentSection,aSectionList):
+def getRecersiveSection(oCurrentSection, aSectionList):
         desktop=getDesktop()
         doc =desktop.getCurrentComponent()
         oParEnum=doc.getText().createEnumeration()
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/lib/logreport.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/lib/logreport.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/lib/logreport.py	2012-03-05 18:43:21 +0000
@@ -39,7 +39,7 @@
     logger.setLevel(logging.INFO)
 
 class Logger(object):
-    def log_write(self,name,level,msg):
+    def log_write(self, name, level, msg):
         log = logging.getLogger(name)
         getattr(log,level)(msg)
 
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/lib/rpc.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/lib/rpc.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/lib/rpc.py	2012-03-05 18:43:21 +0000
@@ -52,7 +52,7 @@
 
 
 class RPCSession(object):
-    def __init__(self,url):
+    def __init__(self, url):
 
         m = re.match('^(http[s]?://|socket://)([\w.\-]+):(\d{1,5})$', url or '')
 
@@ -152,7 +152,7 @@
 
         return res
 
-    def execute(self, sDatabase,UID,sPassword,obj, method, *args):
+    def execute(self, sDatabase, UID, sPassword, obj, method, *args):
         global rpc_url
 
         sock = xmlrpclib.ServerProxy(rpc_url + 'object')
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/lib/tools.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/lib/tools.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/lib/tools.py	2012-03-05 18:43:21 +0000
@@ -21,19 +21,19 @@
 ##############################################################################
 import urllib
 
-def get_absolute_file_path( url ):
+def get_absolute_file_path(url):
 	url_unquoted = urllib.unquote(url)
 	return os.name == 'nt' and url_unquoted[1:] or url_unquoted 
 
 # This function reads the content of a file and return it to the caller
-def read_data_from_file( filename ):
+def read_data_from_file(filename):
 	fp = file( filename, "rb" )
 	data = fp.read()
 	fp.close()
 	return data
 
 # This function writes the content to a file
-def write_data_to_file( filename, data ):
+def write_data_to_file(filename, data):
 	fp = file( filename, 'wb' )
 	fp.write( data )
 	fp.close()
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/modify.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/modify.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/modify.py	2012-03-05 18:43:21 +0000
@@ -61,7 +61,7 @@
     uid = 3
 
 class modify(unohelper.Base, XJobExecutor ):
-    def __init__( self, ctx ):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/test/test_fields.py'
--- base_report_designer/plugin/openerp_report_designer/test/test_fields.py	2011-12-19 16:54:40 +0000
+++ base_report_designer/plugin/openerp_report_designer/test/test_fields.py	2012-03-05 18:43:21 +0000
@@ -10,7 +10,11 @@
 
 sock = xmlrpclib.ServerProxy('http://localhost:8069/xmlrpc/object')
 
-def get(object, level=3, ending=[], ending_excl=[], recur=[], root=''):
+def get(object, level=3, ending=None, ending_excl=None, recur=None, root=''):
+	if ending is None:
+	    ending = []
+	if ending_excl is None:
+	    ending_excl = []
 	res = sock.execute('terp', 3, 'admin', 'account.invoice', 'fields_get')
 	key = res.keys()
 	key.sort()
 
=== modified file 'base_setup/base_setup.py'
--- base_setup/base_setup.py	2011-12-19 16:54:40 +0000
+++ base_setup/base_setup.py	2012-03-05 18:43:21 +0000
@@ -84,7 +84,7 @@
     res = [(r['code'], r['name']) for r in res]
     return res
 
-def _tz_get(self,cr,uid, context=None):
+def _tz_get(self, cr, uid, context=None):
     return [(x, x) for x in pytz.all_timezones]
 
 class user_preferences_config(osv.osv_memory):
 
=== modified file 'base_synchro/base_synchro_obj.py'
--- base_synchro/base_synchro_obj.py	2012-02-28 11:16:26 +0000
+++ base_synchro/base_synchro_obj.py	2012-03-05 18:43:21 +0000
@@ -65,10 +65,12 @@
     # Return a list of changes: [ (date, id) ]
     #
 
-    def get_ids(self, cr, uid, object, dt, domain=[], context=None):
-        return self._get_ids(cr, uid, object, dt, domain, context=context)
+    def get_ids(self, cr, uid, object, dt, domain=None, context=None):
+        return self._get_ids(cr, uid, object, dt, domain=domain, context=context)
 
-    def _get_ids(self, cr, uid, object, dt, domain=[], context=None):
+    def _get_ids(self, cr, uid, object, dt, domain=None, context=None):
+        if domain is None:
+            domain = []
         result = []
         if dt:
             domain2 = domain+[('write_date','>=',dt)]
 
=== modified file 'base_vat/__init__.py'
--- base_vat/__init__.py	2011-12-19 16:54:40 +0000
+++ base_vat/__init__.py	2012-03-05 18:43:21 +0000
@@ -22,4 +22,4 @@
 import res_company
 import base_vat
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_vat/res_company.py'
--- base_vat/res_company.py	2011-11-28 16:40:14 +0000
+++ base_vat/res_company.py	2012-03-05 18:43:21 +0000
@@ -29,4 +29,4 @@
                                               "rather than via a simple format validation (checksum)."),
     }
 
-    
 \ No newline at end of file
+    
 
=== modified file 'caldav/caldav_node.py'
--- caldav/caldav_node.py	2011-10-27 21:11:24 +0000
+++ caldav/caldav_node.py	2012-03-05 18:43:21 +0000
@@ -42,7 +42,7 @@
     DAV_M_NS = dict_merge2(nodes.node_dir.DAV_M_NS,
             { "http://calendarserver.org/ns/" : '_get_dav', } )
 
-    def _file_get(self,cr, nodename=False):
+    def _file_get(self, cr, nodename=False):
         return []
 
     def _child_get(self, cr, name=False, parent_id=False, domain=None):
@@ -99,7 +99,7 @@
     DAV_M_NS = dict_merge2(nodes.node_res_obj.DAV_M_NS,
             { "http://calendarserver.org/ns/" : '_get_dav', } )
 
-    def _file_get(self,cr, nodename=False):
+    def _file_get(self, cr, nodename=False):
         return []
 
     def _child_get(self, cr, name=False, parent_id=False, domain=None):
@@ -180,7 +180,7 @@
 
     http_options = { 'DAV': ['calendar-access'] }
 
-    def __init__(self,path, parent, context, calendar):
+    def __init__(self, path, parent, context, calendar):
         super(node_calendar,self).__init__(path, parent,context)
         self.calendar_id = calendar.id
         self.mimetype = 'application/x-directory'
@@ -271,7 +271,7 @@
     def children(self, cr, domain=None):
         return self._child_get(cr, domain=domain)
 
-    def child(self,cr, name, domain=None):
+    def child(self, cr, name, domain=None):
         res = self._child_get(cr, name, domain=domain)
         if res:
             return res[0]
@@ -353,16 +353,16 @@
         return None
 
 
-    def set_data(self, cr, data, fil_obj = None):
+    def set_data(self, cr, data, fil_obj=None):
         uid = self.context.uid
         calendar_obj = self.context._dirobj.pool.get('basic.calendar')
         res = calendar_obj.import_cal(cr, uid, data, self.calendar_id)
         return res
 
-    def get_data_len(self, cr, fil_obj = None):
+    def get_data_len(self, cr, fil_obj=None):
         return self.content_length
 
-    def _get_ttag(self,cr):
+    def _get_ttag(self, cr):
         return 'calendar-%d' % (self.calendar_id,)
 
     def rmcol(self, cr):
@@ -441,7 +441,7 @@
 
     http_options = { 'DAV': ['calendar-access'] }
 
-    def __init__(self,path, parent, context, res_obj, res_model=None, res_id=None):
+    def __init__(self, path, parent, context, res_obj, res_model=None, res_id=None):
         super(res_node_calendar,self).__init__(path, parent, context)
         self.mimetype = 'text/calendar'
         self.create_date = parent.create_date
@@ -474,10 +474,10 @@
     def _get_caldav_calendar_data(self, cr):
         return self.get_data(cr)
 
-    def get_data_len(self, cr, fil_obj = None):
+    def get_data_len(self, cr, fil_obj=None):
         return self.content_length
 
-    def set_data(self, cr, data, fil_obj = None):
+    def set_data(self, cr, data, fil_obj=None):
         uid = self.context.uid
         context = self.context.context.copy()
         context.update(self.dctx)
@@ -486,7 +486,7 @@
         res =  calendar_obj.import_cal(cr, uid, data, self.calendar_id, context=context)
         return res
 
-    def _get_ttag(self,cr):
+    def _get_ttag(self, cr):
         res = False
         if self.model and self.res_id:
             res = '%s_%d' % (self.model, self.res_id)
 
=== modified file 'crm/crm.py'
--- crm/crm.py	2011-12-31 07:57:20 +0000
+++ crm/crm.py	2012-03-05 18:43:21 +0000
@@ -331,7 +331,9 @@
     And object that inherit (orm inheritance) from a class the overwrite copy 
     """
 
-    def stage_find(self, cr, uid, section_id, domain=[], order='sequence'):
+    def stage_find(self, cr, uid, section_id, domain=None, order='sequence'):
+        if domain is None:
+            domain = []
         domain = list(domain)
         if section_id:
             domain.append(('section_ids', '=', section_id))
 
=== modified file 'crm/crm_lead.py'
--- crm/crm_lead.py	2012-01-31 13:36:57 +0000
+++ crm/crm_lead.py	2012-03-05 18:43:21 +0000
@@ -239,7 +239,7 @@
     def on_change_optout(self, cr, uid, ids, optout):
         return {'value':{'optout':optout,'optin':False}}
 
-    def onchange_stage_id(self, cr, uid, ids, stage_id, context={}):
+    def onchange_stage_id(self, cr, uid, ids, stage_id, context=None):
         if not stage_id:
             return {'value':{}}
         stage = self.pool.get('crm.case.stage').browse(cr, uid, stage_id, context)
 
=== modified file 'crm_claim/crm_claim.py'
--- crm_claim/crm_claim.py	2011-12-21 22:15:04 +0000
+++ crm_claim/crm_claim.py	2012-03-05 18:43:21 +0000
@@ -157,7 +157,7 @@
         self.write(cr, uid, [res_id], vals, context=context)
         return res_id
 
-    def message_update(self, cr, uid, ids, msg, vals={}, default_act='pending', context=None):
+    def message_update(self, cr, uid, ids, msg, vals=None, default_act='pending', context=None):
         if isinstance(ids, (str, int, long)):
             ids = [ids]
 
 
=== modified file 'crm_helpdesk/crm_helpdesk.py'
--- crm_helpdesk/crm_helpdesk.py	2011-10-16 01:28:00 +0000
+++ crm_helpdesk/crm_helpdesk.py	2012-03-05 18:43:21 +0000
@@ -112,7 +112,7 @@
         self.write(cr, uid, [res_id], vals, context)
         return res_id
 
-    def message_update(self, cr, uid, ids, msg, vals={}, default_act='pending', context=None):
+    def message_update(self, cr, uid, ids, msg, vals=None, default_act='pending', context=None):
         if isinstance(ids, (str, int, long)):
             ids = [ids]
 
 
=== modified file 'crm_profiling/crm_profiling.py'
--- crm_profiling/crm_profiling.py	2011-12-19 16:54:40 +0000
+++ crm_profiling/crm_profiling.py	2012-03-05 18:43:21 +0000
@@ -77,7 +77,7 @@
     return ids_to_check
 
 
-def test_prof(cr, uid, seg_id, pid, answers_ids = []):
+def test_prof(cr, uid, seg_id, pid, answers_ids=None):
 
     """ return True if the partner pid fetch the segmentation rule seg_id
         @param cr: the current row, from the database cursor,
 
=== modified file 'delivery/wizard/delivery_sale_order.py'
--- delivery/wizard/delivery_sale_order.py	2012-02-13 15:27:55 +0000
+++ delivery/wizard/delivery_sale_order.py	2012-03-05 18:43:21 +0000
@@ -44,7 +44,7 @@
              
         return res
     
-    def view_init(self, cr , uid , fields, context=None):
+    def view_init(self, cr, uid, fields, context=None):
          if context is None:
             context = {}
          order_obj = self.pool.get('sale.order')
 
=== modified file 'document/content_index.py'
--- document/content_index.py	2011-12-31 07:57:20 +0000
+++ document/content_index.py	2012-03-05 18:43:21 +0000
@@ -51,7 +51,7 @@
             return mts[0]
         return None
 
-    def indexContent(self, content, filename=None, realfile = None):
+    def indexContent(self, content, filename=None, realfile=None):
         """ Use either content or the real file, to index.
             Some parsers will work better with the actual
             content, others parse a file easier. Try the
@@ -95,10 +95,10 @@
 
         raise NhException('No appropriate method to index file')
 
-    def _doIndexContent(self,content):
+    def _doIndexContent(self, content):
         raise NhException("Content not handled here")
 
-    def _doIndexFile(self,fpath):
+    def _doIndexFile(self, fpath):
         raise NhException("Content not handled here")
 
     def __repr__(self):
@@ -136,7 +136,7 @@
         if not f:
             raise Exception("Your indexer should at least suport a mimetype or extension")
 
-    def doIndex(self, content, filename=None, content_type=None, realfname = None, debug=False):
+    def doIndex(self, content, filename=None, content_type=None, realfname=None, debug=False):
         fobj = None
         fname = None
         mime = None
 
=== modified file 'document/document.py'
--- document/document.py	2011-12-19 16:54:40 +0000
+++ document/document.py	2012-03-05 18:43:21 +0000
@@ -146,7 +146,7 @@
     _sql_constraints = [
         # filename_uniq is not possible in pure SQL
     ]
-    def _check_duplication(self, cr, uid, vals, ids=[], op='create'):
+    def _check_duplication(self, cr, uid, vals, ids=None, op='create'):
         name = vals.get('name', False)
         parent_id = vals.get('parent_id', False)
         res_model = vals.get('res_model', False)
 
=== modified file 'document/document_directory.py'
--- document/document_directory.py	2011-12-19 16:54:40 +0000
+++ document/document_directory.py	2012-03-05 18:43:21 +0000
@@ -69,7 +69,7 @@
     }
 
 
-    def _get_root_directory(self, cr,uid, context=None):
+    def _get_root_directory(self, cr, uid, context=None):
         objid=self.pool.get('ir.model.data')
         try:
             mid = objid._get_id(cr, uid, 'document', 'dir_root')
@@ -224,7 +224,7 @@
                 pass
         return res
 
-    def _locate_child(self, cr, uid, root_id, uri,nparent, ncontext):
+    def _locate_child(self, cr, uid, root_id, uri, nparent, ncontext):
         """ try to locate the node in uri,
             Return a tuple (node_dir, remaining_path)
         """
@@ -237,7 +237,7 @@
         default.update({'name': name+ " (copy)"})
         return super(document_directory,self).copy(cr, uid, id, default, context=context)
 
-    def _check_duplication(self, cr, uid, vals, ids=[], op='create'):
+    def _check_duplication(self, cr, uid, vals, ids=None, op='create'):
         name=vals.get('name',False)
         parent_id=vals.get('parent_id',False)
         ressource_parent_type_id=vals.get('ressource_parent_type_id',False)
 
=== modified file 'document/nodes.py'
--- document/nodes.py	2011-12-19 16:54:40 +0000
+++ document/nodes.py	2012-03-05 18:43:21 +0000
@@ -104,7 +104,7 @@
     def get(self, name, default=None):
         return self.context.get(name, default)
 
-    def get_uri(self, cr,  uri):
+    def get_uri(self, cr, uri):
         """ Although this fn passes back to doc.dir, it is needed since
             it is a potential caching point.
         """
@@ -253,7 +253,7 @@
         print "node_class.children()"
         return [] #stub
 
-    def child(self,cr, name, domain=None):
+    def child(self, cr, name, domain=None):
         print "node_class.child()"
         return None
 
@@ -271,7 +271,7 @@
         print "node_class.path_get()"
         return False
 
-    def get_data(self,cr):
+    def get_data(self, cr):
         raise TypeError('no data for %s'% self.type)
 
     def open_data(self, cr, mode):
@@ -288,10 +288,10 @@
         """
         raise TypeError('no data for %s' % self.type)
 
-    def _get_storage(self,cr):
+    def _get_storage(self, cr):
         raise RuntimeError("no storage for base class")
 
-    def get_etag(self,cr):
+    def get_etag(self, cr):
         """ Get a tag, unique per object + modification.
 
             see. http://tools.ietf.org/html/rfc2616#section-13.3.3 """
@@ -435,7 +435,9 @@
 
     """
     our_type = 'database'
-    def __init__(self, path=[], parent=False, context=None):
+    def __init__(self, path=None, parent=False, context=None):
+        if path is None:
+            path = []
         super(node_database,self).__init__(path, parent, context)
         self.unixperms = 040750
         self.uidperms = 5
@@ -478,11 +480,11 @@
 
         return res
 
-    def _file_get(self,cr, nodename=False):
+    def _file_get(self, cr, nodename=False):
         res = []
         return res
 
-    def _get_ttag(self,cr):
+    def _get_ttag(self, cr):
         return 'db-%s' % cr.dbname
 
 def mkdosname(company_name, default='noname'):
@@ -694,7 +696,7 @@
             fnode.set_data(cr, data, fil)
         return fnode
 
-    def _get_ttag(self,cr):
+    def _get_ttag(self, cr):
         return 'dir-%d' % self.dir_id
 
     def move_to(self, cr, ndir_node, new_name=False, fil_obj=None, ndir_obj=None, in_write=False):
@@ -803,7 +805,7 @@
     def children(self, cr, domain=None):
         return self._child_get(cr, domain=domain)
 
-    def child(self,cr, name, domain=None):
+    def child(self, cr, name, domain=None):
         res = self._child_get(cr, name, domain=domain)
         if res:
             return res[0]
@@ -879,7 +881,7 @@
             res.append(self.res_obj_class(res_name, self.dir_id, self, self.context, self.res_model, bo))
         return res
 
-    def _get_ttag(self,cr):
+    def _get_ttag(self, cr):
         return 'rdir-%d' % self.dir_id
 
 class node_res_obj(node_class):
@@ -890,7 +892,7 @@
         node_dirs (with limited domain).
         """
     our_type = 'collection'
-    def __init__(self, path, dir_id, parent, context, res_model, res_bo, res_id = None):
+    def __init__(self, path, dir_id, parent, context, res_model, res_bo, res_id=None):
         super(node_res_obj,self).__init__(path, parent,context)
         assert parent
         #todo: more info from dirr
@@ -959,7 +961,7 @@
             return res[0]
         return None
 
-    def _file_get(self,cr, nodename=False):
+    def _file_get(self, cr, nodename=False):
         res = []
         is_allowed = self.check_perms((nodename and 1) or 5)
         if not is_allowed:
@@ -1161,7 +1163,7 @@
             fnode.set_data(cr, data, fil)
         return fnode
 
-    def _get_ttag(self,cr):
+    def _get_ttag(self, cr):
         return 'rodir-%d-%d' % (self.dir_id, self.res_id)
 
 node_res_dir.res_obj_class = node_res_obj
@@ -1263,7 +1265,7 @@
         else:
             self.path = dirpath[0]
 
-    def get_data(self, cr, fil_obj = None):
+    def get_data(self, cr, fil_obj=None):
         """ Retrieve the data for some file.
             fil_obj may optionally be specified, and should be a browse object
             for the file. This is useful when the caller has already initiated
@@ -1279,14 +1281,14 @@
         stobj = self.context._dirobj.pool.get('document.storage')
         return stobj.get_data(cr, self.context.uid,stor, self,self.context.context, fil_obj)
 
-    def get_data_len(self, cr, fil_obj = None):
+    def get_data_len(self, cr, fil_obj=None):
         # TODO: verify with the storage object!
         bin_size = self.context.context.get('bin_size', False)
         if bin_size and not self.content_length:
             self.content_length = fil_obj.db_datas
         return self.content_length
 
-    def set_data(self, cr, data, fil_obj = None):
+    def set_data(self, cr, data, fil_obj=None):
         """ Store data at some file.
             fil_obj may optionally be specified, and should be a browse object
             for the file. This is useful when the caller has already initiated
@@ -1300,7 +1302,7 @@
         stobj = self.context._dirobj.pool.get('document.storage')
         return stobj.set_data(cr, self.context.uid,stor, self, data, self.context.context, fil_obj)
 
-    def _get_ttag(self,cr):
+    def _get_ttag(self, cr):
         return 'file-%d' % self.file_id
 
     def move_to(self, cr, ndir_node, new_name=False, fil_obj=None, ndir_obj=None, in_write=False):
@@ -1363,7 +1365,7 @@
 
 class node_content(node_class):
     our_type = 'content'
-    def __init__(self, path, parent, context, cnt, dctx = None, act_id=None):
+    def __init__(self, path, parent, context, cnt, dctx=None, act_id=None):
         super(node_content,self).__init__(path, parent,context)
         self.cnt_id = cnt.id
         self.create_date = False
@@ -1383,7 +1385,7 @@
            self.dctx.update(dctx)
         self.act_id = act_id
 
-    def fill_fields(self, cr, dctx = None):
+    def fill_fields(self, cr, dctx=None):
         """ Try to read the object and fill missing fields, like mimetype,
             dates etc.
             This function must be different from the constructor, because
@@ -1397,7 +1399,7 @@
             self.mimetype = str(res[0][0])
 
 
-    def get_data(self, cr, fil_obj = None):
+    def get_data(self, cr, fil_obj=None):
         cntobj = self.context._dirobj.pool.get('document.directory.content')
         if not self.check_perms(4):
             raise IOError(errno.EPERM, "Permission denied")
@@ -1427,7 +1429,7 @@
         
         return nodefd_content(self, cr, mode, ctx)
 
-    def get_data_len(self, cr, fil_obj = None):
+    def get_data_len(self, cr, fil_obj=None):
         # FIXME : here, we actually generate the content twice!!
         # we should have cached the generated content, but it is
         # not advisable to do keep it in memory, until we have a cache
@@ -1436,7 +1438,7 @@
             self.get_data(cr,fil_obj)
         return self.content_length
 
-    def set_data(self, cr, data, fil_obj = None):
+    def set_data(self, cr, data, fil_obj=None):
         cntobj = self.context._dirobj.pool.get('document.directory.content')
         if not self.check_perms(2):
             raise IOError(errno.EPERM, "Permission denied")
@@ -1445,7 +1447,7 @@
         ctx.update(self.dctx)
         return cntobj.process_write(cr, self.context.uid, self, data, ctx)
 
-    def _get_ttag(self,cr):
+    def _get_ttag(self, cr):
         return 'cnt-%d%s' % (self.cnt_id,(self.act_id and ('-' + str(self.act_id))) or '')
 
     def get_dav_resourcetype(self, cr):
 
=== modified file 'document/odt2txt.py' (properties changed: +x to -x)
--- document/odt2txt.py	2011-09-22 09:55:31 +0000
+++ document/odt2txt.py	2012-03-05 18:43:21 +0000
@@ -24,11 +24,11 @@
 import StringIO
 
 class OpenDocumentTextFile :
-    def __init__ (self, filepath) :
+    def __init__ (self, filepath):
         zip = zipfile.ZipFile(filepath)
         self.content = xml.dom.minidom.parseString(zip.read("content.xml"))
 
-    def toString (self) :
+    def toString (self):
         """ Converts the document to a string. """
         buffer = u""
         for val in ["text:p", "text:h", "text:list"]:
@@ -36,7 +36,7 @@
                 buffer += self.textToString(paragraph) + "\n"
         return buffer
 
-    def textToString(self, element) :
+    def textToString(self, element):
         buffer = u""
         for node in element.childNodes :
             if node.nodeType == xml.dom.Node.TEXT_NODE :
 
=== modified file 'document/std_index.py'
--- document/std_index.py	2011-12-19 16:54:40 +0000
+++ document/std_index.py	2012-03-05 18:43:21 +0000
@@ -38,7 +38,7 @@
             except UnicodeError:
                 return s
 
-def textToString(element) :
+def textToString(element):
     buffer = u""
     for node in element.childNodes :
         if node.nodeType == xml.dom.Node.TEXT_NODE :
@@ -55,7 +55,7 @@
     def _getExtensions(self):
         return ['.txt', '.py']
 
-    def _doIndexContent(self,content):
+    def _doIndexContent(self, content):
         return content
 
 cntIndex.register(TxtIndex())
@@ -67,7 +67,7 @@
     def _getExtensions(self):
         return ['.pptx']
 
-    def _doIndexFile(self,fname):
+    def _doIndexFile(self, fname):
         def toString () :
             """ Converts the document to a string. """
             buffer = u""
@@ -95,7 +95,7 @@
     def _getExtensions(self):
         return ['.doc']
 
-    def _doIndexFile(self,fname):
+    def _doIndexFile(self, fname):
         try:
             pop = Popen(['antiword', fname], shell=False, stdout=PIPE)
             (data, _) = pop.communicate()
@@ -115,7 +115,7 @@
     def _getExtensions(self):
         return ['.docx']
 
-    def _doIndexFile(self,fname):
+    def _doIndexFile(self, fname):
         zip = zipfile.ZipFile(fname)
         content = xml.dom.minidom.parseString(zip.read("word/document.xml"))
         def toString () :
@@ -140,7 +140,7 @@
     def _getExtensions(self):
         return ['.xlsx']
 
-    def _doIndexFile(self,fname):
+    def _doIndexFile(self, fname):
         zip = zipfile.ZipFile(fname)
         content = xml.dom.minidom.parseString(zip.read("xl/sharedStrings.xml"))
         def toString () :
@@ -164,7 +164,7 @@
     def _getExtensions(self):
         return ['.pdf']
 
-    def _doIndexFile(self,fname):
+    def _doIndexFile(self, fname):
         pop = Popen(['pdftotext', '-enc', 'UTF-8', '-nopgbrk', fname, '-'], shell=False, stdout=PIPE)
         (data, _) = pop.communicate()
         return _to_unicode(data)
@@ -180,7 +180,7 @@
         return []
         #return ['.png','.jpg','.gif','.jpeg','.bmp','.tiff']
 
-    def _doIndexContent(self,content):
+    def _doIndexContent(self, content):
         return 'image'
 
 
 
=== modified file 'document_webdav/document_webdav.py'
--- document_webdav/document_webdav.py	2011-12-19 16:54:40 +0000
+++ document_webdav/document_webdav.py	2012-03-05 18:43:21 +0000
@@ -57,7 +57,7 @@
         # TODO group
         return
 
-    def _locate_child(self, cr, uid, root_id, uri,nparent, ncontext):
+    def _locate_child(self, cr, uid, root_id, uri, nparent, ncontext):
         """ try to locate the node in uri,
             Return a tuple (node_dir, remaining_path)
         """
 
=== modified file 'document_webdav/redirect.py'
--- document_webdav/redirect.py	2011-12-19 16:54:40 +0000
+++ document_webdav/redirect.py	2012-03-05 18:43:21 +0000
@@ -30,7 +30,7 @@
     _HTTP_OPTIONS = { 'Allow': ['OPTIONS', 'GET', 'HEAD', 'PROPFIND'] }
     redirect_paths = {}
 
-    def __init__(self,request, client_address, server):
+    def __init__(self, request, client_address, server):
         HTTPHandler.__init__(self,request,client_address,server)
 
     def send_head(self):
 
=== modified file 'document_webdav/test_davclient.py' (properties changed: +x to -x)
--- document_webdav/test_davclient.py	2011-12-19 16:54:40 +0000
+++ document_webdav/test_davclient.py	2012-03-05 18:43:21 +0000
@@ -682,7 +682,7 @@
             assert d2 == d, "Data does not match"
         return ctype, rrange, d
 
-    def gd_put(self, path, body=None, srcpath=None, mime=None, noclobber=False, ):
+    def gd_put(self, path, body=None, srcpath=None, mime=None, noclobber=False):
         """ HTTP PUT 
             @param noclobber will prevent overwritting a resource (If-None-Match)
             @param mime will set the content-type
 
=== modified file 'document_webdav/webdav.py'
--- document_webdav/webdav.py	2011-12-19 16:54:40 +0000
+++ document_webdav/webdav.py	2012-03-05 18:43:21 +0000
@@ -236,7 +236,7 @@
     return re
 
 
-def mk_propname_response(self,uri,propnames,doc):
+def mk_propname_response(self, uri, propnames, doc):
     """ make a new <prop> result element for a PROPNAME request
 
     This will simply format the propnames list.
 
=== modified file 'document_webdav/webdav_server.py'
--- document_webdav/webdav_server.py	2012-02-09 09:06:22 +0000
+++ document_webdav/webdav_server.py	2012-03-05 18:43:21 +0000
@@ -81,7 +81,7 @@
                             'DELETE', 'TRACE', 'REPORT', ]
                     }
 
-    def get_userinfo(self,user,pw):
+    def get_userinfo(self, user, pw):
         return False
 
     def _log(self, message):
@@ -167,7 +167,7 @@
             self.close_connection = 1
         DAVRequestHandler.send_header(self, key, value)
 
-    def send_body(self, DATA, code = None, msg = None, desc = None, ctype='application/octet-stream', headers=None):
+    def send_body(self, DATA, code=None, msg=None, desc=None, ctype='application/octet-stream', headers=None):
         if headers and 'Connection' in headers:
             pass
         elif self.request_version in ('HTTP/1.0', 'HTTP/0.9'):
@@ -441,10 +441,10 @@
     def __init__(self, parent):
         self.parent = parent
 
-    def get_propnames(self,uri):
+    def get_propnames(self, uri):
         return self.PROPS
 
-    def get_prop(self,uri,ns,propname):
+    def get_prop(self, uri, ns, propname):
         if self.M_NS.has_key(ns):
             prefix=self.M_NS[ns]
         else:
@@ -460,10 +460,10 @@
     def get_data(self, uri, range=None):
         raise DAV_NotFound
 
-    def _get_dav_creationdate(self,uri):
+    def _get_dav_creationdate(self, uri):
         return time.strftime("%Y-%m-%dT%H:%M:%SZ", time.gmtime())
 
-    def _get_dav_getlastmodified(self,uri):
+    def _get_dav_getlastmodified(self, uri):
         return time.strftime("%a, %d %b %Y %H:%M:%S GMT", time.gmtime())
 
     def _get_dav_displayname(self, uri):
 
=== modified file 'email_template/html2text.py' (properties changed: +x to -x)
--- email_template/html2text.py	2011-12-19 16:54:40 +0000
+++ email_template/html2text.py	2012-03-05 18:43:21 +0000
@@ -158,7 +158,7 @@
         self.abbr_list = {} # stack of abbreviations to write later
         self.baseurl = baseurl
     
-    def outtextf(self, s): 
+    def outtextf(self, s):
         self.outtext += s
     
     def close(self):
@@ -338,7 +338,7 @@
     def pbr(self):
         if self.p_p == 0: self.p_p = 1
 
-    def p(self): self.p_p = 2
+    def p(self):
     
     def o(self, data, puredata=0, force=0):
         if self.abbr_data is not None: self.abbr_data += data
@@ -411,7 +411,8 @@
         if r'\/script>' in data: self.quiet -= 1
         self.o(data, 1)
     
-    def unknown_decl(self, data): pass
+    def unknown_decl(self, data):
+        pass
 
 def wrapwrite(text): sys.stdout.write(text.encode('utf8'))
 
 
=== modified file 'email_template/wizard/mail_compose_message.py'
--- email_template/wizard/mail_compose_message.py	2012-01-31 13:36:57 +0000
+++ email_template/wizard/mail_compose_message.py	2012-03-05 18:43:21 +0000
@@ -27,7 +27,7 @@
 import tools
 
 
-def _reopen(self,res_id,model):
+def _reopen(self, res_id, model):
     return {'type': 'ir.actions.act_window',
             'view_mode': 'form',
             'view_type': 'form',
 
=== modified file 'event/event.py'
--- event/event.py	2012-03-01 11:39:44 +0000
+++ event/event.py	2012-03-05 18:43:21 +0000
@@ -66,7 +66,7 @@
             res.append((record['id'], name))
         return res
 
-    def _name_get_fnc(self, cr, uid, ids,prop,unknow, context=None):
+    def _name_get_fnc(self, cr, uid, ids, prop, unknow, context=None):
         res = self.name_get(cr, uid, ids, context=context)
         return dict(res)
 
 
=== modified file 'event/report/__init__.py'
--- event/report/__init__.py	2011-01-14 00:11:01 +0000
+++ event/report/__init__.py	2012-03-05 18:43:21 +0000
@@ -21,4 +21,4 @@
 
 import report_event_registration
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'event/res_partner.py'
--- event/res_partner.py	2012-02-24 13:20:14 +0000
+++ event/res_partner.py	2012-03-05 18:43:21 +0000
@@ -32,4 +32,4 @@
 
 res_partner()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'event/wizard/event_confirm.py'
--- event/wizard/event_confirm.py	2011-01-14 00:11:01 +0000
+++ event/wizard/event_confirm.py	2012-03-05 18:43:21 +0000
@@ -34,4 +34,4 @@
 
 event_confirm()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'event_moodle/event_moodle.py'
--- event_moodle/event_moodle.py	2012-02-28 14:48:22 +0000
+++ event_moodle/event_moodle.py	2012-03-05 18:43:21 +0000
@@ -125,7 +125,7 @@
         passwd = passwd + '+'
         return passwd
 
-    def check_email(self,email):
+    def check_email(self, email):
     
         """
         check if email is correct
 
=== modified file 'event_project/event_project.py'
--- event_project/event_project.py	2011-01-14 00:11:01 +0000
+++ event_project/event_project.py	2012-03-05 18:43:21 +0000
@@ -55,4 +55,4 @@
     
 event()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'event_project/wizard/event_project_retro.py'
--- event_project/wizard/event_project_retro.py	2011-01-14 00:11:01 +0000
+++ event_project/wizard/event_project_retro.py	2012-03-05 18:43:21 +0000
@@ -80,4 +80,4 @@
 
 event_project()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_attendance/wizard/__init__.py'
--- hr_attendance/wizard/__init__.py	2011-01-14 00:11:01 +0000
+++ hr_attendance/wizard/__init__.py	2012-03-05 18:43:21 +0000
@@ -24,4 +24,4 @@
 import hr_attendance_byweek
 import hr_attendance_bymonth
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_attendance/wizard/hr_attendance_bymonth.py'
--- hr_attendance/wizard/hr_attendance_bymonth.py	2011-12-16 05:52:38 +0000
+++ hr_attendance/wizard/hr_attendance_bymonth.py	2012-03-05 18:43:21 +0000
@@ -50,4 +50,4 @@
 
 hr_attendance_bymonth()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_attendance/wizard/hr_attendance_byweek.py'
--- hr_attendance/wizard/hr_attendance_byweek.py	2011-01-14 00:11:01 +0000
+++ hr_attendance/wizard/hr_attendance_byweek.py	2012-03-05 18:43:21 +0000
@@ -48,4 +48,4 @@
 
 hr_attendance_byweek()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_attendance/wizard/hr_attendance_error.py'
--- hr_attendance/wizard/hr_attendance_error.py	2011-01-14 00:11:01 +0000
+++ hr_attendance/wizard/hr_attendance_error.py	2012-03-05 18:43:21 +0000
@@ -66,4 +66,4 @@
 
 hr_attendance_error()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_evaluation/hr_evaluation.py'
--- hr_evaluation/hr_evaluation.py	2012-02-13 15:27:55 +0000
+++ hr_evaluation/hr_evaluation.py	2012-03-05 18:43:21 +0000
@@ -244,7 +244,7 @@
                 raise osv.except_osv(_('Warning !'),_("You cannot change state, because some appraisal in waiting answer or draft state"))
         return True
 
-    def button_done(self,cr, uid, ids, context=None):
+    def button_done(self, cr, uid, ids, context=None):
         self.write(cr, uid, ids,{'progress': 1 * 100}, context=context)
         self.write(cr, uid, ids,{'state':'done', 'date_close': time.strftime('%Y-%m-%d')}, context=context)
         return True
 
=== modified file 'hr_payroll/report/report_contribution_register.py'
--- hr_payroll/report/report_contribution_register.py	2011-06-29 09:20:32 +0000
+++ hr_payroll/report/report_contribution_register.py	2012-03-05 18:43:21 +0000
@@ -72,4 +72,4 @@
 
 report_sxw.report_sxw('report.contribution.register.lines', 'hr.contribution.register', 'hr_payroll/report/report_contribution_register.rml', parser=contribution_register_report)
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_payroll/report/report_payslip.py'
--- hr_payroll/report/report_payslip.py	2011-12-19 16:54:40 +0000
+++ hr_payroll/report/report_payslip.py	2012-03-05 18:43:21 +0000
@@ -46,4 +46,4 @@
 
 report_sxw.report_sxw('report.payslip', 'hr.payslip', 'hr_payroll/report/report_payslip.rml', parser=payslip_report)
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_payroll/report/report_payslip_details.py'
--- hr_payroll/report/report_payslip_details.py	2011-09-14 13:11:06 +0000
+++ hr_payroll/report/report_payslip_details.py	2012-03-05 18:43:21 +0000
@@ -116,4 +116,4 @@
 
 report_sxw.report_sxw('report.paylip.details', 'hr.payslip', 'hr_payroll/report/report_payslip_details.rml', parser=payslip_details_report)
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_payroll_account/__init__.py'
--- hr_payroll_account/__init__.py	2011-09-15 05:45:39 +0000
+++ hr_payroll_account/__init__.py	2012-03-05 18:43:21 +0000
@@ -23,4 +23,4 @@
 import hr_payroll_account
 import wizard
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_payroll_account/wizard/__init__.py'
--- hr_payroll_account/wizard/__init__.py	2011-09-15 05:45:39 +0000
+++ hr_payroll_account/wizard/__init__.py	2012-03-05 18:43:21 +0000
@@ -22,4 +22,4 @@
 
 import hr_payroll_payslips_by_employees
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_payroll_account/wizard/hr_payroll_payslips_by_employees.py'
--- hr_payroll_account/wizard/hr_payroll_payslips_by_employees.py	2011-09-15 05:45:39 +0000
+++ hr_payroll_account/wizard/hr_payroll_payslips_by_employees.py	2012-03-05 18:43:21 +0000
@@ -37,4 +37,4 @@
 
 hr_payslip_employees()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_recruitment/hr_recruitment.py'
--- hr_recruitment/hr_recruitment.py	2011-12-31 07:57:20 +0000
+++ hr_recruitment/hr_recruitment.py	2012-03-05 18:43:21 +0000
@@ -213,7 +213,7 @@
     }
 
 
-    def onchange_job(self,cr, uid, ids, job, context=None):
+    def onchange_job(self, cr, uid, ids, job, context=None):
         result = {}
 
         if job:
 
=== modified file 'hr_recruitment/wizard/hr_recruitment_create_partner_job.py'
--- hr_recruitment/wizard/hr_recruitment_create_partner_job.py	2011-01-14 00:11:01 +0000
+++ hr_recruitment/wizard/hr_recruitment_create_partner_job.py	2012-03-05 18:43:21 +0000
@@ -29,7 +29,7 @@
         'close': fields.boolean('Close job request'),
                 }
 
-    def view_init(self, cr , uid , fields_list, context=None):
+    def view_init(self, cr, uid, fields_list, context=None):
         case_obj = self.pool.get('hr.applicant')
         if context is None:
             context = {}
@@ -88,4 +88,4 @@
 
 hr_recruitment_partner_create()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_recruitment/wizard/hr_recruitment_employee_hired.py'
--- hr_recruitment/wizard/hr_recruitment_employee_hired.py	2011-11-17 10:53:01 +0000
+++ hr_recruitment/wizard/hr_recruitment_employee_hired.py	2012-03-05 18:43:21 +0000
@@ -26,7 +26,7 @@
     _name = 'hired.employee'
     _description = 'Create Employee'
 
-    def case_close(self, cr, uid,ids, context=None):
+    def case_close(self, cr, uid, ids, context=None):
         """
         @param self: The object pointer
         @param cr: the current row, from the database cursor,
@@ -39,7 +39,7 @@
         self.pool.get('hr.applicant').case_close(cr, uid,context.get('active_ids',[]))
         return {}
 
-    def case_close_with_emp(self, cr, uid,ids, context=None):
+    def case_close_with_emp(self, cr, uid, ids, context=None):
         """
         @param self: The object pointer
         @param cr: the current row, from the database cursor,
 
=== modified file 'hr_timesheet_invoice/hr_timesheet_invoice.py'
--- hr_timesheet_invoice/hr_timesheet_invoice.py	2012-01-03 12:33:39 +0000
+++ hr_timesheet_invoice/hr_timesheet_invoice.py	2012-03-05 18:43:21 +0000
@@ -77,7 +77,7 @@
     _defaults = {
         'pricelist_id': lambda self, cr, uid, ctx: ctx.get('pricelist_id', False),
     }
-    def on_change_partner_id(self, cr, uid, id, partner_id, context={}):
+    def on_change_partner_id(self, cr, uid, id, partner_id, context=None):
         res = super(account_analytic_account, self).on_change_partner_id(cr, uid, id, partner_id, context)
         if (not res.get('value', False)) or not partner_id:
             return res
 
=== modified file 'hr_timesheet_invoice/report/account_analytic_profit.py'
--- hr_timesheet_invoice/report/account_analytic_profit.py	2011-01-14 00:11:01 +0000
+++ hr_timesheet_invoice/report/account_analytic_profit.py	2012-03-05 18:43:21 +0000
@@ -120,4 +120,4 @@
 
 report_sxw.report_sxw('report.account.analytic.profit', 'account.analytic.line', 'addons/hr_timesheet_invoice/report/account_analytic_profit.rml', parser=account_analytic_profit)
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'hr_timesheet_invoice/wizard/hr_timesheet_invoice_create.py'
--- hr_timesheet_invoice/wizard/hr_timesheet_invoice_create.py	2011-11-16 14:59:58 +0000
+++ hr_timesheet_invoice/wizard/hr_timesheet_invoice_create.py	2012-03-05 18:43:21 +0000
@@ -38,7 +38,7 @@
     #   'price': boolean
     #   'product': many2one id
     # }
-    def invoice_cost_create(self, cr, uid, ids, data={}, context=None):
+    def invoice_cost_create(self, cr, uid, ids, data=None, context=None):
         analytic_account_obj = self.pool.get('account.analytic.account')
         res_partner_obj = self.pool.get('res.partner')
         account_payment_term_obj = self.pool.get('account.payment.term')
 
=== modified file 'hr_timesheet_sheet/hr_timesheet_sheet.py'
--- hr_timesheet_sheet/hr_timesheet_sheet.py	2012-02-13 15:27:55 +0000
+++ hr_timesheet_sheet/hr_timesheet_sheet.py	2012-03-05 18:43:21 +0000
@@ -374,7 +374,7 @@
         'department_id':fields.many2one('hr.department','Department'),
     }
 
-    def _default_date_from(self,cr, uid, context=None):
+    def _default_date_from(self, cr, uid, context=None):
         user = self.pool.get('res.users').browse(cr, uid, uid, context=context)
         r = user.company_id and user.company_id.timesheet_range or 'month'
         if r=='month':
@@ -385,7 +385,7 @@
             return time.strftime('%Y-01-01')
         return time.strftime('%Y-%m-%d')
 
-    def _default_date_to(self,cr, uid, context=None):
+    def _default_date_to(self, cr, uid, context=None):
         user = self.pool.get('res.users').browse(cr, uid, uid, context=context)
         r = user.company_id and user.company_id.timesheet_range or 'month'
         if r=='month':
 
=== modified file 'hr_timesheet_sheet/wizard/hr_timesheet_current.py'
--- hr_timesheet_sheet/wizard/hr_timesheet_current.py	2011-01-14 00:11:01 +0000
+++ hr_timesheet_sheet/wizard/hr_timesheet_current.py	2012-03-05 18:43:21 +0000
@@ -61,4 +61,4 @@
 
 hr_timesheet_current_open()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'idea/idea.py'
--- idea/idea.py	2011-12-31 07:57:20 +0000
+++ idea/idea.py	2012-03-05 18:43:21 +0000
@@ -234,7 +234,7 @@
         res_id = super(idea_idea, self).create(cr, user, vals, context=context)
         return res_id
 
-    def copy(self, cr, uid, id, default={}, context=None):
+    def copy(self, cr, uid, id, default=None, context=None):
         """
         Create the new record in idea_idea model from existing one
         @param cr: A database cursor
 
=== modified file 'import_base/import_framework.py'
--- import_base/import_framework.py	2011-12-31 07:57:20 +0000
+++ import_base/import_framework.py	2012-03-05 18:43:21 +0000
@@ -228,7 +228,7 @@
         return map
 
 
-    def _fields_mapp(self,dict_sugar, openerp_dict, table):
+    def _fields_mapp(self, dict_sugar, openerp_dict, table):
         """
             call all the mapper and transform data
             to be compatible with import_data
 
=== modified file 'import_sugarcrm/import_sugarcrm.py'
--- import_sugarcrm/import_sugarcrm.py	2012-02-16 17:29:41 +0000
+++ import_sugarcrm/import_sugarcrm.py	2012-03-05 18:43:21 +0000
@@ -253,7 +253,7 @@
             val['datas_fname'] = Filename
         return val    
     
-    def get_history_mapping(self): 
+    def get_history_mapping(self):
         return { 
                 'model' : 'ir.attachment',
                 'dependencies' : [self.TABLE_USER, self.TABLE_ACCOUNT, self.TABLE_CONTACT, self.TABLE_LEAD, self.TABLE_OPPORTUNITY, self.TABLE_MEETING, self.TABLE_CALL, self.TABLE_EMAIL],
@@ -301,7 +301,7 @@
         val['email_from'] = partner_email
         return val
     
-    def get_crm_claim_mapping(self): 
+    def get_crm_claim_mapping(self):
         return { 
                 'model' : 'crm.claim',
                 'dependencies' : [self.TABLE_USER, self.TABLE_ACCOUNT, self.TABLE_CONTACT, self.TABLE_LEAD],
@@ -604,7 +604,7 @@
                     partner_contact_email = address.email
         return partner_contact_id, partner_contact_email
 
-    def import_opp(self, val):    
+    def import_opp(self, val):
         partner_contact_id, partner_contact_email = self.import_opportunity_contact(val)
         val['partner_address_id/id'] = partner_contact_id
         val['email_from'] = partner_contact_email
 
=== modified file 'l10n_be/__init__.py'
--- l10n_be/__init__.py	2011-01-14 00:11:01 +0000
+++ l10n_be/__init__.py	2012-03-05 18:43:21 +0000
@@ -22,4 +22,4 @@
 import company
 import wizard
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'l10n_be_invoice_bba/invoice.py'
--- l10n_be_invoice_bba/invoice.py	2011-12-19 16:54:40 +0000
+++ l10n_be_invoice_bba/invoice.py	2012-03-05 18:43:21 +0000
@@ -35,7 +35,7 @@
 class account_invoice(osv.osv):
     _inherit = 'account.invoice'
 
-    def _get_reference_type(self, cursor, user, context=None):
+    def _get_reference_type(self, cursor, user, context=None):
         """Add BBA Structured Communication Type and change labels from 'reference' into 'communication' """  
         res = super(account_invoice, self)._get_reference_type(cursor, user,
                 context=context)
@@ -44,7 +44,7 @@
         #logger.notifyChannel('addons.'+self._name, netsvc.LOG_WARNING, 'reference_type =  %s' %res ) 
         return res
 
-    def check_bbacomm(self, val):
+    def check_bbacomm(self, val):
         supported_chars = '0-9+*/ '
         pattern = re.compile('[^' + supported_chars + ']')
         if pattern.findall(val or ''):
@@ -57,7 +57,7 @@
                 return True
         return False
 
-    def _check_communication(self, cr, uid, ids):
+    def _check_communication(self, cr, uid, ids):
         for inv in self.browse(cr, uid, ids):
             if inv.reference_type == 'bba':
                 return self.check_bbacomm(inv.reference)
@@ -86,7 +86,7 @@
         result['value'].update(res_update)
         return result                    
 
-    def generate_bbacomm(self, cr, uid, ids, type, reference_type, algorithm, partner_id, reference):
+    def generate_bbacomm(self, cr, uid, ids, type, reference_type, algorithm, partner_id, reference):
         partner_obj =  self.pool.get('res.partner')
         reference = reference or ''  
         if (type == 'out_invoice'):
@@ -157,7 +157,7 @@
                           "\nPlease contact your OpenERP support channel.") % algorithm)   
         return {'value': {'reference': reference}}    
     
-    def create(self, cr, uid, vals, context=None):
+    def create(self, cr, uid, vals, context=None):
         if vals.has_key('reference_type'):
             reference_type = vals['reference_type']
             if reference_type == 'bba':               
@@ -179,7 +179,7 @@
                               '\nPlease create manually a unique BBA Structured Communication.'))                 
         return super(account_invoice, self).create(cr, uid, vals, context=context)     
 
-    def write(self, cr, uid, ids, vals, context={}):
+    def write(self, cr, uid, ids, vals, context=None):
         if isinstance(ids, (int, long)):
             ids = [ids]
         for inv in self.browse(cr, uid, ids, context):    
 
=== modified file 'l10n_br/l10n_br.py'
--- l10n_br/l10n_br.py	2011-10-03 13:32:03 +0000
+++ l10n_br/l10n_br.py	2012-03-05 18:43:21 +0000
@@ -81,4 +81,4 @@
     
 l10n_br_account_cst()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'l10n_ch/partner.py'
--- l10n_ch/partner.py	2011-09-16 14:32:55 +0000
+++ l10n_ch/partner.py	2012-03-05 18:43:21 +0000
@@ -30,4 +30,4 @@
     }
 
 res_partner()
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'l10n_ch/payment.py'
--- l10n_ch/payment.py	2011-01-14 12:41:54 +0000
+++ l10n_ch/payment.py	2012-03-05 18:43:21 +0000
@@ -24,7 +24,7 @@
 class payment_order(osv.osv):
     _inherit = 'payment.order'
 
-    def get_wizard(self,mode):
+    def get_wizard(self, mode):
         if mode == 'dta':
             return 'l10n_ch', 'action_dta_create'
 
 
=== modified file 'l10n_ch/report/report_webkit_html.py'
--- l10n_ch/report/report_webkit_html.py	2012-02-13 14:44:56 +0000
+++ l10n_ch/report/report_webkit_html.py	2012-03-05 18:43:21 +0000
@@ -67,7 +67,7 @@
             self._check(ids)
         return super(l10n_ch_report_webkit_html, self).set_context(objects, data, ids, report_type=report_type)
     
-    def police_absolute_path(self, inner_path) :
+    def police_absolute_path(self, inner_path):
         """Will get the ocrb police absolute path"""
         path = addons.get_module_resource(os.path.join('l10n_ch', 'report', inner_path))
         return  path
 
=== modified file 'l10n_ch/wizard/create_dta.py'
--- l10n_ch/wizard/create_dta.py	2012-01-17 17:21:53 +0000
+++ l10n_ch/wizard/create_dta.py	2012-03-05 18:43:21 +0000
@@ -41,7 +41,7 @@
     (u'ä','a'),
 ]
 
-def _u2a(text) :
+def _u2a(text):
     """Tries to convert unicode charactere to asci equivalence"""
     if not text : return ""
     txt = ""
 
=== modified file 'l10n_fr/l10n_fr.py'
--- l10n_fr/l10n_fr.py	2011-01-14 00:11:01 +0000
+++ l10n_fr/l10n_fr.py	2012-03-05 18:43:21 +0000
@@ -51,4 +51,4 @@
 
 l10n_fr_line()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'l10n_fr/report/base_report.py'
--- l10n_fr/report/base_report.py	2011-01-14 00:11:01 +0000
+++ l10n_fr/report/base_report.py	2012-03-05 18:43:21 +0000
@@ -117,4 +117,4 @@
                         break
         self._set_variable(code, sum)
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'l10n_fr/wizard/fr_report_bilan.py'
--- l10n_fr/wizard/fr_report_bilan.py	2011-01-14 00:11:01 +0000
+++ l10n_fr/wizard/fr_report_bilan.py	2012-03-05 18:43:21 +0000
@@ -54,4 +54,4 @@
 
 account_bilan_report()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'l10n_fr/wizard/fr_report_compute_resultant.py'
--- l10n_fr/wizard/fr_report_compute_resultant.py	2011-01-14 00:11:01 +0000
+++ l10n_fr/wizard/fr_report_compute_resultant.py	2012-03-05 18:43:21 +0000
@@ -54,4 +54,4 @@
 
 account_cdr_report()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'l10n_lu/wizard/pdf_ext.py'
--- l10n_lu/wizard/pdf_ext.py	2011-01-14 00:11:01 +0000
+++ l10n_lu/wizard/pdf_ext.py	2012-03-05 18:43:21 +0000
@@ -57,7 +57,7 @@
 %%EOF
 """
 
-def output_field( f ):
+def output_field(f):
     return "\xfe\xff" + "".join( [ "\x00"+c for c in f ] )
 
 def extract_keys(lines):
 
=== modified file 'lunch/report/order.py'
--- lunch/report/order.py	2011-01-14 00:11:01 +0000
+++ lunch/report/order.py	2012-03-05 18:43:21 +0000
@@ -26,14 +26,14 @@
 
 class order(report_sxw.rml_parse):
 
-    def get_lines(self, user,objects):
+    def get_lines(self, user, objects):
         lines=[]
         for obj in objects:
             if user.id==obj.user_id.id:
                 lines.append(obj)
         return lines
 
-    def get_total(self, user,objects):
+    def get_total(self, user, objects):
         lines=[]
         for obj in objects:
             if user.id==obj.user_id.id:
 
=== modified file 'mail/mail_thread.py'
--- mail/mail_thread.py	2011-12-19 16:54:40 +0000
+++ mail/mail_thread.py	2012-03-05 18:43:21 +0000
@@ -134,7 +134,7 @@
         self.message_append_dict(cr, uid, [res_id], msg_dict, context=context)
         return res_id
 
-    def message_update(self, cr, uid, ids, msg_dict, vals={}, default_act=None, context=None):
+    def message_update(self, cr, uid, ids, msg_dict, vals=None, default_act=None, context=None):
         """Called by ``message_process`` when a new message is received
            for an existing thread. The default behavior is to create a
            new mail.message in the given thread (by calling
 
=== modified file 'mail/static/scripts/openerp_mailgate.py' (properties changed: +x to -x)
=== modified file 'membership/report/__init__.py'
--- membership/report/__init__.py	2011-01-14 00:11:01 +0000
+++ membership/report/__init__.py	2012-03-05 18:43:21 +0000
@@ -20,4 +20,4 @@
 ##############################################################################
 
 import report_membership
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'membership/report/report_membership.py'
--- membership/report/report_membership.py	2011-12-19 16:54:40 +0000
+++ membership/report/report_membership.py	2012-03-05 18:43:21 +0000
@@ -141,4 +141,4 @@
 
 report_membership()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'membership/wizard/__init__.py'
--- membership/wizard/__init__.py	2011-01-14 00:11:01 +0000
+++ membership/wizard/__init__.py	2012-03-05 18:43:21 +0000
@@ -20,4 +20,4 @@
 ##############################################################################
 
 import membership_invoice
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'mrp/mrp.py'
--- mrp/mrp.py	2012-02-13 18:07:41 +0000
+++ mrp/mrp.py	2012-03-05 18:43:21 +0000
@@ -271,13 +271,15 @@
             return {'value': {'name': prod.name, 'product_uom': prod.uom_id.id}}
         return {}
 
-    def _bom_find(self, cr, uid, product_id, product_uom, properties=[]):
+    def _bom_find(self, cr, uid, product_id, product_uom, properties=None):
         """ Finds BoM for particular product and product uom.
         @param product_id: Selected product.
         @param product_uom: Unit of measure of a product.
         @param properties: List of related properties.
         @return: False or BoM id.
         """
+        if properties is None:
+            properties = []
         cr.execute('select id from mrp_bom where product_id=%s and bom_id is null order by sequence', (product_id,))
         ids = map(lambda x: x[0], cr.fetchall())
         max_prop = 0
@@ -292,7 +294,7 @@
                 max_prop = prop
         return result
 
-    def _bom_explode(self, cr, uid, bom, factor, properties=[], addthis=False, level=0, routing_id=False):
+    def _bom_explode(self, cr, uid, bom, factor, properties=None, addthis=False, level=0, routing_id=False):
         """ Finds Products and Work Centers for related BoM for manufacturing order.
         @param bom: BoM of particular product.
         @param factor: Factor of product UoM.
@@ -577,7 +579,7 @@
         self.write(cr, uid, ids, {'state': 'picking_except'})
         return True
 
-    def action_compute(self, cr, uid, ids, properties=[], context=None):
+    def action_compute(self, cr, uid, ids, properties=None, context=None):
         """ Computes bills of material of a product.
         @param properties: List containing dictionaries of properties.
         @return: No. of products.
 
=== modified file 'mrp/procurement.py'
--- mrp/procurement.py	2011-09-17 16:41:25 +0000
+++ mrp/procurement.py	2012-03-05 18:43:21 +0000
@@ -33,7 +33,7 @@
         'property_ids': fields.many2many('mrp.property', 'procurement_property_rel', 'procurement_id','property_id', 'Properties'),
     }
     
-    def check_produce_product(self, cr, uid, procurement, context=[]):
+    def check_produce_product(self, cr, uid, procurement, context=None):
         """ Finds the bill of material for the product from procurement order.
         @return: True or False
         """
 
=== modified file 'mrp/report/bom_structure.py'
--- mrp/report/bom_structure.py	2011-01-14 00:11:01 +0000
+++ mrp/report/bom_structure.py	2012-03-05 18:43:21 +0000
@@ -35,7 +35,7 @@
     def get_children(self, object, level=0):
         result = []
 
-        def _get_rec(object,level):
+        def _get_rec(object, level):
             for l in object:
                 res = {}
                 res['name'] = l.name
@@ -61,4 +61,4 @@
 report_sxw.report_sxw('report.bom.structure','mrp.bom','mrp/report/bom_structure.rml',parser=bom_structure,header='internal')
 
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'mrp/stock.py'
--- mrp/stock.py	2012-02-13 15:27:55 +0000
+++ mrp/stock.py	2012-03-05 18:43:21 +0000
@@ -102,7 +102,7 @@
                     wf_service.trg_validate(uid, 'procurement.order', m, 'button_wait_done', cr)
         return processed_ids
     
-    def action_consume(self, cr, uid, ids, product_qty, location_id=False, context=None): 
+    def action_consume(self, cr, uid, ids, product_qty, location_id=False, context=None):
         """ Consumed product with specific quatity from specific source location.
         @param product_qty: Consumed product quantity
         @param location_id: Source location
 
=== modified file 'mrp_operations/mrp_operations.py'
--- mrp_operations/mrp_operations.py	2012-01-31 13:36:57 +0000
+++ mrp_operations/mrp_operations.py	2012-03-05 18:43:21 +0000
@@ -372,7 +372,7 @@
                 pass
         return result
 
-    def action_compute(self, cr, uid, ids, properties=[], context=None):
+    def action_compute(self, cr, uid, ids, properties=None, context=None):
         """ Computes bills of material of a product and planned date of work order.
         @param properties: List containing dictionaries of properties.
         @return: No. of products.
 
=== modified file 'plugin/plugin_handler.py'
--- plugin/plugin_handler.py	2011-12-26 17:12:29 +0000
+++ plugin/plugin_handler.py	2012-03-05 18:43:21 +0000
@@ -132,7 +132,7 @@
         return ('res.partner', partner_id, url)
 
     # Specific to outlook rfc822 is not available so we split in arguments headerd,body,attachemnts
-    def push_message_outlook(self, cr, uid, model, headers,res_id=0 ,body_text=False, body_html=False, attachments=False):
+    def push_message_outlook(self, cr, uid, model, headers, res_id=0 , body_text=False, body_html=False, attachments=False):
         # ----------------------------------------
         # solution 1
         # construct a fake rfc822 from the separated arguement
 
=== modified file 'point_of_sale/report/account_statement.py'
--- point_of_sale/report/account_statement.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/account_statement.py	2012-03-05 18:43:21 +0000
@@ -48,4 +48,4 @@
 
 report_sxw.report_sxw('report.account.statement', 'account.bank.statement', 'addons/statement/report/account_statement.rml', parser=account_statement,header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/all_closed_cashbox_of_the_day.py'
--- point_of_sale/report/all_closed_cashbox_of_the_day.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/all_closed_cashbox_of_the_day.py	2012-03-05 18:43:21 +0000
@@ -38,13 +38,13 @@
                 'get_net_total_starting':self._get_net_total_starting,
         })
 
-    def _get_user(self,line_ids):
+    def _get_user(self, line_ids):
         sql = "select name from res_users where id = %d"%(line_ids['create_uid'])
         self.cr.execute(sql)
         user = self.cr.fetchone()
         return user[0]
 
-    def _get_data(self,user):
+    def _get_data(self, user):
         data = {}
         sql = """ SELECT abs.journal_id,abs.id,abs.date,abs.closing_date,abs.name as statement,aj.name as journal,ap.name as period,ru.name as user,rc.name as company,
                        abs.state,abs.balance_end_real FROM account_bank_statement as abs
@@ -57,7 +57,7 @@
         data = self.cr.dictfetchall()
         return data
 
-    def _get_lines(self,statement):
+    def _get_lines(self, statement):
         data = {}
         sql = """ select absl.* from account_bank_statement_line as absl, account_bank_statement as abs
                            where absl.statement_id = abs.id and abs.id = %d"""%(statement['id'])
@@ -65,7 +65,7 @@
         data = self.cr.dictfetchall()
         return data
 
-    def _get_bal(self,data):
+    def _get_bal(self, data):
         res = {}
         sql =""" select sum(pieces*number) as bal from account_cashbox_line where starting_id = %d """%(data['id'])
         self.cr.execute(sql)
@@ -75,7 +75,7 @@
         else:
             return False
 
-    def _get_sub_total(self,user,data,date):
+    def _get_sub_total(self, user, data, date):
         res={}
         self.cr.execute(""" select sum(absl.amount) from account_bank_statement as abs
                             LEFT JOIN account_bank_statement_line as absl ON abs.id = absl.statement_id
@@ -90,7 +90,7 @@
         else:
             return False
 
-    def _get_partner(self,statement):
+    def _get_partner(self, statement):
         res = {}
         if statement['pos_statement_id']:
             sql =""" select rp.name  from account_bank_statement_line as absl,res_partner as rp
@@ -102,7 +102,7 @@
         else:
             return 0.00
 
-    def _get_net_total_starting(self,user):
+    def _get_net_total_starting(self, user):
         lst = []
         res={}
         total_ending_bal = 0.0
@@ -124,7 +124,7 @@
         lst.append(total_starting_bal)
         return lst
 
-    def _get_net_total(self,user):
+    def _get_net_total(self, user):
         res={}
         sql = """select sum(absl.amount) as net_total from account_bank_statement as abs
                     LEFT JOIN account_bank_statement_line as absl ON abs.id = absl.statement_id
@@ -137,4 +137,4 @@
 
 report_sxw.report_sxw('report.all.closed.cashbox.of.the.day', 'account.bank.statement', 'addons/point_of_sale/report/all_closed_cashbox_of_the_day.rml', parser=all_closed_cashbox_of_the_day,header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/pos_details.py'
--- point_of_sale/report/pos_details.py	2011-12-29 16:58:23 +0000
+++ point_of_sale/report/pos_details.py	2012-03-05 18:43:21 +0000
@@ -24,7 +24,7 @@
 
 class pos_details(report_sxw.rml_parse):
 
-    def _get_invoice(self,inv_id):
+    def _get_invoice(self, inv_id):
         res={}
         if inv_id:
             self.cr.execute("select number from account_invoice as ac where id = %s", (inv_id,))
@@ -37,7 +37,7 @@
         user_obj = self.pool.get('res.users')
         return user_obj.search(self.cr, self.uid, [])
 
-    def _pos_sales_details(self,form):
+    def _pos_sales_details(self, form):
         pos_obj = self.pool.get('pos.order')
         user_obj = self.pool.get('res.users')
         data = []
@@ -74,7 +74,7 @@
     def _get_sales_total_2(self):
         return self.total
 
-    def _get_sum_invoice_2(self,form):
+    def _get_sum_invoice_2(self, form):
         pos_obj = self.pool.get('pos.order')
         user_obj = self.pool.get('res.users')
         user_ids = form['user_ids'] or self._get_all_users()
 
=== modified file 'point_of_sale/report/pos_details_summary.py'
--- point_of_sale/report/pos_details_summary.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/pos_details_summary.py	2012-03-05 18:43:21 +0000
@@ -42,7 +42,7 @@
             'getcompany':self.get_company
         })
 
-    def get_company(self,objects):
+    def get_company(self, objects):
         comp=[obj.company_id.name for obj in objects]
         return '%s' % (comp[0])
 
@@ -136,4 +136,4 @@
                                             parser=pos_details_summary,
                                             header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/pos_lines.py'
--- point_of_sale/report/pos_lines.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/pos_lines.py	2012-03-05 18:43:21 +0000
@@ -41,7 +41,7 @@
         self.total = tot
         return self.total
 
-    def __taxes__(self,obj):
+    def __taxes__(self, obj):
         self.cr.execute ( " Select acct.name from pos_order as po " \
                               " LEFT JOIN pos_order_line as pol ON po.id = pol.order_id " \
                               " LEFT JOIN product_taxes_rel as ptr ON pol.product_id = ptr.prod_id " \
@@ -52,4 +52,4 @@
 
 report_sxw.report_sxw('report.pos.lines', 'pos.order', 'addons/point_of_sale/report/pos_lines.rml', parser=pos_lines,header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/pos_payment_report.py'
--- point_of_sale/report/pos_payment_report.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/pos_payment_report.py	2012-03-05 18:43:21 +0000
@@ -33,7 +33,7 @@
             'pos_payment_total':self._pos_payment_total,
         })
 
-    def _pos_payment(self,obj):
+    def _pos_payment(self, obj):
         data={}
         sql = """ select id from pos_order where id = %d"""%(obj.id)
         self.cr.execute(sql)
@@ -56,9 +56,9 @@
             self.total += d['price_unit'] * d['qty']
         return data
 
-    def _pos_payment_total(self,o):
+    def _pos_payment_total(self, o):
         return self.total
 
 report_sxw.report_sxw('report.pos.payment.report', 'pos.order', 'addons/point_of_sale/report/pos_payment_report.rml', parser=pos_payment_report,header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/pos_payment_report_user.py'
--- point_of_sale/report/pos_payment_report_user.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/pos_payment_report_user.py	2012-03-05 18:43:21 +0000
@@ -33,7 +33,7 @@
             'pos_payment_user_total':self.__pos_payment_user__total__,
         })
 
-    def __pos_payment_user__(self,form):
+    def __pos_payment_user__(self, form):
         data={}
         ids = form['user_id']
         sql = "select pt.name,pp.default_code as code,pol.qty,pu.name as uom,pol.discount,pol.price_unit, " \
@@ -61,4 +61,4 @@
 
 report_sxw.report_sxw('report.pos.payment.report.user', 'pos.order', 'addons/point_of_sale/report/pos_payment_report_user.rml', parser=pos_payment_report_user,header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/pos_receipt.py'
--- point_of_sale/report/pos_receipt.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/pos_receipt.py	2012-03-05 18:43:21 +0000
@@ -74,4 +74,4 @@
 
 report_sxw.report_sxw('report.pos.receipt', 'pos.order', 'addons/point_of_sale/report/pos_receipt.rml', parser=order, header=False)
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/pos_sales_user.py'
--- point_of_sale/report/pos_sales_user.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/pos_sales_user.py	2012-03-05 18:43:21 +0000
@@ -33,7 +33,7 @@
 
         })
 
-    def _get_data(self,form):
+    def _get_data(self, form):
         dt1 = form['date_start'] + ' 00:00:00'
         dt2 = form['date_end'] + ' 23:59:59'
         data={}
@@ -47,4 +47,4 @@
 
 report_sxw.report_sxw('report.pos.sales.user', 'pos.order', 'addons/point_of_sale/report/pos_sales_user.rml', parser=pos_sales_user,header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/pos_sales_user_today.py'
--- point_of_sale/report/pos_sales_user_today.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/pos_sales_user_today.py	2012-03-05 18:43:21 +0000
@@ -33,7 +33,7 @@
 
         })
 
-    def _get_data(self,form):
+    def _get_data(self, form):
         data={}
         ids = form['user_id']
 
@@ -47,4 +47,4 @@
 
 report_sxw.report_sxw('report.pos.sales.user.today', 'pos.order', 'addons/point_of_sale/report/pos_sales_user_today.rml', parser=pos_sales_user_today,header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/pos_users_product.py'
--- point_of_sale/report/pos_users_product.py	2011-01-14 00:11:01 +0000
+++ point_of_sale/report/pos_users_product.py	2012-03-05 18:43:21 +0000
@@ -60,9 +60,9 @@
             data = self.cr.fetchone()
             return data[0]
 
-    def _get_total(self,o):
+    def _get_total(self, o):
         return self.total
 
 report_sxw.report_sxw('report.pos.user.product', 'account.bank.statement', 'addons/statement/report/pos_users_product.rml', parser=pos_user_product,header='internal')
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'point_of_sale/report/report_cash_register.py'
--- point_of_sale/report/report_cash_register.py	2012-02-10 09:11:03 +0000
+++ point_of_sale/report/report_cash_register.py	2012-03-05 18:43:21 +0000
@@ -65,4 +65,4 @@
 
 report_cash_register()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'procurement/company.py'
--- procurement/company.py	2011-01-14 00:11:01 +0000
+++ procurement/company.py	2012-03-05 18:43:21 +0000
@@ -35,4 +35,4 @@
 
 company()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'product/product.py'
--- product/product.py	2012-02-22 11:53:32 +0000
+++ product/product.py	2012-03-05 18:43:21 +0000
@@ -349,7 +349,7 @@
             res = False
         return res
 
-    def onchange_uom(self, cursor, user, ids, uom_id,uom_po_id):
+    def onchange_uom(self, cursor, user, ids, uom_id, uom_po_id):
         if uom_id:
             return {'value': {'uom_po_id': uom_id}}
         return {}
@@ -540,7 +540,7 @@
         self.pool.get('product.template').unlink(cr, uid, unlink_product_tmpl_ids, context=context)
         return super(product_product, self).unlink(cr, uid, unlink_ids, context=context)
 
-    def onchange_uom(self, cursor, user, ids, uom_id,uom_po_id):
+    def onchange_uom(self, cursor, user, ids, uom_id, uom_po_id):
         if uom_id and uom_po_id:
             uom_obj=self.pool.get('product.uom')
             uom=uom_obj.browse(cursor,user,[uom_id])[0]
 
=== modified file 'product/report/product_pricelist.py'
--- product/report/product_pricelist.py	2011-01-17 08:00:48 +0000
+++ product/report/product_pricelist.py	2012-03-05 18:43:21 +0000
@@ -39,7 +39,7 @@
             'get_titles': self._get_titles,
         })
 
-    def _get_titles(self,form):
+    def _get_titles(self, form):
         lst = []
         vals = {}
         qtys = 1
@@ -51,7 +51,7 @@
         lst.append(vals)
         return lst
 
-    def _set_quantity(self,form):
+    def _set_quantity(self, form):
         for i in range(1,6):
             q = 'qty%d'%i
             if form[q] >0 and form[q] not in self.quantity:
@@ -76,7 +76,7 @@
         symbol = pool.get('res.currency').read(self.cr, self.uid, [pricelist['currency_id'][0]], ['symbol'], context=self.localcontext)[0]
         return symbol['symbol'] or ''
 
-    def _get_categories(self, products,form):
+    def _get_categories(self, products, form):
         cat_ids=[]
         res=[]
         self.pricelist = form['price_list']
@@ -111,7 +111,7 @@
             res.append({'name':cat[1],'products': products})
         return res
 
-    def _get_price(self,pricelist_id, product_id,qty):
+    def _get_price(self, pricelist_id, product_id, qty):
         sale_price_digits = self.get_digits(dp='Sale Price')
         pool = pooler.get_pool(self.cr.dbname)
         price_dict = pool.get('product.pricelist').price_get(self.cr, self.uid, [pricelist_id], product_id, qty, context=self.localcontext)
 
=== modified file 'product_manufacturer/product_manufacturer.py'
--- product_manufacturer/product_manufacturer.py	2011-01-14 00:11:01 +0000
+++ product_manufacturer/product_manufacturer.py	2012-03-05 18:43:21 +0000
@@ -40,4 +40,4 @@
     }
 product_attribute()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'product_visible_discount/__init__.py'
--- product_visible_discount/__init__.py	2011-01-14 00:11:01 +0000
+++ product_visible_discount/__init__.py	2012-03-05 18:43:21 +0000
@@ -20,4 +20,4 @@
 
 import product_visible_discount
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'project/project.py'
--- project/project.py	2012-02-28 14:08:16 +0000
+++ project/project.py	2012-03-05 18:43:21 +0000
@@ -265,7 +265,7 @@
         task_obj.duplicate_task(cr, uid, map_task_id, context=context)
         return True
 
-    def copy(self, cr, uid, id, default={}, context=None):
+    def copy(self, cr, uid, id, default=None, context=None):
         if context is None:
             context = {}
 
@@ -545,12 +545,12 @@
         return res
 
 
-    def onchange_remaining(self, cr, uid, ids, remaining=0.0, planned = 0.0):
+    def onchange_remaining(self, cr, uid, ids, remaining=0.0, planned=0.0):
         if remaining and not planned:
             return {'value':{'planned_hours': remaining}}
         return {}
 
-    def onchange_planned(self, cr, uid, ids, planned = 0.0, effective = 0.0):
+    def onchange_planned(self, cr, uid, ids, planned=0.0, effective=0.0):
         return {'value':{'remaining_hours': planned - effective}}
 
     def onchange_project(self, cr, uid, id, project_id):
@@ -581,7 +581,9 @@
             #FIXME why there is already the copy and the old one
             self.write(cr, uid, new, {'parent_ids':[(6,0,set(parent_ids))], 'child_ids':[(6,0, set(child_ids))]})
 
-    def copy_data(self, cr, uid, id, default={}, context=None):
+    def copy_data(self, cr, uid, id, default=None, context=None):
+        if default is None:
+            default = {}
         default = default or {}
         default.update({'work_ids':[], 'date_start': False, 'date_end': False, 'date_deadline': False})
         if not default.get('remaining_hours', False):
@@ -810,7 +812,7 @@
            }
         return res
 
-    def do_close(self, cr, uid, ids, context={}):
+    def do_close(self, cr, uid, ids, context=None):
         """
         Close Task
         """
@@ -868,7 +870,7 @@
             self.write(cr, uid, [task.id], {'state': 'open'}, context=context)
         return True
 
-    def do_cancel(self, cr, uid, ids, context={}):
+    def do_cancel(self, cr, uid, ids, context=None):
         request = self.pool.get('res.request')
         tasks = self.browse(cr, uid, ids, context=context)
         self._check_child_task(cr, uid, ids, context=context)
@@ -889,7 +891,7 @@
             self.write(cr, uid, [task.id], {'state': 'cancelled', 'remaining_hours':0.0}, context=context)
         return True
 
-    def do_open(self, cr, uid, ids, context={}):
+    def do_open(self, cr, uid, ids, context=None):
         if not isinstance(ids,list): ids = [ids]
         tasks= self.browse(cr, uid, ids, context=context)
         for t in tasks:
@@ -901,7 +903,7 @@
             self.log(cr, uid, t.id, message)
         return True
 
-    def do_draft(self, cr, uid, ids, context={}):
+    def do_draft(self, cr, uid, ids, context=None):
         self.write(cr, uid, ids, {'state': 'draft'}, context=context)
         return True
 
@@ -915,10 +917,12 @@
         return new_attachment_ids
         
 
-    def do_delegate(self, cr, uid, ids, delegate_data={}, context=None):
+    def do_delegate(self, cr, uid, ids, delegate_data=None, context=None):
         """
         Delegate Task to another users.
         """
+        if delegate_data is None:
+            delegate_data = {}
         assert delegate_data['user_id'], _("Delegated User should be specified")
         delegated_tasks = {}
         for task in self.browse(cr, uid, ids, context=context):
@@ -950,7 +954,7 @@
             delegated_tasks[task.id] = delegated_task_id
         return delegated_tasks
 
-    def do_pending(self, cr, uid, ids, context={}):
+    def do_pending(self, cr, uid, ids, context=None):
         self.write(cr, uid, ids, {'state': 'pending'}, context=context)
         for (id, name) in self.name_get(cr, uid, ids):
             message = _("The task '%s' is pending.") % name
 
=== modified file 'project/wizard/project_task_reevaluate.py'
--- project/wizard/project_task_reevaluate.py	2011-12-19 16:54:40 +0000
+++ project/wizard/project_task_reevaluate.py	2012-03-05 18:43:21 +0000
@@ -26,7 +26,7 @@
 class project_task_reevaluate(osv.osv_memory):
     _name = 'project.task.reevaluate'
 
-    def _get_remaining(self,cr, uid, context=None):
+    def _get_remaining(self, cr, uid, context=None):
         if context is None:
             context = {}
         active_id = context.get('active_id', False)
 
=== modified file 'project_gtd/project_gtd.py'
--- project_gtd/project_gtd.py	2011-12-21 22:15:04 +0000
+++ project_gtd/project_gtd.py	2012-03-05 18:43:21 +0000
@@ -67,7 +67,7 @@
         default['context_id'] = False
         return super(project_task,self).copy_data(cr, uid, id, default, context)
 
-    def _get_context(self,cr, uid, context=None):
+    def _get_context(self, cr, uid, context=None):
         ids = self.pool.get('project.gtd.context').search(cr, uid, [], context=context)
         return ids and ids[0] or False
 
 
=== modified file 'project_gtd/wizard/project_gtd_empty.py'
--- project_gtd/wizard/project_gtd_empty.py	2011-01-14 00:11:01 +0000
+++ project_gtd/wizard/project_gtd_empty.py	2012-03-05 18:43:21 +0000
@@ -30,7 +30,7 @@
         'name': fields.char('Name', size=32)
     }
 
-    def view_init(self, cr , uid , fields_list, context=None):
+    def view_init(self, cr, uid, fields_list, context=None):
         if context is None:
             context = {}
         self._empty(cr, uid, context=context)
 
=== modified file 'project_mailgate/project_mailgate.py'
--- project_mailgate/project_mailgate.py	2011-09-07 22:38:59 +0000
+++ project_mailgate/project_mailgate.py	2012-03-05 18:43:21 +0000
@@ -47,7 +47,9 @@
         self.write(cr, uid, [res_id], data, context)
         return res_id
 
-    def message_update(self, cr, uid, ids, msg, data={}, default_act='pending'):
+    def message_update(self, cr, uid, ids, msg, data=None, default_act='pending'):
+        if data is None:
+            data = {}
         data.update({
             'description': msg['body_text'],
         })
 
=== modified file 'project_mrp/project_procurement.py'
--- project_mrp/project_procurement.py	2011-12-19 16:54:40 +0000
+++ project_mrp/project_procurement.py	2012-03-05 18:43:21 +0000
@@ -40,7 +40,7 @@
         return all(proc.product_id.type != 'service' or (proc.task_id and proc.task_id.state in ('done', 'cancelled')) \
                     for proc in self.browse(cr, uid, ids, context=context))
 
-    def check_produce_service(self, cr, uid, procurement, context=None):    
+    def check_produce_service(self, cr, uid, procurement, context=None):
         return True
 
     def _convert_qty_company_hours(self, cr, uid, procurement, context=None):
 
=== modified file 'project_timesheet/project_timesheet.py'
--- project_timesheet/project_timesheet.py	2012-01-31 13:36:57 +0000
+++ project_timesheet/project_timesheet.py	2012-03-05 18:43:21 +0000
@@ -197,7 +197,7 @@
 
         return super(task,self).unlink(cr, uid, ids, *args, **kwargs)
 
-    def write(self, cr, uid, ids,vals,context=None):
+    def write(self, cr, uid, ids, vals, context=None):
         if context is None:
             context = {}
         if vals.get('project_id',False) or vals.get('name',False):
 
=== modified file 'project_timesheet/report/task_report.py'
--- project_timesheet/report/task_report.py	2011-07-01 23:41:24 +0000
+++ project_timesheet/report/task_report.py	2012-03-05 18:43:21 +0000
@@ -29,7 +29,7 @@
     _auto = False
     _order = "name"
 
-    def _get_task_hours(self, cr, uid, ids, name,args,context):
+    def _get_task_hours(self, cr, uid, ids, name, args, context):
         result = {}
         for record in self.browse(cr, uid, ids,context):
             last_date = datetime.strptime(record.name, '%Y-%m-%d') + relativedelta(months=1) - relativedelta(days=1)
@@ -42,7 +42,7 @@
             result[record.id] = total
         return result
 
-    def get_hrs_timesheet(self, cr, uid, ids, name,args,context):
+    def get_hrs_timesheet(self, cr, uid, ids, name, args, context):
         result = {}
         sum = 0.0
         for record in self.browse(cr, uid, ids, context):
 
=== modified file 'purchase/purchase.py'
--- purchase/purchase.py	2012-02-28 14:08:16 +0000
+++ purchase/purchase.py	2012-03-05 18:43:21 +0000
@@ -395,7 +395,7 @@
             res = inv_id
         return res
 
-    def has_stockable_product(self,cr, uid, ids, *args):
+    def has_stockable_product(self, cr, uid, ids, *args):
         for order in self.browse(cr, uid, ids):
             for order_line in order.order_line:
                 if order_line.product_id and order_line.product_id.product_tmpl_id.type in ('product', 'consu'):
@@ -498,7 +498,7 @@
         wf_service.trg_validate(uid, 'stock.picking', picking_id, 'button_confirm', cr)
         return [picking_id]
 
-    def action_picking_create(self,cr, uid, ids, context=None):
+    def action_picking_create(self, cr, uid, ids, context=None):
         picking_ids = []
         for order in self.browse(cr, uid, ids):
             picking_ids.extend(self._create_pickings(cr, uid, order, order.order_line, None, context=context))
 
=== modified file 'purchase/wizard/purchase_order_group.py'
--- purchase/wizard/purchase_order_group.py	2011-05-19 13:12:28 +0000
+++ purchase/wizard/purchase_order_group.py	2012-03-05 18:43:21 +0000
@@ -88,4 +88,4 @@
 
 purchase_order_group()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'purchase_requisition/purchase_requisition.py'
--- purchase_requisition/purchase_requisition.py	2011-12-23 11:22:41 +0000
+++ purchase_requisition/purchase_requisition.py	2012-03-05 18:43:21 +0000
@@ -183,7 +183,7 @@
         'company_id': fields.related('requisition_id','company_id',type='many2one',relation='res.company',string='Company', store=True, readonly=True),
     }
 
-    def onchange_product_id(self, cr, uid, ids, product_id,product_uom_id, context=None):
+    def onchange_product_id(self, cr, uid, ids, product_id, product_uom_id, context=None):
         """ Changes UoM and name if product_id changes.
         @param name: Name of the field
         @param product_id: Changed product_id
 
=== modified file 'report_webkit/ir_report.py'
--- report_webkit/ir_report.py	2011-12-19 16:54:40 +0000
+++ report_webkit/ir_report.py	2012-03-05 18:43:21 +0000
@@ -35,7 +35,7 @@
 from report.report_sxw import rml_parse
 
 def register_report(name, model, tmpl_path, parser=rml_parse):
-    "Register the report into the services"
+    """Register the report into the services"""
     name = 'report.%s' % name
     if netsvc.Service._services.get(name, False):
         service = netsvc.Service._services[name]
 
=== modified file 'report_webkit/report_helper.py'
--- report_webkit/report_helper.py	2011-12-19 16:54:40 +0000
+++ report_webkit/report_helper.py	2012-03-05 18:43:21 +0000
@@ -75,7 +75,7 @@
         head = header_obj.browse(self.cursor, self.uid, header_img_id)
         return (head.img, head.type)
             
-    def embed_logo_by_name(self, name, width=0, height=0) :
+    def embed_logo_by_name(self, name, width=0, height=0):
         """Return HTML embedded logo by name"""
         img, type = self.get_logo_by_name(name)
         return self.embed_image(type, img, width, height)
 
=== modified file 'resource/faces/task.py'
--- resource/faces/task.py	2011-12-19 16:54:40 +0000
+++ resource/faces/task.py	2012-03-05 18:43:21 +0000
@@ -567,7 +567,8 @@
         return result
     #@-node:_cmp
     #@+node:__getattr__
-    def __getattr__(self, name): return getattr(self._value, name)
+    def __getattr__(self, name):
+        return getattr(self._value, name)
     #@-node:__getattr__
     #@+node:__getitem__
     def __getitem__(self, slice):
 
=== modified file 'resource/resource.py'
--- resource/resource.py	2011-12-21 12:37:24 +0000
+++ resource/resource.py	2012-03-05 18:43:21 +0000
@@ -438,7 +438,7 @@
         (check_dates, 'Error! leave start-date must be lower then leave end-date.', ['date_from', 'date_to'])
     ]
 
-    def onchange_resource(self,cr, uid, ids, resource, context=None):
+    def onchange_resource(self, cr, uid, ids, resource, context=None):
         result = {}
         if resource:
             resource_pool = self.pool.get('resource.resource')
 
=== modified file 'sale/edi/sale_order.py'
--- sale/edi/sale_order.py	2011-11-05 01:56:31 +0000
+++ sale/edi/sale_order.py	2012-03-05 18:43:21 +0000
@@ -219,4 +219,4 @@
             edi_doc_list.append(edi_doc)
         return edi_doc_list
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'sale/report/__init__.py'
--- sale/report/__init__.py	2011-01-14 00:11:01 +0000
+++ sale/report/__init__.py	2012-03-05 18:43:21 +0000
@@ -22,4 +22,4 @@
 import sale_order
 import sale_report
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'sale/sale.py'
--- sale/sale.py	2012-02-28 14:08:16 +0000
+++ sale/sale.py	2012-03-05 18:43:21 +0000
@@ -326,7 +326,7 @@
             self.log(cr, uid, id, message)
         return True
 
-    def onchange_pricelist_id(self, cr, uid, ids, pricelist_id, order_lines, context={}):
+    def onchange_pricelist_id(self, cr, uid, ids, pricelist_id, order_lines, context=None):
         if (not pricelist_id) or (not order_lines):
             return {}
         warning = {
@@ -335,7 +335,7 @@
         }
         return {'warning': warning}
 
-    def onchange_partner_order_id(self, cr, uid, ids, order_id, invoice_id=False, shipping_id=False, context={}):
+    def onchange_partner_order_id(self, cr, uid, ids, order_id, invoice_id=False, shipping_id=False, context=None):
         if not order_id:
             return {}
         val = {}
@@ -501,7 +501,9 @@
             'res_id': inv_ids and inv_ids[0] or False,
         }
 
-    def action_invoice_create(self, cr, uid, ids, grouped=False, states=['confirmed', 'done', 'exception'], date_inv = False, context=None):
+    def action_invoice_create(self, cr, uid, ids, grouped=False, states=None, date_inv=False, context=None):
+        if states is None:
+            states = ['confirmed', 'done', 'exception']
         res = False
         invoices = {}
         invoice_ids = []
 
=== modified file 'sale/wizard/__init__.py'
--- sale/wizard/__init__.py	2011-01-14 00:11:01 +0000
+++ sale/wizard/__init__.py	2012-03-05 18:43:21 +0000
@@ -23,4 +23,4 @@
 import sale_line_invoice
 import sale_make_invoice_advance
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'sale/wizard/sale_make_invoice.py'
--- sale/wizard/sale_make_invoice.py	2011-12-26 08:41:35 +0000
+++ sale/wizard/sale_make_invoice.py	2012-03-05 18:43:21 +0000
@@ -68,4 +68,4 @@
 
 sale_make_invoice()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'sale_crm/__init__.py'
--- sale_crm/__init__.py	2011-01-14 00:11:01 +0000
+++ sale_crm/__init__.py	2012-03-05 18:43:21 +0000
@@ -22,4 +22,4 @@
 import wizard
 import sale_crm
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'sale_layout/sale_layout.py'
--- sale_layout/sale_layout.py	2012-02-14 23:24:48 +0000
+++ sale_layout/sale_layout.py	2012-03-05 18:43:21 +0000
@@ -53,7 +53,7 @@
             seq += 1
         return invoice_line_ids
 
-    def onchange_sale_order_line_view(self, cr, uid, id, type, context={}, *args):
+    def onchange_sale_order_line_view(self, cr, uid, id, type, context=None, *args):
         temp = {}
         temp['value'] = {}
         if (not type):
 
=== modified file 'sale_order_dates/sale_order_dates.py'
--- sale_order_dates/sale_order_dates.py	2011-07-01 23:41:24 +0000
+++ sale_order_dates/sale_order_dates.py	2012-03-05 18:43:21 +0000
@@ -61,4 +61,4 @@
 
 sale_order_dates()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'stock/report/lot_overview.py'
--- stock/report/lot_overview.py	2011-01-14 00:11:01 +0000
+++ stock/report/lot_overview.py	2012-03-05 18:43:21 +0000
@@ -34,7 +34,7 @@
             'grand_total_price':self._grand_total,
         })
 
-    def process(self,location_id):
+    def process(self, location_id):
         location_obj = pooler.get_pool(self.cr.dbname).get('stock.location')
         data = location_obj._product_get_report(self.cr,self.uid, [location_id])
 
 
=== modified file 'stock/report/lot_overview_all.py'
--- stock/report/lot_overview_all.py	2011-01-14 00:11:01 +0000
+++ stock/report/lot_overview_all.py	2012-03-05 18:43:21 +0000
@@ -34,7 +34,7 @@
             'grand_total_price':self._grand_total,
         })
 
-    def process(self,location_id):
+    def process(self, location_id):
         location_obj = pooler.get_pool(self.cr.dbname).get('stock.location')
         data = location_obj._product_get_all_report(self.cr,self.uid, [location_id])
         data['location_name'] = location_obj.read(self.cr, self.uid, [location_id],['complete_name'])[0]['complete_name']
 
=== modified file 'stock/report/product_stock.py'
--- stock/report/product_stock.py	2011-01-14 00:11:01 +0000
+++ stock/report/product_stock.py	2012-03-05 18:43:21 +0000
@@ -42,7 +42,9 @@
 
 
 class report_stock(report_int):
-    def create(self, cr, uid, ids, datas, context={}):
+    def create(self, cr, uid, ids, datas, context=None):
+        if context is None:
+            context = {}
         product_ids = ids
         if 'location_id' in context:
             location_id = context['location_id']
 
=== modified file 'stock/report/report_stock.py'
--- stock/report/report_stock.py	2011-12-19 16:54:40 +0000
+++ stock/report/report_stock.py	2012-03-05 18:43:21 +0000
@@ -74,7 +74,7 @@
                 group by location_id, product_id, prodlot_id
             )""")
 
-    def unlink(self, cr, uid, ids, context={}):
+    def unlink(self, cr, uid, ids, context=None):
         raise osv.except_osv(_('Error !'), _('You cannot delete any record!'))
 
 
@@ -131,7 +131,7 @@
                 group by location_id, product_id, tracking_id
             )""")
         
-    def unlink(self, cr, uid, ids, context={}):
+    def unlink(self, cr, uid, ids, context=None):
         raise osv.except_osv(_('Error !'), _('You cannot delete any record!'))
 
 stock_report_tracklots()
 
=== modified file 'stock/report/stock_by_location.py'
--- stock/report/stock_by_location.py	2011-01-14 00:11:01 +0000
+++ stock/report/stock_by_location.py	2012-03-05 18:43:21 +0000
@@ -26,7 +26,7 @@
 #FIXME: we should use toxml
 
 class report_custom(report_rml):
-    def create_xml(self, cr, uid, ids, datas, context={}):
+    def create_xml(self, cr, uid, ids, datas, context=None):
         config = """
         <config>
             <date>09/09/2005</date>
 
=== modified file 'stock/report/stock_inventory_move_report.py'
--- stock/report/stock_inventory_move_report.py	2011-01-14 00:11:01 +0000
+++ stock/report/stock_inventory_move_report.py	2012-03-05 18:43:21 +0000
@@ -30,7 +30,7 @@
              'qty_total':self._qty_total
         })
 
-    def _qty_total(self,objects):
+    def _qty_total(self, objects):
         total = 0.0
         uom = objects[0].product_uom.name
         for obj in objects:
 
=== modified file 'stock/stock.py'
--- stock/stock.py	2012-02-28 14:08:16 +0000
+++ stock/stock.py	2012-03-05 18:43:21 +0000
@@ -345,22 +345,28 @@
         })
         return product_obj.get_product_available(cr, uid, product_ids, context=context)
 
-    def _product_get(self, cr, uid, id, product_ids=False, context=None, states=['done']):
+    def _product_get(self, cr, uid, id, product_ids=False, context=None, states=None):
         """
         @param product_ids:
         @param states:
         @return:
         """
+        if states is None:
+            states = ['done']
         ids = id and [id] or []
         return self._product_get_multi_location(cr, uid, ids, product_ids, context=context, states=states)
 
-    def _product_all_get(self, cr, uid, id, product_ids=False, context=None, states=['done']):
+    def _product_all_get(self, cr, uid, id, product_ids=False, context=None, states=None):
+        if states is None:
+            states = ['done']
         # build the list of ids of children of the location given by id
         ids = id and [id] or []
         location_ids = self.search(cr, uid, [('location_id', 'child_of', ids)])
         return self._product_get_multi_location(cr, uid, location_ids, product_ids, context, states)
 
-    def _product_virtual_get(self, cr, uid, id, product_ids=False, context=None, states=['done']):
+    def _product_virtual_get(self, cr, uid, id, product_ids=False, context=None, states=None):
+        if states is None:
+            states = ['done']
         return self._product_all_get(cr, uid, id, product_ids, context, ['confirmed', 'waiting', 'assigned', 'done'])
 
     def _product_reserve(self, cr, uid, ids, product_id, product_qty, context=None, lock=False):
@@ -518,7 +524,7 @@
     def unlink(self, cr, uid, ids, context=None):
         raise osv.except_osv(_('Error'), _('You can not remove a lot line !'))
 
-    def action_traceability(self, cr, uid, ids, context={}):
+    def action_traceability(self, cr, uid, ids, context=None):
         """ It traces the information of a product
         @param self: The object pointer.
         @param cr: A database cursor
 
=== modified file 'stock_invoice_directly/wizard/stock_invoice.py'
--- stock_invoice_directly/wizard/stock_invoice.py	2011-10-16 01:28:00 +0000
+++ stock_invoice_directly/wizard/stock_invoice.py	2012-03-05 18:43:21 +0000
@@ -47,4 +47,4 @@
 
 invoice_directly()
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'stock_location/procurement_pull.py'
--- stock_location/procurement_pull.py	2011-12-19 16:54:40 +0000
+++ stock_location/procurement_pull.py	2012-03-05 18:43:21 +0000
@@ -45,7 +45,7 @@
                     return (line.type_proc=='move') and (line.location_src_id)
         return False
 
-    def action_move_create(self, cr, uid, ids,context=None):
+    def action_move_create(self, cr, uid, ids, context=None):
         proc_obj = self.pool.get('procurement.order')
         move_obj = self.pool.get('stock.move')
         picking_obj=self.pool.get('stock.picking')
 
=== modified file 'stock_planning/stock_planning.py'
--- stock_planning/stock_planning.py	2011-12-21 22:15:04 +0000
+++ stock_planning/stock_planning.py	2012-03-05 18:43:21 +0000
@@ -406,7 +406,7 @@
                 res[val.id] = 'Future'
         return res
 
-    def _get_op(self, cr, uid, ids, field_names, arg, context=None):  # op = OrderPoint
+    def _get_op(self, cr, uid, ids, field_names, arg, context=None):
         res = {}
         for val in self.browse(cr, uid, ids, context=context):
             res[val.id]={}
 
=== modified file 'stock_planning/wizard/stock_planning_createlines.py'
--- stock_planning/wizard/stock_planning_createlines.py	2011-09-21 07:37:15 +0000
+++ stock_planning/wizard/stock_planning_createlines.py	2012-03-05 18:43:21 +0000
@@ -49,7 +49,7 @@
         'company_id': lambda self, cr, uid, c: self.pool.get('res.company')._company_default_get(cr, uid, 'stock.planning', context=c),
     }
 
-    def create_planning(self,cr, uid, ids, context=None):
+    def create_planning(self, cr, uid, ids, context=None):
         if context is None:
             context = {}
         product_obj = self.pool.get('product.product')
 
=== modified file 'survey/survey.py'
--- survey/survey.py	2012-02-13 15:27:55 +0000
+++ survey/survey.py	2012-03-05 18:43:21 +0000
@@ -514,7 +514,7 @@
     _description = 'Survey Question Column Heading'
     _rec_name = 'title'
 
-    def _get_in_visible_rating_weight(self,cr, uid, context=None):
+    def _get_in_visible_rating_weight(self, cr, uid, context=None):
         if context is None:
             context = {}
         if context.get('in_visible_rating_weight', False):
@@ -567,7 +567,7 @@
             }
         return val
 
-    def _get_in_visible_answer_type(self,cr, uid, context=None):
+    def _get_in_visible_answer_type(self, cr, uid, context=None):
         if context is None:
             context = {}
         return context.get('in_visible_answer_type', False)
 
=== modified file 'survey/wizard/__init__.py'
--- survey/wizard/__init__.py	2011-01-14 00:11:01 +0000
+++ survey/wizard/__init__.py	2012-03-05 18:43:21 +0000
@@ -27,4 +27,4 @@
 import survey_answer
 import survey_print
 
-# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 \ No newline at end of file
+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'survey/wizard/survey_answer.py'
--- survey/wizard/survey_answer.py	2012-02-13 15:27:55 +0000
+++ survey/wizard/survey_answer.py	2012-03-05 18:43:21 +0000
@@ -997,7 +997,7 @@
 
         return survey_question_wiz_id
 
-    def action_new_question(self,cr, uid, ids, context=None):
+    def action_new_question(self, cr, uid, ids, context=None):
         """
         New survey.Question form.
         """
@@ -1039,7 +1039,7 @@
             'context': context
         }
 
-    def action_edit_page(self,cr, uid, ids, context=None):
+    def action_edit_page(self, cr, uid, ids, context=None):
         """
         Edit survey.page.
         """
@@ -1061,7 +1061,7 @@
             'context': context
         }
 
-    def action_delete_page(self,cr, uid, ids, context=None):
+    def action_delete_page(self, cr, uid, ids, context=None):
         """
         Delete survey.page.
         """
@@ -1087,7 +1087,7 @@
             'context': context
         }
 
-    def action_edit_question(self,cr, uid, ids, context=None):
+    def action_edit_question(self, cr, uid, ids, context=None):
         """
         Edit survey.question.
         """
@@ -1109,7 +1109,7 @@
             'context': context
         }
 
-    def action_delete_question(self,cr, uid, ids, context=None):
+    def action_delete_question(self, cr, uid, ids, context=None):
         """
         Delete survey.question.
         """
 
=== modified file 'wiki/web/widgets/rss/feedparser.py' (properties changed: +x to -x)
--- wiki/web/widgets/rss/feedparser.py	2011-12-19 16:54:40 +0000
+++ wiki/web/widgets/rss/feedparser.py	2012-03-05 18:43:21 +0000
@@ -2446,8 +2446,10 @@
     data = doctype_pattern.sub('', data)
     return version, data
 
-def parse(url_file_stream_or_string, etag=None, modified=None, agent=None, referrer=None, handlers=[]):
+def parse(url_file_stream_or_string, etag=None, modified=None, agent=None, referrer=None, handlers=None):
     '''Parse a feed from a URL, file, stream, or string'''
+    if handlers is None:
+        handlers = []
     result = FeedParserDict()
     result['feed'] = FeedParserDict()
     result['entries'] = []
 
=== modified file 'wiki/web/widgets/wikimarkup/__init__.py'
--- wiki/web/widgets/wikimarkup/__init__.py	2011-12-19 16:54:40 +0000
+++ wiki/web/widgets/wikimarkup/__init__.py	2012-03-05 18:43:21 +0000
@@ -489,7 +489,7 @@
 			return text.encode("utf-8")
 		return text
 
-	def strip(self, text, stripcomments=False, dontstrip=[]):
+	def strip(self, text, stripcomments=False, dontstrip=None):
 		render = True
 
 		commentState = {}
@@ -2087,7 +2087,9 @@
 # <quote cite="person">quote</quote>
 from cgi import escape
 
-def hook_quote(env, body, attributes={}):
+def hook_quote(env, body, attributes=None):
+	if attributes is None:
+	    attributes = {}
 	text = [u'<div class="blockquote">']
 	if 'cite' in attributes:
 		text.append(u"<strong class=\"cite\">%s wrote:</strong>\n" % escape(attributes['cite']))