chore: Add matomo optout

Infomaniak · Feb 25, 2025 · e943af4 · e943af4
1 parent d7bbef8
commit e943af4
Show file tree

Hide file tree

Showing 4 changed files with 33 additions and 7 deletions.
diff --git a/Project.swift b/Project.swift
@@ -65,6 +65,7 @@ let project = Project(name: "kDrive",
                                       .external(name: "Realm"),
                                       .external(name: "MyKSuite"),
                                       .external(name: "InfomaniakPrivacyManagement"),
+                                      .external(name: "InfomaniakCoreCommonUI"),
                                       .sdk(name: "StoreKit", type: .framework, status: .required)
                                   ],
                                   settings: .settings(base: Constants.baseSettings),

diff --git a/kDrive/UI/Controller/Menu/AboutTableViewController.swift b/kDrive/UI/Controller/Menu/AboutTableViewController.swift
@@ -24,7 +24,7 @@ import UIKit
 
 class AboutTableViewController: BaseGroupedTableViewController {
     private enum AboutRow: CaseIterable {
-        case privacy, sourceCode, license, version, dataPrivacy
+        case privacy, dataPrivacy, sourceCode, license, version
 
         var url: URL? {
             switch self {

diff --git a/kDrive/UI/Controller/SwiftUI/AboutPrivacyViewBridgeController.swift b/kDrive/UI/Controller/SwiftUI/AboutPrivacyViewBridgeController.swift
@@ -16,22 +16,42 @@
  along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-import Foundation
+import InfomaniakCoreCommonUI
+import InfomaniakDI
 import InfomaniakPrivacyManagement
 import kDriveCore
-import UIKit
 import SwiftUI
 
 enum AboutPrivacyViewBridgeController {
     static func instantiate() -> UIViewController {
-        let swiftUIView = PrivacyManagementView(
+        let swiftUIView = AboutPrivacyView()
+        return UIHostingController(rootView: swiftUIView)
+    }
+}
+
+struct AboutPrivacyView: View {
+    @AppStorage(UserDefaults.shared.key(.matomoAuthorized)) private var matomoAuthorized = DefaultPreferences.matomoAuthorized
+
+    var body: some View {
+        PrivacyManagementView(
             urlRepository: URLConstants.sourceCode.url,
             backgroundColor: KDriveAsset.backgroundColor.swiftUIColor,
             illustration: KDriveAsset.documentSignaturePencilBulb.swiftUIImage,
             userDefaultStore: .shared,
             userDefaultKeyMatomo: UserDefaults.shared.key(.matomoAuthorized),
             userDefaultKeySentry: UserDefaults.shared.key(.sentryAuthorized)
         )
-        return UIHostingController(rootView: swiftUIView)
+        .onChange(of: matomoAuthorized) { newValue in
+            @InjectService var matomo: InfomaniakCoreCommonUI.MatomoUtils
+            #if DEBUG && !TEST
+            matomo.optOut(true)
+            #else
+            matomo.optOut(!newValue)
+            #endif
+        }
     }
 }
+
+#Preview {
+    AboutPrivacyView()
+}
diff --git a/kDriveCore/Utils/DriveUserDefaults+Extension.swift b/kDriveCore/Utils/DriveUserDefaults+Extension.swift
@@ -343,7 +343,7 @@ public extension UserDefaults {
         get {
             bool(forKey: key(.matomoAuthorized))
             if object(forKey: key(.matomoAuthorized)) == nil {
-                set(true, forKey: key(.matomoAuthorized))
+                set(DefaultPreferences.matomoAuthorized, forKey: key(.matomoAuthorized))
             }
             return bool(forKey: key(.matomoAuthorized))
         }
@@ -356,7 +356,7 @@ public extension UserDefaults {
         get {
             bool(forKey: key(.sentryAuthorized))
             if object(forKey: key(.sentryAuthorized)) == nil {
-                set(true, forKey: key(.sentryAuthorized))
+                set(DefaultPreferences.sentryAuthorized, forKey: key(.sentryAuthorized))
             }
             return bool(forKey: key(.sentryAuthorized))
         }
@@ -365,3 +365,8 @@ public extension UserDefaults {
         }
     }
 }
+
+public enum DefaultPreferences {
+    public static let matomoAuthorized = true
+    public static let sentryAuthorized = true
+}