QSqlTableModel::setRecord(): do not try to detect value changes
authorMark Brand <mabrand@mabrand.nl>
Mon, 6 Feb 2012 14:03:25 +0000 (15:03 +0100)
committerQt by Nokia <qt-info@nokia.com>
Wed, 8 Feb 2012 13:43:18 +0000 (14:43 +0100)
commit5953304bc5f50eca77f78fd0623479de6bebacbe
treef410b506565b85a03c81c40c0441328b2b73972f
parentf5e58a1f69e5d17369198bbf1e7bb8946e3c7b5d
QSqlTableModel::setRecord(): do not try to detect value changes

In an apparent attempt to be economical with emitting dataChanged()
and submitting SQL to the databse, setRecord() compares each field
value of the record with the old value, taking action only when
a difference is detected.  Several complaints against this code are:

-The comparision does not work on float type.

-It is really up to the application and database to decide this. The
model should make few assumptions. The application has the option to
omit fields from the record that should be ignored.

-The current behavior seems to assume that the "old" values are the
current state of the database, but the database may have changed since
the model was last refreshed.

-The code compares the value from record(), which probably
corresponds to the EditRole, with the DisplayRole value from data().

Change-Id: I11477c185eb411d442144dc682893d0df12d03d5
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
src/sql/models/qsqltablemodel.cpp
tests/auto/sql/models/qsqltablemodel/tst_qsqltablemodel.cpp