actualbudget / actual

A local-first personal finance app
https://actualbudget.org
MIT License
14k stars 1.12k forks source link

[Bug]: ofx amount parsing does not work for foreign number formats #2800

Closed Guekka closed 2 months ago

Guekka commented 4 months ago

Verified issue does not already exist?

What happened?

When importing an OFX file, amounts all appear as NaN. This is due to the code using the naive Number parser: source.

Solution

Using a different parser. I saw number-parsing recommended on SO.

youngcw commented 4 months ago

Can you provide an example ofx file?

Guekka commented 4 months ago

Of course, gimme a minute. But any number such as -5,74 will cause a NaN

Guekka commented 4 months ago

Here you go @youngcw parsing-number-locale-issue.zip