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
1=== modified file 'account_iban_preserve_domestic/__openerp__.py'
2--- account_iban_preserve_domestic/__openerp__.py 2013-02-26 21:06:36 +0000
3+++ account_iban_preserve_domestic/__openerp__.py 2013-02-26 21:20:32 +0000
4@@ -1,61 +1,53 @@
5 ##############################################################################
6 #
7-# Copyright (C) 2012 Therp BV (<http://therp.nl>).
8+# Copyright (C) 2012 - 2013 Therp BV (<http://therp.nl>).
9 #
10 # All other contributions are (C) by their respective contributors
11 #
12 # All Rights Reserved
13 #
14-# WARNING: This program as such is intended to be used by professional
15-# programmers who take the whole responsability of assessing all potential
16-# consequences resulting from its eventual inadequacies and bugs
17-# End users who are looking for a ready-to-use solution with commercial
18-# garantees and support are strongly adviced to contract EduSense BV
19-#
20 # This program is free software: you can redistribute it and/or modify
21-# it under the terms of the GNU General Public License as published by
22-# the Free Software Foundation, either version 3 of the License, or
23-# (at your option) any later version.
24+# it under the terms of the GNU Affero General Public License as
25+# published by the Free Software Foundation, either version 3 of the
26+# License, or (at your option) any later version.
27 #
28 # This program is distributed in the hope that it will be useful,
29 # but WITHOUT ANY WARRANTY; without even the implied warranty of
30 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
31-# GNU General Public License for more details.
32+# GNU Affero General Public License for more details.
33 #
34-# You should have received a copy of the GNU General Public License
35+# You should have received a copy of the GNU Affero General Public License
36 # along with this program. If not, see <http://www.gnu.org/licenses/>.
37 #
38 ##############################################################################
39+
40 {
41 'name': 'Domestic bank account number',
42- 'version': '0.1.134',
43+ 'version': '0.1.163',
44 'license': 'AGPL-3',
45 'author': 'Therp BV',
46 'website': 'https://launchpad.net/banking-addons',
47 'category': 'Banking addons',
48- 'depends': ['base_iban','account'],
49- 'init_xml': [],
50- 'update_xml': [
51+ 'depends': [
52+ 'base_iban',
53+ 'account',
54+ ],
55+ 'data': [
56 'res_partner_bank_view.xml'
57 ],
58- 'demo_xml': [],
59 'description': '''
60-This module is compatible with OpenERP 6.1.
61+This module is compatible with OpenERP 7.0.
62
63-The IBAN module in OpenERP 6.1 registers the IBAN
64+The IBAN module in OpenERP 6.1/7.0 registers the IBAN
65 on the same field as the domestic account number,
66 instead of keeping both on separate fields as is the
67 case in 6.0.
68
69 This module adds a field to register the domestic account
70-number on IBANs, while the domestic account number is
71-still widely in use in certain regions.
72-
73-Note that an upgrade to OpenERP 6.1 makes you lose the
74-domestic account numbers on IBANs that were already in
75-your system, unless you installed the 6.0 version of this
76-module prior to the upgrade to OpenERP 6.1.
77+number on IBANs, as the domestic account number is
78+still in use in certain regions. This should make for a
79+smoother migration to SEPA.
80 ''',
81 'active': False,
82- 'installable': False,
83+ 'installable': True,
84 }
85
86=== modified file 'account_iban_preserve_domestic/res_partner_bank.py'
87--- account_iban_preserve_domestic/res_partner_bank.py 2012-04-14 08:58:58 +0000
88+++ account_iban_preserve_domestic/res_partner_bank.py 2013-02-26 21:20:32 +0000
89@@ -1,5 +1,28 @@
90-from osv import fields,osv
91-class res_partner_bank(osv.osv):
92+# -*- coding: utf-8 -*-
93+##############################################################################
94+#
95+# OpenERP, Open Source Management Solution
96+# This module copyright (C) 2012 - 2013 Therp BV (<http://therp.nl>).
97+#
98+# This program is free software: you can redistribute it and/or modify
99+# it under the terms of the GNU Affero General Public License as
100+# published by the Free Software Foundation, either version 3 of the
101+# License, or (at your option) any later version.
102+#
103+# This program is distributed in the hope that it will be useful,
104+# but WITHOUT ANY WARRANTY; without even the implied warranty of
105+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
106+# GNU Affero General Public License for more details.
107+#
108+# You should have received a copy of the GNU Affero General Public License
109+# along with this program. If not, see <http://www.gnu.org/licenses/>.
110+#
111+##############################################################################
112+
113+from openerp.osv import fields, orm
114+
115+
116+class res_partner_bank(orm.Model):
117 ''' Adds a field for domestic account numbers '''
118 _inherit = "res.partner.bank"
119
120
121=== modified file 'account_iban_preserve_domestic/res_partner_bank_view.xml'
122--- account_iban_preserve_domestic/res_partner_bank_view.xml 2012-02-19 21:10:20 +0000
123+++ account_iban_preserve_domestic/res_partner_bank_view.xml 2013-02-26 21:20:32 +0000
124@@ -18,20 +18,5 @@
125 </field>
126 </record>
127
128- <!-- add the field to the partner form, as defined in
129- the account module -->
130- <record id="view_partner_account_form" model="ir.ui.view">
131- <field name="name">res.partner.account.form</field>
132- <field name="model">res.partner</field>
133- <field name="inherit_id" ref="account.view_partner_property_form"/>
134- <field name="arch" type="xml">
135- <field name="acc_number" position="after">
136- <newline/>
137- <field name="acc_number_domestic"
138- attrs="{'invisible': [('state', '!=', 'iban')]}"
139- />
140- </field>
141- </field>
142- </record>
143 </data>
144 </openerp>

Subscribers

People subscribed via source and target branches

to status/vote changes: