diff --git a/Dalamud/Interface/Internal/Windows/PluginInstallerWindow.cs b/Dalamud/Interface/Internal/Windows/PluginInstallerWindow.cs index 2cc8aeb9f..f663cd168 100644 --- a/Dalamud/Interface/Internal/Windows/PluginInstallerWindow.cs +++ b/Dalamud/Interface/Internal/Windows/PluginInstallerWindow.cs @@ -909,6 +909,7 @@ namespace Dalamud.Interface.Internal.Windows ImGui.SameLine(); var cursor = ImGui.GetCursorPos(); + // Name ImGui.Text(label); @@ -1166,12 +1167,19 @@ namespace Dalamud.Interface.Internal.Windows } // Outdated API level - if (plugin.Manifest.DalamudApiLevel < PluginManager.DalamudApiLevel) + if (plugin.IsOutdated) { label += Locs.PluginTitleMod_OutdatedError; trouble = true; } + // Banned + if (plugin.IsBanned) + { + label += Locs.PluginTitleMod_BannedError; + trouble = true; + } + ImGui.PushID($"installed{index}{plugin.Manifest.InternalName}"); if (this.DrawPluginCollapsingHeader(label, plugin, plugin.Manifest, plugin.Manifest.IsThirdParty, trouble, availablePluginUpdate != default, false, () => this.DrawInstalledPluginContextMenu(plugin), index)) @@ -1216,13 +1224,20 @@ namespace Dalamud.Interface.Internal.Windows ImGui.TextWrapped(manifest.Description); } - if (plugin.Manifest.DalamudApiLevel < PluginManager.DalamudApiLevel) + if (plugin.IsOutdated) { ImGui.PushStyleColor(ImGuiCol.Text, ImGuiColors.DalamudRed); ImGui.TextWrapped(Locs.PluginBody_Outdated); ImGui.PopStyleColor(); } + if (plugin.IsBanned) + { + ImGui.PushStyleColor(ImGuiCol.Text, ImGuiColors.DalamudRed); + ImGui.TextWrapped(Locs.PluginBody_Banned); + ImGui.PopStyleColor(); + } + // Available commands (if loaded) if (plugin.IsLoaded) { @@ -1243,6 +1258,7 @@ namespace Dalamud.Interface.Internal.Windows // Controls this.DrawPluginControlButton(plugin); this.DrawDevPluginButtons(plugin); + this.DrawDeletePluginButton(plugin); this.DrawVisitRepoUrlButton(plugin.Manifest.RepoUrl); if (availablePluginUpdate != default) @@ -1304,7 +1320,7 @@ namespace Dalamud.Interface.Internal.Windows var disabled = this.updateStatus == OperationStatus.InProgress || this.installStatus == OperationStatus.InProgress; // Disable everything if the plugin is outdated - disabled = disabled || (plugin.Manifest.DalamudApiLevel < PluginManager.DalamudApiLevel && !configuration.LoadAllApiLevels); + disabled = disabled || (plugin.IsOutdated && !configuration.LoadAllApiLevels) || plugin.IsBanned; if (plugin.State == PluginState.InProgress) { @@ -1452,7 +1468,6 @@ namespace Dalamud.Interface.Internal.Windows private void DrawDevPluginButtons(LocalPlugin localPlugin) { var configuration = Service.Get(); - var pluginManager = Service.Get(); if (localPlugin is LocalDevPlugin plugin) { @@ -1495,33 +1510,40 @@ namespace Dalamud.Interface.Internal.Windows { ImGui.SetTooltip(Locs.PluginButtonToolTip_AutomaticReloading); } + } + } - // Delete - if (plugin.State == PluginState.Unloaded) + private void DrawDeletePluginButton(LocalPlugin plugin) + { + var unloaded = plugin.State == PluginState.Unloaded; + var showButton = unloaded && (plugin.IsDev || plugin.IsOutdated || plugin.IsBanned); + + if (!showButton) + return; + + var pluginManager = Service.Get(); + + ImGui.SameLine(); + if (ImGuiComponents.IconButton(FontAwesomeIcon.TrashAlt)) + { + try { - ImGui.SameLine(); - if (ImGuiComponents.IconButton(FontAwesomeIcon.TrashAlt)) - { - try - { - plugin.DllFile.Delete(); - pluginManager.RemovePlugin(plugin); - } - catch (Exception ex) - { - Log.Error(ex, $"Plugin installer threw an error during removal of {plugin.Name}"); - - this.errorModalMessage = Locs.ErrorModal_DeleteFail(plugin.Name); - this.errorModalDrawing = true; - this.errorModalOnNextFrame = true; - } - } - - if (ImGui.IsItemHovered()) - { - ImGui.SetTooltip(Locs.PluginBody_DeleteDevPlugin); - } + plugin.DllFile.Delete(); + pluginManager.RemovePlugin(plugin); } + catch (Exception ex) + { + Log.Error(ex, $"Plugin installer threw an error during removal of {plugin.Name}"); + + this.errorModalMessage = Locs.ErrorModal_DeleteFail(plugin.Name); + this.errorModalDrawing = true; + this.errorModalOnNextFrame = true; + } + } + + if (ImGui.IsItemHovered()) + { + ImGui.SetTooltip(Locs.PluginButtonToolTip_DeletePlugin); } } @@ -1993,6 +2015,8 @@ namespace Dalamud.Interface.Internal.Windows public static string PluginTitleMod_OutdatedError => Loc.Localize("InstallerOutdatedError", " (outdated)"); + public static string PluginTitleMod_BannedError => Loc.Localize("InstallerOutdatedError", " (banned)"); + public static string PluginTitleMod_New => Loc.Localize("InstallerNewPlugin ", " New!"); #endregion @@ -2027,6 +2051,8 @@ namespace Dalamud.Interface.Internal.Windows public static string PluginBody_Outdated => Loc.Localize("InstallerOutdatedPluginBody ", "This plugin is outdated and incompatible at the moment. Please wait for it to be updated by its author."); + public static string PluginBody_Banned => Loc.Localize("InstallerBannedPluginBody ", "This plugin version is banned and not available at the moment. Please wait for it to be updated by its author."); + #endregion #region Plugin buttons diff --git a/Dalamud/Plugin/Internal/Exceptions/BannedPluginException.cs b/Dalamud/Plugin/Internal/Exceptions/BannedPluginException.cs new file mode 100644 index 000000000..e4418b6d3 --- /dev/null +++ b/Dalamud/Plugin/Internal/Exceptions/BannedPluginException.cs @@ -0,0 +1,22 @@ +namespace Dalamud.Plugin.Internal.Exceptions +{ + /// + /// This represents a banned plugin that attempted an operation. + /// + internal class BannedPluginException : PluginException + { + /// + /// Initializes a new instance of the class. + /// + /// The message describing the invalid operation. + public BannedPluginException(string message) + { + this.Message = message; + } + + /// + /// Gets the message describing the invalid operation. + /// + public override string Message { get; } + } +} diff --git a/Dalamud/Plugin/Internal/LocalDevPlugin.cs b/Dalamud/Plugin/Internal/LocalDevPlugin.cs index d8e0d94e0..3615c01a9 100644 --- a/Dalamud/Plugin/Internal/LocalDevPlugin.cs +++ b/Dalamud/Plugin/Internal/LocalDevPlugin.cs @@ -145,19 +145,17 @@ namespace Dalamud.Plugin.Internal return; } + var notificationManager = Service.Get(); + try { this.Reload(); - Service.Get() - .AddNotification( - $"The DevPlugin '{this.Name} was reloaded successfully.", "Plugin reloaded!", NotificationType.Success); + notificationManager.AddNotification($"The DevPlugin '{this.Name} was reloaded successfully.", "Plugin reloaded!", NotificationType.Success); } catch (Exception ex) { Log.Error(ex, "DevPlugin reload failed."); - Service.Get() - .AddNotification( - $"The DevPlugin '{this.Name} could not be reloaded.", "Plugin reload failed!", NotificationType.Error); + notificationManager.AddNotification($"The DevPlugin '{this.Name} could not be reloaded.", "Plugin reload failed!", NotificationType.Error); } }, this.fileWatcherTokenSource.Token); diff --git a/Dalamud/Plugin/Internal/LocalPlugin.cs b/Dalamud/Plugin/Internal/LocalPlugin.cs index d51b14303..c06c12c08 100644 --- a/Dalamud/Plugin/Internal/LocalPlugin.cs +++ b/Dalamud/Plugin/Internal/LocalPlugin.cs @@ -86,10 +86,8 @@ namespace Dalamud.Plugin.Internal var assemblyVersion = this.pluginAssembly.GetName().Version; - // Files that may or may not exist + // Although it is conditionally used here, we need to set the initial value regardless. this.manifestFile = LocalPluginManifest.GetManifestFile(this.DllFile); - this.disabledFile = LocalPluginManifest.GetDisabledFile(this.DllFile); - this.testingFile = LocalPluginManifest.GetTestingFile(this.DllFile); // If the parameter manifest was null if (manifest == null) @@ -108,28 +106,32 @@ namespace Dalamud.Plugin.Internal // Save the manifest to disk so there won't be any problems later. // We'll update the name property after it can be retrieved from the instance. - var manifestFile = LocalPluginManifest.GetManifestFile(this.DllFile); - this.Manifest.Save(manifestFile); + this.Manifest.Save(this.manifestFile); } else { this.Manifest = manifest; } - // This bit converts from ".disabled" functionality to using the manifest. + // This converts from the ".disabled" file feature to the manifest instead. + this.disabledFile = LocalPluginManifest.GetDisabledFile(this.DllFile); if (this.disabledFile.Exists) { this.Manifest.Disabled = true; this.disabledFile.Delete(); } - // This bit converts from ".testing" functionality to using the manifest. + // This converts from the ".testing" file feature to the manifest instead. + this.testingFile = LocalPluginManifest.GetTestingFile(this.DllFile); if (this.testingFile.Exists) { this.Manifest.Testing = true; this.testingFile.Delete(); } + var pluginManager = Service.Get(); + this.IsBanned = pluginManager.IsManifestBanned(this.Manifest); + this.SaveManifest(); } @@ -174,11 +176,21 @@ namespace Dalamud.Plugin.Internal /// public bool IsDisabled => this.Manifest.Disabled; + /// + /// Gets a value indicating whether this plugin's API level is out of date. + /// + public bool IsOutdated => this.Manifest.DalamudApiLevel < PluginManager.DalamudApiLevel; + /// /// Gets a value indicating whether the plugin is for testing use only. /// public bool IsTesting => this.Manifest.IsTestingExclusive || this.Manifest.Testing; + /// + /// Gets a value indicating whether this plugin has been banned. + /// + public bool IsBanned { get; } + /// /// Gets a value indicating whether this plugin is dev plugin. /// @@ -223,6 +235,9 @@ namespace Dalamud.Plugin.Internal throw new InvalidPluginOperationException($"Unable to load {this.Name}, unload previously faulted, restart Dalamud"); } + if (pluginManager.IsManifestBanned(this.Manifest)) + throw new BannedPluginException($"Unable to load {this.Name}, banned"); + if (this.Manifest.ApplicableVersion < startInfo.GameVersion) throw new InvalidPluginOperationException($"Unable to load {this.Name}, no applicable version"); @@ -255,7 +270,9 @@ namespace Dalamud.Plugin.Internal { var manifestFile = LocalPluginManifest.GetManifestFile(this.DllFile); if (manifestFile.Exists) + { this.Manifest = LocalPluginManifest.Load(manifestFile); + } } } diff --git a/Dalamud/Plugin/Internal/PluginManager.cs b/Dalamud/Plugin/Internal/PluginManager.cs index 81d5b92a8..d8bf6b802 100644 --- a/Dalamud/Plugin/Internal/PluginManager.cs +++ b/Dalamud/Plugin/Internal/PluginManager.cs @@ -548,18 +548,23 @@ namespace Dalamud.Plugin.Internal PluginLocations.Remove(plugin.AssemblyName.FullName); throw; } + catch (BannedPluginException) + { + // Out of date plugins get added so they can be updated. + Log.Information($"Plugin was banned, adding anyways: {dllFile.Name}"); + } catch (Exception ex) { if (plugin.IsDev) { // Dev plugins always get added to the list so they can be fiddled with in the UI - Log.Information(ex, $"Dev plugin failed to load, adding anyways: {dllFile.Name}"); + Log.Information(ex, $"Dev plugin failed to load, adding anyways: {dllFile.Name}"); plugin.Disable(); // Disable here, otherwise you can't enable+load later } - else if (plugin.Manifest.DalamudApiLevel < DalamudApiLevel) + else if (plugin.IsOutdated) { // Out of date plugins get added so they can be updated. - Log.Information(ex, $"Plugin was outdated, adding anyways: {dllFile.Name}"); + Log.Information(ex, $"Plugin was outdated, adding anyways: {dllFile.Name}"); // plugin.Disable(); // Don't disable, or it gets deleted next boot. } else @@ -936,7 +941,12 @@ namespace Dalamud.Plugin.Internal return true; } - private bool IsManifestBanned(PluginManifest manifest) + /// + /// Determine if a plugin has been banned by inspecting the manifest. + /// + /// Manifest to inspect. + /// A value indicating whether the plugin/manifest has been banned. + public bool IsManifestBanned(PluginManifest manifest) { return this.bannedPlugins.Any(ban => ban.Name == manifest.InternalName && ban.AssemblyVersion == manifest.AssemblyVersion); }