Bug 363082 - Brokerage account isn't s selectable during import of investment statement
Summary: Brokerage account isn't s selectable during import of investment statement
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: importer (show other bugs)
Version: git (master)
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-14 19:35 UTC by wojnilowicz
Modified: 2016-06-24 18:56 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 5.0
Sentry Crash Report:


Attachments
New way of selecting brokerage account (82.82 KB, application/pdf)
2016-05-14 19:36 UTC, wojnilowicz
Details

Note You need to log in before you can comment on or make changes to this bug.
Description wojnilowicz 2016-05-14 19:35:09 UTC
It's not a bug, it's proposal of improvement. If during import of CSV investment statement, brokerage account isn't automatically detected user has to type it in manually. User can make a typo and if he doesn't have brokerage account yet, he has no possibility to create it at site.

Reproducible: Always

Steps to Reproduce:
1. File->Import->CSV... 
2. choose Investing, 
3. proceed with importing till question about brokerage account

Actual Results:  
KMM asks user about brokerage account through simple input window.

Expected Results:  
KMM should ask user about brokerage like in case of investment account i.e. through account selector because ui is more consistent with it and account selector is comfortable to handle.
Comment 1 wojnilowicz 2016-05-14 19:36:08 UTC
Created attachment 98976 [details]
New way of selecting brokerage account
Comment 2 wojnilowicz 2016-06-24 18:45:36 UTC
Git commit daffd1caf5c52656b1fbc520a497cad5dc8a6501 by Łukasz Wojniłowicz.
Committed on 24/06/2016 at 18:43.
Pushed by wojnilowicz into branch 'master'.

Get rid of inputParameter for brokerage account

Functionality of inputParameter is replaced with KAccountSelectDlg.
REVIEW: 128101

M  +0    -10   kmymoney/plugins/csvimport/investprocessing.cpp
M  +0    -9    kmymoney/plugins/csvimport/redefinedlg.cpp

http://commits.kde.org/kmymoney/daffd1caf5c52656b1fbc520a497cad5dc8a6501
Comment 3 wojnilowicz 2016-06-24 18:56:33 UTC
Git commit b1c0ef8a2553ab44938829dc0870ea6796e27c8f by Łukasz Wojniłowicz.
Committed on 24/06/2016 at 18:45.
Pushed by wojnilowicz into branch 'master'.

Use KAccountSelectDlg to select brokerage account

If there is no "Account (Brokerage)" user should be asked to select
brokerage account.
REVIEW: 127920

M  +14   -0    kmymoney/converter/mymoneystatementreader.cpp
M  +8    -0    kmymoney/converter/mymoneystatementreader.h

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