> On July 24, 2016, 12:54 p.m., Thomas Baumgart wrote: > > kmymoney/plugins/csvimport/csvwizard.cpp, line 514 > > <https://git.reviewboard.kde.org/r/128511/diff/1/?file=472248#file472248line514> > > > > Remove if not needed
Ups! that shouldn't be commented out, but works anyhow :). > On July 24, 2016, 12:54 p.m., Thomas Baumgart wrote: > > kmymoney/plugins/csvimport/investprocessing.cpp, line 2461 > > <https://git.reviewboard.kde.org/r/128511/diff/1/?file=472250#file472250line2461> > > > > Why not simply 'm_endColumn++'? Same question applies to the line above. I'll do that way. - Łukasz ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128511/#review97792 ----------------------------------------------------------- On July 24, 2016, 11:07 a.m., Łukasz Wojniłowicz wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/128511/ > ----------------------------------------------------------- > > (Updated July 24, 2016, 11:07 a.m.) > > > Review request for KMymoney. > > > Repository: kmymoney > > > Description > ------- > > Purpose of displayLine in tableWidget shouldn't differ whether it's > investment or banking statement, so take of responsibilities of parsing > data into columns and creating memo field from displayLine and put them > into separate functions. > > In csvdialog.cpp and investprocessing.cpp, I introduced createMemoField (it > simplifies memo concatenating and allows to remove some redundant boolean > variables), which in both places looks the same. I think it will be easy to > move it into csvwizard.cpp after trying to move e.g. readFile there, so that > will be next target. > > > Diffs > ----- > > kmymoney/plugins/csvimport/csvdialog.h 69cca6e > kmymoney/plugins/csvimport/csvdialog.cpp fa70b04 > kmymoney/plugins/csvimport/csvwizard.h 28eea62 > kmymoney/plugins/csvimport/csvwizard.cpp 7aee196 > kmymoney/plugins/csvimport/investprocessing.h 38f622c > kmymoney/plugins/csvimport/investprocessing.cpp 328a79b > > Diff: https://git.reviewboard.kde.org/r/128511/diff/ > > > Testing > ------- > > Banking and investment statement CSV imports; with and without setup. > > > Thanks, > > Łukasz Wojniłowicz > >
