Merge lp:~openerp-dev/openobject-addons/trunk-unmutable-defaults-abo into lp:openobject-addons

[Anto]

=== modified file 'account/account.py'
--- account/account.py	2012-09-30 12:46:54 +0000
+++ account/account.py	2012-10-02 11:29:53 +0000
@@ -595,12 +595,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, 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 = []
         account = self.browse(cr, uid, id, context=context)
         new_child_ids = []
-        if not default:
-            default = {}
-        default = default.copy()
         default.update(code=_("%s (copy)") % (account['code'] or ''))
         if not local:
             done_list = []
@@ -777,11 +780,14 @@
         (_check_currency, 'Configuration error!\nThe 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.update(
             code=_("%s (copy)") % (journal['code'] or ''),
             name=_("%s (copy)") % (journal['name'] or ''),
@@ -1178,7 +1184,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
@@ -1437,9 +1443,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':'/',
@@ -2269,7 +2281,10 @@
     _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, data=None, context=None):
+        if data is None:
+            data = {}
         move_ids = []
         entry = {}
         account_move_obj = self.pool.get('account.move')
@@ -2280,8 +2295,8 @@
         if context is None:
             context = {}
 
-        if datas.get('date', False):
-            context.update({'date': datas['date']})
+        if data.get('date', False):
+            context.update({'date': data['date']})
 
         move_date = context.get('date', time.strftime('%Y-%m-%d'))
         move_date = datetime.strptime(move_date,"%Y-%m-%d")
@@ -2467,10 +2482,10 @@
         all_moves = []
         obj_model = self.pool.get('account.model')
         for line in self.browse(cr, uid, ids, context=context):
-            datas = {
+            data = {
                 'date': line.date,
             }
-            move_ids = obj_model.generate(cr, uid, [line.subscription_id.model_id.id], datas, context)
+            move_ids = obj_model.generate(cr, uid, [line.subscription_id.model_id.id], data, context)
             tocheck[line.subscription_id.id] = True
             self.write(cr, uid, [line.id], {'move_id':move_ids[0]})
             all_moves.extend(move_ids)
@@ -3236,7 +3251,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.
 
@@ -3254,6 +3269,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)
@@ -3266,7 +3287,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
 
@@ -3284,6 +3305,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-09-13 11:46:09 +0000
+++ account/account_bank.py	2012-10-02 11:29:53 +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
@@ -53,7 +53,7 @@
             data['currency_name'] = data['currency_id'] and currency_name[data['currency_id'][0]] or ''
         return super(bank, self)._prepare_name_get(cr, uid, bank_dicts, context=context)
 
-    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-09-29 12:34:49 +0000
+++ account/account_move_line.py	2012-10-02 11:29:53 +0000
@@ -1105,7 +1105,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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-06-05 11:53:52 +0000
+++ account/report/account_report.py	2012-10-02 11:29:53 +0000
@@ -284,4 +284,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-10-02 11:29:53 +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	2012-03-30 13:30:01 +0000
+++ account/res_currency.py	2012-10-02 11:29:53 +0000
@@ -45,4 +45,3 @@
 res_currency_account()
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
-
 
=== modified file 'account/wizard/account_invoice_state.py'
--- account/wizard/account_invoice_state.py	2012-08-06 15:44:10 +0000
+++ account/wizard/account_invoice_state.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-07-25 07:33:57 +0000
+++ account/wizard/account_move_bank_reconcile.py	2012-10-02 11:29:53 +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	2012-09-18 13:27:27 +0000
+++ account/wizard/account_move_journal.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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_reconcile.py'
--- account/wizard/account_reconcile.py	2012-09-29 09:14:16 +0000
+++ account/wizard/account_reconcile.py	2012-10-02 11:29:53 +0000
@@ -160,4 +160,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_state_open.py'
--- account/wizard/account_state_open.py	2012-07-25 07:33:57 +0000
+++ account/wizard/account_state_open.py	2012-10-02 11:29:53 +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	2012-06-25 13:33:53 +0000
+++ account/wizard/account_subscription_generate.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-06-13 09:13:22 +0000
+++ account_analytic_plans/report/crossovered_analytic.py	2012-10-02 11:29:53 +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')
@@ -112,7 +112,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_anglo_saxon/__init__.py'
--- account_anglo_saxon/__init__.py	2011-01-14 00:11:01 +0000
+++ account_anglo_saxon/__init__.py	2012-10-02 11:29:53 +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-09-29 13:21:28 +0000
+++ account_asset/account_asset.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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_followup/__init__.py'
--- account_followup/__init__.py	2012-09-17 15:10:31 +0000
+++ account_followup/__init__.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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-09-05 15:14:51 +0000
+++ account_followup/wizard/account_followup_print.py	2012-10-02 11:29:53 +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_payment/__init__.py'
--- account_payment/__init__.py	2011-01-14 00:11:01 +0000
+++ account_payment/__init__.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-08-07 11:31:37 +0000
+++ account_payment/account_move_line.py	2012-10-02 11:29:53 +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-07-25 11:39:00 +0000
+++ account_payment/account_payment.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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_pay.py'
--- account_payment/wizard/account_payment_pay.py	2012-01-31 13:36:57 +0000
+++ account_payment/wizard/account_payment_pay.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-09-30 11:20:34 +0000
+++ account_voucher/account_voucher.py	2012-10-02 11:29:53 +0000
@@ -849,7 +849,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.
 
@@ -1275,7 +1275,9 @@
                 self.reconcile_send_note(cr, uid, [voucher.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 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-10-02 11:29:53 +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 'anonymization/anonymization.py'
--- anonymization/anonymization.py	2012-09-20 09:08:13 +0000
+++ anonymization/anonymization.py	2012-10-02 11:29:53 +0000
@@ -363,7 +363,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:
@@ -498,7 +498,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 'audittrail/audittrail.py'
--- audittrail/audittrail.py	2012-09-30 12:53:37 +0000
+++ audittrail/audittrail.py	2012-10-02 11:29:53 +0000
@@ -202,7 +202,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
 
@@ -211,6 +211,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')
@@ -349,9 +351,9 @@
             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 
+        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
         log those differences.
 
@@ -375,9 +377,11 @@
                 (model.id, resource_id): []
               }
 
-        The reason why the structure returned is build as above is because when modifying an existing 
+        The reason why the structure returned is build as above is because when modifying an existing
         record, we may have to log a change done in a x2many field of that object
         """
+        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
@@ -435,6 +439,8 @@
             on specific fields only.
         :return: True
         """
+        if field_list is None:
+            field_list = []
         # loop on all the given ids
         for res_id in res_ids:
             # compare old and new values and get audittrail log lines accordingly
 
=== modified file 'base_crypt/crypt.py'
--- base_crypt/crypt.py	2012-08-10 14:12:19 +0000
+++ base_crypt/crypt.py	2012-10-02 11:29:53 +0000
@@ -47,7 +47,9 @@
 magic_md5 = '$1$'
 _logger = logging.getLogger(__name__)
 
-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-10-02 11:29:53 +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_record/wizard/base_module_record_objects.py'
--- base_module_record/wizard/base_module_record_objects.py	2012-03-07 06:01:32 +0000
+++ base_module_record/wizard/base_module_record_objects.py	2012-10-02 11:29:53 +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_report_designer/openerp_sxw2rml/openerp_sxw2rml.py'
--- base_report_designer/openerp_sxw2rml/openerp_sxw2rml.py	2012-08-06 16:28:17 +0000
+++ base_report_designer/openerp_sxw2rml/openerp_sxw2rml.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-08-06 16:28:17 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/AddAttachment.py	2012-10-02 11:29:53 +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( "You have to select a 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	2012-07-25 11:55:11 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Change.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +0000
@@ -3,31 +3,31 @@
 # Portions of this file are under the following copyright and license:
 #
 #
-#   Copyright (c) 2003-2004 Danny Brewer 
-#   [e-mail] 
-# 
-#   This library is free software; you can redistribute it and/or 
-#   modify it under the terms of the GNU Lesser General Public 
-#   License as published by the Free Software Foundation; either 
-#   version 2.1 of the License, or (at your option) any later version. 
-# 
-#   This library 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 
-#   Lesser General Public License for more details. 
-# 
-#   You should have received a copy of the GNU Lesser General Public 
-#   License along with this library; if not, write to the Free Software 
-#   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA 
-# 
-#   See:  http://www.gnu.org/licenses/lgpl.html 
-#
-# 
+#   Copyright (c) 2003-2004 Danny Brewer
+#   [e-mail]
+#
+#   This library is free software; you can redistribute it and/or
+#   modify it under the terms of the GNU Lesser General Public
+#   License as published by the Free Software Foundation; either
+#   version 2.1 of the License, or (at your option) any later version.
+#
+#   This library 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
+#   Lesser General Public License for more details.
+#
+#   You should have received a copy of the GNU Lesser General Public
+#   License along with this library; if not, write to the Free Software
+#   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307 USA
+#
+#   See:  http://www.gnu.org/licenses/lgpl.html
+#
+#
 # and other portions are under the following copyright and license:
 #
 #
 #    OpenERP, Open Source Management Solution>..
-#    Copyright (C) 2004-2010 OpenERP SA (<http://openerp.com>). 
+#    Copyright (C) 2004-2010 OpenERP SA (<http://openerp.com>).
 #
 #    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
@@ -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,9 @@
                 sObject = self.getRes(sock,res[myval]['relation'], sVar[sVar.find("/")+1:])
         return sObject
 
-    def getBraces(self,aReportSyntex=[]):
+    def getBraces(self, aReportSyntex=None):
+        if aReportSyntex is None:
+            aReportSyntex = []
         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-10-02 11:29:53 +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	2012-08-06 16:28:17 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/ExportToRML.py	2012-10-02 11:29:53 +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	2012-07-25 11:55:11 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Expression.py	2012-10-02 11:29:53 +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 in Expression 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	2012-07-25 12:53:32 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Fields.py	2012-10-02 11:29:53 +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 E.g. http://localhost:8069 \nOR \nField-4 E.g. 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 particular 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	2012-08-07 11:06:16 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/ModifyExistingReport.py	2012-10-02 11:29:53 +0000
@@ -65,7 +65,7 @@
     uid = 3
 
 class ModifyExistingReport(unohelper.Base, XJobExecutor):
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
@@ -177,10 +177,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-10-02 11:29:53 +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	2012-07-25 11:55:11 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Repeatln.py	2012-10-02 11:29:53 +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 particular 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	2012-08-07 11:06:16 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/SendToServer.py	2012-10-02 11:29:53 +0000
@@ -72,7 +72,7 @@
         'HTML' : 'html'
     }
 
-    def __init__(self,ctx):
+    def __init__(self, ctx):
         self.ctx     = ctx
         self.module  = "openerp_report"
         self.version = "0.1"
@@ -136,10 +136,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):
@@ -221,7 +221,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	2012-08-07 11:06:16 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/ServerParameter.py	2012-10-02 11:29:53 +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,8 +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")
         global url
