Bug 236169

Summary:	QIF Import - Empty payee fields
Product:	[Applications] kmymoney	Reporter:	heart1010 <heart1010>
Component:	general	Assignee:	KMyMoney Devel Mailing List <kmymoney-devel>
Status:	RESOLVED FIXED
Severity:	normal	CC:	agander93
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:
Attachments:	Test QIF-File

Description heart1010 2010-05-03 15:25:41 UTC

Version:           3.97.2 (using KDE 4.3.5)
OS:                Linux
Installed from:    openSUSE RPMs

Tested an import of a qif file and here are problems with the payee field. 

A few payee fields are completely blank also there is text in the qif file!?

see picture http://www.abload.de/img/kmm_qif_importp53s.png

P.S.: Importing the same qif file in skroogle for example is ok
(so I don't think that's a problem of my qif file I tried to import)

Comment 1 Alvaro Soliverez 2010-05-03 16:38:41 UTC

Are the payee fields complete or are they in the memo fields?

Can you provide an example of a payee field that is not being imported?
Having more data will help us solve the problem, since QIF is not a standard nor easy format to deal with.

Comment 2 heart1010 2010-05-03 17:05:52 UTC

Created attachment 43194 [details]
Test QIF-File

This qif file includes 13 transactions and when I import this qif file 4 payee fields get blank, one of that 4 is correct because in the qif file there is "P" on that transaction.

Tried to reproduce the error and filled this empty payee line in the qif file but nothing changed after import. So I have no clue why KMM doesn't fetch some payee lines correctly.

Comment 3 allan 2010-05-03 17:12:33 UTC

Can I but in here, as I was in contact with the bug originator off-list and
confirmed his problem with this file - 

!Type:Bank
D28-08-09
PNET INTEREST       TO 27JUL2009
T0.08
^
D28-07-09
P
T0.07
^
D28-06-09
PNET INTEREST       TO 27JUN2009
T0.08
^

The 28 Aug entry gets in OK, but the other two have blank payee and memo,
similar to the pic. referred to above.

After a blank payee entry, following payees become blank.

Comment 4 Cristian Oneț 2010-05-03 19:29:29 UTC

SVN commit 1122361 by conet:

BUG: 236169
When a QIF field is missing don't return a string containing the space character. 
All logic in the importer should interpret an empty string as a missing field. I've checked the commit log and the code was added in 2003-10-02 and the only QIF related comment is 'improved QIF import performance'. Returning the space character as a payee was causing all subsequent payees that contained a space in their names to be matched as the 'space' payee.
Note that users affected by this bug should delete their payee having only a space for a name (or disable the matching for it) to be able to import other payees containing spaces.

 M  +0 -2      mymoneyqifreader.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1122361