AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[QtSql]QIBASE]add support for prepared queries in numRowsAffected

Browse Source 
Prepared queries need to be handled as procedures.

Change-Id: I4b4d7743fe45c416ca83ceb942aa48d79731f2f8
Reviewed-by: Mark Brand <mabrand@mabrand.nl>
This commit is contained in:
Israel Lins Albuquerque 2014-03-26 12:45:41 -03:00 committed by The Qt Project
parent d432091622
commit 075c876f9a
2 changed files with 24 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/sql/drivers/ibase/qsql_ibase.cpp
View File

@ -1313,6 +1313,7 @@ int QIBaseResult::numRowsAffected()
{
static char acCountInfo[] = {isc_info_sql_records};
char cCountType;
bool bIsProcedure = false;
switch (d->queryType) {
case isc_info_sql_stmt_select:
@ -1327,6 +1328,9 @@ int QIBaseResult::numRowsAffected()
case isc_info_sql_stmt_insert:
cCountType = isc_info_req_insert_count;
break;
case isc_info_sql_stmt_exec_procedure:
bIsProcedure = true; // will sum all changes
break;
default:
qWarning() << "numRowsAffected: Unknown statement type (" << d->queryType << ")";
return -1;
@ -1344,8 +1348,14 @@ int QIBaseResult::numRowsAffected()
pcBuf += 2;
int iValue = isc_vax_integer (pcBuf, sLength);
pcBuf += sLength;
if (cType == cCountType) {
if (bIsProcedure) {
if (cType == isc_info_req_insert_count || cType == isc_info_req_update_count
|| cType == isc_info_req_delete_count) {
if (iResult == -1)
iResult = 0;
iResult += iValue;
}
} else if (cType == cCountType) {
iResult = iValue;
break;
}

12
tests/auto/sql/kernel/qsqlquery/tst_qsqlquery.cpp
View File

@ -1069,6 +1069,18 @@ void tst_QSqlQuery::numRowsAffected()
QCOMPARE( q.numRowsAffected(), i );
QCOMPARE( q.numRowsAffected(), i ); // yes, we check twice
QVERIFY_SQL( q, prepare( "update " + qtest + " set id = id + :newid" ) );
q.bindValue(":newid", 100);
QVERIFY_SQL( q, exec() );
QCOMPARE( q.numRowsAffected(), i );
QCOMPARE( q.numRowsAffected(), i ); // yes, we check twice
QVERIFY_SQL( q, prepare( "update " + qtest + " set id = id + :newid where NOT(1 = 1)" ) );
q.bindValue(":newid", 100);
QVERIFY_SQL( q, exec() );
QCOMPARE( q.numRowsAffected(), 0 );
QCOMPARE( q.numRowsAffected(), 0 ); // yes, we check twice
QVERIFY_SQL( q, exec( "insert into " + qtest + " values (42000, 'homer', 'marge')" ) );
QCOMPARE( q.numRowsAffected(), 1 );
QCOMPARE( q.numRowsAffected(), 1 ); // yes, we check twice