@@ -157,11 +156,10 @@
             self.logobj.log_write('successful login',LOG_INFO, ': successful login from %s using database %s' % (sLogin, sDatabase))
             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()
@@ -172,5 +170,4 @@
 elif __name__=="package":
     g_ImplementationHelper.addImplementation( ServerParameter, "org.openoffice.openerp.report.serverparam", ("com.sun.star.task.Job",),)
 
-
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'base_report_designer/plugin/openerp_report_designer/bin/script/Translation.py'
--- base_report_designer/plugin/openerp_report_designer/bin/script/Translation.py	2012-08-06 16:28:17 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/Translation.py	2012-10-02 11:29:53 +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 E.g. http://localhost:8069 \nOR \nField-4 E.g. 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 particular 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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-06-22 06:48:54 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/lib/logreport.py	2012-10-02 11:29:53 +0000
@@ -39,11 +39,10 @@
     _logger.setLevel(logging.INFO)
 
 class Logger(object):
-    def log_write(self,name,level,msg):
+    def log_write(self, name, level, msg):
         getattr(_logger,level)(msg)
 
     def shutdown(self):
         logging.shutdown()
 
-
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== 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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-08-06 16:28:17 +0000
+++ base_report_designer/plugin/openerp_report_designer/bin/script/modify.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +0000
@@ -10,7 +10,13 @@
 
 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 = []
+	if recur is None:
+	    recur = []
 	res = sock.execute('terp', 3, 'admin', 'account.invoice', 'fields_get')
 	key = res.keys()
 	key.sort()
 
=== modified file 'base_vat/__init__.py'
--- base_vat/__init__.py	2011-12-19 16:54:40 +0000
+++ base_vat/__init__.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-07-25 12:05:41 +0000
+++ caldav/caldav_node.py	2012-10-02 11:29:53 +0000
@@ -43,7 +43,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):
@@ -100,7 +100,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):
@@ -181,7 +181,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	2012-09-24 12:51:39 +0000
+++ crm/crm.py	2012-10-02 11:29:53 +0000
@@ -58,7 +58,7 @@
 
 class crm_case_stage(osv.osv):
     """ Model for case stages. This models the main stages of a document
-        management flow. Main CRM objects (leads, opportunities, project 
+        management flow. Main CRM objects (leads, opportunities, project
         issues, ...) will now use only stages, instead of state and stages.
         Stages are for example used to display the kanban view of records.
     """
@@ -126,11 +126,11 @@
         'note': fields.text('Description'),
         'working_hours': fields.float('Working Hours', digits=(16,2 )),
         'stage_ids': fields.many2many('crm.case.stage', 'section_stage_rel', 'section_id', 'stage_id', 'Stages'),
-        'alias_id': fields.many2one('mail.alias', 'Alias', ondelete="cascade", required=True, 
+        'alias_id': fields.many2one('mail.alias', 'Alias', ondelete="cascade", required=True,
                                     help="The email address associated with this team. New emails received will automatically "
                                          "create new leads assigned to the team."),
     }
-    
+
     def _get_stage_common(self, cr, uid, context):
         ids = self.pool.get('crm.case.stage').search(cr, uid, [('case_default','=',1)], context=context)
         return ids
@@ -165,12 +165,12 @@
                 name = record['parent_id'][1] + ' / ' + name
             res.append((record['id'], name))
         return res
-    
+
     def create(self, cr, uid, vals, context=None):
         mail_alias = self.pool.get('mail.alias')
         if not vals.get('alias_id'):
             vals.pop('alias_name', None) # prevent errors during copy()
-            alias_id = mail_alias.create_unique_alias(cr, uid, 
+            alias_id = mail_alias.create_unique_alias(cr, uid,
                     {'alias_name': vals['name']},
                     model_name="crm.lead",
                     context=context)
@@ -178,7 +178,7 @@
         res = super(crm_case_section, self).create(cr, uid, vals, context)
         mail_alias.write(cr, uid, [vals['alias_id']], {'alias_defaults': {'section_id': res, 'type':'lead'}}, context)
         return res
-        
+
     def unlink(self, cr, uid, ids, context=None):
         # Cascade-delete mail aliases as well, as they should not exist without the sales team.
         mail_alias = self.pool.get('mail.alias')
@@ -217,7 +217,6 @@
         'section_id': fields.many2one('crm.case.section', 'Sales Team'),
     }
 
-
 def _links_get(self, cr, uid, context=None):
     """Gets links value for reference field"""
     obj = self.pool.get('res.request.link')
 
=== modified file 'crm/crm_lead.py'
--- crm/crm_lead.py	2012-09-28 09:53:31 +0000
+++ crm/crm_lead.py	2012-10-02 11:29:53 +0000
@@ -311,7 +311,6 @@
             }
         return {'value' : values}
 
-
     def _check(self, cr, uid, ids=False, context=None):
         """ Override of the base.stage method.
             Function called by the scheduler to process cases for date actions
@@ -801,7 +800,7 @@
             if section_id:
                 vals.setdefault('message_follower_ids', [])
                 vals['message_follower_ids'] += [(4, follower.id) for follower in section_id.message_follower_ids]
-        return super(crm_lead,self).write(cr, uid, ids, vals, context) 
+        return super(crm_lead,self).write(cr, uid, ids, vals, context)
 
     # ----------------------------------------
     # Mail Gateway
 
=== modified file 'crm_claim/crm_claim.py'
--- crm_claim/crm_claim.py	2012-09-10 04:17:26 +0000
+++ crm_claim/crm_claim.py	2012-10-02 11:29:53 +0000
@@ -35,7 +35,7 @@
 
 class crm_claim_stage(osv.osv):
     """ Model for claim stages. This models the main stages of a claim
