[MTGOSDK/API] Client: Separate disposal of client from caches

Migrate all RemoteClient disposal to the `Client.Dispose` method instead. Partially reverts 80d1b17 to better accommodate lazy instantiation of remote objects.

MTGOSDK/src/API/Client.cs

@@ -245,17 +245,10 @@ public static bool ValidateVersion(bool assert = false)
   /// <summary>
   /// Cleans up any cached remote objects pinning objects in client memory.
   /// </summary>
-  /// <param name="disconnect">Whether to disconnect the client.</param>
-  public void ClearCaches(bool disconnect = false)
+  public void ClearCaches()
   {
-    if (disconnect)
-    {
-      RemoteClient.Dispose();
-      ObjectProvider.RequiresRefresh = true;
-    }
-    m_currentUser = null;
-
     Log.Debug("Disposing all pinned remote objects registered with the client.");
+    m_currentUser = null;
     UserManager.Users.Clear();
     CollectionManager.Cards.Clear();
   }
@@ -265,7 +258,9 @@ public void ClearCaches(bool disconnect = false)
   /// </summary>
   public void Dispose()
   {
-    ClearCaches(disconnect: true);
+    ClearCaches();
+    RemoteClient.Dispose();
+    ObjectProvider.RequiresRefresh = true;
   }
 
   //