Merge lp:~akretion-team/openobject-addons/trunk-partner-bank-currency-related into lp:openobject-addons

Proposed by Alexis de Lattre
Status: Merged
Merged at revision: 6463
Proposed branch: lp:~akretion-team/openobject-addons/trunk-partner-bank-currency-related
Merge into: lp:openobject-addons
Diff against target: 44 lines (+17/-0)
2 files modified
account/account_bank.py (+3/-0)
account/account_bank_view.xml (+14/-0)
To merge this branch: bzr merge lp:~akretion-team/openobject-addons/trunk-partner-bank-currency-related
Reviewer Review Type Date Requested Status
Raphael Collet (OpenERP) (community) Approve
Review via email: mp+91038@code.launchpad.net

Description of the change

This merge proposal adds a currency_id related field on res.partner.bank. It displays the field on the form and tree view of res.partner.bank. With this addition, we are now able to easily identify the currency of the bank account when we select it on a customer invoice for example. For a company who has several bank accounts in different currencies in the same bank, it's much easier to differenciate the bank accounts by their currency than by their number.

This proposal implements the suggestion of Raphaël Collet on this (refused) merge proposal :
https://code.launchpad.net/~alexis-via/openobject-server/server-trunk-partner-bank-currency/+merge/90713

To post a comment you must log in.
Revision history for this message
Raphael Collet (OpenERP) (rco-openerp) wrote :

Excellent!

Thanks,
Raphael

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account/account_bank.py'
--- account/account_bank.py 2011-12-30 10:20:07 +0000
+++ account/account_bank.py 2012-02-01 10:15:31 +0000
@@ -26,6 +26,9 @@
26 _inherit = "res.partner.bank"26 _inherit = "res.partner.bank"
27 _columns = {27 _columns = {
28 'journal_id': fields.many2one('account.journal', 'Account Journal', help="This journal will be created automatically for this bank account when you save the record"),28 'journal_id': fields.many2one('account.journal', 'Account Journal', help="This journal will be created automatically for this bank account when you save the record"),
29 'currency_id': fields.related('journal_id', 'currency', type="many2one",
30 relation='res.currency', string="Currency", readonly=True,
31 help="Currency of the related account journal."),
29 }32 }
30 def create(self, cr, uid, data, context={}):33 def create(self, cr, uid, data, context={}):
31 result = super(bank, self).create(cr, uid, data, context=context)34 result = super(bank, self).create(cr, uid, data, context=context)
3235
=== modified file 'account/account_bank_view.xml'
--- account/account_bank_view.xml 2011-10-16 01:28:00 +0000
+++ account/account_bank_view.xml 2012-02-01 10:15:31 +0000
@@ -16,12 +16,26 @@
16 <group name="accounting" col="2" colspan="2" attrs="{'invisible': [('company_id','=', False)]}" groups="base.group_extended">16 <group name="accounting" col="2" colspan="2" attrs="{'invisible': [('company_id','=', False)]}" groups="base.group_extended">
17 <separator string="Accounting Information" colspan="2"/>17 <separator string="Accounting Information" colspan="2"/>
18 <field name="journal_id"/>18 <field name="journal_id"/>
19 <field name="currency_id"/>
19 </group>20 </group>
20 </group>21 </group>
21 </field>22 </field>
22 </record>23 </record>
2324
2425
26 <record id="view_partner_bank_tree_add_currency" model="ir.ui.view">
27 <field name="name">Partner Bank Accounts - Add currency on tree</field>
28 <field name="model">res.partner.bank</field>
29 <field name="type">tree</field>
30 <field name="inherit_id" ref="base.view_partner_bank_tree"/>
31 <field name="arch" type="xml">
32 <field name="acc_number" position="after">
33 <field name="currency_id"/>
34 </field>
35 </field>
36 </record>
37
38
25 <record id="action_bank_tree" model="ir.actions.act_window">39 <record id="action_bank_tree" model="ir.actions.act_window">
26 <field name="name">Setup your Bank Accounts</field>40 <field name="name">Setup your Bank Accounts</field>
27 <field name="res_model">res.partner.bank</field>41 <field name="res_model">res.partner.bank</field>

Subscribers

People subscribed via source and target branches

to all changes: