[Kmymoney-devel] Review Request 122138: [GCI2014] Find the maximum in a set of numeric strings.

[Daniel Ring]

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

(Updated Jan. 24, 2015, 1:14 a.m.)


Review request for KMymoney and Christian David.


Changes
-------

I moved the SQL query generation to a set of functions in the `MyMoneyDbDriver` class and modified `highestIdNum` to call them rather than building the query itself. I implemented the query for Sqlite3 (the default), MySQL, PostgreSQL, and Oracle. The Sqlite3 implementation should work in SqlCipher3 without modification. I noticed there were also subclasses for DB2, ODBC, Interbase, and Sybase, but they didn't appear to be fully implemented.  Are the other RDBMSs not fully supported, or are they simply compatible with the default implemetations in most cases? If the latter, should I test the query in each of them as well?


Bugs: 339103
    http://bugs.kde.org/show_bug.cgi?id=339103


Repository: kmymoney


Description
-------

This adds a function to retrieve the highest ID number from the database. The IDs are prefixed by a short identifier string, and are stored in the database as strings.

This review request was created as a part of Google Code-In 2014. The specific task it refers to can be found here: https://www.google-melange.com/gci/task/view/google/gci2014/5774859095244800


Diffs (updated)
-----

  kmymoney/mymoney/storage/mymoneydatabasemgrtest.h 07fc175cd9381ef0fc7ad314cf905c22a43fca81 
  kmymoney/mymoney/storage/mymoneydatabasemgrtest.cpp 642fc53d9e220f34e463b2a27ab9a5d4b0c1e3ec 
  kmymoney/mymoney/storage/mymoneydbdriver.h e5fe0e9a0058dac1573b6b08087c93c74a06d9a6 
  kmymoney/mymoney/storage/mymoneydbdriver.cpp ab7449238bd55e3f4dc543790cd0f38eaa59d787 
  kmymoney/mymoney/storage/mymoneystoragesql.h 469c0f47d61df783f48787660f526fccdf5264a4 
  kmymoney/mymoney/storage/mymoneystoragesql.cpp fa9fbdf7d77eaa4922302d1f583317212af45ddf 

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


Testing
-------

I created a small sample database with SQLite and verified that the SQL statment worked properly. I was able to successfully build and run KMyMoney with this function included in Debian Stable (using cmake 2.8.11.1 from backports). However, I was unsure how best to test the function itself, so please let me know if any issues arise.


Thanks,

Daniel Ring

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20150124/684455a2/attachment.html>