Fix QuickOpen with Qt 5.11 (bug 395039)

Authored by brauch on Jun 13 2018, 2:33 PM.



Apparently the item model changes in Qt 5.11 cause our quickopen list to be empty.

I do not know why the old code does not work, to be honest, to me it seems okay. The only thing which is definitely wrong is the call to mapFromSource; the index passed to this belongs to the model which has since been replaced, and not to the actual model attached to the proxy model. That issue is unrelated though, commenting out that code (and thus removing the feature) doesn't fix it. With the suggested patch it's not an issue any more either because we don't change the model.

I like the suggested implementation better anyways, so I think we should change this in either case, but does anyone have an idea why the old code is broken? Is that maybe even a regression in Qt itself? @mwolff?

Diff Detail

R40 Kate
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
brauch created this revision.Jun 13 2018, 2:33 PM
Restricted Application added a project: Kate. · View Herald TranscriptJun 13 2018, 2:33 PM
Restricted Application added a subscriber: kwrite-devel. · View Herald Transcript
brauch requested review of this revision.Jun 13 2018, 2:33 PM
brauch edited the summary of this revision. (Show Details)Jun 13 2018, 2:38 PM
mwolff accepted this revision.Jun 13 2018, 3:00 PM

no clue, I wouldn't be surprised by bugs/regressions in Qt - I've seen a lot of breakage in item/view code recently

This revision is now accepted and ready to land.Jun 13 2018, 3:00 PM
cullmann accepted this revision.Jun 13 2018, 3:04 PM

If that change works in old Qt and new, I am happy with it.

I'll dig around a bit and see if I can find the change in Qt causing this. In the meantime, do you want this in applications/18.04 or master only?

Looks like it was already fixed for Qt 5.11.1, e15fc26e9fdbff141890a3e2e8dc4ef935d022a0 sounds a lot like it:

commit e15fc26e9fdbff141890a3e2e8dc4ef935d022a0
Author: Christian Ehrlicher <>
Date: Fri May 25 21:06:43 2018 +0200

QSortFilterProxyModel: don't assert when old model gets destroyed

When a new model was set with setSourceModel() and the mapping was
built up, the destruction of the old model caused a reset in the
QSortFilterProxyModel which lead to an empty view or an assertion.
Now we properly disconnect the old model again and also clean up the old
mapping/persistent indexes when a new source model is set.

Task-number: QTBUG-44962
Task-number: QTBUG-67948
Task-number: QTBUG-68427
Change-Id: I2e0612899c210bde3ac0cfa59aefd78269deee5b
Reviewed-by: David Faure <>

That also changes the code I think is relevant for this bug, i.e. QSortFilterProxyModel's setSourceModel() method.

This revision was automatically updated to reflect the committed changes.

Hmm, we want to have it in master, too, or?

Yes sorry, I felt too tired yesterday to correctly resolve the version number merge conflict ;) Done now.

Nothing to be sorry about ;=)
Thanks for taking care!