[17.0][FIX] account_statement_import_sheet_file: fix none decimal separator

[jbaudoux]

Allow to parse "123456" as 1234.56 when using "none" as decimal separator

Before it was returning "1.2.3.4.5.6" and crashing

[OCA-git-bot]

Hi @alexey-pelykh,
some modules you are maintaining are being modified, check this out!

[alexey-pelykh]

I'm not sure I'm aboard this the fix - if there's no decimal separator and 123456 is given, it should be parsed as 123456 - why should it work otherwise?

[jbaudoux]

I'm not sure I'm aboard this the fix - if there's no decimal separator and 123456 is given, it should be parsed as 123456 - why should it work otherwise?

It will be parsed as 123456 if you configure there should be a separator and the value doesn't contain any.
The bank provides a file without separator and the currency has 2 decimals. When the amount 123456 is given, it means 1234.56
Before this it was adding a "." between each figure so I guess that option never worked and has never been used

[alexey-pelykh]

It will be parsed as 123456 if you configure there should be a separator and the value doesn't contain any.

I think it calls for a documentation extension that "If no decimal delimiter is configured, ...... Example: ...."
Otherwise I'm good with the PR

[github-actions]

There hasn't been any activity on this pull request in the past 4 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 PR to never become stale, please ask a PSC member to apply the "no stale" label.

[jbaudoux]

help added on decimal separator field