Login is smarter now

Author: mnadareski

RedumpTool/Features/PacksFeature.cs

@@ -65,8 +65,7 @@ public override bool Execute()
                 _client.Timeout = TimeSpan.FromSeconds(timeout.Value);
 
             // Login to Redump, if necessary
-            if (!_client.LoggedIn)
-                _client.Login(username, password).Wait();
+            _client.Login(username, password).Wait();
 
             // Start the processing
             var processingTask = _client.DownloadPacks(outDir, useSubfolders);

RedumpTool/Features/QueryFeature.cs

@@ -89,8 +89,7 @@ public override bool Execute()
                 _client.Timeout = TimeSpan.FromSeconds(timeout.Value);
 
             // Login to Redump, if necessary
-            if (!_client.LoggedIn)
-                _client.Login(username, password).Wait();
+            _client.Login(username, password).Wait();
 
             // Start the processing
             Task<List<int>> processingTask;

RedumpTool/Features/SiteFeature.cs

@@ -89,8 +89,7 @@ public override bool Execute()
                 _client.Timeout = TimeSpan.FromSeconds(timeout.Value);
 
             // Login to Redump, if necessary
-            if (!_client.LoggedIn)
-                _client.Login(username, password).Wait();
+            _client.Login(username, password).Wait();
 
             // Start the processing
             Task<List<int>> processingTask;

RedumpTool/Features/UserFeature.cs

@@ -72,8 +72,7 @@ public override bool Execute()
                 _client.Timeout = TimeSpan.FromSeconds(timeout.Value);
 
             // Login to Redump, if necessary
-            if (!_client.LoggedIn)
-                _client.Login(username, password).Wait();
+            _client.Login(username, password).Wait();
 
             // Start the processing
             Task<List<int>> processingTask;

RedumpTool/Features/WIPFeature.cs

@@ -84,8 +84,7 @@ public override bool Execute()
                 _client.Timeout = TimeSpan.FromSeconds(timeout.Value);
 
             // Login to Redump, if necessary
-            if (!_client.LoggedIn)
-                _client.Login(username, password).Wait();
+            _client.Login(username, password).Wait();
 
             // Start the processing
             Task<List<int>> processingTask;

SabreTools.RedumpLib/Web/RedumpClient.cs

@@ -116,6 +116,13 @@ public RedumpClient()
         /// <returns>True if the user could be logged in, false otherwise, null on error</returns>
         public async Task<bool?> Login(string username, string password)
         {
+            // Check for already logged in
+            if (LoggedIn)
+            {
+                Console.WriteLine("Already logged in!");
+                return true;
+            }
+
             // Credentials verification
             if (!string.IsNullOrEmpty(username) && !string.IsNullOrEmpty(password))
             {