Skip to content

Commit 55aa432

Browse files
committed
Merge login & defaults feature flags into one
1 parent c4363cf commit 55aa432

File tree

5 files changed

+9
-14
lines changed

5 files changed

+9
-14
lines changed

WordPress/Classes/Stores/UserPersistentStoreFactory.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,12 @@ import Foundation
33
@objc
44
final class UserPersistentStoreFactory: NSObject {
55
static func instance() -> UserPersistentRepository {
6-
FeatureFlag.sharedUserDefaults.enabled ? UserPersistentStore.standard : UserDefaults.standard
6+
FeatureFlag.contentMigration.enabled ? UserPersistentStore.standard : UserDefaults.standard
77
}
88

99
@objc
1010
static func userDefaultsInstance() -> UserDefaults {
11-
guard FeatureFlag.sharedUserDefaults.enabled, let defaults = UserDefaults(suiteName: WPAppGroupName) else {
11+
guard FeatureFlag.contentMigration.enabled, let defaults = UserDefaults(suiteName: WPAppGroupName) else {
1212
return UserDefaults.standard
1313
}
1414

WordPress/Classes/System/WordPressAppDelegate.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ class WordPressAppDelegate: UIResponder, UIApplicationDelegate {
159159
}
160160

161161
private func copyToSharedDefaultsIfNeeded() {
162-
if !AppConfiguration.isJetpack && FeatureFlag.sharedUserDefaults.enabled && !UserPersistentStore.standard.isOneOffMigrationComplete {
162+
if !AppConfiguration.isJetpack && FeatureFlag.contentMigration.enabled && !UserPersistentStore.standard.isOneOffMigrationComplete {
163163
let dict = UserDefaults.standard.dictionaryRepresentation()
164164
for (key, value) in dict {
165165
UserPersistentStore.standard.set(value, forKey: key)

WordPress/Classes/Utility/Blogging Reminders/BloggingRemindersScheduler.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ class BloggingRemindersScheduler {
147147
}
148148

149149
static func handleRemindersMigration() {
150-
guard FeatureFlag.sharedLogin.enabled else {
150+
guard FeatureFlag.contentMigration.enabled else {
151151
return
152152
}
153153

WordPress/Classes/Utility/BuildInformation/FeatureFlag.swift

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,7 @@ enum FeatureFlag: Int, CaseIterable, OverrideableFlag {
3232
case featureHighlightTooltip
3333
case jetpackPowered
3434
case jetpackPoweredBottomSheet
35-
case sharedUserDefaults
36-
case sharedLogin
35+
case contentMigration
3736
case newJetpackLandingScreen
3837
case newWordPressLandingScreen
3938
case newCoreDataContext
@@ -115,9 +114,7 @@ enum FeatureFlag: Int, CaseIterable, OverrideableFlag {
115114
return true
116115
case .jetpackPoweredBottomSheet:
117116
return false
118-
case .sharedUserDefaults:
119-
return false
120-
case .sharedLogin:
117+
case .contentMigration:
121118
return false
122119
case .newJetpackLandingScreen:
123120
return true
@@ -239,10 +236,8 @@ extension FeatureFlag {
239236
return "Jetpack powered banners and badges"
240237
case .jetpackPoweredBottomSheet:
241238
return "Jetpack powered bottom sheet"
242-
case .sharedUserDefaults:
243-
return "Shared User Defaults"
244-
case .sharedLogin:
245-
return "Shared Login"
239+
case .contentMigration:
240+
return "Content Migration"
246241
case .newJetpackLandingScreen:
247242
return "New Jetpack landing screen"
248243
case .newWordPressLandingScreen:

WordPress/Classes/Utility/LegacyContextFactory.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ - (void)createWriterContext
3535
NSManagedObjectContext *context = [[NSManagedObjectContext alloc] initWithConcurrencyType:NSPrivateQueueConcurrencyType];
3636
context.persistentStoreCoordinator = self.container.persistentStoreCoordinator;
3737

38-
if ([Feature enabled:FeatureFlagSharedLogin]) {
38+
if ([Feature enabled:FeatureFlagContentMigration]) {
3939
context.mergePolicy = NSMergeByPropertyStoreTrumpMergePolicy;
4040
}
4141

0 commit comments

Comments
 (0)