jseutter / ofxparse

Ofx file format parser for Python
http://sites.google.com/site/ofxparse/
MIT License
204 stars 121 forks source link

Support decimal comma for transaction amounts, #83 #91

Closed takis closed 8 years ago

takis commented 8 years ago

BNP Paribas Fortis in Belgium uses commas as the decimal separator instead of points.

takis commented 8 years ago

According to the OFX 2.1.1 specification, comma's are allowed, while separators for thousands and millions so on are not.

Amounts that do not represent whole numbers (for example, 540.32), must include a decimal point or comma to indicate the start of the fractional amount. Amounts should not include any punctuation separating thousands, millions, and so forth.

jseutter commented 8 years ago

Handling commas as decimal separators is one of those things that is obvious in retrospect but never occurred to me before. Thank you for your fixes and tests!