Move items in a QSortFilterProxyModel

[JonB]

@Valerian
Fair enough. Then I can only suggest you read that doc page carefully to understand what they are saying is required in each direction, and play with the parameters/which row/how far you move the row till you see something happening, and figure from there.

[VRonin]

• moveRow must already call beginMoveRows and endMoveRows inside itself so you are duplicating calls
• moveRow is not implemented in any of Qt's model (as far as I'm aware) so it will probably just do nothing and return false (check the return value)

What model are you using as source to this proxy?

[Valerian]

@VRonin said in Move items in a QSortFilterProxyModel:

What model are you using as source to this proxy?

QAbstractTableModel

[JonB]

@VRonin
Don't forget OP is claiming that one of his moveUp/moveDown does work, while the other does not....

moveRow must already call beginMoveRows and endMoveRows inside itself so you are duplicating calls

If that's true that might explain....!

[VRonin]

@Valerian said in Move items in a QSortFilterProxyModel:

QAbstractTableModel

Did you reimplement moveRow?

[Valerian]

@VRonin I re-implemeted it in the QSortFilterProxyModel. Didn't want to move rows in the main model

[VRonin]

Can you post the implementation?

[JonB]

@Valerian
I now realise you got your code from https://stackoverflow.com/a/43683619/489865, didn't you? Wasn't actually confirmed to be correct solution. I still don't get where the + 2 comes from, but there you are.

IF @VRonin does not sort you out (so to speak :) ), I don't know, but maybe you should have a careful read of https://stackoverflow.com/a/12254935/489865. Not sure whether the following might affect you:

(2) Yes if you don't use sorting, just filtering. Qt in such case erases entire internal mapping and re-builds it from scratch so the data will be just as they appear in source model

[Valerian]

@VRonin said in Move items in a QSortFilterProxyModel:

Can you post the implementation?

I have already posted the implementation of the QSortFilterProxyModel. Are you seeking the implementation of the QAbstractTableModel?

[VRonin]

@VRonin said in Move items in a QSortFilterProxyModel:

Did you reimplement moveRow?

@Valerian said in Move items in a QSortFilterProxyModel:

I re-implemeted it in the QSortFilterProxyModel.

I don't see an implementation of FilterProxyModel::moveRow (or rather FilterProxyModel::moveRows) above

[Valerian]

@JonB said in Move items in a QSortFilterProxyModel:

I now realise you got your code from https://stackoverflow.com/a/43683619/489865, didn't you? Wasn't actually confirmed to be correct solution. I still don't get where the + 2 comes from, but there you are.

Yes I did pick it from there.

Thanks for the other stackflow link. Will have a read on that.

[Valerian]

@VRonin said in Move items in a QSortFilterProxyModel:

I don't see an implementation of FilterProxyModel::moveRow (or rather FilterProxyModel::moveRows) above

I haven't implement that. Not sure if I need to write that

[VRonin]

@VRonin said in Move items in a QSortFilterProxyModel:

Did you reimplement moveRow?

@Valerian said in Move items in a QSortFilterProxyModel:

I re-implemeted it in the QSortFilterProxyModel

As I mentioned, the default implementation does nothing and returns false so calling it is pointless unless you reimplement it

[JonB]

@VRonin [I don't get what's happening in the pic? :( ]

[VRonin]

@JonB https://www.youtube.com/watch?v=xXdD9yFZGv8

[JonB]

@VRonin
Dear Mr VRonin,

Thank you for that link. I opened it from Firefox from the Ubuntu I sit inside as a VirtualBox under Windows Server.

The video started, and then froze. My whole Linux guest box froze. Nothing other than a Windows TaskMan kill of the whole VBox was possible, killing my session and any work I had done but not committed!

Please remind me never to risk clicking on such a link again! :) I shall have to manage without knowing the meaning of the pic now...

[VRonin]

@JonB Sorry mate, it was just a stupid YouTube video of a guy saying carrots are good for eyesight, another guy shoving 2 carrots in his eyes and then the image above

[Valerian]

@VRonin Thank you for your feedback. Will re-implement those functions and try

[JonB]

@VRonin Ohhhhh, I would never have figured that's what it was! All OK :)

[fanyha]

hi, can anybody tell me how to implement moveRows() for the class derived from QSortFilterProxyModel:
moveRows(const QModelIndex &sourceParent, int sourceRow, int count, const QModelIndex &destinationParent, int destinationChild)

it looks like we need to exchange the proxy indexes, but we can't access to the private members in the subclass