Merge lp:~camptocamp/c2c-addons/accounting-profiles into lp:c2c-addons

Proposed by Vincent Renaville@camptocamp
Status: Merged
Merged at revision: 5
Proposed branch: lp:~camptocamp/c2c-addons/accounting-profiles
Merge into: lp:c2c-addons
Diff against target: 167 lines (+147/-0)
3 files modified
common_accounting_profile/__openerp__.py (+59/-0)
l10n_ch_profile/__openerp__.py (+48/-0)
l10n_fr_profile/__openerp__.py (+40/-0)
To merge this branch: bzr merge lp:~camptocamp/c2c-addons/accounting-profiles
Reviewer Review Type Date Requested Status
Guewen Baconnier @ Camptocamp Approve
Alexandre Fayolle - camptocamp code review, no test Approve
Frederic Clementi - Camptocamp functional Approve
Review via email: mp+205935@code.launchpad.net

Description of the change

Hello,

This proposal purpose is to add profile for setup accounting in l10n_ch & l10n_fr

Vincent

To post a comment you must log in.
Revision history for this message
Frederic Clementi - Camptocamp (frederic-clementi) wrote :

LGTM

review: Approve (functional)
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Totally agree with your idea of creating profiles with the modules we use (next steps: sphinx docs?).

Do we really want html_invoice_product_note in the common_accounting_profile? I have in head at least one project which do not use it on purpose.
The profiles should install what is strictly required (in our vision ;-) and this module does not apply IMO (it changes product_template.description from a text to a HTML field which can bring complications, especially when dealing with external systems).

Apart that, seems good to me.

Thanks!

review: Needs Information
Revision history for this message
Frederic Clementi - Camptocamp (frederic-clementi) wrote :

easy... just uninstall it, Guewen :)

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

> easy... just uninstall it, Guewen :)

You see the profiles as a script to install a set of modules: we have oerpscenario for that.

In my sense, a profile gathers a set of modules that must be there to accomplish a (large) task and *shouldn't be removed*. If I install common_accounting_profile and then uninstall html_invoice_product_note, common_accounting_profile will be uninstalled as well and then we lose all the benefits of using a profile.

4. By Vincent Renaville@camptocamp

[ADD] transaction modules

5. By Vincent Renaville@camptocamp

[FIX] remove module html_invoice_product_note depedencie

Revision history for this message
Vincent Renaville@camptocamp (vrenaville-c2c) wrote :

Hello Guewen,

I have just remove html_invoice_product_note according to your review at rev 5

Thanks for your review and sorry for the delay

Vincent

Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

LGTM

