Review Request 128734: Rewrite intro page in CSV Importer
    Łukasz Wojniłowicz 
    lukasz.wojnilowicz at gmail.com
       
    Mon Aug 22 05:51:45 UTC 2016
    
    
  
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128734/
-----------------------------------------------------------
Review request for KMymoney.
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.
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
-------
Thanks,
Łukasz Wojniłowicz
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20160822/b605a0e1/attachment-0001.html>
    
    
More information about the KMyMoney-devel
mailing list