Review Request 121655: Add overloads (non-raw queries and params) for convenience KexiDB functions

Jarosław Staniek staniek at kde.org
Wed Dec 31 09:38:57 GMT 2014


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

(Updated Dec. 31, 2014, 9:38 a.m.)


Status
------

This change has been marked as submitted.


Review request for Calligra, Andrius da Costa Ribas, Adam Pigg, Roman Shtemberko, Radosław Wicik, and Wojciech Kosowicz.


Repository: calligra


Description
-------

Add overloads (non-raw queries and params) for convenience KexiDB functions

For:
- Utils::rowCount()
- Connection::querySingleRecord()
- Connection::querySingleString()
- Connection::querySingleNumber()
- Connection::queryStringList()

By the way, remove unsafe use of static variables in querySingleNumber().

Needed to fix the "Can't export data for parametrized query to a .csv file" bug (151504)
see https://git.reviewboard.kde.org/r/121443/


Diffs
-----

  libs/db/connection.h 0ed171d857b93060398787226698b3c67758b024 
  libs/db/connection.cpp c422f78ca0f6c8a5f767e6892d0468821cccc567 
  libs/db/utils.h 9bea691a4af95447983ddc0f093c1ae4b4aeb7e6 
  libs/db/utils.cpp 923a401f7f4265d19c699db3d35046f21914747f 

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


Testing
-------

Tested with large .kexi files; these methods are used there and Kexi still works OK


Thanks,

Jarosław Staniek

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20141231/ffd86963/attachment.htm>


More information about the calligra-devel mailing list