-        management flow. Main CRM objects (leads, opportunities, project 
+        management flow. Main CRM objects (leads, opportunities, project
         issues, ...) will now use only stages, instead of state and stages.
         Stages are for example used to display the kanban view of records.
     """
@@ -104,7 +104,7 @@
         'email_from': fields.char('Email', size=128, help="These people will receive email."),
         'partner_phone': fields.char('Phone', size=32),
         'stage_id': fields.many2one ('crm.claim.stage', 'Stage',
-                        domain="['|', ('section_ids', '=', section_id), ('case_default', '=', True)]"), 
+                        domain="['|', ('section_ids', '=', section_id), ('case_default', '=', True)]"),
         'cause': fields.text('Root Cause'),
         'state': fields.related('stage_id', 'state', type="selection", store=True,
                 selection=crm.AVAILABLE_STATES, string="State", readonly=True,
 
=== modified file 'crm_helpdesk/crm_helpdesk.py'
--- crm_helpdesk/crm_helpdesk.py	2012-09-17 10:11:09 +0000
+++ crm_helpdesk/crm_helpdesk.py	2012-10-02 11:29:53 +0000
@@ -69,9 +69,9 @@
             'probability': fields.float('Probability (%)'),
             'categ_id': fields.many2one('crm.case.categ', 'Category', \
                             domain="['|',('section_id','=',False),('section_id','=',section_id),\
-                            ('object_id.model', '=', 'crm.helpdesk')]"), 
-            'duration': fields.float('Duration', states={'done': [('readonly', True)]}), 
-            'state': fields.selection(crm.AVAILABLE_STATES, 'Status', size=16, readonly=True, 
+                            ('object_id.model', '=', 'crm.helpdesk')]"),
+            'duration': fields.float('Duration', states={'done': [('readonly', True)]}),
+            'state': fields.selection(crm.AVAILABLE_STATES, 'Status', size=16, readonly=True,
                                   help='The state is set to \'Draft\', when a case is created.\
                                   \nIf the case is in progress the state is set to \'Open\'.\
                                   \nWhen the case is over, the state is set to \'Done\'.\
@@ -139,9 +139,9 @@
 
         return super(crm_helpdesk,self).message_update(cr, uid, ids, msg, update_vals=update_vals, context=context)
 
-    # ******************************
+    # ---------------------------------------------------
     # OpenChatter
-    # ******************************
+    # ---------------------------------------------------
 
     def case_get_note_msg_prefix(self, cr, uid, id, context=None):
         """ override of default base_state method. """
@@ -152,5 +152,4 @@
         self.message_post(cr, uid, ids, body=msg, context=context)
         return True
 
-
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'crm_profiling/crm_profiling.py'
--- crm_profiling/crm_profiling.py	2012-07-12 05:50:38 +0000
+++ crm_profiling/crm_profiling.py	2012-10-02 11:29:53 +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 'document/content_index.py'
--- document/content_index.py	2012-08-06 17:08:41 +0000
+++ document/content_index.py	2012-10-02 11:29:53 +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,16 +95,15 @@
 
         raise NhException('No appropriate method to index file.')
 
-    def _doIndexContent(self,content):
+    def _doIndexContent(self, content):
         raise NhException("Content cannot be handled here.")
 
-    def _doIndexFile(self,fpath):
+    def _doIndexFile(self, fpath):
         raise NhException("Content cannot be handled here.")
 
     def __repr__(self):
         return "<indexer %s.%s>" %(self.__module__, self.__class__.__name__)
 
-
 def mime_match(mime, mdict):
     if mdict.has_key(mime):
         return (mime, mdict[mime])
@@ -136,7 +135,7 @@
         if not f:
             raise Exception("Your indexer should at least support 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	2012-09-24 16:26:45 +0000
+++ document/document.py	2012-10-02 11:29:53 +0000
@@ -149,7 +149,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	2012-09-24 16:26:45 +0000
+++ document/document_directory.py	2012-10-02 11:29:53 +0000
@@ -221,7 +221,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)
         """
@@ -234,7 +234,7 @@
         default.update(name=_("%s (copy)") % (name))
         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	2012-08-06 17:08:41 +0000
+++ document/nodes.py	2012-10-02 11:29:53 +0000
@@ -103,7 +103,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.
         """
@@ -252,7 +252,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
 
@@ -270,7 +270,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):
@@ -287,10 +287,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 """
@@ -434,7 +434,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
@@ -477,11 +479,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'):
@@ -666,7 +668,6 @@
 
         return dirobj.create(cr, uid, val)
 
-
     def create_child(self, cr, path, data=None):
         """ API function to create a child file object and node
             Return the node_* created
@@ -693,7 +694,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):
@@ -802,7 +803,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]
@@ -878,7 +879,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):
@@ -889,7 +890,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
@@ -958,7 +959,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:
@@ -1160,7 +1161,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
@@ -1210,7 +1211,6 @@
             return False
         return self.file_id == other.file_id
 
-
     def open_data(self, cr, mode):
         stor = self.storage_id
         assert stor, "No storage for file #%s." % self.file_id
@@ -1262,7 +1262,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
@@ -1278,14 +1278,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
@@ -1299,7 +1299,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):
@@ -1362,7 +1362,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
@@ -1382,7 +1382,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
@@ -1395,8 +1395,7 @@
         if res and res[0][0]:
             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.")
@@ -1426,7 +1425,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
@@ -1435,7 +1434,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.")
@@ -1444,7 +1443,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):
@@ -1558,5 +1557,4 @@
             cr.close()
         StringIO.close(self)
 
-#eof
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'document/odt2txt.py' (properties changed: +x to -x)
--- document/odt2txt.py	2011-09-22 09:55:31 +0000
+++ document/odt2txt.py	2012-10-02 11:29:53 +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	2012-07-25 10:33:34 +0000
+++ document/std_index.py	2012-10-02 11:29:53 +0000
@@ -39,7 +39,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 :
@@ -56,7 +56,7 @@
     def _getExtensions(self):
         return ['.txt', '.py']
 
-    def _doIndexContent(self,content):
+    def _doIndexContent(self, content):
         return content
 
 cntIndex.register(TxtIndex())
@@ -68,7 +68,7 @@
     def _getExtensions(self):
         return ['.pptx']
 
-    def _doIndexFile(self,fname):
+    def _doIndexFile(self, fname):
         def toString () :
             """ Converts the document to a string. """
             buffer = u""
@@ -96,7 +96,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()
@@ -116,7 +116,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 () :
@@ -141,7 +141,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 () :
@@ -165,7 +165,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)
@@ -181,7 +181,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	2012-08-06 17:08:41 +0000
+++ document_webdav/document_webdav.py	2012-10-02 11:29:53 +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	2012-06-22 06:48:39 +0000
+++ document_webdav/redirect.py	2012-10-02 11:29:53 +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	2012-08-31 13:51:36 +0000
+++ document_webdav/test_davclient.py	2012-10-02 11:29:53 +0000
@@ -683,7 +683,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
@@ -705,5 +705,4 @@
         etag = m.getheader('ETag')
         return etag or True
 
-#eof
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'document_webdav/webdav.py'
--- document_webdav/webdav.py	2012-07-13 05:29:57 +0000
+++ document_webdav/webdav.py	2012-10-02 11:29:53 +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-08-06 17:08:41 +0000
+++ document_webdav/webdav_server.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +0000
@@ -41,13 +41,13 @@
         if k.startswith("&#") and k.endswith(";"): return int(k[2:-1]) # not in latin-1
         return ord(codecs.latin_1_decode(k)[0])
 
-unifiable = {'rsquo':"'", 'lsquo':"'", 'rdquo':'"', 'ldquo':'"', 
+unifiable = {'rsquo':"'", 'lsquo':"'", 'rdquo':'"', 'ldquo':'"',
 'copy':'(C)', 'mdash':'--', 'nbsp':' ', 'rarr':'->', 'larr':'<-', 'middot':'*',
 'ndash':'-', 'oelig':'oe', 'aelig':'ae',
-'agrave':'a', 'aacute':'a', 'acirc':'a', 'atilde':'a', 'auml':'a', 'aring':'a', 
-'egrave':'e', 'eacute':'e', 'ecirc':'e', 'euml':'e', 
+'agrave':'a', 'aacute':'a', 'acirc':'a', 'atilde':'a', 'auml':'a', 'aring':'a',
+'egrave':'e', 'eacute':'e', 'ecirc':'e', 'euml':'e',
 'igrave':'i', 'iacute':'i', 'icirc':'i', 'iuml':'i',
-'ograve':'o', 'oacute':'o', 'ocirc':'o', 'otilde':'o', 'ouml':'o', 
+'ograve':'o', 'oacute':'o', 'ocirc':'o', 'otilde':'o', 'ouml':'o',
 'ugrave':'u', 'uacute':'u', 'ucirc':'u', 'uuml':'u'}
 
 unifiable_n = {}
@@ -60,7 +60,7 @@
         c = int(name[1:], 16)
     else:
         c = int(name)
-    
+
     if not UNICODE_SNOB and c in unifiable_n.keys():
         return unifiable_n[c]
     else:
@@ -76,14 +76,14 @@
 
 def replaceEntities(s):
     s = s.group(1)
-    if s[0] == "#": 
+    if s[0] == "#":
         return charref(s[1:])
     else: return entityref(s)
 
 r_unescape = re.compile(r"&(#?[xX]?(?:[0-9a-fA-F]+|\w{1,8}));")
 def unescape(s):
     return r_unescape.sub(replaceEntities, s)
-    
+
 def fixattrs(attrs):
     # Fix bug in sgmllib.py
     if not attrs: return attrs
@@ -105,7 +105,7 @@
     """Wrap all paragraphs in the provided text."""
     if not BODY_WIDTH:
         return text
-    
+
     assert wrap, "Requires Python 2.3."
     result = ''
     newlines = 0
@@ -136,7 +136,7 @@
 class _html2text(sgmllib.SGMLParser):
     def __init__(self, out=sys.stdout.write, baseurl=''):
         sgmllib.SGMLParser.__init__(self)
-        
+
         if out is None: self.out = self.outtextf
         else: self.out = out
         self.outtext = u''
@@ -157,43 +157,43 @@
         self.abbr_data = None # last inner HTML (for abbr being defined)
         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):
         sgmllib.SGMLParser.close(self)
-        
+
         self.pbr()
         self.o('', 0, 'end')
-        
+
         return self.outtext
-        
+
     def handle_charref(self, c):
         self.o(charref(c))
 
     def handle_entityref(self, c):
         self.o(entityref(c))
-            
+
     def unknown_starttag(self, tag, attrs):
         self.handle_tag(tag, attrs, 1)
-    
+
     def unknown_endtag(self, tag):
         self.handle_tag(tag, None, 0)
-        
+
     def previousIndex(self, attrs):
         """ returns the index of certain set of attributes (of a link) in the
             self.a list
- 
+
             If the set of attributes is not found, returns None
         """
         if not attrs.has_key('href'): return None
-        
+
         i = -1
         for a in self.a:
             i += 1
             match = 0
-            
+
             if a.has_key('href') and a['href'] == attrs['href']:
                 if a.has_key('title') or attrs.has_key('title'):
                         if (a.has_key('title') and attrs.has_key('title') and
@@ -206,13 +206,13 @@
 
     def handle_tag(self, tag, attrs, start):
         attrs = fixattrs(attrs)
-    
+
         if hn(tag):
             self.p()
             if start: self.o(hn(tag)*"#" + ' ')
 
         if tag in ['p', 'div']: self.p()
-        
+
         if tag == "br" and start: self.o("  \n")
 
         if tag == "hr" and start:
@@ -220,21 +220,21 @@
             self.o("* * *")
             self.p()
 
-        if tag in ["head", "style", 'script']: 
+        if tag in ["head", "style", 'script']:
             if start: self.quiet += 1
             else: self.quiet -= 1
 
         if tag in ["body"]:
             self.quiet = 0 # sites like 9rules.com never close <head>
-        
+
         if tag == "blockquote":
-            if start: 
+            if start:
                 self.p(); self.o('> ', 0, 1); self.start = 1
                 self.blockquote += 1
             else:
                 self.blockquote -= 1
                 self.p()
-        
+
         if tag in ['em', 'i', 'u']: self.o("_")
         if tag in ['strong', 'b']: self.o("**")
         if tag == "code" and not self.pre: self.o('`') #TODO: `` `this` ``
@@ -243,7 +243,7 @@
                 attrsD = {}
                 for (x, y) in attrs: attrsD[x] = y
                 attrs = attrsD
-                
+
                 self.abbr_title = None
                 self.abbr_data = ''
                 if attrs.has_key('title'):
@@ -253,13 +253,13 @@
                     self.abbr_list[self.abbr_data] = self.abbr_title
                     self.abbr_title = None
                 self.abbr_data = ''
-        
+
         if tag == "a":
             if start:
                 attrsD = {}
                 for (x, y) in attrs: attrsD[x] = y
                 attrs = attrsD
-                if attrs.has_key('href') and not (SKIP_INTERNAL_LINKS and attrs['href'].startswith('#')): 
+                if attrs.has_key('href') and not (SKIP_INTERNAL_LINKS and attrs['href'].startswith('#')):
                     self.astack.append(attrs)
                     self.o("[")
                 else:
@@ -277,7 +277,7 @@
                             a['outcount'] = self.outcount
                             self.a.append(a)
                         self.o("][" + `a['count']` + "]")
-        
+
         if tag == "img" and start:
             attrsD = {}
             for (x, y) in attrs: attrsD[x] = y
@@ -296,20 +296,20 @@
                 self.o("![")
                 self.o(alt)
                 self.o("]["+`attrs['count']`+"]")
-        
+
         if tag == 'dl' and start: self.p()
         if tag == 'dt' and not start: self.pbr()
         if tag == 'dd' and start: self.o('    ')
         if tag == 'dd' and not start: self.pbr()
-        
+
         if tag in ["ol", "ul"]:
             if start:
                 self.list.append({'name':tag, 'num':0})
             else:
                 if self.list: self.list.pop()
-            
+
             self.p()
-        
+
         if tag == 'li':
             if start:
                 self.pbr()
@@ -323,10 +323,10 @@
                 self.start = 1
             else:
                 self.pbr()
-        
+
         if tag in ["table", "tr"] and start: self.p()
         if tag == 'td': self.pbr()
-        
+
         if tag == "pre":
             if start:
                 self.startpre = 1
@@ -334,34 +334,35 @@
             else:
                 self.pre = 0
             self.p()
-            
+
     def pbr(self):
         if self.p_p == 0: self.p_p = 1
 
-    def p(self): self.p_p = 2
-    
+    def p(self):
+        self.p_p = 2
+
     def o(self, data, puredata=0, force=0):
         if self.abbr_data is not None: self.abbr_data += data
-        
-        if not self.quiet: 
+
+        if not self.quiet:
             if puredata and not self.pre:
                 data = re.sub('\s+', ' ', data)
                 if data and data[0] == ' ':
                     self.space = 1
                     data = data[1:]
             if not data and not force: return
-            
+
             if self.startpre:
                 #self.out(" :") #TODO: not output when already one there
                 self.startpre = 0
-            
+
             bq = (">" * self.blockquote)
             if not (force and data and data[0] == ">") and self.blockquote: bq += " "
-            
+
             if self.pre:
                 bq += "    "
                 data = data.replace("\n", "\n"+bq)
-            
+
             if self.start:
                 self.space = 0
                 self.p_p = 0
@@ -377,7 +378,7 @@
             if self.p_p:
                 self.out(('\n'+bq)*self.p_p)
                 self.space = 0
-                
+
             if self.space:
                 if not self.lastWasNL: self.out(' ')
                 self.space = 0
@@ -388,7 +389,7 @@
                 newa = []
                 for link in self.a:
                     if self.outcount > link['outcount']:
-                        self.out("   ["+`link['count']`+"]: " + urlparse.urljoin(self.baseurl, link['href'])) 
+                        self.out("   ["+`link['count']`+"]: " + urlparse.urljoin(self.baseurl, link['href']))
                         if link.has_key('title'): self.out(" ("+link['title']+")")
                         self.out("\n")
                     else:
@@ -397,7 +398,7 @@
                 if self.a != newa: self.out("\n") # Don't need an extra line when nothing was done.
 
                 self.a = newa
-            
+
             if self.abbr_list and force == "end":
                 for abbr, definition in self.abbr_list.items():
                     self.out("  *[" + abbr + "]: " + definition + "\n")
@@ -410,8 +411,9 @@
     def handle_data(self, data):
         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 'event/event.py'
--- event/event.py	2012-09-17 16:33:01 +0000
+++ event/event.py	2012-10-02 11:29:53 +0000
@@ -223,6 +223,7 @@
         'company_id': lambda self,cr,uid,c: self.pool.get('res.company')._company_default_get(cr, uid, 'event.event', context=c),
         'user_id': lambda obj, cr, uid, context: uid,
     }
+
     def subscribe_to_event(self, cr, uid, ids, context=None):
         register_pool = self.pool.get('event.registration')
         user_pool = self.pool.get('res.users')
@@ -252,6 +253,7 @@
     _constraints = [
         (_check_closing_date, 'Error ! Closing Date cannot be set before Beginning Date.', ['date_end']),
     ]
+
     def onchange_event_type(self, cr, uid, ids, type_event, context=None):
         if type_event:
             type_info =  self.pool.get('event.type').browse(cr,uid,type_event,context)
@@ -347,7 +349,6 @@
         'phone': fields.char('Phone', size=64),
         'name': fields.char('Name', size=128, select=True),
     }
-
     _defaults = {
         'nb_register': 1,
         'state': 'draft',
 
=== modified file 'event/report/__init__.py'
--- event/report/__init__.py	2011-01-14 00:11:01 +0000
+++ event/report/__init__.py	2012-10-02 11:29:53 +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/wizard/event_confirm.py'
--- event/wizard/event_confirm.py	2011-01-14 00:11:01 +0000
+++ event/wizard/event_confirm.py	2012-10-02 11:29:53 +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-09-26 07:45:27 +0000
+++ event_moodle/event_moodle.py	2012-10-02 11:29:53 +0000
@@ -127,7 +127,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-10-02 11:29:53 +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-10-02 11:29:53 +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/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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-07-25 12:13:54 +0000
+++ hr_attendance/wizard/hr_attendance_error.py	2012-10-02 11:29:53 +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-09-27 04:10:11 +0000
+++ hr_evaluation/hr_evaluation.py	2012-10-02 11:29:53 +0000
@@ -43,6 +43,7 @@
         'month_next': 12,
         'company_id': lambda s,cr,uid,c: s.pool.get('res.company')._company_default_get(cr, uid, 'account.account', context=c),
     }
+
 hr_evaluation_plan()
 
 class hr_evaluation_plan_phase(osv.osv):
@@ -95,6 +96,7 @@
 
         '''),
     }
+
 hr_evaluation_plan_phase()
 
 class hr_employee(osv.osv):
@@ -210,8 +212,6 @@
                 elif phase.action == "self":
                     children = [evaluation.employee_id]
                 for child in children:
-#                    if not child.user_id:
-#                        continue
 
                     int_id = hr_eval_inter_obj.create(cr, uid, {
                         'evaluation_id': evaluation.id,
@@ -250,7 +250,7 @@
                 raise osv.except_osv(_('Warning!'),_("You cannot change state, because some appraisal(s) are 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,{'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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-07-19 12:03:11 +0000
+++ hr_payroll/report/report_payslip_details.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-08-16 10:42:15 +0000
+++ hr_payroll_account/wizard/hr_payroll_payslips_by_employees.py	2012-10-02 11:29:53 +0000
@@ -38,4 +38,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	2012-09-16 15:18:26 +0000
+++ hr_recruitment/hr_recruitment.py	2012-10-02 11:29:53 +0000
@@ -243,7 +243,7 @@
         'stage_id': _read_group_stage_ids
     }
 
-    def onchange_job(self,cr, uid, ids, job, context=None):
+    def onchange_job(self, cr, uid, ids, job, context=None):
         result = {}
 
         if job:
@@ -494,7 +494,6 @@
         message = _("Applicant has been <b>created</b>.")
         return self.message_post(cr, uid, ids, body=message, context=context)
 
-
 class hr_job(osv.osv):
     _inherit = "hr.job"
     _name = "hr.job"
@@ -505,7 +504,6 @@
                                     help="Email alias for this job position. New emails will automatically "
                                          "create new applicants for this job position."),
     }
-
     _defaults = {
         'alias_domain': False, # always hide alias during creation
     }
@@ -548,12 +546,12 @@
         datas['model'] = 'survey.print'
         context.update({'response_id': [0], 'response_no': 0,})
         return {
-                'type': 'ir.actions.report.xml',
-                'report_name': 'survey.form',
-                'datas': datas,
-                'context' : context,
-                'nodestroy':True,
-            }
+            'type': 'ir.actions.report.xml',
+            'report_name': 'survey.form',
+            'datas': datas,
+            'context' : context,
+            'nodestroy':True,
+        }
 
 class applicant_category(osv.osv):
     """ Category of applicant """
 
=== modified file 'hr_recruitment/wizard/hr_recruitment_create_partner_job.py'
--- hr_recruitment/wizard/hr_recruitment_create_partner_job.py	2012-08-07 11:31:37 +0000
+++ hr_recruitment/wizard/hr_recruitment_create_partner_job.py	2012-10-02 11:29:53 +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 = {}
 
=== 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-10-02 11:29:53 +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-09-07 07:31:52 +0000
+++ hr_timesheet_invoice/hr_timesheet_invoice.py	2012-10-02 11:29:53 +0000
@@ -84,7 +84,7 @@
             res['value']['to_invoice'] = ir_model_obj.get_object_reference(cr, uid, 'hr_timesheet_invoice', 'timesheet_invoice_factor1')[1]
         return res
 
-    def on_change_partner_id(self, cr, uid, ids,partner_id, name, context=None):
+    def on_change_partner_id(self, cr, uid, ids, partner_id, name, context=None):
         res = super(account_analytic_account,self).on_change_partner_id(cr, uid, ids,partner_id, name, context=context)
         part = self.pool.get('res.partner').browse(cr, uid, partner_id, context=context)
         pricelist = part.property_product_pricelist and part.property_product_pricelist.id or False
@@ -199,6 +199,7 @@
 
 hr_analytic_timesheet()
 
+
 class account_invoice(osv.osv):
     _inherit = "account.invoice"
 
@@ -218,6 +219,7 @@
 
 account_invoice()
 
+
 class account_move_line(osv.osv):
     _inherit = "account.move.line"
 
@@ -234,4 +236,3 @@
 account_move_line()
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
-
 
=== 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-10-02 11:29:53 +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	2012-09-29 13:29:07 +0000
+++ hr_timesheet_invoice/wizard/hr_timesheet_invoice_create.py	2012-10-02 11:29:53 +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')
@@ -52,6 +52,8 @@
         invoices = []
         if context is None:
             context = {}
+        if data is None:
+            data = {}
 
         account_ids = {}
         for line in self.pool.get('account.analytic.line').browse(cr, uid, ids, context=context):
@@ -86,14 +88,14 @@
                 'date_due': date_due,
                 'fiscal_position': account.partner_id.property_account_position.id
             }
-            
+
             context2 = context.copy()
             context2['lang'] = partner.lang
-            # set company_id in context, so the correct default journal will be selected 
+            # set company_id in context, so the correct default journal will be selected
             context2['force_company'] = curr_invoice['company_id']
-            # set force_company in context so the correct product properties are selected (eg. income account) 
+            # set force_company in context so the correct product properties are selected (eg. income account)
             context2['company_id'] = curr_invoice['company_id']
-            
+
             last_invoice = invoice_obj.create(cr, uid, curr_invoice, context=context2)
             invoices.append(last_invoice)
 
 
=== modified file 'hr_timesheet_sheet/hr_timesheet_sheet.py'
--- hr_timesheet_sheet/hr_timesheet_sheet.py	2012-09-27 19:10:59 +0000
+++ hr_timesheet_sheet/hr_timesheet_sheet.py	2012-10-02 11:29:53 +0000
@@ -358,7 +358,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':
@@ -369,7 +369,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	2012-08-07 11:31:37 +0000
+++ hr_timesheet_sheet/wizard/hr_timesheet_current.py	2012-10-02 11:29:53 +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	2012-08-17 10:03:02 +0000
+++ idea/idea.py	2012-10-02 11:29:53 +0000
@@ -39,6 +39,7 @@
         ('name', 'unique(name)', 'The name of the category must be unique' )
     ]
     _order = 'name asc'
+
 idea_category()
 
 class idea_idea(osv.osv):
@@ -65,24 +66,24 @@
     }
     _order = 'name asc'
 
-    def idea_cancel(self, cr, uid, ids, context={}):
+    def idea_cancel(self, cr, uid, ids, context=None):
         self.write(cr, uid, ids, { 'state': 'cancel' })
         self.message_post(cr, uid, ids, body=_('Idea canceled.'), context=context)
         return True
 
-    def idea_open(self, cr, uid, ids, context={}):
+    def idea_open(self, cr, uid, ids, context=None):
         self.write(cr, uid, ids, { 'state': 'open'})
         self.message_post(cr, uid, ids, body=_('Idea accepted.'), context=context)
         return True
 
-    def idea_close(self, cr, uid, ids, context={}):
+    def idea_close(self, cr, uid, ids, context=None):
         self.message_post(cr, uid, ids, body=_('Idea done.'), context=context)
         self.write(cr, uid, ids, { 'state': 'close' })
         return True
 
-    def idea_draft(self, cr, uid, ids, context={}):
+    def idea_draft(self, cr, uid, ids, context=None):
         self.message_post(cr, uid, ids, body=_('Idea reset to draft.'), context=context)
         self.write(cr, uid, ids, { 'state': 'draft' })
         return True
+
 idea_idea()
-
 
=== modified file 'import_base/import_framework.py'
--- import_base/import_framework.py	2012-09-13 09:37:17 +0000
+++ import_base/import_framework.py	2012-10-02 11:29:53 +0000
@@ -227,7 +227,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-09-07 11:44:36 +0000
+++ import_sugarcrm/import_sugarcrm.py	2012-10-02 11:29:53 +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],
@@ -602,7 +602,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	2012-09-13 09:28:14 +0000
+++ l10n_be/__init__.py	2012-10-02 11:29:53 +0000
@@ -21,4 +21,4 @@
 
 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_coda/l10n_be_coda.py'
--- l10n_be_coda/l10n_be_coda.py	2012-09-25 07:32:01 +0000
+++ l10n_be_coda/l10n_be_coda.py	2012-10-02 11:29:53 +0000
@@ -171,6 +171,7 @@
         'parent_id': fields.many2one('account.coda.trans.type', 'Parent'),
         'description': fields.text('Description', translate=True),
     }
+
 account_coda_trans_type()
 
 class account_coda_trans_code(osv.osv):
@@ -187,6 +188,7 @@
         'description': fields.char('Description', size=128, translate=True, select=2),
         'comment': fields.text('Comment', translate=True),
     }
+
 account_coda_trans_code()
 
 class account_coda_trans_category(osv.osv):
@@ -197,6 +199,7 @@
         'category': fields.char('Transaction Category', size=3, required=True),
         'description': fields.char('Description', size=256, translate=True),
     }
+
 account_coda_trans_category()
 
 class account_coda_comm_type(osv.osv):
@@ -209,14 +212,17 @@
     }
     _sql_constraints = [
         ('code_uniq', 'unique (code)','The Structured Communication Code must be unique !')
-        ]
+    ]
+
 account_coda_comm_type()
 
 class coda_bank_statement(osv.osv):
     _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
@@ -232,7 +238,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]
@@ -404,6 +410,7 @@
     _columns = {
         'coda_statement_line_ids': fields.one2many('coda.bank.statement.line', 'globalisation_id', 'CODA Bank Statement Lines', readonly=True),
     }
+
 account_bank_statement_line_global()
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'l10n_be_invoice_bba/invoice.py'
--- l10n_be_invoice_bba/invoice.py	2012-07-18 12:26:36 +0000
+++ l10n_be_invoice_bba/invoice.py	2012-10-02 11:29:53 +0000
@@ -2,9 +2,9 @@
 ##############################################################################
 #
 #    OpenERP, Open Source Management Solution
-#    
+#
 #    Copyright (c) 2011 Noviat nv/sa (www.noviat.be). All rights reserved.
-# 
+#
 #    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
@@ -36,23 +36,23 @@
     _inherit = 'account.invoice'
 
     def _get_reference_type(self, cursor, user, context=None):
-        """Add BBA Structured Communication Type and change labels from 'reference' into 'communication' """  
+        """Add BBA Structured Communication Type and change labels from 'reference' into 'communication' """
         res = super(account_invoice, self)._get_reference_type(cursor, user,
                 context=context)
         res[[i for i,x in enumerate(res) if x[0] == 'none'][0]] = ('none', 'Free Communication')
         res.append(('bba', 'BBA Structured Communication'))
-        #l_logger.warning('reference_type =  %s' %res ) 
+        #l_logger.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 ''):
-            return False                
+            return False
         bbacomm = re.sub('\D', '', val or '')
         if len(bbacomm) == 12:
             base = int(bbacomm[:10])
-            mod = base % 97 or 97      
+            mod = base % 97 or 97
             if mod == int(bbacomm[-2:]):
                 return True
         return False
@@ -64,29 +64,29 @@
         return True
 
     def onchange_partner_id(self, cr, uid, ids, type, partner_id,
-            date_invoice=False, payment_term=False, partner_bank_id=False, company_id=False):       
+            date_invoice=False, payment_term=False, partner_bank_id=False, company_id=False):
         result = super(account_invoice, self).onchange_partner_id(cr, uid, ids, type, partner_id,
             date_invoice, payment_term, partner_bank_id, company_id)
 #        reference_type = self.default_get(cr, uid, ['reference_type'])['reference_type']
 #        _logger.warning('partner_id %s' % partner_id)
         reference = False
         reference_type = 'none'
-        if partner_id:        
+        if partner_id:
             if (type == 'out_invoice'):
                 reference_type = self.pool.get('res.partner').browse(cr, uid, partner_id).out_inv_comm_type
                 if reference_type:
                     reference = self.generate_bbacomm(cr, uid, ids, type, reference_type, partner_id, '', context={})['value']['reference']
-        res_update = {       
+        res_update = {
             'reference_type': reference_type or 'none',
             'reference': reference,
         }
         result['value'].update(res_update)
-        return result                    
+        return result
 
     def generate_bbacomm(self, cr, uid, ids, type, reference_type, partner_id, reference, context=None):
         partner_obj =  self.pool.get('res.partner')
-        reference = reference or ''
-        algorithm = False
+        reference = reference or ''
+        algorithm = False
         if partner_id:
             algorithm = partner_obj.browse(cr, uid, partner_id, context=context).out_inv_comm_algorithm
         algorithm = algorithm or 'random'
@@ -97,7 +97,7 @@
                         doy = time.strftime('%j')
                         year = time.strftime('%Y')
                         seq = '001'
-                        seq_ids = self.search(cr, uid, 
+                        seq_ids = self.search(cr, uid,
                             [('type', '=', 'out_invoice'), ('reference_type', '=', 'bba'),
                              ('reference', 'like', '+++%s/%s/%%' % (doy, year))], order='reference')
                         if seq_ids:
@@ -110,7 +110,7 @@
                                       '\nPlease create manually a unique BBA Structured Communication.'))
                         bbacomm = doy + year + seq
                         base = int(bbacomm)
-                        mod = base % 97 or 97   
+                        mod = base % 97 or 97
                         reference = '+++%s/%s/%s%02d+++' % (doy, year, seq, mod)
                 elif algorithm == 'partner_ref':
                     if not self.check_bbacomm(reference):
@@ -119,13 +119,13 @@
                         if (len(partner_ref_nr) < 3) or (len(partner_ref_nr) > 7):
                             raise osv.except_osv(_('Warning!'),
                                 _('The Partner should have a 3-7 digit Reference Number for the generation of BBA Structured Communications!' \
-                                  '\nPlease correct the Partner record.'))                            
+                                  '\nPlease correct the Partner record.'))
                         else:
                             partner_ref_nr = partner_ref_nr.ljust(7, '0')
                             seq = '001'
-                            seq_ids = self.search(cr, uid, 
+                            seq_ids = self.search(cr, uid,
                                 [('type', '=', 'out_invoice'), ('reference_type', '=', 'bba'),
-                                 ('reference', 'like', '+++%s/%s/%%' % (partner_ref_nr[:3], partner_ref_nr[3:]))], order='reference')                            
+                                 ('reference', 'like', '+++%s/%s/%%' % (partner_ref_nr[:3], partner_ref_nr[3:]))], order='reference')
                             if seq_ids:
                                 prev_seq = int(self.browse(cr, uid, seq_ids[-1]).reference[12:15])
                                 if prev_seq < 999:
@@ -133,10 +133,10 @@
                                 else:
                                     raise osv.except_osv(_('Warning!'),
                                         _('The daily maximum of outgoing invoices with an automatically generated BBA Structured Communications has been exceeded!' \
-                                          '\nPlease create manually a unique BBA Structured Communication.'))                            
+                                          '\nPlease create manually a unique BBA Structured Communication.'))
                         bbacomm = partner_ref_nr + seq
                         base = int(bbacomm)
-                        mod = base % 97 or 97                           
+                        mod = base % 97 or 97
                         reference = '+++%s/%s/%s%02d+++' % (partner_ref_nr[:3], partner_ref_nr[3:], seq, mod)
                 elif algorithm == 'random':
                     if not self.check_bbacomm(reference):
@@ -144,67 +144,66 @@
                         bbacomm = str(base).rjust(7, '0')
                         base = int(bbacomm)
                         mod = base % 97 or 97
-                        mod = str(mod).rjust(2, '0')               
+                        mod = str(mod).rjust(2, '0')
                         reference = '+++%s/%s/%s%s+++' % (bbacomm[:3], bbacomm[3:7], bbacomm[7:], mod)
                 else:
                     raise osv.except_osv(_('Error!'),
                         _("Unsupported Structured Communication Type Algorithm '%s' !" \
-                          "\nPlease contact your OpenERP support channel.") % algorithm)   
-        return {'value': {'reference': reference}}    
-    
+                          "\nPlease contact your OpenERP support channel.") % algorithm)
+        return {'value': {'reference': reference}}
+
     def create(self, cr, uid, vals, context=None):
         if vals.has_key('reference_type'):
             reference_type = vals['reference_type']
-            if reference_type == 'bba':               
+            if reference_type == 'bba':
                 if vals.has_key('reference'):
                     bbacomm = vals['reference']
                 else:
                     raise osv.except_osv(_('Warning!'),
                         _('Empty BBA Structured Communication!' \
-                          '\nPlease fill in a unique BBA Structured Communication.'))       
+                          '\nPlease fill in a unique BBA Structured Communication.'))
                 if self.check_bbacomm(bbacomm):
                     reference = re.sub('\D', '', bbacomm)
-                    vals['reference'] = '+++' + reference[0:3] + '/' + reference[3:7] + '/' + reference[7:] + '+++'     
-                    same_ids = self.search(cr, uid, 
+                    vals['reference'] = '+++' + reference[0:3] + '/' + reference[3:7] + '/' + reference[7:] + '+++'
+                    same_ids = self.search(cr, uid,
                         [('type', '=', 'out_invoice'), ('reference_type', '=', 'bba'),
                          ('reference', '=', vals['reference'])])
                     if same_ids:
                         raise osv.except_osv(_('Warning!'),
                             _('The BBA Structured Communication has already been used!' \
-                              '\nPlease create manually a unique BBA Structured Communication.'))                 
-        return super(account_invoice, self).create(cr, uid, vals, context=context)     
+                              '\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):    
+        for inv in self.browse(cr, uid, ids, context):
             if vals.has_key('reference_type'):
                 reference_type = vals['reference_type']
-            else:    
+            else:
                 reference_type = inv.reference_type or ''
-            if reference_type == 'bba':               
+            if reference_type == 'bba':
                 if vals.has_key('reference'):
                     bbacomm = vals['reference']
                 else:
                     bbacomm = inv.reference or ''
                 if self.check_bbacomm(bbacomm):
                     reference = re.sub('\D', '', bbacomm)
-                    vals['reference'] = '+++' + reference[0:3] + '/' + reference[3:7] + '/' + reference[7:] + '+++'     
-                    same_ids = self.search(cr, uid, 
-                        [('id', '!=', inv.id), ('type', '=', 'out_invoice'), 
+                    vals['reference'] = '+++' + reference[0:3] + '/' + reference[3:7] + '/' + reference[7:] + '+++'
+                    same_ids = self.search(cr, uid,
+                        [('id', '!=', inv.id), ('type', '=', 'out_invoice'),
                          ('reference_type', '=', 'bba'), ('reference', '=', vals['reference'])])
                     if same_ids:
                         raise osv.except_osv(_('Warning!'),
                             _('The BBA Structured Communication has already been used!' \
-                              '\nPlease create manually a unique BBA Structured Communication.'))                 
-        return super(account_invoice, self).write(cr, uid, ids, vals, context)    
+                              '\nPlease create manually a unique BBA Structured Communication.'))
+        return super(account_invoice, self).write(cr, uid, ids, vals, context)
 
     _columns = {
         'reference': fields.char('Communication', size=64, help="The partner reference of this invoice."),
         'reference_type': fields.selection(_get_reference_type, 'Communication Type',
             required=True),
     }
-    
     _constraints = [
         (_check_communication, 'Invalid BBA Structured Communication !', ['Communication']),
         ]
 
=== 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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-08-07 11:06:16 +0000
+++ l10n_ch/report/report_webkit_html.py	2012-10-02 11:29:53 +0000
@@ -76,7 +76,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-08-07 11:31:37 +0000
+++ l10n_ch/wizard/create_dta.py	2012-10-02 11:29:53 +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	2012-04-13 10:39:57 +0000
+++ l10n_fr/l10n_fr.py	2012-10-02 11:29:53 +0000
@@ -29,7 +29,6 @@
         'name': fields.char('Name', size=128),
         'line_ids': fields.one2many('l10n.fr.line', 'report_id', 'Lines'),
     }
-
     _sql_constraints = [
         ('code_uniq', 'unique (code)','The code report must be unique !')
     ]
@@ -61,7 +60,4 @@
 
 res_company()
 
-
-
-
 # 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-10-02 11:29:53 +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_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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-09-18 15:59:05 +0000
+++ mail/mail_thread.py	2012-10-02 11:29:53 +0000
@@ -590,7 +590,7 @@
         context = context or {}
         attachments = attachments or []
         assert (not thread_id) or isinstance(thread_id, (int,long)) or \
-            (isinstance(thread_id, (list, tuple)) and len(thread_id) == 1), "Invalid thread_id" 
+            (isinstance(thread_id, (list, tuple)) and len(thread_id) == 1), "Invalid thread_id"
         if isinstance(thread_id, (list, tuple)):
             thread_id = thread_id and thread_id[0]
 
@@ -618,7 +618,7 @@
             'parent_id': parent_id,
             'attachment_ids': attachment_ids,
         })
-        for x in ('from', 'to', 'cc'): values.pop(x, None) # Avoid warnings 
+        for x in ('from', 'to', 'cc'): values.pop(x, None) # Avoid warnings
         return self.pool.get('mail.message').create(cr, uid, values, context=context)
 
     #------------------------------------------------------
 
=== 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-10-02 11:29:53 +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	2012-05-04 12:13:26 +0000
+++ membership/report/report_membership.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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-09-30 12:57:57 +0000
+++ mrp/mrp.py	2012-10-02 11:29:53 +0000
@@ -274,13 +274,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
@@ -295,7 +297,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.
@@ -609,11 +611,13 @@
         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.
         """
+        if properties is None:
+            properties = []
         results = []
         bom_obj = self.pool.get('mrp.bom')
         uom_obj = self.pool.get('product.uom')
@@ -832,7 +836,7 @@
                         'product_id': wc.product_id.id,
                         'unit_amount': wc_line.hour,
                         'product_uom_id': wc.product_id.id and wc.product_id.uom_id.id or False
-                        
+
                     } )
             if wc.costs_journal_id and wc.costs_general_account_id:
                 value = wc_line.cycle * wc.costs_cycle
@@ -849,7 +853,7 @@
                         'product_id': wc.product_id.id,
                         'unit_amount': wc_line.cycle,
                         'product_uom_id': wc.product_id.id and wc.product_id.uom_id.id or False
-                        
+
                     } )
         return amount
 
 
=== modified file 'mrp/procurement.py'
--- mrp/procurement.py	2012-08-17 10:03:02 +0000
+++ mrp/procurement.py	2012-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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-08-17 10:03:02 +0000
+++ mrp_operations/mrp_operations.py	2012-10-02 11:29:53 +0000
@@ -422,7 +422,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	2012-09-13 08:25:35 +0000
+++ plugin/plugin_handler.py	2012-10-02 11:29:53 +0000
@@ -6,7 +6,6 @@
 
 from osv import osv, fields
 
-
 class plugin_handler(osv.osv_memory):
     _name = 'plugin.handler'
 
@@ -37,7 +36,7 @@
             @param email: email is a standard RFC2822 email message
             @return Dictionary which contain id and the model name of the document linked with the mail
                 if no document is found the id = 0
-                (model_name, res_id, url, name_get) 
+                (model_name, res_id, url, name_get)
         """
         mail_message_obj = self.pool.get('mail.message')
         model = ""
@@ -52,7 +51,7 @@
             msg_ids = mail_message_obj.search(cr, uid, [('message_id','=', message_id)])
             msg_id = len(msg_ids) and msg_ids[0] or False
         if not msg_id and parent_id:
-            msg_id = parent_id 
+            msg_id = parent_id
         if msg_id:
             msg = mail_message_obj.browse(cr, uid, msg_id)
             res_id = msg.res_id
@@ -61,7 +60,6 @@
             name =  self.pool.get(model).name_get(cr, uid, [res_id])[0][1]
         return (model,res_id, url,name)
 
-
     def document_type(self, cr, uid, context=None):
         """
             Return the list of available model to push
