Fix bad model hygiene in Positioner::move()

Authored by hein on Jul 20 2018, 9:04 AM.


Fix bad model hygiene in Positioner::move()

Don't change the proxy maps prior to insert/remove transactions so
rowCount() at transaction start time matches the count passed into
the method. The old code hits an ASSERT in Qt 5.11 otherwise.

Care is taken not to emit dataChanged() in the middle of another

Also cleaned up vestiges of caching lastRow(): This cache was
evicted all over the places but never actually filled anymore, so
maybe we don't need it.


Reviewers: davidedmundson

Subscribers: plasma-devel

Tags: Plasma

Differential Revision: