search

OCA / l10n-switzerland

Odoo Swiss localization
GNU Affero General Public License v3.0
53 stars 164 forks source link

l10n_ch_base_bank should not depend on l10n_ch #528

Closed Martronic-SA closed 2 years ago

Martronic-SA commented 5 years ago

a swiss company may have a completely different accounting plan than the default l10n_ch but still need the swiss banks. if l10n_ch_base_bank is not really technically depending on l10n_ch, that dependency should be removed from the manifest file.

yvaucher commented 5 years ago

@Martronic-SA Issue is that the Odoo core module now includes more than the Chart of account.

Your assertion was true for version <12.0

https://github.com/odoo/odoo/blob/91560fae56f1cba09141eefa1d0da03f4a7b3f72/addons/l10n_ch/models/res_bank.py#L26-L36

yvaucher commented 5 years ago

What you want is to avoid to generate automatically the CoA. The template on itself is not an issue.

It's a bit tricky but possible to avoid Odoo to install automatically the CoA from the template on module's installation.

What you can do is to create a single account.account on your company. Before installing l10n_ch. It will consider you already have a CoA installed and won't copy the template as CoA.

Warning, installing account module with your company set with CH country will also automatically install l10n_ch https://github.com/odoo/odoo/blob/12.0/addons/account/__init__.py#L13

Martronic-SA commented 5 years ago

Ok I see. But banks and CoA might (should?) have been added in two separate modules as long as they concern two different things not linked together (except that they are both for Switzerland). In my particular case I just wanted to uninstall the swiss CoA but keep swiss banks.

yvaucher commented 5 years ago

I know, Odoo have been a bit un-flexible about it due to their stable policy, you can try to contact them about this but they will most likely change nothing before Odoo 14.0.

Martronic-SA commented 5 years ago

Yes I understand that it's difficult to change now that te module l10n_ch has been published like that. I keep that issue open for the moment as so we can think about it for the future releases.

github-actions[bot] commented 2 years ago

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.