BookmarkModel::insert method should not advertise it is inserting.

This method calls QSqlTableModel::insertRecord which goes on to
call beginInsertRows.

In practice this fix prevents BookmarkModel from emitting two
countChanged() signals when inserting bookmarks.

Reviewed-by: Hugo Parente Lima

1 files changed, 0 insertions, 3 deletions

diff --git a/src/core/BookmarkModel.cpp b/src/core/BookmarkModel.cpp
index 0d271fe..9726902 100644
--- a/src/core/BookmarkModel.cpp
+++ b/src/core/BookmarkModel.cpp
@@ -62,10 +62,7 @@ void BookmarkModel::insert(const QString& name, const QString& url)
     record.setValue(QLatin1String("url"), url);
     record.setValue(QLatin1String("dateAdded"), QDateTime::currentDateTime().toTime_t());
 
-    QModelIndex index = QModelIndex();
-    beginInsertRows(index, rowCount(index), rowCount(index));
     insertRecord(-1, record);
-    endInsertRows();
     submitAll();
 }