[Notifications] Fixup setSourceModel
ClosedPublic

Authored by broulik on Thu, Jan 30, 2:58 PM.

Details

Reviewers
None
Group Reviewers
Plasma
Commits
R120:1c0c8ddef430: [Notifications] Fixup setSourceModel
Summary

Qt says to do a proper model reset in this method.
I also noticed something funky going on with the LimitedRowCountProxyModel not showing any data which went away with this.

Test Plan

5.18

Diff Detail

Repository
R120 Plasma Workspace
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Thu, Jan 30, 2:58 PM
Restricted Application added a project: Plasma. · View Herald TranscriptThu, Jan 30, 2:58 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Thu, Jan 30, 2:58 PM

QAbstractSortFilterProxyModel has docs that say you need it, but QSortFilterProxyModel which you inherit from does reset the model in setSourceModel.

This revision was not accepted when it landed; it landed in state Needs Review.Tue, Feb 4, 8:51 AM
This revision was automatically updated to reflect the committed changes.