Review Request 128734: Rewrite intro page in CSV Importer

Łukasz Wojniłowicz lukasz.wojnilowicz at gmail.com
Sat Sep 3 07:56:18 UTC 2016


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

(Updated Sept. 3, 2016, 9:56 a.m.)


Status
------

This change has been marked as submitted.


Review request for KMymoney.


Changes
-------

Submitted with commit 8ae1ebcbe7d59058cc39ece143efa3197fc7565e by ?ukasz Wojni?owicz to branch master.


Repository: kmymoney


Description
-------

1) slotFileDialogClicked was duplicated in csvdialog.cpp and
investprocessing.cpp so moved it to csvwizard.cppp,
2) introduced getInFileName for asking user about his statement file,
3) createProfile, editProfileName, addProfileName was complicated and
replaced by single shorter profileTypeChanged
4) slotRadioButton_bankClicked and slotRadioButton_investClicked was
complicted and replaced by profileTypeChange
5) adding/removing/renaming profile was not user friendly and
informative so new buttons added,
6) no need to have template csvimporterrc, all can be done through
validateConfigFile
7) cleaned up csvimporterrc so its more informative and easier to get
data from it,
8) introduced updateConfigFile which is framework for migrating
csvimporterrc file between different KMM versions,
9) removed enableInputs as it wasn't doing anything usefull,
10) introduced m_config which allows asking for configuration file only
once at the beginning,
11) removed lots of redundant variables.

new csvimporterrc schema
======

[Bank-a1]
Directory=~/

[Invest-a1]
Directory=~/

[Misc]
Height=400
KMMVer=5,0,0
Width=800

[ProfileNames]
Bank=a1
Invest=a1
PriorBank=
PriorInvest=

[Securities]
SecurityNameList=


Diffs
-----

  kmymoney/plugins/csvimport/csvdialog.h ef7ba53 
  kmymoney/plugins/csvimport/csvdialog.cpp 859cdc2 
  kmymoney/plugins/csvimport/csvwizard.h cbc70fb 
  kmymoney/plugins/csvimport/csvwizard.cpp 3ff38ed 
  kmymoney/plugins/csvimport/introwizardpage.ui a91a737 
  kmymoney/plugins/csvimport/investprocessing.h 9374212 
  kmymoney/plugins/csvimport/investprocessing.cpp c306173 

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


Testing
-------


File Attachments
----------------

intro page.png
  https://git.reviewboard.kde.org/media/uploaded/files/2016/08/22/855272f4-8710-4790-902e-5bba44c37739__intro_page.png


Thanks,

Łukasz Wojniłowicz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20160903/3aa0aa88/attachment.html>


More information about the KMyMoney-devel mailing list