Review Request 128155: Require only name or symbol of securities from investment statement

[Łukasz Wojniłowicz]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128155/
-----------------------------------------------------------

(Updated June 25, 2016, 4:20 p.m.)


Status
------

This change has been marked as submitted.


Review request for KMymoney.


Changes
-------

Submitted with commit 2466548926730c86d06449c07807c9bacba186eb by ?ukasz Wojni?owicz to branch master.


Repository: kmymoney


Description
-------

During CSV import of investment statement, symbol and detail (ie. name) columns are required. This patch makes posible to import investment statement with only one column selected. If security is new, user input is required (ie. provide either symbol or name). User input is done through existing securities window.


Diffs
-----

  kmymoney/plugins/csvimport/csvdialog.cpp 6799df7 
  kmymoney/plugins/csvimport/csvwizard.cpp d17622a 
  kmymoney/plugins/csvimport/investprocessing.cpp 34e53c8 
  kmymoney/plugins/csvimport/symboltabledlg.h 246c36a 
  kmymoney/plugins/csvimport/symboltabledlg.cpp 585ec62 
  kmymoney/plugins/csvimport/symboltabledlg.ui f93bddd 

Diff: https://git.reviewboard.kde.org/r/128155/diff/


Testing
-------

My CSV test files with securities already in KMM database and some new securities which were identified by name or by symbol.


Thanks,

Łukasz Wojniłowicz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20160625/60fd41c6/attachment.html>