Merge lp:~banking-addons-team/banking-addons/ba70-mig_account_iban_preserve_domestic into lp:banking-addons

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 163
Proposed branch: lp:~banking-addons-team/banking-addons/ba70-mig_account_iban_preserve_domestic
Merge into: lp:banking-addons
Diff against target: 144 lines (+44/-44)
3 files modified
account_iban_preserve_domestic/__openerp__.py (+19/-27)
account_iban_preserve_domestic/res_partner_bank.py (+25/-2)
account_iban_preserve_domestic/res_partner_bank_view.xml (+0/-15)
To merge this branch: bzr merge lp:~banking-addons-team/banking-addons/ba70-mig_account_iban_preserve_domestic
Reviewer Review Type Date Requested Status
Guewen Baconnier @ Camptocamp code review, no test Approve
Review via email: mp+150668@code.launchpad.net

Commit message

[MIG] account_iban_preserve_domestic 7.0

Description of the change

The new series lp:banking-addons/banking-addons-70 is a copy of lp:banking-addons/6.1, but set to uninstallable. This simple module is the first to be migrated to OpenERP 7.0.

To post a comment you must log in.
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

LGTM

review: Approve (code review, no test)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account_iban_preserve_domestic/__openerp__.py'
--- account_iban_preserve_domestic/__openerp__.py 2013-02-26 21:06:36 +0000
+++ account_iban_preserve_domestic/__openerp__.py 2013-02-26 21:20:32 +0000
@@ -1,61 +1,53 @@
1##############################################################################1##############################################################################
2#2#
3# Copyright (C) 2012 Therp BV (<http://therp.nl>).3# Copyright (C) 2012 - 2013 Therp BV (<http://therp.nl>).
4# 4#
5# All other contributions are (C) by their respective contributors5# All other contributions are (C) by their respective contributors
6#6#
7# All Rights Reserved7# All Rights Reserved
8#8#
9# WARNING: This program as such is intended to be used by professional
10# programmers who take the whole responsability of assessing all potential
11# consequences resulting from its eventual inadequacies and bugs
12# End users who are looking for a ready-to-use solution with commercial
13# garantees and support are strongly adviced to contract EduSense BV
14#
15# This program is free software: you can redistribute it and/or modify9# This program is free software: you can redistribute it and/or modify
16# it under the terms of the GNU General Public License as published by10# it under the terms of the GNU Affero General Public License as
17# the Free Software Foundation, either version 3 of the License, or11# published by the Free Software Foundation, either version 3 of the
18# (at your option) any later version.12# License, or (at your option) any later version.
19#13#
20# This program is distributed in the hope that it will be useful,14# This program is distributed in the hope that it will be useful,
21# but WITHOUT ANY WARRANTY; without even the implied warranty of15# but WITHOUT ANY WARRANTY; without even the implied warranty of
22# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the16# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
23# GNU General Public License for more details.17# GNU Affero General Public License for more details.
24#18#
25# You should have received a copy of the GNU General Public License19# You should have received a copy of the GNU Affero General Public License
26# along with this program. If not, see <http://www.gnu.org/licenses/>.20# along with this program. If not, see <http://www.gnu.org/licenses/>.
27#21#
28##############################################################################22##############################################################################
23
29{24{
30 'name': 'Domestic bank account number',25 'name': 'Domestic bank account number',
31 'version': '0.1.134',26 'version': '0.1.163',
32 'license': 'AGPL-3',27 'license': 'AGPL-3',
33 'author': 'Therp BV',28 'author': 'Therp BV',
34 'website': 'https://launchpad.net/banking-addons',29 'website': 'https://launchpad.net/banking-addons',
35 'category': 'Banking addons',30 'category': 'Banking addons',
36 'depends': ['base_iban','account'],31 'depends': [
37 'init_xml': [],32 'base_iban',
38 'update_xml': [33 'account',
34 ],
35 'data': [
39 'res_partner_bank_view.xml'36 'res_partner_bank_view.xml'
40 ],37 ],
41 'demo_xml': [],
42 'description': '''38 'description': '''
43This module is compatible with OpenERP 6.1.39This module is compatible with OpenERP 7.0.
4440
45The IBAN module in OpenERP 6.1 registers the IBAN41The IBAN module in OpenERP 6.1/7.0 registers the IBAN
46on the same field as the domestic account number, 42on the same field as the domestic account number,
47instead of keeping both on separate fields as is the43instead of keeping both on separate fields as is the
48case in 6.0.44case in 6.0.
4945
50This module adds a field to register the domestic account46This module adds a field to register the domestic account
51number on IBANs, while the domestic account number is47number on IBANs, as the domestic account number is
52still widely in use in certain regions.48still in use in certain regions. This should make for a
5349smoother migration to SEPA.
54Note that an upgrade to OpenERP 6.1 makes you lose the
55domestic account numbers on IBANs that were already in
56your system, unless you installed the 6.0 version of this
57module prior to the upgrade to OpenERP 6.1.
58 ''',50 ''',
59 'active': False,51 'active': False,
60 'installable': False,52 'installable': True,
61}53}
6254
=== modified file 'account_iban_preserve_domestic/res_partner_bank.py'
--- account_iban_preserve_domestic/res_partner_bank.py 2012-04-14 08:58:58 +0000
+++ account_iban_preserve_domestic/res_partner_bank.py 2013-02-26 21:20:32 +0000
@@ -1,5 +1,28 @@
1from osv import fields,osv1# -*- coding: utf-8 -*-
2class res_partner_bank(osv.osv):2##############################################################################
3#
4# OpenERP, Open Source Management Solution
5# This module copyright (C) 2012 - 2013 Therp BV (<http://therp.nl>).
6#
7# This program is free software: you can redistribute it and/or modify
8# it under the terms of the GNU Affero General Public License as
9# published by the Free Software Foundation, either version 3 of the
10# License, or (at your option) any later version.
11#
12# This program is distributed in the hope that it will be useful,
13# but WITHOUT ANY WARRANTY; without even the implied warranty of
14# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15# GNU Affero General Public License for more details.
16#
17# You should have received a copy of the GNU Affero General Public License
18# along with this program. If not, see <http://www.gnu.org/licenses/>.
19#
20##############################################################################
21
22from openerp.osv import fields, orm
23
24
25class res_partner_bank(orm.Model):
3 ''' Adds a field for domestic account numbers '''26 ''' Adds a field for domestic account numbers '''
4 _inherit = "res.partner.bank"27 _inherit = "res.partner.bank"
528
629
=== modified file 'account_iban_preserve_domestic/res_partner_bank_view.xml'
--- account_iban_preserve_domestic/res_partner_bank_view.xml 2012-02-19 21:10:20 +0000
+++ account_iban_preserve_domestic/res_partner_bank_view.xml 2013-02-26 21:20:32 +0000
@@ -18,20 +18,5 @@
18 </field>18 </field>
19 </record>19 </record>
2020
21 <!-- add the field to the partner form, as defined in
22 the account module -->
23 <record id="view_partner_account_form" model="ir.ui.view">
24 <field name="name">res.partner.account.form</field>
25 <field name="model">res.partner</field>
26 <field name="inherit_id" ref="account.view_partner_property_form"/>
27 <field name="arch" type="xml">
28 <field name="acc_number" position="after">
29 <newline/>
30 <field name="acc_number_domestic"
31 attrs="{'invisible': [('state', '!=', 'iban')]}"
32 />
33 </field>
34 </field>
35 </record>
36 </data>21 </data>
37</openerp>22</openerp>

Subscribers

People subscribed via source and target branches

to status/vote changes: