Bug 310786 - CSV Plugin- Bad import of file with debit field of several blanks
Summary: CSV Plugin- Bad import of file with debit field of several blanks
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: git (master)
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-27 22:19 UTC by allan
Modified: 2012-12-04 11:03 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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