Fix a bug in QSqlQuery::isNull documentation

the method returns true if there is not such field.

Change-Id: I25db8de4561d3e0604f3e64edc1810140ba4aad2
Reviewed-by: Andy Shaw <andy.shaw@digia.com>
Reviewed-by: Mark Brand <mabrand@mabrand.nl>

1 files changed, 9 insertions, 8 deletions

diff --git a/src/sql/kernel/qsqlquery.cpp b/src/sql/kernel/qsqlquery.cpp
index 661a582afe..e5f4c5fc56 100644
--- a/src/sql/kernel/qsqlquery.cpp
+++ b/src/sql/kernel/qsqlquery.cpp
@@ -309,19 +309,20 @@ QSqlQuery& QSqlQuery::operator=(const QSqlQuery& other)
 }
 
 /*!
-  Returns \c true if the query is \l{isActive()}{active} and positioned
-  on a valid record and the \a field is NULL; otherwise returns
-  false. Note that for some drivers, isNull() will not return accurate
-  information until after an attempt is made to retrieve data.
+    Returns \c true if the query is not \l{isActive()}{active},
+    the query is not positioned on a valid record,
+    there is no such field, or the field is null; otherwise \c false.
+    Note that for some drivers, isNull() will not return accurate
+    information until after an attempt is made to retrieve data.
 
-  \sa isActive(), isValid(), value()
+    \sa isActive(), isValid(), value()
 */
 
 bool QSqlQuery::isNull(int field) const
 {
-    if (d->sqlResult->isActive() && d->sqlResult->isValid())
-        return d->sqlResult->isNull(field);
-    return true;
+    return !d->sqlResult->isActive()
+             || !d->sqlResult->isValid()
+             || d->sqlResult->isNull(field);
 }
 
 /*!