Skip to content
Commit 84765ba6 authored by Aleix Pol Gonzalez's avatar Aleix Pol Gonzalez 🐧
Browse files

Notifier: don't show update notifications too often 

BUG: 416193
BUG: 436927
parent db089e0d
Hide whitespace changes
Inline Side-by-side
  • Adam Williamson @adamwill ·
    Contributor

    Hum. I have questions about this. It seems to only actually set LastNotificationTime at the end, after using it in the timeSinceLastRefresh calculation and returning if the result of that calculation is invalid. Does this not give us a catch-22 where LastNotificationTime will never be set, because we'll never reach the point where it's set because we first fail because it isn't set? Or will the result of the first timeSinceLastRefresh calculation run on a given user profile be 'valid' and cause us to get to the end of the function?

  • Aleix Pol Gonzalez 🐧 @apol ·
    Author Developer

    Good catch! !404 (merged)

0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment