diff --git a/addons/hr_recruitment/wizard/hr_recruitment_phonecall.py b/addons/hr_recruitment/wizard/hr_recruitment_phonecall.py index d87aeb7b93f..ecb6a7cb3a4 100644 --- a/addons/hr_recruitment/wizard/hr_recruitment_phonecall.py +++ b/addons/hr_recruitment/wizard/hr_recruitment_phonecall.py @@ -81,7 +81,7 @@ class job2phonecall(osv.osv_memory): if id3: id3 = data_obj.browse(cr, uid, id3, context=context).res_id - for job in job_case_obj.browse(cr, uid, context.get('active_id', False), context=context): + for job in job_case_obj.browse(cr, uid, context.get('active_ids', []), context=context): #TODO: Take other info from job new_phonecall_id = phonecall_case_obj.create(cr, uid, { 'name': job.name, diff --git a/addons/mrp_repair/mrp_repair.py b/addons/mrp_repair/mrp_repair.py index 728d38036c9..6de8b4f5492 100644 --- a/addons/mrp_repair/mrp_repair.py +++ b/addons/mrp_repair/mrp_repair.py @@ -192,6 +192,8 @@ class mrp_repair(osv.osv): def copy(self, cr, uid, id, default=None, context=None): if not default: default = {} + if not context: + context = {} default.update({ 'state':'draft', 'repaired':False, diff --git a/addons/outlook/installer.py b/addons/outlook/installer.py index 143567dd7cf..6b571b7b70a 100755 --- a/addons/outlook/installer.py +++ b/addons/outlook/installer.py @@ -30,6 +30,8 @@ class outlook_installer(osv.osv_memory): _inherit = 'res.config.installer' def default_get(self, cr, uid, fields, context=None): + if not context: + context = {} data = super(outlook_installer, self).default_get(cr, uid, fields, context=context) data['doc_file'] = 'http://doc.openerp.com/book/2/2_6_Comms/2_6_Comms_outlook.html' file = open(addons.get_module_resource('outlook','plugin','openerp-outlook-addin.exe'), 'r') diff --git a/addons/point_of_sale/account_bank_statement.py b/addons/point_of_sale/account_bank_statement.py index 552687f7bdb..fa54ab265b2 100644 --- a/addons/point_of_sale/account_bank_statement.py +++ b/addons/point_of_sale/account_bank_statement.py @@ -52,6 +52,8 @@ class account_cash_statement(osv.osv): return True def _user_allow(self, cr, uid, statement_id, context=None): + if not context: + context = {} statement = self.browse(cr, uid, statement_id, context=context) if (not statement.journal_id.journal_users) and uid == 1: return True for user in statement.journal_id.journal_users: diff --git a/addons/point_of_sale/point_of_sale.py b/addons/point_of_sale/point_of_sale.py index c53cc5daf23..939f5dbe0e0 100644 --- a/addons/point_of_sale/point_of_sale.py +++ b/addons/point_of_sale/point_of_sale.py @@ -60,6 +60,8 @@ class pos_order(osv.osv): _order = "date_order, create_date desc" def unlink(self, cr, uid, ids, context=None): + if not context: + context = {} for rec in self.browse(cr, uid, ids, context=context): for rec_statement in rec.statement_ids: if (rec_statement.statement_id and rec_statement.statement_id.state == 'confirm') or rec.state == 'done': @@ -70,6 +72,8 @@ class pos_order(osv.osv): """ Changed price list on_change of partner_id""" if not part: return {'value': {}} + if not context: + context = {} pricelist = self.pool.get('res.partner').browse(cr, uid, part, context=context).property_product_pricelist.id return {'value': {'pricelist_id': pricelist}} @@ -77,6 +81,8 @@ class pos_order(osv.osv): """ Calculates amount_tax of order line @param field_names: Names of fields. @return: Dictionary of values """ + if not context: + context = {} cr.execute(""" SELECT p.id, @@ -122,14 +128,12 @@ class pos_order(osv.osv): res[order.id] = val return res - def _get_date_payment(self, cr, uid, ids, context=None, *a): + def _get_date_payment(self, cr, uid, ids, context, *a): """ Find Validation Date @return: Dictionary of values """ res = {} val = None - if not context: - context = {} - for order in self.browse(cr, uid, ids, context=context): + for order in self.browse(cr, uid, ids): cr.execute("SELECT date_validation FROM pos_order WHERE id = %s", (order.id,)) date_p = cr.fetchone() date_p = date_p and date_p[0] or None @@ -199,6 +203,8 @@ class pos_order(osv.osv): def copy(self, cr, uid, id, default=None, context=None): if not default: default = {} + if not context: + context = {} default.update({ 'state': 'draft', 'partner_id': False, @@ -293,6 +299,8 @@ class pos_order(osv.osv): @param name: Names of fields. @return: pricelist ID """ + if not context: + context = {} pricelist = self.pool.get('product.pricelist').search(cr, uid, [('name', '=', 'Public Pricelist')], context=context) return pricelist and pricelist[0] or False @@ -341,6 +349,8 @@ class pos_order(osv.osv): """ Test all amount is paid for this order @return: True """ + if not context: + context = {} for order in self.browse(cr, uid, ids, context=context): if order.lines and not order.amount_total: return True @@ -506,6 +516,8 @@ class pos_order(osv.osv): """ Changes order state to cancel @return: True """ + if not context: + context = {} self.write(cr, uid, ids, {'state': 'cancel'}, context=context) self.cancel_picking(cr, uid, ids, context=context) return True @@ -516,6 +528,8 @@ class pos_order(osv.osv): statement_line_obj = self.pool.get('account.bank.statement.line') prod_obj = self.pool.get('product.product') property_obj = self.pool.get('ir.property') + if not context: + context = {} curr_c = self.pool.get('res.users').browse(cr, uid, uid, context=context).company_id curr_company = curr_c.id order = self.browse(cr, uid, order_id, context=context) @@ -678,6 +692,8 @@ class pos_order(osv.osv): account_tax_obj = self.pool.get('account.tax') res_obj=self.pool.get('res.users') property_obj=self.pool.get('ir.property') + if not context: + context = {} period = account_period_obj.find(cr, uid, context=context)[0] for order in self.browse(cr, uid, ids, context=context): @@ -904,10 +920,14 @@ class pos_order(osv.osv): return True def action_cancel(self, cr, uid, ids, context=None): + if not context: + context = {} self.write(cr, uid, ids, {'state': 'cancel'}, context=context) return True def action_done(self, cr, uid, ids, context=None): + if not context: + context = {} for order in self.browse(cr, uid, ids, context=context): if not order.journal_entry: self.create_account_move(cr, uid, ids, context=None) @@ -936,11 +956,9 @@ account_bank_statement() class account_bank_statement_line(osv.osv): _inherit = 'account.bank.statement.line' - def _get_statement_journal(self, cr, uid, ids, context=None, *a): + def _get_statement_journal(self, cr, uid, ids, context, *a): res = {} - if not context: - context = {} - for line in self.browse(cr, uid, ids, context=context): + for line in self.browse(cr, uid, ids): res[line.id] = line.statement_id and line.statement_id.journal_id and line.statement_id.journal_id.name or None return res _columns= { @@ -1137,6 +1155,8 @@ class pos_order_line(osv.osv): return {'value': {'notice': 'No Discount', 'price_ded': price * discount * 0.01 or 0.0}} def onchange_qty(self, cr, uid, ids, discount, qty, price, context=None): + if not context: + context = {} subtotal = qty * price if discount: subtotal = subtotal - (subtotal * discount / 100) diff --git a/addons/point_of_sale/wizard/all_closed_cashbox_of_the_day.py b/addons/point_of_sale/wizard/all_closed_cashbox_of_the_day.py index c7f2240d81f..139eea8301a 100644 --- a/addons/point_of_sale/wizard/all_closed_cashbox_of_the_day.py +++ b/addons/point_of_sale/wizard/all_closed_cashbox_of_the_day.py @@ -34,6 +34,8 @@ class all_closed_cashbox_of_the_day(osv.osv_memory): @param context: A standard dictionary @return : retrun report """ + if not context: + context = {} datas = {'ids': context.get('active_ids', [])} return { 'type': 'ir.actions.report.xml', diff --git a/addons/point_of_sale/wizard/pos_box_entries.py b/addons/point_of_sale/wizard/pos_box_entries.py index dcc9779a903..d6441c467f9 100644 --- a/addons/point_of_sale/wizard/pos_box_entries.py +++ b/addons/point_of_sale/wizard/pos_box_entries.py @@ -61,6 +61,8 @@ class pos_box_entries(osv.osv_memory): @param context: A standard dictionary @return :Return of operation of product """ + if not context: + context = {} product_obj = self.pool.get('product.product') ids = product_obj.search(cr, uid, [('income_pdt', '=', True)], context=context) res = product_obj.read(cr, uid, ids, ['id', 'name'], context=context) @@ -95,6 +97,8 @@ class pos_box_entries(osv.osv_memory): res_obj = self.pool.get('res.users') product_obj = self.pool.get('product.product') bank_statement = self.pool.get('account.bank.statement.line') + if not context: + context = {} for data in self.read(cr, uid, ids, context=context): vals = {} curr_company = res_obj.browse(cr, uid, uid, context=context).company_id.id diff --git a/addons/point_of_sale/wizard/pos_box_out.py b/addons/point_of_sale/wizard/pos_box_out.py index a05c5f579e1..80e0ae7fe61 100644 --- a/addons/point_of_sale/wizard/pos_box_out.py +++ b/addons/point_of_sale/wizard/pos_box_out.py @@ -41,6 +41,8 @@ class pos_box_out(osv.osv_memory): @return :Return of operation of product """ product_obj = self.pool.get('product.product') + if not context: + context = {} company_id = self.pool.get('res.users').browse(cr, uid, uid, context=context).company_id.id ids = product_obj.search(cr, uid, ['&', ('expense_pdt', '=', True), '|', ('company_id', '=', company_id), ('company_id', '=', None)], context=context) res = product_obj.read(cr, uid, ids, ['id', 'name'], context=context) @@ -75,6 +77,8 @@ class pos_box_out(osv.osv_memory): product_obj = self.pool.get('product.template') productp_obj = self.pool.get('product.product') res_obj = self.pool.get('res.users') + if not context: + context = {} for data in self.read(cr, uid, ids, context=context): curr_company = res_obj.browse(cr, uid, uid, context=context).company_id.id statement_id = statement_obj.search(cr, uid, [('journal_id', '=', data['journal_id']), ('company_id', '=', curr_company), ('user_id', '=', uid), ('state', '=', 'open')], context=context) diff --git a/addons/point_of_sale/wizard/pos_close_statement.py b/addons/point_of_sale/wizard/pos_close_statement.py index 222338f6e10..03f26b7f0d8 100644 --- a/addons/point_of_sale/wizard/pos_close_statement.py +++ b/addons/point_of_sale/wizard/pos_close_statement.py @@ -43,6 +43,8 @@ class pos_close_statement(osv.osv_memory): j_ids = map(lambda x1: x1[0], cr.fetchall()) journal_ids = journal_obj.search(cr, uid, [('auto_cash', '=', True), ('type', '=', 'cash'), ('id', 'in', j_ids)], context=context) ids = statement_obj.search(cr, uid, [('state', '!=', 'confirm'), ('user_id', '=', uid), ('journal_id', 'in', journal_ids)], context=context) + if not context: + context = {} for journal in journal_obj.browse(cr, uid, journal_ids, context=context): if not ids: raise osv.except_osv(_('Message'), _('Cash registers are already closed.')) diff --git a/addons/point_of_sale/wizard/pos_discount.py b/addons/point_of_sale/wizard/pos_discount.py index f0ea93e3d7a..149f6260ad3 100644 --- a/addons/point_of_sale/wizard/pos_discount.py +++ b/addons/point_of_sale/wizard/pos_discount.py @@ -43,6 +43,8 @@ class pos_discount(osv.osv_memory): @param context: A standard dictionary @return: New arch of view with new columns. """ + if not context: + context = {} super(pos_discount, self).view_init(cr, uid, fields_list, context=context) record_id = context and context.get('active_id', False) or False order = self.pool.get('pos.order').browse(cr, uid, record_id, context=context) diff --git a/addons/point_of_sale/wizard/pos_open_statement.py b/addons/point_of_sale/wizard/pos_open_statement.py index 954f95070c8..0b10f0cab42 100644 --- a/addons/point_of_sale/wizard/pos_open_statement.py +++ b/addons/point_of_sale/wizard/pos_open_statement.py @@ -40,6 +40,8 @@ class pos_open_statement(osv.osv_memory): statement_obj = self.pool.get('account.bank.statement') sequence_obj = self.pool.get('ir.sequence') journal_obj = self.pool.get('account.journal') + if not context: + context = {} company_id = self.pool.get('res.users').browse(cr, uid, uid, context=context).company_id.id cr.execute("SELECT DISTINCT journal_id FROM pos_journal_users " "WHERE user_id = %s ORDER BY journal_id"% (uid, )) diff --git a/addons/point_of_sale/wizard/pos_payment.py b/addons/point_of_sale/wizard/pos_payment.py index c7ae5d6eb27..b8043d658ef 100644 --- a/addons/point_of_sale/wizard/pos_payment.py +++ b/addons/point_of_sale/wizard/pos_payment.py @@ -81,6 +81,8 @@ class pos_make_payment(osv.osv_memory): return res def view_init(self, cr, uid, fields_list, context=None): + if not context: + context = {} super(pos_make_payment, self).view_init(cr, uid, fields_list, context=context) active_id = context and context.get('active_id', False) or False if active_id: @@ -140,13 +142,15 @@ class pos_make_payment(osv.osv_memory): if the order is paid print invoice (if wanted) or ticket. """ order_obj = self.pool.get('pos.order') + if not context: + context = {} active_id = context and context.get('active_id', False) order = order_obj.browse(cr, uid, active_id, context=context) amount = order.amount_total - order.amount_paid data = self.read(cr, uid, ids, context=context)[0] # Todo need to check ... if data['is_acc']: - line_id, price = order_obj.add_product(cr, uid, order.id, data['product_id'], -1.0, context) + line_id, price = order_obj.add_product(cr, uid, order.id, data['product_id'], -1.0, context=context) amount = order.amount_total - order.amount_paid - price if amount != 0.0: @@ -155,28 +159,28 @@ class pos_make_payment(osv.osv_memory): if order_obj.test_paid(cr, uid, [active_id]): if data['partner_id'] and data['invoice_wanted']: - order_obj.action_invoice(cr, uid, [active_id], context) - order_obj.create_picking(cr, uid, [active_id], context) + order_obj.action_invoice(cr, uid, [active_id], context=context) + order_obj.create_picking(cr, uid, [active_id], context=context) if context.get('return', False): order_obj.write(cr, uid, [active_id], {'state':'done'}, context=context) else: order_obj.write(cr, uid, [active_id],{'state':'paid'}, context=context) - return self.create_invoice(cr, uid, ids, context) + return self.create_invoice(cr, uid, ids, context=context) else: context.update({'flag': True}) - order_obj.action_paid(cr, uid, [active_id], context) + order_obj.action_paid(cr, uid, [active_id], context=context) if context.get('return', False): order_obj.write(cr, uid, [active_id], {'state':'done'}, context=context) else: order_obj.write(cr, uid, [active_id], {'state':'paid'}, context=context) - return self.print_report(cr, uid, ids, context) + return self.print_report(cr, uid, ids, context=context) if order.amount_paid > 0.0: context.update({'flag': True}) # Todo need to check - order_obj.action_paid(cr, uid, [active_id], context) + order_obj.action_paid(cr, uid, [active_id], context=context) order_obj.write(cr, uid, [active_id], {'state': 'advance'}, context=context) - return self.print_report(cr, uid, ids, context) + return self.print_report(cr, uid, ids, context=context) return {} @@ -184,6 +188,8 @@ class pos_make_payment(osv.osv_memory): """ Create a invoice """ + if not context: + context = {} active_ids = [context and context.get('active_id', False)] datas = {'ids': active_ids} return { diff --git a/addons/point_of_sale/wizard/pos_receipt.py b/addons/point_of_sale/wizard/pos_receipt.py index 39e38bdfa3c..19ce8c2c359 100644 --- a/addons/point_of_sale/wizard/pos_receipt.py +++ b/addons/point_of_sale/wizard/pos_receipt.py @@ -35,6 +35,8 @@ class pos_receipt(osv.osv_memory): @param context: A standard dictionary @return: New arch of view with new columns. """ + if not context: + context = {} order_lst = self. pool.get('pos.order').browse(cr, uid, context['active_id'], context=context) def print_report(self, cr, uid, ids, context=None): @@ -46,6 +48,8 @@ class pos_receipt(osv.osv_memory): @param context: A standard dictionary @return : retrun report """ + if not context: + context = {} datas = {'ids': context.get('active_ids', [])} return { 'type': 'ir.actions.report.xml', diff --git a/addons/point_of_sale/wizard/pos_return.py b/addons/point_of_sale/wizard/pos_return.py index c0e37d84ef3..9d85d3c84fe 100644 --- a/addons/point_of_sale/wizard/pos_return.py +++ b/addons/point_of_sale/wizard/pos_return.py @@ -47,7 +47,7 @@ class pos_return(osv.osv_memory): active_ids = context.get('active_ids') if not context: context={} - for order in order_obj.browse(cr, uid, active_ids): + for order in order_obj.browse(cr, uid, active_ids, context=context): for line in order.lines: if 'return%s'%(line.id) in fields: res['return%s'%(line.id)] = line.qty @@ -68,7 +68,7 @@ class pos_return(osv.osv_memory): context={} active_ids=context.get('active_ids') - for order in order_obj.browse(cr, uid, active_ids): + for order in order_obj.browse(cr, uid, active_ids, context=context): for line in order.lines: if 'return%s'%(line.id) not in self._columns: self._columns['return%s'%(line.id)] = fields.float("Quantity") @@ -101,7 +101,7 @@ class pos_return(osv.osv_memory):