diff options
| author | Mark Brand <mabrand@mabrand.nl> | 2012-02-06 17:19:19 +0100 |
|---|---|---|
| committer | Qt by Nokia <qt-info@nokia.com> | 2012-02-07 22:05:51 +0100 |
| commit | 269ef14215d3c3f89fe6239879a5db111dc1a7b4 (patch) | |
| tree | 0825c2a4ba8d39b8d12b40f933147e44365e8135 /dist | |
| parent | f5e1da12f0e7bdeee4db74acc52dfabeb12a4e31 (diff) | |
QSqlTableModel::removeRows(): require valid full range of rows
If an invalid range of rows is specified, it's likely to be a
programming or user error. The old behavior of ignoring out of range
rows seems dangerous and complicates the code.
Also implement the documented behavior of returning false if
changes are unsuccessful for OnFieldChange and OnRowChange.
Previously the return value of submit() was ignored.
Updated and improved documentation.
Change-Id: Iaaf51c6d9a0c8c06fd5d186b4b88358fbeab9936
Reviewed-by: Yunqiao Yin <charles.yin@nokia.com>
Diffstat (limited to 'dist')
| -rw-r--r-- | dist/changes-5.0.0 | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/dist/changes-5.0.0 b/dist/changes-5.0.0 index 6bacccc34a..5317137059 100644 --- a/dist/changes-5.0.0 +++ b/dist/changes-5.0.0 @@ -331,6 +331,11 @@ QTestLib * removeRows() no longer emits extra beforeDelete signal for out of range row. +* removeRows() now requires the whole range of targetted rows to be valid +before doing anything. Previously, it would remove what it could and +ignore the rest of the range. + + **************************************************************************** * Database Drivers * **************************************************************************** |