From 440fb56b3095b96b08b5401568eca4c3307270a0 Mon Sep 17 00:00:00 2001 From: Bea Lam Date: Mon, 29 Aug 2011 13:58:05 +1000 Subject: Fix conversion of QAbstractItemModel::rowsMoved() parameters VisualDataModel was emitting itemsMoved() signal with incorrect "to" value when multiple items were moved forwards. (change cherry picked from b64817858484db6e7c280d41ed81d4c87dff2275) Change-Id: I684eb1716d2ad6e0f702e95d9c0cd5c6fe2c9cbb Reviewed-on: http://codereview.qt.nokia.com/3694 Reviewed-by: Bea Lam Reviewed-by: Qt Sanity Bot --- src/qtquick1/graphicsitems/qdeclarativevisualitemmodel.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/qtquick1/graphicsitems/qdeclarativevisualitemmodel.cpp') diff --git a/src/qtquick1/graphicsitems/qdeclarativevisualitemmodel.cpp b/src/qtquick1/graphicsitems/qdeclarativevisualitemmodel.cpp index 5d8a3d4b37..f76a4d9f08 100644 --- a/src/qtquick1/graphicsitems/qdeclarativevisualitemmodel.cpp +++ b/src/qtquick1/graphicsitems/qdeclarativevisualitemmodel.cpp @@ -1379,7 +1379,7 @@ void QDeclarative1VisualDataModel::_q_rowsMoved(const QModelIndex &sourceParent, Q_D(QDeclarative1VisualDataModel); const int count = sourceEnd - sourceStart + 1; if (destinationParent == d->m_root && sourceParent == d->m_root) { - _q_itemsMoved(sourceStart, sourceStart > destinationRow ? destinationRow : destinationRow-1, count); + _q_itemsMoved(sourceStart, sourceStart > destinationRow ? destinationRow : destinationRow-count, count); } else if (sourceParent == d->m_root) { _q_itemsRemoved(sourceStart, count); } else if (destinationParent == d->m_root) { -- cgit v1.2.3