[kmymoney] [Bug 328050] KMymoney does not support space as thousand separator

[Ralf Habacker]

https://bugs.kde.org/show_bug.cgi?id=328050

--- Comment #15 from Ralf Habacker <ralf.habacker at freenet.de> ---
(In reply to Thomas Baumgart from comment #11)
> Git commit 0e75cd0973d7e19d09efd4ac99a4eee0e521f6c5 by Thomas Baumgart.
> Committed on 04/06/2018 at 09:40.
> Pushed by tbaumgart into branch 'master'.
> 
> Adjust usage of thousands separator
> 
> Modifications according to the analysis in bug 328050
> 
> OFX: the modified code is commented out and only used for test purposes
> CSV: just a simplification of the code, no change in functionality
> 
> M  +2    -7    kmymoney/plugins/csv/export/csvwriter.cpp
> M  +13   -13   kmymoney/plugins/ofx/import/dialogs/mymoneyofxconnector.cpp
> 
> https://commits.kde.org/kmymoney/0e75cd0973d7e19d09efd4ac99a4eee0e521f6c5
You fixed this in CsvWriter::writeTransactionEntry() but not in 
CsvWriter::writeSplitEntry(), see
https://cgit.kde.org/kmymoney.git/tree/kmymoney/plugins/csv/export/csvwriter.cpp#n256

BTW: It looks that removing this replacement, which intention was to fixes
clash with field separator, introduces a new bug resulting into invalid csv
file in case the memo field contains the field delimiter.

-- 
You are receiving this mail because:
You are the assignee for the bug.