review: Approve (code review, no test)
Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added directory 'common_accounting_profile'
2=== added file 'common_accounting_profile/__init__.py'
3=== added file 'common_accounting_profile/__openerp__.py'
4--- common_accounting_profile/__openerp__.py 1970-01-01 00:00:00 +0000
5+++ common_accounting_profile/__openerp__.py 2014-03-28 10:26:37 +0000
6@@ -0,0 +1,59 @@
7+# -*- coding: utf-8 -*-
8+##############################################################################
9+#
10+# Author Vincent Renaville. Copyright 2013 Camptocamp SA
11+#
12+# This program is free software: you can redistribute it and/or modify
13+# it under the terms of the GNU Affero General Public License as
14+# published by the Free Software Foundation, either version 3 of the
15+# License, or (at your option) any later version.
16+#
17+# This program is distributed in the hope that it will be useful,
18+# but WITHOUT ANY WARRANTY; without even the implied warranty of
19+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
20+# GNU Affero General Public License for more details.
21+#
22+# You should have received a copy of the GNU Affero General Public License
23+# along with this program. If not, see <http://www.gnu.org/licenses/>.
24+#
25+##############################################################################
26+{
27+ "name" : "Common accounting profile",
28+ "description" : """Install all modules required for common accounting
29+ """,
30+ "version" : "1.0",
31+ "author" : "Camptocamp",
32+ "category" : "Accounting & Finance",
33+ "website": "http://www.camptocamp.com",
34+ "depends" : [
35+ 'account_accountant',
36+ 'account_constraints',
37+ 'account_default_draft_move',
38+ 'account_draft_invoice_print',
39+ 'account_financial_report_webkit',
40+ 'account_financial_report_webkit_xls',
41+ 'account_journal_report_xls',
42+ 'account_move_line_report_xls',
43+ 'report_xls',
44+ 'account_export_csv',
45+ 'account_reversal',
46+ 'currency_rate_update',
47+ 'invoice_webkit',
48+ 'account_compute_tax_amount',
49+ 'base_iban',
50+ 'account_move_validation_improvement',
51+ 'account_statement_base_completion',
52+ 'account_statement_base_import',
53+ 'account_statement_ext',
54+ 'account_statement_commission',
55+ 'invoicing_voucher_killer',
56+ 'statement_voucher_killer',
57+ 'account_advanced_reconcile',
58+ ],
59+ "data" :[
60+ ],
61+ "active": False,
62+ "installable": True
63+}
64+
65+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
66
67=== added directory 'l10n_ch_profile'
68=== added file 'l10n_ch_profile/__init__.py'
69=== added file 'l10n_ch_profile/__openerp__.py'
70--- l10n_ch_profile/__openerp__.py 1970-01-01 00:00:00 +0000
71+++ l10n_ch_profile/__openerp__.py 2014-03-28 10:26:37 +0000
72@@ -0,0 +1,48 @@
73+# -*- coding: utf-8 -*-
74+##############################################################################
75+#
76+# Author Vincent Renaville. Copyright 2013 Camptocamp SA
77+#
78+# This program is free software: you can redistribute it and/or modify
79+# it under the terms of the GNU Affero General Public License as
80+# published by the Free Software Foundation, either version 3 of the
81+# License, or (at your option) any later version.
82+#
83+# This program is distributed in the hope that it will be useful,
84+# but WITHOUT ANY WARRANTY; without even the implied warranty of
85+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
86+# GNU Affero General Public License for more details.
87+#
88+# You should have received a copy of the GNU Affero General Public License
89+# along with this program. If not, see <http://www.gnu.org/licenses/>.
90+#
91+##############################################################################
92+{
93+ "name" : "L10n_ch accounting profile",
94+ "description" : """Install all modules required for accounting in Switzerland
95+ """,
96+ "version" : "1.0",
97+ "author" : "Camptocamp",
98+ "category" : "Accounting & Finance",
99+ "website": "http://www.camptocamp.com",
100+ "depends" : [
101+ 'common_accounting_profile',
102+ 'l10n_ch',
103+ 'l10n_ch_bank',
104+ 'l10n_ch_base_bank',
105+ 'l10n_ch_dta',
106+ 'l10n_ch_payment_slip',
107+ 'l10n_ch_zip',
108+ 'l10n_multilang',
109+ 'payment_term_rounding',
110+ 'l10n_ch_dta_base_transaction_id',
111+ 'l10n_ch_payment_slip_base_transaction_id',
112+ 'l10n_ch_payment_slip_account_statement_base_completion'
113+ ],
114+ "data" :[
115+ ],
116+ "active": False,
117+ "installable": True
118+}
119+
120+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
121
122=== added directory 'l10n_fr_profile'
123=== added file 'l10n_fr_profile/__init__.py'
124=== added file 'l10n_fr_profile/__openerp__.py'
125--- l10n_fr_profile/__openerp__.py 1970-01-01 00:00:00 +0000
126+++ l10n_fr_profile/__openerp__.py 2014-03-28 10:26:37 +0000
127@@ -0,0 +1,40 @@
128+# -*- coding: utf-8 -*-
129+##############################################################################
130+#
131+# Author Vincent Renaville. Copyright 2013 Camptocamp SA
132+#
133+# This program is free software: you can redistribute it and/or modify
134+# it under the terms of the GNU Affero General Public License as
135+# published by the Free Software Foundation, either version 3 of the
136+# License, or (at your option) any later version.
137+#
138+# This program is distributed in the hope that it will be useful,
139+# but WITHOUT ANY WARRANTY; without even the implied warranty of
140+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
141+# GNU Affero General Public License for more details.
142+#
143+# You should have received a copy of the GNU Affero General Public License
144+# along with this program. If not, see <http://www.gnu.org/licenses/>.
145+#
146+##############################################################################
147+{
148+ "name" : "l10n_fr accounting profile",
149+ "description" : """Install all modules required for accounting in France
150+ """,
151+ "version" : "1.0",
152+ "author" : "Camptocamp",
153+ "category" : "Accounting & Finance",
154+ "website": "http://www.camptocamp.com",
155+ "depends" : [
156+ 'common_accounting_profile',
157+ 'l10n_fr',
158+ 'l10n_fr_rib',
159+ 'l10n_fr_siret',
160+ ],
161+ "data" :[
162+ ],
163+ "active": False,
164+ "installable": True
165+}
166+
167+# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

Subscribers

People subscribed via source and target branches

to all changes: