Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Many bugs with QIF import -- Test cases inside #2132

Closed
t4777sd opened this issue Apr 8, 2019 · 5 comments

Comments

Projects
None yet
2 participants
@t4777sd
Copy link

commented Apr 8, 2019

Using the very latest alpha version.

Describe the bug
Import from valid QIF files generated from quicken does not produce desired results

To Reproduce

Here is the QIF file:

!Option:AutoSwitch
!Account
NChecking 1
TBank
^
NChecking 2
TBank
^
!Clear:AutoSwitch
!Option:AutoSwitch
!Account
NChecking 1
TBank
^
!Type:Bank 
D1/ 1'18
U0.00
T0.00
CX
POpening Balance
L[Checking 1]
^
D1/ 5'18
U85,000.00
T85,000.00
PPayer #1
LConsulting Income (Business)
^
D1/ 5'18
U-12,000.00
T-12,000.00
PBill #1
LMisc. Expense (Business)
^
D1/ 5'18
U-11,000.00
T-11,000.00
PBank Transfer
L[Checking 2]
^
D1/ 5'18
U-11,039.00
T-11,039.00
PSplit Expense
LAuto & Transport:Gas & Fuel
SAuto & Transport:Gas & Fuel
$-39.00
SHealth & Fitness:Doctor
$-11,000.00
^
D1/ 5'19
U-14,000.00
T-14,000.00
PBank Transfer
L[Checking 2]
^
D1/17'19
U17,500.00
T17,500.00
PSplit Income
LOther Income (Business)
SOther Income (Business)
$5,000.00
SNet Salary
$12,500.00
^
D1/17'19
U-23,200.00
T-23,200.00
PSplit With Transfer
LHealth & Fitness:Doctor
SHealth & Fitness:Doctor
$-12,000.00
S[Checking 2]
$-11,200.00
^
!Account
NChecking 2
TBank
^
!Type:Bank 
D1/ 1'18
U0.00
T0.00
CX
POpening Balance
L[Checking 2]
^
D1/ 5'18
U11,000.00
T11,000.00
PBank Transfer
L[Checking 1]
^
D1/ 5'19
U14,000.00
T14,000.00
PBank Transfer
L[Checking 1]
^
D1/17'19
U11,200.00
T11,200.00
PSplit With Transfer
L[Checking 1]
^
D1/17'19
U-12,500.00
T-12,500.00
PSelf Transfer Payment
L[Checking 2]
^
D3/17'19
U12,650.00
T12,650.00
PSelf Transfer Deposit
L[Checking 2]
^
D4/ 8'19
U137,000.00
T137,000.00
PPayer #2
LNet Salary Spouse
^
D4/ 8'19
U-17,000.00
T-17,000.00
PBill #2
LHealth & Fitness:Dentist
^

The QIf file and images of what the results SHOULD be are here: https://send.firefox.com/download/490fe390140b628e/#S2SQQXe-UJ-zgWwH6SCq8A

Summary of correct results ending balance:
Checking 1: $31,261
Checking 2: $156,350

Summary of actual results in mm:
Checking 1: $0.00
Checking 2: $120,000

@vomikan vomikan self-assigned this Apr 9, 2019

@vomikan vomikan added bug import labels Apr 9, 2019

@vomikan

This comment has been minimized.

Copy link
Contributor

commented Apr 9, 2019

Seems mmex skips 'Self Transfer Payment'
It looks like transfer from 'Checking 2' to the same 'Checking 2' account.

D1/17'19
U-12,500.00
T-12,500.00
PSelf Transfer Payment
L[Checking 2]
^
D3/17'19
U12,650.00
T12,650.00
PSelf Transfer Deposit
L[Checking 2]
@vomikan

This comment has been minimized.

Copy link
Contributor

commented Apr 9, 2019

I've changed a bit QIF import. Result:
image

image

image

image

vomikan added a commit to vomikan/moneymanagerex that referenced this issue Apr 9, 2019

vomikan added a commit that referenced this issue Apr 9, 2019

Merge pull request #2134 from vomikan/master
fix(#2132): self transfer support
@t4777sd

This comment has been minimized.

Copy link
Author

commented Apr 9, 2019

Shouldn't the Checking #1 ending balance be $31,261. Maybe I am misreading the image, but it appears the 2nd to last line is $31,261, but then it gets reduced down to $20,061 which is different than the ending balance it should be.

@vomikan

This comment has been minimized.

Copy link
Contributor

commented Apr 10, 2019

The last transaction of acc1 is transfer of 11200 to acc2. Transfers in mmex shown in both acc - which money goes out and second one where the money comes.
(Look at the transaction ID)

@vomikan

This comment has been minimized.

Copy link
Contributor

commented Apr 10, 2019

Seems 'Split With Transfer' is some (magic for me) type of transactions that parsed invalid.

@vomikan vomikan closed this May 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.