Skip to content

Commit

Permalink
Clean some things up
Browse files Browse the repository at this point in the history
- Remove unnecessary PathMap from csproj
- Clean up duplicate import in SetupNag
- Fix deprecations in CollectionStrategy
  • Loading branch information
KazWolfe committed Feb 19, 2024
1 parent 9527fe2 commit f33552c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 11 deletions.
6 changes: 3 additions & 3 deletions FFXIVPlugin/ActionExecutor/Strategies/CollectionStrategy.cs
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@

namespace XIVDeck.FFXIVPlugin.ActionExecutor.Strategies;

[ActionStrategy(HotbarSlotType.Collection)]
[ActionStrategy(HotbarSlotType.McGuffin)]
public class CollectionStrategy : IActionStrategy {
private static readonly ExcelSheet<McGuffin> Sheet = Injections.DataManager.Excel.GetSheet<McGuffin>()!;

Expand All @@ -23,7 +23,7 @@ private static ExecutableAction GetExecutableAction(McGuffin mcguffin) {
ActionName = uiData.Name.ToString(),
IconId = (int) uiData.Icon,
Category = null,
HotbarSlotType = HotbarSlotType.Collection,
HotbarSlotType = HotbarSlotType.McGuffin,
SortOrder = uiData.Order
};
}
Expand Down Expand Up @@ -52,7 +52,7 @@ public void Execute(uint actionId, ActionPayload? _) {
}

Injections.Framework.RunOnFrameworkThread(delegate {
HotbarManager.ExecuteHotbarAction(HotbarSlotType.Collection, actionId);
HotbarManager.ExecuteHotbarAction(HotbarSlotType.McGuffin, actionId);
});
}

Expand Down
2 changes: 0 additions & 2 deletions FFXIVPlugin/UI/Windows/Nags/SetupNag.cs
Original file line number Diff line number Diff line change
@@ -1,10 +1,8 @@
using Dalamud.Interface;
using Dalamud.Interface.Colors;
using Dalamud.Interface.Components;
using Dalamud.Interface.Utility;
using ImGuiNET;
using XIVDeck.FFXIVPlugin.Resources.Localization;
using XIVDeck.FFXIVPlugin.Resources.Localization;
using XIVDeck.FFXIVPlugin.Utils;

namespace XIVDeck.FFXIVPlugin.UI.Windows.Nags;
Expand Down
7 changes: 1 addition & 6 deletions FFXIVPlugin/XIVDeck.FFXIVPlugin.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
<Copyright>(c) 2023, KazWolfe. Licensed under AGPL-3.0.</Copyright>
<IsPackable>false</IsPackable>
<NeutralLanguage>en</NeutralLanguage>
<Nullable>enable</Nullable>
<PackageLicenseExpression>AGPL-3.0</PackageLicenseExpression>
</PropertyGroup>

Expand All @@ -36,11 +35,7 @@
</PropertyGroup>
<PropertyGroup Label="Feature">
<AllowUnsafeBlocks>true</AllowUnsafeBlocks>
</PropertyGroup>

<!-- Anonymize and shorten build paths for release -->
<PropertyGroup Condition="'$(Configuration)' == 'Release'">
<PathMap>$([System.IO.Path]::GetFullPath('$(MSBuildThisFileDirectory)'))=S:\</PathMap>
<Nullable>enable</Nullable>
</PropertyGroup>

<PropertyGroup>
Expand Down

0 comments on commit f33552c

Please sign in to comment.