[Kmymoney-devel] Re: Review Request: Use KComboBox instead of KLineEdit for investment security name use.

Allan Anderson agander93 at gmail.com
Sat Mar 26 21:56:07 CET 2011

This is an automatically generated e-mail. To reply, visit:

(Updated March 26, 2011, 8:56 p.m.)

Review request for kmymoney.


Made the suggested changes, plus removed quite a few other 'return on same line' issues,
mainly from early code.


Previously, it was necessary to type in the name whenever a security was to be imported.  Instead, 
a KComboBox is used, and security names entered are saved in the resource file, to save inputting 
each time.  A button has been added to allow removal of any unwanted or erroneous names from the 
selection list.

Also, a decimal separator now has to be selected before import, to avoid accidental use of a previous,
wrong type, which could affect imported values.

Diffs (updated)

  /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvimporterdlg.h 1226119 
  /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvimporterdlg.cpp 1226119 
  /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvimporterdlgdecl.ui 1226119 
  /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvprocessing.cpp 1226119 
  /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/csvutil.cpp 1226119 
  /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/investmentdlg.cpp 1226119 
  /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/investprocessing.h 1226119 
  /trunk/extragear/office/kmymoney/kmymoney/plugins/csvimport/investprocessing.cpp 1226119 

Diff: http://svn.reviewboard.kde.org/r/6633/diff


Numerous test files used, plus production files.

Astyle and Krazy2 run.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kmymoney-devel/attachments/20110326/00d32a63/attachment.htm 

More information about the KMyMoney-devel mailing list