Skip to content

Commit

Permalink
Merge pull request #438 from FFXIV-CombatReborn/more-redtext-fixes
Browse files Browse the repository at this point in the history
Update ECommons submodule and improve null safety
  • Loading branch information
LTS-FFXIV authored Oct 23, 2024
2 parents 5d8164a + b4286a2 commit 94b3f69
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 8 deletions.
2 changes: 1 addition & 1 deletion ECommons
Submodule ECommons updated 82 files
+6 −1 ECommons/UIHelpers/AddonMasterImplementations/!AddonMasterBase.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/AirShipExploration.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/AirShipExplorationResult.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/Bank.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/BankaCraftworksSupply.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/BannerEditor.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/BannerList.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/CollectablesShop.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ColorantColoring.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/CompanyCraftRecipeNoteBook.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/CompanyCraftSupply.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ContentsFinderConfirm.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ContentsFinderSetting.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ContextMenu.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/Dialogue.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/DifficultySelectYesNo.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/FreeCompanyCreditShop.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/Gathering.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/GatheringMasterpiece.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/GcArmyChangeClass.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/GcArmyExpedition.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/GcArmyExpeditionResult.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/GcArmyMenberProfile.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/GcArmyTraining.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/GearSetList.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/GrandCompanySupplyReward.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/InputNumeric.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ItemFinder.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ItemInspectionResult.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ItemSearchResult.cs
+3 −0 ECommons/UIHelpers/AddonMasterImplementations/JournalDetail.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/JournalResult.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/LetterHistory.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/LetterList.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/LetterViewer.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/LookingForGroup.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/LookingForGroupCondition.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/LookingForGroupDetail.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/LookingForGroupPrivate.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/LotteryWeeklyInput.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/LotteryWeeklyRewardList.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ManeuversArmorBoarding.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ManeuversRecord.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/MateriaAttachDialog.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/MateriaRetrieveDialog.cs
+2 −12 ECommons/UIHelpers/AddonMasterImplementations/MaterializeDialog.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/MiragePrismExecute.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/MiragePrismRemove.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/PurifyAutoDialog.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/PurifyResult.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/PvpProfile.cs
+3 −0 ECommons/UIHelpers/AddonMasterImplementations/PvpReward.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/RecipeNote.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/RecommendEquip.cs
+2 −12 ECommons/UIHelpers/AddonMasterImplementations/Repair.cs
+2 −12 ECommons/UIHelpers/AddonMasterImplementations/Request.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/RetainerItemTransferList.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/RetainerItemTransferProgress.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/RetainerList.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/RetainerSell.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/RetainerTaskAsk.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/RetainerTaskResult.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ReturnerDialog.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/SalvageAutoDialog.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/SalvageDialog.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/SalvageResult.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/SelectIconString.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/SelectOk.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/SelectString.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/SelectYesno.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ShopCardDialog.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ShopExchangeCurrencyDialog.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/ShopExchangeItemDialog.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/Talk.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/TitleDCWorldMap.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/TripleTriadRequest.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/TripleTriadResult.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/VoteMvp.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/WeeklyBingoResult.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/_CharaSelectListMenu.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/_CharaSelectWorldServer.cs
+2 −0 ECommons/UIHelpers/AddonMasterImplementations/_TitleMenu.cs
12 changes: 6 additions & 6 deletions RotationSolver.Basic/DataCenter.cs
Original file line number Diff line number Diff line change
Expand Up @@ -463,12 +463,12 @@ public static IBattleChara? DispelTarget
{
get
{
var weakenPeople =
DataCenter.PartyMembers.Where(o => o is IBattleChara b && b.StatusList.Any(StatusHelper.CanDispel));
var weakenNPC =
DataCenter.FriendlyNPCMembers.Where(o => o is IBattleChara b && b.StatusList.Any(StatusHelper.CanDispel));
var dyingPeople =
weakenPeople.Where(o => o is IBattleChara b && b.StatusList.Any(StatusHelper.IsDangerous));
var weakenPeople = DataCenter.PartyMembers
.Where(o => o is IBattleChara b && b.StatusList != null && b.StatusList.Any(StatusHelper.CanDispel));
var weakenNPC = DataCenter.FriendlyNPCMembers
.Where(o => o is IBattleChara b && b.StatusList != null && b.StatusList.Any(StatusHelper.CanDispel));
var dyingPeople = weakenPeople
.Where(o => o is IBattleChara b && b.StatusList != null && b.StatusList.Any(StatusHelper.IsDangerous));

return dyingPeople.OrderBy(ObjectHelper.DistanceToPlayer).FirstOrDefault()
?? weakenPeople.OrderBy(ObjectHelper.DistanceToPlayer).FirstOrDefault()
Expand Down
5 changes: 4 additions & 1 deletion RotationSolver/RotationSolverPlugin.cs
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,10 @@ public sealed class RotationSolverPlugin : IDalamudPlugin, IDisposable
public RotationSolverPlugin(IDalamudPluginInterface pluginInterface)
{
ECommonsMain.Init(pluginInterface, this, ECommons.Module.DalamudReflector, ECommons.Module.ObjectFunctions);
ThreadLoadImageHandler.TryGetIconTextureWrap(0, true, out _);
Svc.Framework.RunOnTick(() =>
{
ThreadLoadImageHandler.TryGetIconTextureWrap(0, true, out _);
});
IconSet.Init();

_dis.Add(new Service());
Expand Down

0 comments on commit 94b3f69

Please sign in to comment.