maxsoch/BrewBar
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases 3 Wiki Activity

cleanup: remove dead refreshOutdatedList and unused lastOutdatedFetch

Browse source 
refreshOutdatedList (the 5-second cache path) was never called by any
menu item or code path, so the cache logic never ran. lastOutdatedFetch
was only written. cachedOutdated stays: the Upgrade All confirmation
dialog uses it to list pending packages.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
maxsoch 2026-07-06 21:30:12 +02:00
parent e47455b3f8
commit d1f4c0cb79
1 changed files with 1 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
BrewBar/BrewBarApp.swift
View file

@ -35,9 +35,8 @@ class AppDelegate: NSObject, NSApplicationDelegate {
}
}
// ✅ cache to prevent duplicate outdated calls
/// last known outdated list, shown in the Upgrade All confirmation
var cachedOutdated: [String] = []
var lastOutdatedFetch: Date?
// MARK: - Menu
@ -272,17 +271,6 @@ class AppDelegate: NSObject, NSApplicationDelegate {
return alert.runModal() == .alertFirstButtonReturn
}
/// ✅ SMART refresh (uses cache)
@objc func refreshOutdatedList() {
// if fetched in last 5s → uses cache
if let last = lastOutdatedFetch, Date().timeIntervalSince(last) < 5 {
updateOutdatedMenu(with: cachedOutdated)
return
}
fetchOutdated()
}
/// ✅ FORCE refresh (menu button)
@objc func forceRefreshOutdated() {
fetchOutdated()
@ -295,7 +283,6 @@ class AppDelegate: NSObject, NSApplicationDelegate {
let lines = BrewParser.parseOutdated(output)
self.cachedOutdated = lines
self.lastOutdatedFetch = Date()
self.updateOutdatedMenu(with: lines)
self.updateStatus(count: lines.count)