Fix NetworkManager initialization to use lazy delegate

mollyim · Aug 6, 2024 · bc78172 · bc78172
1 parent d8a24c7
commit bc78172
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/app/src/main/java/org/thoughtcrime/securesms/dependencies/AppDependencies.kt b/app/src/main/java/org/thoughtcrime/securesms/dependencies/AppDependencies.kt
@@ -217,8 +217,9 @@ object AppDependencies {
   val webSocketObserver: Observable<WebSocketConnectionState> = _webSocketObserver
 
   @JvmStatic
-  val networkManager: NetworkManager
-    get() = provider.provideNetworkManager()
+  val networkManager: NetworkManager by lazy {
+    provider.provideNetworkManager()
+  }
 
   private val _networkModule = resettableLazy {
     NetworkDependenciesModule(application, provider, _webSocketObserver)

diff --git a/app/src/main/java/org/thoughtcrime/securesms/net/NetworkManager.java b/app/src/main/java/org/thoughtcrime/securesms/net/NetworkManager.java
@@ -53,10 +53,6 @@ public static NetworkManager create(@NonNull Context context) {
     return new NetworkManager(ApplicationContext.getInstance(), OrbotHelper.get(context));
   }
 
-  public boolean isNetworkEnabled() {
-    return Networking.isEnabled();
-  }
-
   public void setNetworkEnabled(boolean enabled) {
     Networking.setEnabled(enabled);
     if (enabled) {