Review Request 127712: Bug 360747 CSV Importer detects more columns than are assigned

[Łukasz Wojniłowicz]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127712/
-----------------------------------------------------------

(Updated May 8, 2016, 7:12 a.m.)


Status
------

This change has been marked as submitted.


Review request for KMymoney.


Changes
-------

Submitted with commit 4debce51fe4afa867db9d33baaa1a0185b031379 by ?ukasz Wojni?owicz to branch master.


Bugs: 360747
    http://bugs.kde.org/show_bug.cgi?id=360747


Repository: kmymoney


Description
-------

Fixes bug #360747. Current routine doesn't calculate columns well when
FieldDelimiter=DecimalSymbol. parseLine() from csvutil.cpp does it
properly.


Diffs
-----

  kmymoney/plugins/csvimport/investprocessing.cpp 6826122 

Diff: https://git.reviewboard.kde.org/r/127712/diff/


Testing
-------

1) CSV file where FieldDelimiter=DecimalSymbol=comma (Test file from bug #360747)
2) CSV file where FieldDelimiter=comma and DecimalSymbol=dot


Thanks,

Łukasz Wojniłowicz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20160508/24943f12/attachment-0001.html>