Skip to content
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.

Commit

Permalink
fix: revert pet detection change for now to make summoner usable again.
Browse files Browse the repository at this point in the history
Does not pickup a pet as the pet, in this case Carbuncle at least, returns empty (BattleNpcSubKind.None / 0). BattleNpcSubKind.Pet is 2, so this whole function returns false leading to summoner repeatedly summoning Carbuncle.
  • Loading branch information
RiotNOR committed May 12, 2023
1 parent 78c0ff5 commit 953011f
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion RotationSolver/Updaters/TargetUpdater.cs
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,9 @@ private unsafe static void UpdateFriends(IEnumerable<BattleChara> allTargets)
DataCenter.PartyMembers = GetPartyMembers(allTargets);
DataCenter.AllianceMembers = allTargets.OfType<PlayerCharacter>();

DataCenter.HasPet = HasPet();
var mayPet = allTargets.OfType<BattleNpc>().Where(npc => npc.OwnerId == Service.Player.ObjectId);
DataCenter.HasPet = mayPet.Any(npc => npc.BattleNpcKind == BattleNpcSubKind.Pet);
//DataCenter.HasPet = HasPet();

DataCenter.PartyTanks = DataCenter.PartyMembers.GetJobCategory(JobRole.Tank);
DataCenter.PartyHealers = DataCenter.PartyMembers.GetJobCategory(JobRole.Healer);
Expand Down

0 comments on commit 953011f

Please sign in to comment.