Bug 310786

Summary: CSV Plugin- Bad import of file with debit field of several blanks
Product: [Applications] kmymoney Reporter: allan <agander93>
Component: generalAssignee: KMyMoney Devel Mailing List <kmymoney-devel>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: git (master)   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: http://commits.kde.org/kmymoney/881755b4d1dec31f0b3b87a0e14fec5fd5927854 Version Fixed/Implemented In:
Sentry Crash Report:

Description allan 2012-11-27 22:19:24 UTC 
When importing a CSV file derived from a pdf file, on clicking 'Import', the supposedly empty debit column in the plugin contained several '-.00' values.  After import, the ledger also showed these, but didn't show the credit values which should have been in those transactions.

Looking at the 'empty' debit fields, they did appear empty, but in fact contained all blanks.  A test for QString.isEmpty() therefore failed, and the debit processing was entered, resulting in the credit values being overwritten.


Reproducible: Always

Steps to Reproduce:
1. As above.
2.
3.
Actual Results:  
No money values were imported.

Expected Results:  
When a credit field is valid, don't import bogus debit entries.
Comment 1 allan 2012-12-04 11:03:44 UTC 
Git commit 881755b4d1dec31f0b3b87a0e14fec5fd5927854 by Allan Anderson.
Committed on 27/11/2012 at 23:24.
Pushed by allananderson into branch 'master'.

M  +1    -1    kmymoney/plugins/csvimport/csvdialog.cpp
M  +5    -3    kmymoney/plugins/csvimport/csvutil.cpp

http://commits.kde.org/kmymoney/881755b4d1dec31f0b3b87a0e14fec5fd5927854