D13596: Fix there being more security updates than total updates in notifier

Chris Holland noreply at phabricator.kde.org
Tue Jun 19 00:26:04 UTC 2018


Zren created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
Zren requested review of this revision.

REVISION SUMMARY
  We currently read the previous `m_securityCount` instead of using the current `securityCount`.
  
  BUG: 392056
  FIXED-IN 5.13.1 (or 5.13.2?)
  FIXED-IN 5.12.6 (This should be backported right?) https://github.com/KDE/discover/blame/Plasma/5.12/notifier/DiscoverNotifier.cpp#L82
  
  The bug report is from 5.12.3, and was also reported on reddit today. Both have a screenshot of the bug.
  https://bugs.kde.org/show_bug.cgi?id=392056
  https://www.reddit.com/r/kde/comments/8rzrr6/so_whats_the_logic_behind_updater/
  
  We could also start the counter at 0 and add `securityCount` afterward.
  
    -    uint count = securityUpdatesCount();
    +    uint count = 0;
         foreach(BackendNotifierModule* module, m_backends)
             count += module->updatesCount();
    +    count += securityCount;

TEST PLAN
  I haven't attempted to compile this or test it. I don't have any security updates atm anyways.
  I haven't pushed a bugfix to a `Plasma/5.__` branch before either. Do I make a separate commit for 5.12, 5.13, and master?

REPOSITORY
  R134 Discover Software Store

REVISION DETAIL
  https://phabricator.kde.org/D13596

AFFECTED FILES
  notifier/DiscoverNotifier.cpp

To: Zren
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180619/112cb751/attachment.html>


More information about the Plasma-devel mailing list