From ec21879cebda856ecb862bc78281538db6705d31 Mon Sep 17 00:00:00 2001
From: Michal Rentka <michalrentka@gmail.com>
Date: Thu, 21 Nov 2024 10:58:10 +0100
Subject: [PATCH] CR fixes

---
 Zotero/Controllers/IdleTimerController.swift | 21 +++++++++++---------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/Zotero/Controllers/IdleTimerController.swift b/Zotero/Controllers/IdleTimerController.swift
index ebd2a0b3b..1bf311901 100644
--- a/Zotero/Controllers/IdleTimerController.swift
+++ b/Zotero/Controllers/IdleTimerController.swift
@@ -13,7 +13,7 @@ import CocoaLumberjackSwift
 import RxSwift
 
 final class IdleTimerController {
-    private static let customIdleTimerTimemout = 1200
+    private static let customIdleTimerTimeout: DispatchTimeInterval = .seconds(1200)
     private let disposeBag: DisposeBag
     /// Processes which require idle timer disabled
     private var activeProcesses: Int = 0
@@ -40,7 +40,7 @@ final class IdleTimerController {
             guard let activeTimer = self?.activeTimer else { return }
             DDLogInfo("IdleTimerController: reset idle timer")
             activeTimer.suspend()
-            activeTimer.schedule(deadline: .now() + DispatchTimeInterval.seconds(Self.customIdleTimerTimemout))
+            activeTimer.schedule(deadline: .now() + Self.customIdleTimerTimeout)
             activeTimer.resume()
         }
     }
@@ -58,7 +58,7 @@ final class IdleTimerController {
 
         func startTimer(controller: IdleTimerController) {
             let timer = DispatchSource.makeTimerSource(flags: [], queue: .main)
-            timer.schedule(deadline: .now() + DispatchTimeInterval.seconds(Self.customIdleTimerTimemout))
+            timer.schedule(deadline: .now() + Self.customIdleTimerTimeout)
             timer.setEventHandler(handler: { [weak controller] in
                 controller?.forceStopIdleTimer()
             })
@@ -69,15 +69,18 @@ final class IdleTimerController {
 
     func stopCustomIdleTimer() {
         inMainThread { [weak self] in
-            guard let self, activeProcesses > 0 else {
+            guard let self else { return }
+
+            if activeProcesses > 0 {
+                activeProcesses -= 1
+                DDLogInfo("IdleTimerController: enable idle timer \(activeProcesses)")
+            } else {
                 DDLogWarn("IdleTimerController: tried to enable idle timer with no active processes")
-                return
+                activeProcesses = 0
             }
-            activeProcesses -= 1
-
-            DDLogInfo("IdleTimerController: enable idle timer \(activeProcesses)")
 
             guard activeProcesses == 0 else { return }
+
             set(disabled: false)
             activeTimer?.suspend()
             activeTimer?.setEventHandler(handler: nil)
@@ -97,7 +100,7 @@ final class IdleTimerController {
 
     private func forceStopIdleTimer() {
         DDLogInfo("IdleTimerController: force stop timer")
-        activeProcesses = 0
+        activeProcesses = 1
         stopCustomIdleTimer()
     }
 }