Differential D15682

Fix showing update notification
ClosedPublic
Actions

Authored by broulik on Sep 21 2018, 3:12 PM.

Details

Reviewers
apol
Group Reviewers
Plasma
Commits
R134:8b2cf5d81a61: Fix showing update notification
Summary

It would check whether the old count was greater than or equal to the new count which is the wrong way round and as such the notification was never shown.

BUG: 396426
BUG: 390322

Test Plan

5.12 branch

Started plasmashell, got a notification about how many updates are waiting to be installed

Diff Detail

Repository
R134 Discover Software Store
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.Sep 21 2018, 3:12 PM
Restricted Application added a project: Plasma. · View Herald TranscriptSep 21 2018, 3:12 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
broulik requested review of this revision.Sep 21 2018, 3:12 PM
apol accepted this revision.Sep 23 2018, 10:14 PM
This revision is now accepted and ready to land.Sep 23 2018, 10:14 PM
apol edited the summary of this revision. (Show Details)Sep 23 2018, 10:15 PM
Closed by commit R134:8b2cf5d81a61: Fix showing update notification (authored by broulik). · Explain WhySep 24 2018, 6:35 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mnotifier/DiscoverNotifier.cpp (2 lines)

Diff 42215

View Options

notifier/DiscoverNotifier.cpp

Loading...