@@ -74,13 +72,13 @@
         doc_dict['res.partner'] = "Partner"
         return doc_dict.items()
 
-    # Can be used where search record was used 
+    # Can be used where search record was used
     def list_document_get(self, cr, uid, model, name):
         """
             This function return the result of name_search on the object model
-            @param model: the name of the model 
+            @param model: the name of the model
             @param : the name of the document
-            @return : the result of name_search a list of tuple 
+            @return : the result of name_search a list of tuple
             [(id, 'name')]
         """
         return self.pool.get(model).name_search(cr, uid, name)
@@ -89,7 +87,7 @@
         """
             @param email: email is a standard RFC2822 email message
             @param model: On which model the message is pushed
-            @param thread_id: on which document the message is pushed, if thread_id = 0 a new document is created 
+            @param thread_id: on which document the message is pushed, if thread_id = 0 a new document is created
             @return Dictionary which contain model , url and resource id.
         """
         mail_message = self.pool.get('mail.message')
@@ -97,7 +95,7 @@
         msg = self.pool.get('mail.thread').message_parse(cr, uid, email)
         message_id = msg.get('message-id')
         mail_ids = mail_message.search(cr, uid, [('message_id','=',message_id),('res_id','=',res_id),('model','=',model)])
-        
+
         if message_id and mail_ids :
             mail_record = mail_message.browse(cr, uid, mail_ids)[0]
             res_id = mail_record.res_id
@@ -109,11 +107,11 @@
                 res_id = model_obj.message_process(cr, uid, model, email)
                 notify = "Mail successfully pushed, a new %s has been created " % model
         else:
-            model_obj.message_post(cr, uid, [res_id], 
-                            body= msg.get('body'), 
-                            subject= msg.get('subject'), 
-                            type= 'email', 
-                            parent_id= msg.get('parent_id'), 
+            model_obj.message_post(cr, uid, [res_id],
+                            body= msg.get('body'),
+                            subject= msg.get('subject'),
+                            type= 'email',
+                            parent_id= msg.get('parent_id'),
                             attachments= msg.get('attachments'))
             notify = "Mail successfully pushed"
         url = self._make_url(cr, uid, res_id, model)
@@ -123,7 +121,7 @@
         """
             @param data : the data use to create the res.partner
                 [('field_name', value)], field name is required
-            @param partner_id : On which partner the address is attached 
+            @param partner_id : On which partner the address is attached
              if partner_id = 0 then create a new partner with the same name that the address
             @return : the partner_id sended or created, this allow the plugin to open the right partner page
         """
@@ -138,7 +136,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=False, body_html=False, attachments=False):
+    def push_message_outlook(self, cr, uid, model, headers, res_id=0, body=False, body_html=False, attachments=False):
         # ----------------------------------------
         # solution 1
         # construct a fake rfc822 from the separated arguement
