Skip to content

Commit

Permalink
Merge pull request #470 from kaedys/quickfix/level-check-gte
Browse files Browse the repository at this point in the history
[QUICKFIX] Level checks greater-than-or-equal-to
  • Loading branch information
MKhayle authored Nov 23, 2024
2 parents fb8dbad + b8bb8d0 commit 1c93b05
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 9 deletions.
4 changes: 2 additions & 2 deletions XIVComboExpanded/Combos/BLM.cs
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ protected override uint Invoke(uint actionID, uint lastComboMove, float comboTim
return BLM.Despair;
if (HasEffect(BLM.Buffs.Firestarter)) // rare case usually caused by rotation error or clipping
return BLM.Fire3;
if (level > BLM.Levels.Paradox && gauge.IsParadoxActive) // typical use case
if (level >= BLM.Levels.Paradox && gauge.IsParadoxActive) // typical use case
return BLM.Paradox;
if (level >= BLM.Levels.Despair && LocalPlayer?.CurrentMp >= 800) // suboptimal despair usage, but still better than Blizzard3
return BLM.Despair;
Expand Down Expand Up @@ -165,7 +165,7 @@ protected override uint Invoke(uint actionID, uint lastComboMove, float comboTim
if (IsEnabled(CustomComboPreset.BlackEnochianUmbralIceFeature) && gauge.UmbralIceStacks < 3)
{
if (level >= BLM.Levels.Blizzard3 && (HasEffect(BLM.Buffs.Swiftcast) || HasEffect(BLM.Buffs.Triplecast)))
return BLM.Blizzard3;
return BLM.Blizzard3;
return OriginalHook(BLM.Blizzard);
}

Expand Down
2 changes: 1 addition & 1 deletion XIVComboExpanded/Combos/MNK.cs
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ protected override uint Invoke(uint actionID, uint lastComboMove, float comboTim
return OriginalHook(MNK.MasterfulBlitz);
}

if (gauge.RaptorFury == 0 && level > MNK.Levels.TwinSnakes)
if (gauge.RaptorFury == 0 && level >= MNK.Levels.TwinSnakes)
return MNK.TwinSnakes;
}

Expand Down
12 changes: 6 additions & 6 deletions XIVComboExpanded/Combos/RPR.cs
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ protected override uint Invoke(uint actionID, uint lastComboMove, float comboTim
return RPR.Perfectio;
}

if (level > RPR.Levels.BloodStalk && IsEnabled(CustomComboPreset.ReaperAutoSoulReaverFeature))
if (level >= RPR.Levels.BloodStalk && IsEnabled(CustomComboPreset.ReaperAutoSoulReaverFeature))
{
// Blood Stalk if we're over 90 gauge, or if we're over 50 and the overcap-only feature isn't enabled,
// or if the auto-Soul Slice feature is enabled and our next action would otherwise be Soul Slice.
Expand All @@ -200,7 +200,7 @@ protected override uint Invoke(uint actionID, uint lastComboMove, float comboTim
}
}

if (level > RPR.Levels.SoulSlice && IsEnabled(CustomComboPreset.ReaperAutoSoulSliceFeature))
if (level >= RPR.Levels.SoulSlice && IsEnabled(CustomComboPreset.ReaperAutoSoulSliceFeature))
{
if (IsCooldownUsable(RPR.SoulSlice) && gauge.Soul <= 50)
return RPR.SoulSlice;
Expand Down Expand Up @@ -281,7 +281,7 @@ protected override uint Invoke(uint actionID, uint lastComboMove, float comboTim
return RPR.HarvestMoon;
}

if (level > RPR.Levels.GrimSwathe && IsEnabled(CustomComboPreset.ReaperAutoSoulReaverFeature))
if (level >= RPR.Levels.GrimSwathe && IsEnabled(CustomComboPreset.ReaperAutoSoulReaverFeature))
{
// Blood Stalk if we're over 90 gauge, or if we're over 50 and the overcap-only feature isn't enabled,
// or if the auto-Soul Slice feature is enabled and our next action would otherwise be Soul Slice.
Expand All @@ -298,7 +298,7 @@ protected override uint Invoke(uint actionID, uint lastComboMove, float comboTim
}
}

if (level > RPR.Levels.SoulScythe && IsEnabled(CustomComboPreset.ReaperAutoSoulSliceFeature))
if (level >= RPR.Levels.SoulScythe && IsEnabled(CustomComboPreset.ReaperAutoSoulSliceFeature))
{
if (IsCooldownUsable(RPR.SoulScythe) && gauge.Soul <= 50)
return RPR.SoulScythe;
Expand Down Expand Up @@ -461,7 +461,7 @@ protected override uint Invoke(uint actionID, uint lastComboMove, float comboTim
return OriginalHook(RPR.Gallows);
}

if (level > RPR.Levels.BloodStalk && IsEnabled(CustomComboPreset.ReaperAutoSoulReaverFeature))
if (level >= RPR.Levels.BloodStalk && IsEnabled(CustomComboPreset.ReaperAutoSoulReaverFeature))
{
if (gauge.Soul >= 50 && (gauge.Soul > 50 ||
!IsEnabled(CustomComboPreset.ReaperReaperAutoBloodStalkOvercapFeature)))
Expand Down Expand Up @@ -520,7 +520,7 @@ protected override uint Invoke(uint actionID, uint lastComboMove, float comboTim
if (HasEffect(RPR.Buffs.SoulReaver) || HasEffect(RPR.Buffs.Executioner))
return OriginalHook(RPR.Guillotine);

if (level > RPR.Levels.GrimSwathe && IsEnabled(CustomComboPreset.ReaperAutoSoulReaverFeature))
if (level >= RPR.Levels.GrimSwathe && IsEnabled(CustomComboPreset.ReaperAutoSoulReaverFeature))
{
if (gauge.Soul >= 50 && (gauge.Soul > 50 ||
!IsEnabled(CustomComboPreset.ReaperReaperAutoBloodStalkOvercapFeature)))
Expand Down

0 comments on commit 1c93b05

Please sign in to comment.