From 701f52c6898903c4bfbc9c1ba4735d6745aefd0e Mon Sep 17 00:00:00 2001
From: Soner Yuksel <grkmyksl@gmail.com>
Date: Mon, 25 Sep 2023 11:24:15 -0400
Subject: [PATCH] Altering sync type change method and deleting unnecessary
 parameter to enable default bookmarks type

---
 .../Sync/BraveCore/BraveSyncAPIExtensions.swift      | 12 +++++-------
 .../Sync/SyncSettingsTableViewController.swift       |  2 +-
 .../Frontend/Sync/SyncWelcomeViewController.swift    | 10 ++++++----
 Sources/Brave/Migration/Migration.swift              |  3 +--
 4 files changed, 13 insertions(+), 14 deletions(-)

diff --git a/Sources/Brave/Frontend/Sync/BraveCore/BraveSyncAPIExtensions.swift b/Sources/Brave/Frontend/Sync/BraveCore/BraveSyncAPIExtensions.swift
index 94fb2e0ed0d..8ac7482ae05 100644
--- a/Sources/Brave/Frontend/Sync/BraveCore/BraveSyncAPIExtensions.swift
+++ b/Sources/Brave/Frontend/Sync/BraveCore/BraveSyncAPIExtensions.swift
@@ -55,9 +55,11 @@ extension BraveSyncAPI {
   }
   
   @discardableResult
-  func joinSyncGroup(codeWords: String, syncProfileService: BraveSyncProfileServiceIOS, shouldEnableBookmarks: Bool) -> Bool {
+  func joinSyncGroup(codeWords: String, syncProfileService: BraveSyncProfileServiceIOS) -> Bool {
     if setSyncCode(codeWords) {
-      enableSyncTypes(syncProfileService: syncProfileService, shouldEnableBookmarks: shouldEnableBookmarks)
+      // Enable default sync type Bookmarks when joining a chain
+      Preferences.Chromium.syncBookmarksEnabled.value = true
+      enableSyncTypes(syncProfileService: syncProfileService)
       requestSync()
       setSetupComplete()
       Preferences.Chromium.syncEnabled.value = true
@@ -92,13 +94,9 @@ extension BraveSyncAPI {
     resetSync()
   }
 
-  func enableSyncTypes(syncProfileService: BraveSyncProfileServiceIOS, shouldEnableBookmarks: Bool) {
+  func enableSyncTypes(syncProfileService: BraveSyncProfileServiceIOS) {
     syncProfileService.userSelectedTypes = []
     
-    // This value is true by default
-    // In some cases while joining a sync chain all values must be disabled
-    Preferences.Chromium.syncBookmarksEnabled.value = shouldEnableBookmarks
-      
     if Preferences.Chromium.syncBookmarksEnabled.value {
       syncProfileService.userSelectedTypes.update(with: .BOOKMARKS)
     }
diff --git a/Sources/Brave/Frontend/Sync/SyncSettingsTableViewController.swift b/Sources/Brave/Frontend/Sync/SyncSettingsTableViewController.swift
index 695a43c306f..9d42f35ff82 100644
--- a/Sources/Brave/Frontend/Sync/SyncSettingsTableViewController.swift
+++ b/Sources/Brave/Frontend/Sync/SyncSettingsTableViewController.swift
@@ -305,7 +305,7 @@ class SyncSettingsTableViewController: SyncViewController, UITableViewDelegate,
         }
       }
 
-      syncAPI.enableSyncTypes(syncProfileService: syncProfileService, shouldEnableBookmarks: false)
+      syncAPI.enableSyncTypes(syncProfileService: syncProfileService)
     }
   }
   
diff --git a/Sources/Brave/Frontend/Sync/SyncWelcomeViewController.swift b/Sources/Brave/Frontend/Sync/SyncWelcomeViewController.swift
index 5bd353bd6ad..b8e868ca18d 100644
--- a/Sources/Brave/Frontend/Sync/SyncWelcomeViewController.swift
+++ b/Sources/Brave/Frontend/Sync/SyncWelcomeViewController.swift
@@ -7,6 +7,7 @@ import BraveShared
 import BraveCore
 import BraveUI
 import os.log
+import Preferences
 
 /// Sometimes during heavy operations we want to prevent user from navigating back, changing screen etc.
 protocol NavigationPrevention {
@@ -261,7 +262,7 @@ class SyncWelcomeViewController: SyncViewController {
         }
       }
       
-      self.syncAPI.joinSyncGroup(codeWords: self.syncAPI.getSyncCode(), syncProfileService: self.syncProfileServices, shouldEnableBookmarks: true)
+      self.syncAPI.joinSyncGroup(codeWords: self.syncAPI.getSyncCode(), syncProfileService: self.syncProfileServices)
       self.handleSyncSetupFailure()
     }
     
@@ -337,7 +338,7 @@ extension SyncWelcomeViewController: SyncPairControllerDelegate {
 
     // In parallel set code words - request sync and setup complete
     // should be called on brave-core side
-    syncAPI.joinSyncGroup(codeWords: codeWords, syncProfileService: syncProfileServices, shouldEnableBookmarks: false)
+    syncAPI.joinSyncGroup(codeWords: codeWords, syncProfileService: syncProfileServices)
     handleSyncSetupFailure()
   }
   
@@ -485,8 +486,9 @@ extension SyncWelcomeViewController: SyncPairControllerDelegate {
   }
   
   private func enableDefaultTypeAndPushSettings() {
-    // Enable Bookmark Syncing and push settings
-    syncAPI.enableSyncTypes(syncProfileService: syncProfileServices, shouldEnableBookmarks: true)
+    // Enable default sync type Bookmarks and push settings
+    Preferences.Chromium.syncBookmarksEnabled.value = true
+    syncAPI.enableSyncTypes(syncProfileService: syncProfileServices)
     pushSettings()
   }
   
diff --git a/Sources/Brave/Migration/Migration.swift b/Sources/Brave/Migration/Migration.swift
index 4b21352ba39..b763446daaf 100644
--- a/Sources/Brave/Migration/Migration.swift
+++ b/Sources/Brave/Migration/Migration.swift
@@ -41,7 +41,6 @@ public class Migration {
     }
 
     // Adding Observer to enable sync types
-
     NotificationCenter.default.addObserver(
       self,
       selector: #selector(enableUserSelectedTypesForSync),
@@ -56,7 +55,7 @@ public class Migration {
       return
     }
 
-    braveCore.syncAPI.enableSyncTypes(syncProfileService: braveCore.syncProfileService, shouldEnableBookmarks: true)
+    braveCore.syncAPI.enableSyncTypes(syncProfileService: braveCore.syncProfileService)
   }
 
   /// Adblock files don't have to be moved, they now have a new directory and will be downloaded there.