[Pager] Fix switching pages on
ClosedPublic

Authored by broulik on Jan 7 2020, 9:17 AM.

Details

Reviewers
hein
mart
Group Reviewers
Plasma
Summary

The code was clearly intending to use the index, as indicated by a comparison with currentPage (which is an int) and changePage taking an int.

BUG: 415423
FIXED-IN: 5.17.5

Test Plan
  • Dragged a file onto pager, waited for desktop switch, plasma no longer crashed
  • Activity pager also works fine the same way

Diff Detail

Repository
R119 Plasma Desktop
Lint
Lint Skipped
Unit
Unit Tests Skipped
broulik created this revision.Jan 7 2020, 9:17 AM
Restricted Application added a project: Plasma. · View Herald TranscriptJan 7 2020, 9:17 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Jan 7 2020, 9:17 AM
mart accepted this revision.Jan 7 2020, 9:27 AM
This revision is now accepted and ready to land.Jan 7 2020, 9:27 AM