8.0 bug, clearPersistRows!!!

As one of the differences between QTitanDataGrid 8.0 Public Beta and Release versions, the clearPersistRows function is called inside void GridModelController::modelReset(). However, this results in updating the model contents and calling

beginResetModel();
qSwap( VecFilesEx, vecFiles_ );
endResetModel();

calls, and after that, some features don't work.

The features that don't work are mouse click selections and keyboard navigation.

What is the role of the function that caused the problem, and why did you call it at that point?

P.S: Do you test and release before you release, I always get default functionality not working.

We cannot comment on the internals of the source code here. To solve the problem, would it be better to add a description of the bug and how it can be reproduced?

calls, and after that, some features don't work.

In general, calling modelReset completely resets the persistence rows configuration, that is, information about selection and the focus row is lost. In fact, we are talking about installing a new model into the grid.

To reiterate,
I call modelReset when I want to refresh the data inside the model. For example, I have a class that inherits from QAbstractItemModel. This class contains the contents of some directory. To refresh the contents of this directory, the data inside , is changed by calling the following code.

beginResetModel();
qSwap( VecFilesEx, vecFiles_ ); c.
endResetModel();
// vecFiles_ is a vector inside the model.
// VecFilesEx is the newly acquired information.

From now on, you won't be able to move items with the keyboard, and you won't be able to select items with a mouse click. However, it is possible to select rows by dragging (when it is a MultiRowSelection).

However, clickEvent, dobuleClick events, etc. will still come in and work as normal.

Thank you so much for your bug-report. To fix the issue please replace the line:

GridModelController::modelReset()
{
....
clearPersistRows();

to

for (QSet<GridPersistentRow*>::const_iterator it = m_persistRows.constBegin(); it != m_persistRows.constEnd(); ++it)
(*it)->clear();
....
}

We will add the bugfix into the 8.1.0. I apologize for the any inconvenience caused.