QSortFilterProxyModel trouble when inserting row

[saknopper]

Hi all,

I'm having trouble using a proxy model in a tableview, everything was just fine when is used the qsqltablemodel directly. Though, using a proxy model, there are some issues.

When I'm adding a row (code below) and edit the row which just has been added, the proxy model takes care of sorting the items again. This causes an empty row to appear, at the position at which the row was initially inserted.

When I restart the application or manually set the model again in the tableview, it shows the items correctly again.

int row = 0;
model->insertRow(row);
QModelIndex index = model->index(row, 1);
 
countriesView->setCurrentIndex(index);
countriesView->edit(index);

int row = 0;
model->insertRow(row);
QModelIndex index = model->index(row, 1);

countriesView->setCurrentIndex(index);
countriesView->edit(index);

To copy to clipboard, switch view to plain text mode 

model = QSortFilterProxyModel
countriesView = QTableView

Like I said, it used to work perfectly when model is an QSqlTableModel.

[jpn]

Sounds like the dynamic sorting causes that the edited model index is actually something different than the inserted one. I think you should add the row into the actual model and then use QAbstractProxyModel::mapFromSource() to get the correct model index.

[saknopper]

Thanks for your suggestion jpn, though it didn't work out as expected unfortunately.

I currently use the following code instead of the code above to create a new row:

int row = 0;
model->sourceModel()->insertRow(row);
QModelIndex index = model->sourceModel()->index(row, 1);

int row = 0;
model->sourceModel()->insertRow(row);
QModelIndex index = model->sourceModel()->index(row, 1);

To copy to clipboard, switch view to plain text mode 

As you can see, I don't call edit() to enable the user to edit the field right away. So after adding a new row an empty row appears in the tableview, which can be edited when you double click on it. This itself works flawless, though when you're done editing and press enter to save the value it still adds an empty row at the top of the tableview. The table view is sorted correctly though.

I hope you haven't run out of suggestions yet...

[jpn]

Does this work as desired? Which exact version of Qt are you using?

PS. Rows and columns can be added through the context menu..

[saknopper]

Yes, it works perfectly.

I'm using Qt 4.2.2

[saknopper]

It's weird that when I calculate the position of the to be inserted row with model->rowCount() and then call insertRow() at that position. It still adds the row at the top of the table. While in your example in inserts the row at the bottom.

[jpn]

Hmm, so maybe it happens only when using QSortFilterProxyModel + QSqlTableModel?

[saknopper]

Perhaps... If that would be true, is there a way I can work around this problem that you know of?

[jpn]

Could you, by any chance, prepare a minimal example reproducing the problem we could play with? You could for example modify the earlier example of mine to use an SQLite db or so..

[saknopper]

Sure, I'll do that.

[saknopper]

I made an example with an in memory sqlite database for you, it illustrates the problem very clearly (in my opinion).

Some steps to reproduce the problem:

1) Sort on country (instead of id)
2) Add a new row
3) Enter for example "Italy" in the textbox
4) Press enter

The view resorts itself, but there's an empty row at the top.

Let me know if I can help you any further.

Thanks for looking into it.

[wysota]

Looks like the edit call causes the malfunction, probably because the "id" field is set to "0" instead of the proper index.

I changed the addRow() method and it works fine now:

void Table::addRow()
{
        int row = proxyModel->sourceModel()->rowCount();
        proxyModel->sourceModel()->insertRow(row);
        QModelIndex idx = proxyModel->sourceModel()->index(row, 1);
        if (idx.isValid())
        {
                proxyModel->sourceModel()->setData(idx.sibling(idx.row(), 0), row); // added here
                idx = proxyModel->mapFromSource(idx);
                setCurrentIndex(idx);
                edit(idx);
        }
}

void Table::addRow()
{
        int row = proxyModel->sourceModel()->rowCount();
        proxyModel->sourceModel()->insertRow(row);
        QModelIndex idx = proxyModel->sourceModel()->index(row, 1);
        if (idx.isValid())
        {
                proxyModel->sourceModel()->setData(idx.sibling(idx.row(), 0), row); // added here
                idx = proxyModel->mapFromSource(idx);
                setCurrentIndex(idx);
                edit(idx);
        }
}

To copy to clipboard, switch view to plain text mode 

[wysota]

Can you provide a minimal compilable example reproducing the problem? I'd like to take a look...