@@ -149,14 +147,14 @@
         # solution 2
         # use self.pushmessage only with header and body
         # add attachemnt yourself after
-        mail_message = self.pool.get('mail.message')        
+        mail_message = self.pool.get('mail.message')
         ir_attachment_obj = self.pool.get('ir.attachment')
         attach_ids = []
         msg = self.pool.get('mail.thread').message_parse(cr, uid, headers)
-        message_id = msg.get('message-id')    
+        message_id = msg.get('message-id')
         push_mail = self.push_message(cr, uid, model, headers, res_id)
         res_id = push_mail[1]
-        model =  push_mail[0]            
+        model =  push_mail[0]
         for name in attachments.keys():
             attachment_ids = ir_attachment_obj.search(cr, uid, [('res_model', '=', model), ('res_id', '=', res_id), ('datas_fname', '=', name)])
             if attachment_ids:
 
=== 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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-05-12 22:37:43 +0000
+++ point_of_sale/report/pos_details_summary.py	2012-10-02 11:29:53 +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])
 
 
=== 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-10-02 11:29:53 +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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-03-30 09:08:37 +0000
+++ point_of_sale/report/pos_receipt.py	2012-10-02 11:29:53 +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	2012-08-21 08:23:43 +0000
+++ point_of_sale/report/pos_sales_user.py	2012-10-02 11:29:53 +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={}
 
=== 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-10-02 11:29:53 +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-10-02 11:29:53 +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 'product/product.py'
--- product/product.py	2012-09-26 12:24:35 +0000
+++ product/product.py	2012-10-02 11:29:53 +0000
@@ -335,7 +335,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 {}
@@ -594,7 +594,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	2012-07-11 13:06:15 +0000
+++ product/report/product_pricelist.py	2012-10-02 11:29:53 +0000
@@ -24,6 +24,7 @@
 from osv import osv
 import pooler
 from tools.translate import _
+
 class product_pricelist(report_sxw.rml_parse):
     def __init__(self, cr, uid, name, context):
         super(product_pricelist, self).__init__(cr, uid, name, context=context)
@@ -39,7 +40,7 @@
             'get_titles': self._get_titles,
         })
 
-    def _get_titles(self,form):
+    def _get_titles(self, form):
         lst = []
         vals = {}
         qtys = 1
@@ -51,7 +52,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 +77,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 +112,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='Product 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)
@@ -123,5 +124,5 @@
         return price
 
 report_sxw.report_sxw('report.product.pricelist','product.product','addons/product/report/product_pricelist.rml',parser=product_pricelist)
+
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
-
 
=== 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-10-02 11:29:53 +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-10-02 11:29:53 +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-09-29 19:58:25 +0000
+++ project/project.py	2012-10-02 11:29:53 +0000
@@ -55,7 +55,6 @@
     }
     _order = 'sequence'
 
-
 def short_name(name):
         """Keep first word(s) of name to make it small enough
            but distinctive"""
