search

OCA / cooperative

GNU Affero General Public License v3.0
11 stars 22 forks source link

l10n_be_cooperator: In tax shelter certificate, the thousands separator and decimal separator do not match the partner's locale #124

Closed carmenbianca closed 6 months ago

carmenbianca commented 6 months ago

Module

l10n_be_cooperator

Describe the bug

The generated PDF of a tax shelter certificate uses the wrong thousands separator and decimal separator. It should be '.' and ',' respectively for a French partner, but ',' and '.' respectively are used.

To Reproduce

Affected versions: 16.0, have not tested others

Steps to reproduce the behavior:

  1. Add French (BE) as language to database, then set the language of the partner whose certificate you want to print as French (BE). In the demo data, this is Virginie Leloup.
  2. Print the tax shelter subscription certificate of a partner.

Example:

Capture d’écran du 2024-04-02 15-40-57

Expected behavior The thousands separator and decimal separator respect the partner's locale.

Additional context Add any other context about the problem here. (e.g. OS, Python version, ...)

carmenbianca commented 6 months ago

I wrote https://github.com/OCA/cooperative/issues/123 shortly before this one. They are probably maybe maybe-not linked.

carmenbianca commented 6 months ago

false alarm; forgot to set the partner to French.