Changing code to load bank data from SQLite DB
Ralf Habacker
ralf.habacker at freenet.de
Sat Jun 13 21:02:35 BST 2020
Am 13.06.20 um 06:52 schrieb Prasun Kumar:
> Hi mentors,
> I have tested the SQLite database to implement
> AccountNumberCheck::findbank() and it worked successfully. Now I'm
> ready to change the present code.
> So how should I make these commits? Should I replace the current code
> by commenting them out
No need for that as changes are recorded in git log
or should I remove them altogether?
yes, see above
> Also, there are many methods which deal solely with textfile reading
> such as dataValidStartDate(), dataValidEndDate(), isDataValidForDate(),
> find_closest_datafile to name a few.
> bool isDataValidForDate(std::time_t date) const;
>
> /** Returns true if a bankdata file valid for the given date is
> * available, otherwise false.
> *
> * Whether this file has already been loaded has to be queried
> * separately through isDataValidForDate().
> *
> * This function was introduced in ktoblzcheck-1.15.
*/
This and the other methods are part of the public api and should not be
removed as clients may depends on them.
> To make the commits small, I would be making a commit for each method
> change.
yes, looks good.
Regards
Ralf
More information about the Kde-finance-apps
mailing list