Merge pull request #201 from mattrubin/prevent-haptic-overload

Prevent haptic overload

Author: mattrubin

Authenticator/Source/AppController.swift

@@ -166,21 +166,11 @@ class AppController {
 
         case let .showErrorMessage(message):
             SVProgressHUD.showError(withStatus: message)
-
-            // Provide haptic feedback
-            if #available(iOS 10.0, *) {
-                let feedbackGenerator = UINotificationFeedbackGenerator()
-                feedbackGenerator.notificationOccurred(.error)
-            }
+            generateHapticFeedback(for: .error)
 
         case let .showSuccessMessage(message):
             SVProgressHUD.showSuccess(withStatus: message)
-
-            // Provide haptic feedback
-            if #available(iOS 10.0, *) {
-                let feedbackGenerator = UINotificationFeedbackGenerator()
-                feedbackGenerator.notificationOccurred(.success)
-            }
+            generateHapticFeedback(for: .success)
 
         case .showApplicationSettings:
             guard let applicationSettingsURL = URL(string: UIApplicationOpenSettingsURLString) else {
@@ -208,6 +198,13 @@ class AppController {
         return topViewController(presentedFrom: presentedViewController)
     }
 
+    private func generateHapticFeedback(for notificationFeedbackType: UINotificationFeedbackType) {
+        if #available(iOS 10.0, *) {
+            let feedbackGenerator = UINotificationFeedbackGenerator()
+            feedbackGenerator.notificationOccurred(notificationFeedbackType)
+        }
+    }
+
     // MARK: - Public
 
     var rootViewController: UIViewController {

Authenticator/Source/TokenScannerViewController.swift

@@ -184,10 +184,18 @@ final class TokenScannerViewController: UIViewController, QRScannerDelegate {
 
     // MARK: QRScannerDelegate
 
+    private var lastScanTime = Date(timeIntervalSince1970: 0)
+    private let minimumScanInterval: TimeInterval = 1
+
     func handleDecodedText(_ text: String) {
         guard viewModel.isScanning else {
             return
         }
-        dispatchAction(.scannerDecodedText(text))
+
+        let now = Date()
+        if now.timeIntervalSince(lastScanTime) > minimumScanInterval {
+            lastScanTime = now
+            dispatchAction(.scannerDecodedText(text))
+        }
     }
 }