From 62d3ab394166e42365b5a2b6b08d712da2a29114 Mon Sep 17 00:00:00 2001 From: niv Date: Fri, 15 Oct 2010 17:46:03 +0200 Subject: [PATCH 001/114] [ADD] Added possibility to set a default account tax to new products. bzr revid: niv@niv-laptop-20101015154603-mm39x167qtoitgxd --- addons/account/account.py | 25 +++++++++++++++++++++++++ addons/account/account_view.xml | 5 ++++- 2 files changed, 29 insertions(+), 1 deletion(-) diff --git a/addons/account/account.py b/addons/account/account.py index 5de913c45a7..0afd89a48a3 100644 --- a/addons/account/account.py +++ b/addons/account/account.py @@ -2537,7 +2537,22 @@ class wizard_multi_charts_accounts(osv.osv_memory): 'bank_accounts_id': fields.one2many('account.bank.accounts.wizard', 'bank_account_id', 'Bank Accounts', required=True), 'code_digits':fields.integer('# of Digits', required=True, help="No. of Digits to use for account code"), 'seq_journal':fields.boolean('Separated Journal Sequences', help="Check this box if you want to use a different sequence for each created journal. Otherwise, all will use the same sequence."), + "sale_tax": fields.many2one("account.tax.template", "Sale Tax"), + "purchase_tax": fields.many2one("account.tax.template", "Purchase Tax"), } + def onchange_chart_template_id(self, cr, uid, ids, chart_template_id=False, context=None): + res = {} + res['value'] = {} + res['value']["sale_tax"] = False + res['value']["purchase_tax"] = False + if chart_template_id: + ids = self.pool.get('account.tax.template').search(cr, uid, [("chart_template_id" + , "=", chart_template_id)], order="sequence") + if len(ids) > 0: + id=ids[0] + res['value']["sale_tax"] = id + res['value']["purchase_tax"] = id + return res def _get_chart(self, cr, uid, context={}): ids = self.pool.get('account.chart.template').search(cr, uid, [], context=context) @@ -2553,6 +2568,7 @@ class wizard_multi_charts_accounts(osv.osv_memory): def execute(self, cr, uid, ids, context=None): obj_multi = self.browse(cr, uid, ids[0]) + obj_acc = self.pool.get('account.account') obj_acc_tax = self.pool.get('account.tax') obj_journal = self.pool.get('account.journal') @@ -2591,6 +2607,7 @@ class wizard_multi_charts_accounts(osv.osv_memory): tax_code_template_ref[tax_code_template.id] = new_tax_code #create all the tax + tax_template_to_tax = {} for tax in obj_multi.chart_template_id.tax_template_ids: #create it vals_tax = { @@ -2619,8 +2636,10 @@ class wizard_multi_charts_accounts(osv.osv_memory): 'type_tax_use': tax.type_tax_use } new_tax = obj_acc_tax.create(cr, uid, vals_tax) + tax_template_to_tax[tax.id] = new_tax #as the accounts have not been created yet, we have to wait before filling these fields todo_dict[new_tax] = { + 'account_collected_id': tax.account_collected_id and tax.account_collected_id.id or False, 'account_paid_id': tax.account_paid_id and tax.account_paid_id.id or False, } @@ -2842,6 +2861,12 @@ class wizard_multi_charts_accounts(osv.osv_memory): 'position_id': new_fp, } obj_ac_fp.create(cr, uid, vals_acc) + + ir_values = self.pool.get('ir.values') + ir_values.set(cr, uid, key='default', key2=False, name="taxes_id", company=obj_multi.company_id.id + , models =[('product.product',False)], value=[tax_template_to_tax[obj_multi.sale_tax.id]]) + ir_values.set(cr, uid, key='default', key2=False, name="supplier_taxes_id", company=obj_multi.company_id.id + , models =[('product.product',False)], value=[tax_template_to_tax[obj_multi.purchase_tax.id]]) wizard_multi_charts_accounts() diff --git a/addons/account/account_view.xml b/addons/account/account_view.xml index 49de367122b..ce0985ab7ec 100644 --- a/addons/account/account_view.xml +++ b/addons/account/account_view.xml @@ -2412,8 +2412,11 @@ - + + +
From e5fb3a0780c8ce5b0fbd9373c167d782c0914db5 Mon Sep 17 00:00:00 2001 From: niv Date: Mon, 18 Oct 2010 11:37:58 +0200 Subject: [PATCH 002/114] [FIX] Fixed incorrect merge bzr revid: niv@niv-laptop-20101018093758-j99jast7cjx934s2 --- addons/account/account_view.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/account/account_view.xml b/addons/account/account_view.xml index 2a7af0b9e5b..ce2053e69a3 100644 --- a/addons/account/account_view.xml +++ b/addons/account/account_view.xml @@ -2415,7 +2415,7 @@ - + From 9e7d695c3f403dacf09d6f105ffeae69552d1f5e Mon Sep 17 00:00:00 2001 From: Mustufa Rangwala Date: Mon, 18 Oct 2010 16:16:39 +0530 Subject: [PATCH 003/114] [FIX] bzr revid: mra@mra-laptop-20101018104639-r9jhqu31ar79nbih --- addons/account/account_move_line.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/account/account_move_line.py b/addons/account/account_move_line.py index 6bed1f0f396..7e5f9742746 100644 --- a/addons/account/account_move_line.py +++ b/addons/account/account_move_line.py @@ -177,7 +177,7 @@ class account_move_line(osv.osv): for item in i[2]: data[item] = i[2][item] if context['journal']: - journal_data = obj_journal.browse(cr, uid, context['journal']) + journal_data = journal_obj.browse(cr, uid, context['journal']) if journal_data.type == 'purchase': if total_new > 0: account = journal_data.default_credit_account_id From ac690626e00fb0a2d53e09647dc54383fed5fdc2 Mon Sep 17 00:00:00 2001 From: Mustufa Rangwala Date: Mon, 18 Oct 2010 16:33:56 +0530 Subject: [PATCH 004/114] [FIX] bzr revid: mra@mra-laptop-20101018110356-8t6yqcdi84zbr8lh --- addons/account/account_move_line.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/addons/account/account_move_line.py b/addons/account/account_move_line.py index 7e5f9742746..41d413fbbf5 100644 --- a/addons/account/account_move_line.py +++ b/addons/account/account_move_line.py @@ -293,6 +293,8 @@ class account_move_line(osv.osv): return data def on_create_write(self, cr, uid, id, context={}): + if not id: + return [] ml = self.browse(cr, uid, id, context) return map(lambda x: x.id, ml.move_id.line_id) From 3e828804235ee9a7865990d9dab12910a2a61185 Mon Sep 17 00:00:00 2001 From: "ARA (OpenERP)" Date: Mon, 18 Oct 2010 18:13:04 +0530 Subject: [PATCH 005/114] [FIX] account coda: if there is not coda statement raise error bzr revid: ara@tinyerp.com-20101018124304-3jl0c1uy39b146xl --- addons/account_coda/account_coda.py | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/addons/account_coda/account_coda.py b/addons/account_coda/account_coda.py index 8dd4067c571..caafe9f30da 100644 --- a/addons/account_coda/account_coda.py +++ b/addons/account_coda/account_coda.py @@ -22,6 +22,7 @@ import time from osv import osv,fields +from tools.translate import _ class account_coda(osv.osv): _name = "account.coda" @@ -41,6 +42,13 @@ class account_coda(osv.osv): 'company_id': lambda s,cr,uid,c: s.pool.get('res.company')._company_default_get(cr, uid, 'account.coda', context=c), } + def search(self, cr, user, args, offset=0, limit=None, order=None, context=None, count=False): + res = super(account_coda, self).search(cr, user, args=args, offset=offset, limit=limit, order=order, + context=context, count=count) + if not res: + raise osv.except_osv('Error', _('Coda file not found for bank statement !!')) + return res + account_coda() class account_bank_statement(osv.osv): From b7eb35f51588db1a1c410a5727bf5ef2c29a0d4a Mon Sep 17 00:00:00 2001 From: "ARA (OpenERP)" Date: Tue, 19 Oct 2010 11:01:59 +0530 Subject: [PATCH 006/114] [FIX] account coda: if there is not coda statement raise error bzr revid: ara@tinyerp.com-20101019053159-kl96unqmg03l6trv --- addons/account_coda/account_coda.py | 2 +- addons/account_coda/account_coda_view.xml | 13 +++++++------ 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/addons/account_coda/account_coda.py b/addons/account_coda/account_coda.py index caafe9f30da..24d928a2fd0 100644 --- a/addons/account_coda/account_coda.py +++ b/addons/account_coda/account_coda.py @@ -45,7 +45,7 @@ class account_coda(osv.osv): def search(self, cr, user, args, offset=0, limit=None, order=None, context=None, count=False): res = super(account_coda, self).search(cr, user, args=args, offset=offset, limit=limit, order=order, context=context, count=count) - if not res: + if context.get('bank_statement', False) and not res: raise osv.except_osv('Error', _('Coda file not found for bank statement !!')) return res diff --git a/addons/account_coda/account_coda_view.xml b/addons/account_coda/account_coda_view.xml index 7c7a10a05a3..a16e68d5ee0 100644 --- a/addons/account_coda/account_coda_view.xml +++ b/addons/account_coda/account_coda_view.xml @@ -80,12 +80,13 @@ id="menu_account_coda_import" sequence="20"/> + domain="[('statement_ids', 'in', [active_id])]" + context="{'bank_statement': 1}" + res_model="account.coda" + src_model="account.bank.statement" + view_type="form" + view_mode="tree,form" + id="act_account_payment_account_bank_statement"/> From 69549b1571f4b6e03e6e91268bf36f7d091efbe0 Mon Sep 17 00:00:00 2001 From: vth Date: Tue, 19 Oct 2010 11:18:51 +0530 Subject: [PATCH 007/114] [FIX] account : accout/configration/miscellanious/recurring models add default value to the company_id bzr revid: vth@tinyerp.com-20101019054851-cqupju2q8vgeil9n --- addons/account/account.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/addons/account/account.py b/addons/account/account.py index ecadfaf37f9..3eedfc7ce65 100644 --- a/addons/account/account.py +++ b/addons/account/account.py @@ -458,7 +458,7 @@ class account_account(osv.osv): ids = self.search(cr, user, [('name', operator, name)]+ args, limit=limit) if not ids and len(name.split()) >= 2: #Separating code and name of account for searching - operand1,operand2 = name.split(' ',1) #name can contain spaces e.g. OpenERP S.A. + operand1,operand2 = name.split(' ',1) #name can contain spaces e.g. OpenERP S.A. ids = self.search(cr, user, [('code', operator, operand1), ('name', operator, operand2)]+ args, limit=limit) else: ids = self.search(cr, user, args, context=context, limit=limit) @@ -2025,6 +2025,7 @@ class account_model(osv.osv): _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'), + 'company_id': lambda self,cr,uid,c: self.pool.get('res.company')._company_default_get(cr, uid, 'account.model', context=c), } def generate(self, cr, uid, ids, datas={}, context=None): move_ids = [] From 3ce4c36d04e3ab5e01041a14691e4ea282279db6 Mon Sep 17 00:00:00 2001 From: mtr Date: Tue, 19 Oct 2010 14:37:55 +0530 Subject: [PATCH 008/114] [IMP] l10n_lu:-improvement in yml bzr revid: mtr@mtr-20101019090755-wwsmqfxojpurmlc6 --- addons/l10n_lu/test/l10n_lu_report.yml | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/addons/l10n_lu/test/l10n_lu_report.yml b/addons/l10n_lu/test/l10n_lu_report.yml index 43b1ce35df1..b6b273c5a76 100644 --- a/addons/l10n_lu/test/l10n_lu_report.yml +++ b/addons/l10n_lu/test/l10n_lu_report.yml @@ -9,6 +9,10 @@ - !python {model: res.partner}: | import netsvc, tools, os - (data, format) = netsvc.LocalService('report.l10n_lu.tax.report.print').create(cr, uid, [], {'model':'ir.ui.menu', 'form':{'tax_code_id': ref('account_tax_code_id_r1'), 'period_id': ref('account.period_1')}}, {}) - if tools.config['test_report_directory']: - file(os.path.join(tools.config['test_report_directory'], 'l10n_lu-vat_report.'+format), 'wb+').write(data) + try: + (data, format) = netsvc.LocalService('report.l10n_lu.tax.report.print').create(cr, uid, [], {'model':'ir.ui.menu', 'form':{'tax_code_id': ref('account_tax_code_id_r1'), 'period_id': ref('account.period_1')}}, {}) + if tools.config['test_report_directory']: + file(os.path.join(tools.config['test_report_directory'], 'l10n_lu-vat_report.'+format), 'wb+').write(data) + except: + pass + From c232ca85b3812004dd1fe5fa73f6fcd08d335f96 Mon Sep 17 00:00:00 2001 From: vth Date: Tue, 19 Oct 2010 14:42:45 +0530 Subject: [PATCH 009/114] [FIX] account: remove warning from account/wizard/account_chart.py bzr revid: vth@tinyerp.com-20101019091245-tepif0h5z8lwwjkq --- addons/account/wizard/account_chart.py | 1 - 1 file changed, 1 deletion(-) diff --git a/addons/account/wizard/account_chart.py b/addons/account/wizard/account_chart.py index b87c11f02f3..fdf684b2522 100644 --- a/addons/account/wizard/account_chart.py +++ b/addons/account/wizard/account_chart.py @@ -19,7 +19,6 @@ # ############################################################################## -import time from osv import fields, osv class account_chart(osv.osv_memory): From 378fb690e3c4fb0a0a7f225b1b11e9c39637a981 Mon Sep 17 00:00:00 2001 From: ssi Date: Tue, 19 Oct 2010 14:43:05 +0530 Subject: [PATCH 010/114] [FIX]account: Open current refunded invoice after invoice refund. bzr revid: ssi@tinyerp.com-20101019091305-lh1uaiykp0bkxc9s --- addons/account/wizard/account_invoice_refund.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/addons/account/wizard/account_invoice_refund.py b/addons/account/wizard/account_invoice_refund.py index 0c866853594..b5059447ebc 100644 --- a/addons/account/wizard/account_invoice_refund.py +++ b/addons/account/wizard/account_invoice_refund.py @@ -210,9 +210,11 @@ class account_invoice_refund(osv.osv_memory): result = mod_obj._get_id(cr, uid, 'account', xml_id) id = mod_obj.read(cr, uid, result, ['res_id'], context=context)['res_id'] result = act_obj.read(cr, uid, id, context=context) - result['res_id'] = created_inv + invoice_domain = eval(result['domain']) + invoice_domain.append(('id', '=', created_inv)) + result['domain'] = invoice_domain return result - + def invoice_refund(self, cr, uid, ids, context=None): data_refund = self.read(cr, uid, ids, [],context=context)[0]['filter_refund'] return self.compute_refund(cr, uid, ids, data_refund, context=context) From 74dc9a43786b367b0c8508e2cf1c46ad50d31e4c Mon Sep 17 00:00:00 2001 From: ssi Date: Tue, 19 Oct 2010 15:08:21 +0530 Subject: [PATCH 011/114] [FIX]hr_evaluation: removed unsued variables bzr revid: ssi@tinyerp.com-20101019093821-ax5rhud1f8cna492 --- addons/hr_evaluation/hr_evaluation.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/addons/hr_evaluation/hr_evaluation.py b/addons/hr_evaluation/hr_evaluation.py index 3c5a150691e..cca922304aa 100644 --- a/addons/hr_evaluation/hr_evaluation.py +++ b/addons/hr_evaluation/hr_evaluation.py @@ -200,14 +200,12 @@ class hr_evaluation(osv.osv): for employee in employee_obj.browse(cr, uid, [employee_id], context=context): if employee and employee.evaluation_plan_id and employee.evaluation_plan_id.id: evaluation_plan_id=employee.evaluation_plan_id.id - employee_ids=employee_obj.search(cr, uid, [('parent_id','=',employee.id)], context=context) return {'value': {'plan_id':evaluation_plan_id}} def button_plan_in_progress(self, cr, uid, ids, context=None): hr_eval_inter_obj = self.pool.get('hr.evaluation.interview') if context is None: context = {} - apprai_id = [] for evaluation in self.browse(cr, uid, ids, context=context): wait = False for phase in evaluation.plan_id.phase_ids: From dafd8d70f9b6949925a04f9100683fbce1114822 Mon Sep 17 00:00:00 2001 From: "ARA (OpenERP)" Date: Tue, 19 Oct 2010 15:53:06 +0530 Subject: [PATCH 012/114] [IMP] Event: Remove the repeated user_id and change the position of state in tree view bzr revid: ara@tinyerp.com-20101019102306-0x89ew6rjzx7tana --- addons/event/event_view.xml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/addons/event/event_view.xml b/addons/event/event_view.xml index 918147b79cb..f900cb7b3a6 100644 --- a/addons/event/event_view.xml +++ b/addons/event/event_view.xml @@ -171,7 +171,6 @@ - @@ -180,8 +179,8 @@ - +