Bug 363954

Summary:	CSV Importer doesn't import fees with comma as decimal separator
Product:	[Applications] kmymoney	Reporter:	NSLW <lukasz.wojnilowicz>
Component:	importer	Assignee:	KMyMoney Devel Mailing List <kmymoney-devel>
Status:	RESOLVED FIXED
Severity:	normal	CC:	onet.cristian, ralf.habacker
Priority:	NOR
Version:	git (master)
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:	http://commits.kde.org/kmymoney/bbdfda2597c502ebcc3a9ce20958132abfdebec4	Version Fixed In:	4.8.1,5.0.0
Attachments:	CSV Test File Column Assignment for CSV file

Description NSLW 2016-06-04 18:56:47 UTC

CSV Importer doesn't import fees with comma as decimal separator

Reproducible: Always

Steps to Reproduce:
1. File->Import->CSV
2. Select "Investments" and create new profile
3. Press select file and choose attached "test file.csv"
4. Set field delimiter to "comma" and text delimiter to "quotes" and press next
5. Assign columns according to attached image and press next
6. Set date format as "y m d" and press next
7. Set decimal symbol to "comma" and press "Import CSV"

Actual Results:  
Fee 3,56 isn't imported for this transaction
2016-01-20,MONNARI,Kupno,20,12,240,MON,"3,56"

Expected Results:  
Fee 3,56 should be imported for this transaction
2016-01-20,MONNARI,Kupno,20,12,240,MON,"3,56"

Comment 1 NSLW 2016-06-04 18:58:37 UTC

Created attachment 99366 [details]
CSV Test File

Comment 2 NSLW 2016-06-04 18:59:44 UTC

Created attachment 99368 [details]
Column Assignment for CSV file

Comment 3 NSLW 2016-06-18 17:23:32 UTC

Git commit 5796089e8cc2b9951ce8e32cfb87f7a1dd9d9f20 by Łukasz Wojniłowicz.
Committed on 18/06/2016 at 17:22.
Pushed by wojnilowicz into branch '4.8'.

Fix importing fees with comma as decimal separator
REVIEW: 128100

M  +21   -11   kmymoney/plugins/csvimport/investprocessing.cpp

http://commits.kde.org/kmymoney/5796089e8cc2b9951ce8e32cfb87f7a1dd9d9f20

Comment 4 Cristian Oneț 2016-06-18 18:09:02 UTC

Git commit bbdfda2597c502ebcc3a9ce20958132abfdebec4 by Cristian Oneț, on behalf of Łukasz Wojniłowicz.
Committed on 18/06/2016 at 18:06.
Pushed by conet into branch 'master'.

Fix importing fees with comma as decimal separator
REVIEW: 128100
(cherry picked from commit 5796089e8cc2b9951ce8e32cfb87f7a1dd9d9f20)

M  +21   -11   kmymoney/plugins/csvimport/investprocessing.cpp

http://commits.kde.org/kmymoney/bbdfda2597c502ebcc3a9ce20958132abfdebec4