@@ -191,7 +190,6 @@
 
     # Lambda indirection method to avoid passing a copy of the overridable method when declaring the field
     _alias_models = lambda self, *args, **kwargs: self._get_alias_models(*args, **kwargs)
-
     _columns = {
         'complete_name': fields.function(_complete_name, string="Project Name", type='char', size=250),
         'active': fields.boolean('Active', help="If the active field is set to False, it will allow you to hide the project without removing it."),
@@ -311,11 +309,12 @@
         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 = {}
+        if default is None:
+            default = {}
 
-        default = default or {}
         context['active_test'] = False
         default['state'] = 'open'
         default['tasks'] = []
@@ -650,12 +649,12 @@
                 res[task.id]['progress'] = 100.0
         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):
@@ -686,7 +685,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):
@@ -699,7 +700,6 @@
                 default.update({'name':new_name})
         return super(task, self).copy_data(cr, uid, id, default, context)
 
-
     def _is_template(self, cr, uid, ids, field_name, arg, context=None):
         res = {}
         for task in self.browse(cr, uid, ids, context=context):
@@ -777,7 +777,6 @@
         'color': fields.integer('Color Index'),
         'user_email': fields.related('user_id', 'email', type='char', string='User Email', readonly=True),
     }
-
     _defaults = {
         'stage_id': _get_default_stage_id,
         'project_id': _get_default_project_id,
@@ -790,7 +789,6 @@
         'user_id': lambda obj, cr, uid, context: uid,
         'company_id': lambda self, cr, uid, c: self.pool.get('res.company')._company_default_get(cr, uid, 'project.task', context=c),
     }
-
     _order = "priority, sequence, date_start, name, id"
 
     def set_priority(self, cr, uid, ids, priority, *args):
@@ -853,9 +851,8 @@
         (_check_recursion, 'Error ! You cannot create recursive tasks.', ['parent_ids']),
         (_check_dates, 'Error ! Task end-date must be greater then task start-date', ['date_start','date_end'])
     ]
-    #
+
     # Override view according to the company definition
-    #
     def fields_view_get(self, cr, uid, view_id=None, view_type='form', context=None, toolbar=False, submenu=False):
         users_obj = self.pool.get('res.users')
         if context is None: context = {}
@@ -887,9 +884,9 @@
                 res['fields'][f]['string'] = res['fields'][f]['string'].replace('Hours',tm)
         return res
 
-    # ****************************************
+    # ----------------------------------------
     # Case management
-    # ****************************************
+    # ----------------------------------------
 
     def stage_find(self, cr, uid, cases, section_id, domain=[], order='sequence', context=None):
         """ Override of the base.stage method
@@ -1021,10 +1018,12 @@
             new_attachment_ids.append(attachment.copy(cr, uid, attachment_id, default={'res_id': delegated_task_id}, context=context))
         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):
@@ -1170,7 +1169,7 @@
         return result
 
     # ---------------------------------------------------
-    # mail gateway
+    # Mail gateway
     # ---------------------------------------------------
 
     def message_new(self, cr, uid, msg, custom_values=None, context=None):
 
=== 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-10-02 11:29:53 +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	2012-06-27 13:14:12 +0000
+++ project_gtd/project_gtd.py	2012-10-02 11:29:53 +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	2012-08-07 11:31:37 +0000
+++ project_gtd/wizard/project_gtd_empty.py	2012-10-02 11:29:53 +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_mrp/project_procurement.py'
--- project_mrp/project_procurement.py	2012-09-30 10:59:00 +0000
+++ project_mrp/project_procurement.py	2012-10-02 11:29:53 +0000
@@ -42,7 +42,7 @@
                 return False
         return True
 
-    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-09-29 20:13:26 +0000
+++ project_timesheet/project_timesheet.py	2012-10-02 11:29:53 +0000
@@ -225,7 +225,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	2012-07-31 14:47:04 +0000
+++ project_timesheet/report/task_report.py	2012-10-02 11:29:53 +0000
@@ -29,7 +29,7 @@
     _auto = False
     _order = "name"
 
-    def get_hrs_timesheet(self, cr, uid, ids, name,args,context):
+    def get_hrs_timesheet(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)
 
=== modified file 'purchase/purchase.py'
--- purchase/purchase.py	2012-09-30 11:40:14 +0000
+++ purchase/purchase.py	2012-10-02 11:29:53 +0000
@@ -31,9 +31,6 @@
 from osv.orm import browse_record, browse_null
 from tools import DEFAULT_SERVER_DATE_FORMAT, DEFAULT_SERVER_DATETIME_FORMAT, DATETIME_FORMATS_MAP
 
-#
-# Model definition
-#
 class purchase_order(osv.osv):
 
     def _amount_all(self, cr, uid, ids, field_name, arg, context=None):
@@ -239,7 +236,6 @@
             else:
                 raise osv.except_osv(_('Invalid Action!'), _('In order to delete a purchase order, you must cancel it first.'))
 
-        # TODO: temporary fix in 5.0, to remove in 5.2 when subflows support
         # automatically sending subflow.delete upon deletion
         wf_service = netsvc.LocalService("workflow")
         for id in unlink_ids:
@@ -333,7 +329,7 @@
                 'view_id': view_id,
                 'res_id': pick_ids[0]
             })
-            
+
         action.update({
             'context': ctx,
         })
@@ -380,7 +376,7 @@
             for line in po.order_line:
                 if line.state=='draft':
                     todo.append(line.id)
-#        current_name = self.name_get(cr, uid, ids)[0][1]
+
         self.pool.get('purchase.order.line').action_confirm(cr, uid, todo, context)
         for id in ids:
             self.write(cr, uid, [id], {'state' : 'confirmed', 'validator' : uid})
@@ -491,7 +487,7 @@
         self.invoice_done_send_note(cr, uid, ids, context=context)
         return True
 
-    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'):
@@ -596,7 +592,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))
@@ -665,7 +661,7 @@
             list_key.sort()
             return tuple(list_key)
 
-    # compute what the new orders should contain
+        # Compute what the new orders should contain
 
         new_orders = {}
 
@@ -1098,4 +1094,5 @@
         return super(mail_mail, self)._postprocess_sent_message(cr, uid, mail=mail, context=context)
 
 mail_mail()
+
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'purchase/wizard/purchase_order_group.py'
--- purchase/wizard/purchase_order_group.py	2012-08-07 11:34:14 +0000
+++ purchase/wizard/purchase_order_group.py	2012-10-02 11:29:53 +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	2012-09-29 09:07:50 +0000
+++ purchase_requisition/purchase_requisition.py	2012-10-02 11:29:53 +0000
@@ -207,7 +207,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-10-02 11:29:53 +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-10-02 11:29:53 +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	2012-07-25 12:49:45 +0000
+++ resource/faces/task.py	2012-10-02 11:29:53 +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	2012-09-24 16:26:45 +0000
+++ resource/resource.py	2012-10-02 11:29:53 +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/report/__init__.py'
--- sale/report/__init__.py	2011-01-14 00:11:01 +0000
+++ sale/report/__init__.py	2012-10-02 11:29:53 +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-09-30 11:40:14 +0000
+++ sale/sale.py	2012-10-02 11:29:53 +0000
@@ -253,7 +253,7 @@
 
         return osv.osv.unlink(self, cr, uid, unlink_ids, context=context)
 
-    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 = {
@@ -262,7 +262,6 @@
         }
         return {'warning': warning}
 
-
     def onchange_partner_id(self, cr, uid, ids, part):
         if not part:
             return {'value': {'partner_invoice_id': False, 'partner_shipping_id': False,  'payment_term': False, 'fiscal_position': False}}
@@ -437,7 +436,9 @@
                 return False
         return True
 
-    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 = []
@@ -648,7 +649,7 @@
     def cancel_send_note(self, cr, uid, ids, context=None):
         for obj in self.browse(cr, uid, ids, context=context):
             self.message_post(cr, uid, [obj.id], body=_("Sale Order for <em>%s</em> <b>cancelled</b>.") % (obj.partner_id.name), context=context)
-            
+
     def done_send_note(self, cr, uid, ids, context=None):
         for obj in self.browse(cr, uid, ids, context=context):
             self.message_post(cr, uid, [obj.id], body=_("Sale Order for <em>%s</em> set to <b>Done</b>") % (obj.partner_id.name), context=context)
@@ -874,12 +875,12 @@
         context_partner = {'lang': lang, 'partner_id': partner_id}
 
         if not product:
-            return {'value': {'th_weight': 0, 
+            return {'value': {'th_weight': 0,
                 'product_uos_qty': qty}, 'domain': {'product_uom': [],
                    'product_uos': []}}
         if not date_order:
             date_order = time.strftime(DEFAULT_SERVER_DATE_FORMAT)
-#
+
         result = {}
         warning_msgs = {}
         product_obj = product_obj.browse(cr, uid, product, context=context)
@@ -995,6 +996,7 @@
             wf_service = netsvc.LocalService("workflow")
             wf_service.trg_validate(uid, 'sale.order', context.get('default_res_id', False), 'quotation_sent', cr)
         return super(mail_compose_message, self).send_mail(cr, uid, ids, context=context)
+
 mail_compose_message()
 
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
 
=== modified file 'sale/wizard/__init__.py'
--- sale/wizard/__init__.py	2012-09-20 13:12:29 +0000
+++ sale/wizard/__init__.py	2012-10-02 11:29:53 +0000
@@ -23,4 +23,4 @@