New method for cleaning old entries added to avoid selector problems (#…

…676)
duckduckgo · Aug 8, 2022 · b4231de · b4231de
1 parent 749386d
commit b4231de
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/DuckDuckGo/History/Model/HistoryCoordinator.swift b/DuckDuckGo/History/Model/HistoryCoordinator.swift
@@ -208,8 +208,14 @@ final class HistoryCoordinator: HistoryCoordinating {
         }
     }
 
-    @objc private func cleanOldAndLoad(completionHandler: ((Error?) -> Void)? = nil) {
-        clean(until: .monthAgo, completionHandler: completionHandler)
+    var cleaningDate: Date { .monthAgo }
+
+    @objc private func cleanOld() {
+        clean(until: cleaningDate)
+    }
+
+    private func cleanOldAndLoad(completionHandler: ((Error?) -> Void)?) {
+        clean(until: cleaningDate, completionHandler: completionHandler)
     }
 
     private func clean(until date: Date,
@@ -310,7 +316,7 @@ final class HistoryCoordinator: HistoryCoordinating {
         let timer = Timer(fireAt: .startOfDayTomorrow,
                           interval: .day,
                           target: self,
-                          selector: #selector(cleanOldAndLoad),
+                          selector: #selector(cleanOld),
                           userInfo: nil,
                           repeats: true)
         RunLoop.main.add(timer, forMode: RunLoop.Mode.common)