NetworkObserver SDK checks inconsistency resolved (#996)

Check for API to be N+ when unregistering network observer
PierfrancescoSoffritti · Jun 8, 2023 · a8d2924 · a8d2924
1 parent ab6efa3
commit a8d2924
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...java/com/pierfrancescosoffritti/androidyoutubeplayer/core/player/utils/NetworkObserver.kt b/...java/com/pierfrancescosoffritti/androidyoutubeplayer/core/player/utils/NetworkObserver.kt
@@ -36,7 +36,8 @@ internal class NetworkObserver(private val context: Context) {
 
   /** Stop observing network changes and cleanup */
   fun destroy() {
-    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
+    // Min API for `unregisterNetworkCallback` is L, but we use `registerDefaultNetworkCallback` only for N and above.
+    if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
       val callback = networkCallback ?: return
       val connectivityManager = context.getSystemService(Context.CONNECTIVITY_SERVICE) as ConnectivityManager
       connectivityManager.unregisterNetworkCallback(callback)