merge in some changes

modules/expo-background-notification-handler/android/build.gradle

@@ -6,88 +6,89 @@ group = 'expo.modules.backgroundnotificationhandler'
 version = '0.5.0'
 
 buildscript {
-  def expoModulesCorePlugin = new File(project(":expo-modules-core").projectDir.absolutePath, "ExpoModulesCorePlugin.gradle")
-  if (expoModulesCorePlugin.exists()) {
-    apply from: expoModulesCorePlugin
-    applyKotlinExpoModulesCorePlugin()
-  }
-
-  // Simple helper that allows the root project to override versions declared by this library.
-  ext.safeExtGet = { prop, fallback ->
-    rootProject.ext.has(prop) ? rootProject.ext.get(prop) : fallback
-  }
-
-  // Ensures backward compatibility
-  ext.getKotlinVersion = {
-    if (ext.has("kotlinVersion")) {
-      ext.kotlinVersion()
-    } else {
-      ext.safeExtGet("kotlinVersion", "1.8.10")
+    def expoModulesCorePlugin = new File(project(":expo-modules-core").projectDir.absolutePath, "ExpoModulesCorePlugin.gradle")
+    if (expoModulesCorePlugin.exists()) {
+        apply from: expoModulesCorePlugin
+        applyKotlinExpoModulesCorePlugin()
     }
-  }
 
-  repositories {
-    mavenCentral()
-  }
+    // Simple helper that allows the root project to override versions declared by this library.
+    ext.safeExtGet = { prop, fallback ->
+        rootProject.ext.has(prop) ? rootProject.ext.get(prop) : fallback
+    }
 
-  dependencies {
-    classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:${getKotlinVersion()}")
-  }
+    // Ensures backward compatibility
+    ext.getKotlinVersion = {
+        if (ext.has("kotlinVersion")) {
+            ext.kotlinVersion()
+        } else {
+            ext.safeExtGet("kotlinVersion", "1.8.10")
+        }
+    }
+
+    repositories {
+        mavenCentral()
+    }
+
+    dependencies {
+        classpath("org.jetbrains.kotlin:kotlin-gradle-plugin:${getKotlinVersion()}")
+    }
 }
 
 afterEvaluate {
-  publishing {
-    publications {
-      release(MavenPublication) {
-        from components.release
-      }
+    publishing {
+        publications {
+            release(MavenPublication) {
+                from components.release
+            }
+        }
+        repositories {
+            maven {
+                url = mavenLocal().url
+            }
+        }
     }
-    repositories {
-      maven {
-        url = mavenLocal().url
-      }
-    }
-  }
 }
 
 android {
-  compileSdkVersion safeExtGet("compileSdkVersion", 33)
+    compileSdkVersion safeExtGet("compileSdkVersion", 33)
 
-  def agpVersion = com.android.Version.ANDROID_GRADLE_PLUGIN_VERSION
-  if (agpVersion.tokenize('.')[0].toInteger() < 8) {
-    compileOptions {
-      sourceCompatibility JavaVersion.VERSION_11
-      targetCompatibility JavaVersion.VERSION_11
+    def agpVersion = com.android.Version.ANDROID_GRADLE_PLUGIN_VERSION
+    if (agpVersion.tokenize('.')[0].toInteger() < 8) {
+        compileOptions {
+            sourceCompatibility JavaVersion.VERSION_11
+            targetCompatibility JavaVersion.VERSION_11
+        }
+
+        kotlinOptions {
+            jvmTarget = JavaVersion.VERSION_11.majorVersion
+        }
     }
 
-    kotlinOptions {
-      jvmTarget = JavaVersion.VERSION_11.majorVersion
+    namespace "expo.modules.backgroundnotificationhandler"
+    defaultConfig {
+        minSdkVersion safeExtGet("minSdkVersion", 21)
+        targetSdkVersion safeExtGet("targetSdkVersion", 34)
+        versionCode 1
+        versionName "0.5.0"
     }
-  }
-
-  namespace "expo.modules.backgroundnotificationhandler"
-  defaultConfig {
-    minSdkVersion safeExtGet("minSdkVersion", 21)
-    targetSdkVersion safeExtGet("targetSdkVersion", 34)
-    versionCode 1
-    versionName "0.5.0"
-  }
-  lintOptions {
-    abortOnError false
-  }
-  publishing {
-    singleVariant("release") {
-      withSourcesJar()
+    lintOptions {
+        abortOnError false
+    }
+    publishing {
+        singleVariant("release") {
+            withSourcesJar()
+        }
     }
-  }
 }
 
 repositories {
-  mavenCentral()
+    mavenCentral()
 }
 
 dependencies {
-  implementation project(':expo-modules-core')
-  implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:${getKotlinVersion()}"
-  implementation 'com.google.firebase:firebase-messaging-ktx:24.0.0'
+    implementation project(':expo-bluesky-swiss-army')
+    implementation project(':expo-modules-core')
+    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:${getKotlinVersion()}"
+    implementation 'com.google.firebase:firebase-messaging-ktx:24.0.0'
 }

modules/expo-background-notification-handler/android/src/main/java/expo/modules/backgroundnotificationhandler/BackgroundNotificationHandler.kt

@@ -2,6 +2,7 @@ package expo.modules.backgroundnotificationhandler
 
 import android.content.Context
 import com.google.firebase.messaging.RemoteMessage
+import expo.modules.blueskyswissarmy.sharedprefs.Preferences
 
 enum class NotificationType(val type: String) {
   Like("like"),
@@ -36,7 +37,7 @@ class BackgroundNotificationHandler(
   }
 
   private fun mutateWithChatMessage(remoteMessage: RemoteMessage) {
-    if (NotificationPrefs(context).getBoolean("playSoundChat")) {
+    if (Preferences(context).getBoolean("playSoundChat") == true) {
       // If oreo or higher
       if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.O) {
         remoteMessage.data["channelId"] = "chat-messages"

modules/expo-background-notification-handler/android/src/main/java/expo/modules/backgroundnotificationhandler/ExpoBackgroundNotificationHandlerModule.kt

@@ -1,8 +1,20 @@
 package expo.modules.backgroundnotificationhandler
 
+import expo.modules.blueskyswissarmy.sharedprefs.Preferences
 import expo.modules.kotlin.modules.Module
 import expo.modules.kotlin.modules.ModuleDefinition
 
+val DEFAULTS = mapOf<String, Any>(
+  "playSoundChat" to true,
+  "playSoundFollow" to false,
+  "playSoundLike" to false,
+  "playSoundMention" to false,
+  "playSoundQuote" to false,
+  "playSoundReply" to false,
+  "playSoundRepost" to false,
+  "mutedThreads" to mapOf<String, List<String>>()
+)
+
 class ExpoBackgroundNotificationHandlerModule : Module() {
   companion object {
     var isForegrounded = false
@@ -12,7 +24,17 @@ class ExpoBackgroundNotificationHandlerModule : Module() {
     Name("ExpoBackgroundNotificationHandler")
 
     OnCreate {
-      NotificationPrefs(appContext.reactContext).initialize()
+      val context = appContext.reactContext ?: throw Error("Context is null")
+      DEFAULTS.forEach { (key, value) ->
+        if (Preferences(context).hasValue(key)) {
+          return@forEach
+        }
+
+        Preferences(context)._(key, value)
+
+        when (value) {
+        }
+      }
     }
 
     OnActivityEntersForeground {
@@ -23,50 +45,6 @@ class ExpoBackgroundNotificationHandlerModule : Module() {
       isForegrounded = false
     }
 
-    AsyncFunction("getAllPrefsAsync") {
-      return@AsyncFunction NotificationPrefs(appContext.reactContext).getAllPrefs()
-    }
-
-    AsyncFunction("getBoolAsync") { forKey: String ->
-      return@AsyncFunction NotificationPrefs(appContext.reactContext).getBoolean(forKey)
-    }
-
-    AsyncFunction("getStringAsync") { forKey: String ->
-      return@AsyncFunction NotificationPrefs(appContext.reactContext).getString(forKey)
-    }
-
-    AsyncFunction("getStringArrayAsync") { forKey: String ->
-      return@AsyncFunction NotificationPrefs(appContext.reactContext).getStringArray(forKey)
-    }
-
-    AsyncFunction("setBoolAsync") { forKey: String, value: Boolean ->
-      NotificationPrefs(appContext.reactContext).setBoolean(forKey, value)
-    }
-
-    AsyncFunction("setStringAsync") { forKey: String, value: String ->
-      NotificationPrefs(appContext.reactContext).setString(forKey, value)
-    }
-
-    AsyncFunction("setStringArrayAsync") { forKey: String, value: Array<String> ->
-      NotificationPrefs(appContext.reactContext).setStringArray(forKey, value)
-    }
-
-    AsyncFunction("addToStringArrayAsync") { forKey: String, string: String ->
-      NotificationPrefs(appContext.reactContext).addToStringArray(forKey, string)
-    }
-
-    AsyncFunction("removeFromStringArrayAsync") { forKey: String, string: String ->
-      NotificationPrefs(appContext.reactContext).removeFromStringArray(forKey, string)
-    }
-
-    AsyncFunction("addManyToStringArrayAsync") { forKey: String, strings: Array<String> ->
-      NotificationPrefs(appContext.reactContext).addManyToStringArray(forKey, strings)
-    }
-
-    AsyncFunction("removeManyFromStringArrayAsync") { forKey: String, strings: Array<String> ->
-      NotificationPrefs(appContext.reactContext).removeManyFromStringArray(forKey, strings)
-    }
-
     AsyncFunction("setBadgeCountAsync") { _: String, _: Int ->
       // This does nothing on Android
     }

modules/expo-background-notification-handler/android/src/main/java/expo/modules/backgroundnotificationhandler/NotificationPrefs.kt

@@ -1,134 +0,0 @@
-package expo.modules.backgroundnotificationhandler
-
-import android.content.Context
-
-val DEFAULTS = mapOf<String, Any>(
-  "playSoundChat" to true,
-  "playSoundFollow" to false,
-  "playSoundLike" to false,
-  "playSoundMention" to false,
-  "playSoundQuote" to false,
-  "playSoundReply" to false,
-  "playSoundRepost" to false,
-  "mutedThreads" to mapOf<String, List<String>>()
-)
-
-class NotificationPrefs (private val context: Context?) {
-  private val prefs = context?.getSharedPreferences("xyz.blueskyweb.app", Context.MODE_PRIVATE)
-    ?: throw Error("Context is null")
-
-  fun initialize() {
-    prefs
-      .edit()
-      .apply {
-        DEFAULTS.forEach { (key, value) ->
-          if (prefs.contains(key)) {
-            return@forEach
-          }
-
-          when (value) {
-            is Boolean -> {
-              putBoolean(key, value)
-            }
-            is String -> {
-              putString(key, value)
-            }
-            is Array<*> -> {
-              putStringSet(key, value.map { it.toString() }.toSet())
-            }
-            is Map<*, *> -> {
-              putStringSet(key, value.map { it.toString() }.toSet())
-            }
-          }
-        }
-      }
-      .apply()
-  }
-
-  fun getAllPrefs(): MutableMap<String, *> {
-    return prefs.all
-  }
-
-  fun getBoolean(key: String): Boolean {
-    return prefs.getBoolean(key, false)
-  }
-
-  fun getString(key: String): String? {
-    return prefs.getString(key, null)
-  }
-
-  fun getStringArray(key: String): Array<String>? {
-    return prefs.getStringSet(key, null)?.toTypedArray()
-  }
-
-  fun setBoolean(key: String, value: Boolean) {
-    prefs
-      .edit()
-      .apply {
-        putBoolean(key, value)
-      }
-      .apply()
-  }
-
-  fun setString(key: String, value: String) {
-    prefs
-      .edit()
-      .apply {
-        putString(key, value)
-      }
-      .apply()
-  }
-
-  fun setStringArray(key: String, value: Array<String>) {
-    prefs
-      .edit()
-      .apply {
-        putStringSet(key, value.toSet())
-      }
-      .apply()
-  }
-
-  fun addToStringArray(key: String, string: String) {
-    prefs
-      .edit()
-      .apply {
-        val set = prefs.getStringSet(key, null)?.toMutableSet() ?: mutableSetOf()
-        set.add(string)
-        putStringSet(key, set)
-      }
-      .apply()
-  }
-
-  fun removeFromStringArray(key: String, string: String) {
-    prefs
-      .edit()
-      .apply {
-        val set = prefs.getStringSet(key, null)?.toMutableSet() ?: mutableSetOf()
-        set.remove(string)
-        putStringSet(key, set)
-      }
-      .apply()
-  }
-
-  fun addManyToStringArray(key: String, strings: Array<String>) {
-    prefs
-      .edit()
-      .apply {
-        val set = prefs.getStringSet(key, null)?.toMutableSet() ?: mutableSetOf()
-        set.addAll(strings.toSet())
-        putStringSet(key, set)
-      }
-      .apply()
-  }
-
-  fun removeManyFromStringArray(key: String, strings: Array<String>) {
-    prefs
-      .edit()
-      .apply {
-        val set = prefs.getStringSet(key, null)?.toMutableSet() ?: mutableSetOf()
-        set.removeAll(strings.toSet())
-        putStringSet(key, set)
-      }
-      .apply()
-  }
-}