Hotfix - Auto-enable plugins after updating them

Author: RobertBeekman

src/Artemis.UI/Services/Updating/WorkshopUpdateService.cs

@@ -20,15 +20,21 @@ public class WorkshopUpdateService : IWorkshopUpdateService
     private readonly ILogger _logger;
     private readonly IWorkshopClient _client;
     private readonly IWorkshopService _workshopService;
+    private readonly IPluginManagementService _pluginManagementService;
     private readonly Lazy<IUpdateNotificationProvider> _updateNotificationProvider;
     private readonly PluginSetting<bool> _showNotifications;
 
-    public WorkshopUpdateService(ILogger logger, IWorkshopClient client, IWorkshopService workshopService, ISettingsService settingsService,
+    public WorkshopUpdateService(ILogger logger,
+        IWorkshopClient client,
+        IWorkshopService workshopService,
+        ISettingsService settingsService,
+        IPluginManagementService pluginManagementService,
         Lazy<IUpdateNotificationProvider> updateNotificationProvider)
     {
         _logger = logger;
         _client = client;
         _workshopService = workshopService;
+        _pluginManagementService = pluginManagementService;
         _updateNotificationProvider = updateNotificationProvider;
         _showNotifications = settingsService.GetSetting("Workshop.ShowNotifications", true);
     }
@@ -88,6 +94,18 @@ public async Task<bool> AutoUpdateEntry(InstalledEntry installedEntry)
             else
                 _logger.Warning("Auto-update failed for entry {Entry}: {Message}", entry, updateResult.Message);
 
+            if (!updateResult.IsSuccess || updateResult.Installed is not Plugin {IsEnabled: false} updatedPlugin)
+                return updateResult.IsSuccess;
+
+            try
+            {
+                _pluginManagementService.EnablePlugin(updatedPlugin, true, true);
+            }
+            catch (Exception e)
+            {
+                _logger.Warning(e, "Failed to auto-enable updated plugin {Plugin}", updatedPlugin);
+            }
+
             return updateResult.IsSuccess;
         }
         catch (Exception e)

src/Artemis.WebClient.Workshop/Handlers/InstallationHandlers/Implementations/PluginEntryInstallationHandler.cs

@@ -101,9 +101,7 @@ public async Task<EntryInstallResult> InstallAsync(IEntrySummary entry, IRelease
             {
                 // ignored, will get cleaned up as an orphaned file
             }
-
-            if (installedEntry.Entity.Id != Guid.Empty)
-                _workshopService.RemoveInstalledEntry(installedEntry);
+            
             return EntryInstallResult.FromException(e);
         }