Skip to content

Commit

Permalink
Merge pull request #567 from FFXIV-CombatReborn/hotforteachingmode
Browse files Browse the repository at this point in the history
Hotforteachingmode
  • Loading branch information
LTS-FFXIV authored Jan 13, 2025
2 parents 1156b97 + cf2f4cb commit cb7dcd8
Show file tree
Hide file tree
Showing 7 changed files with 28 additions and 309 deletions.
6 changes: 3 additions & 3 deletions BasicRotations/Magical/BLM_Beta.cs
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
public sealed class BLM_Beta : BlackMageRotation
{
#region Config Options
[RotationConfig(CombatType.PvE, Name = "Use Infinite Paradox Rotation (Requires Level 90)")]
[RotationConfig(CombatType.PvE, Name = "Use Infinite Paradox Rotation when at level 100)")]
public bool Infinity { get; set; } = false;

[RotationConfig(CombatType.PvE, Name = "(Standard Rotation) Use Leylines in combat when standing still")]
Expand Down Expand Up @@ -140,7 +140,7 @@ protected override bool AttackAbility(IAction nextGCD, out IAction? act)
if (AmplifierPvE.CanUse(out act)) return true;
}

if (ParadoxPvE.EnoughLevel && Infinity)
if (FlareStarPvE.EnoughLevel && Infinity)
{
if (UseMedicine && UseBurstMedicine(out act)) return true;

Expand Down Expand Up @@ -199,7 +199,7 @@ protected override bool GeneralGCD(out IAction? act)
if (MaintainStatus(out act)) return true;
}

if (ParadoxPvE.EnoughLevel && Infinity)
if (FlareStarPvE.EnoughLevel && Infinity)
{

if (InAstralFire && ElementTime < 4 && Player.HasStatus(true, StatusID.Firestarter))
Expand Down
1 change: 0 additions & 1 deletion BasicRotations/RebornRotations.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@
<ItemGroup>
<Compile Include="Duty\EmanationDefault" />
<Compile Include="Magical\ICWA_PCT_BETA" />
<Compile Include="Tank\PLD_Default_Old" />
</ItemGroup>
<ItemGroup>
<None Include="Tank\GNB_Default.cs" />
Expand Down
9 changes: 6 additions & 3 deletions BasicRotations/Tank/PLD_Default.cs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
namespace RebornRotations.Tank;

[Rotation("Beta", CombatType.PvE, GameVersion = "7.15")]
[Rotation("Default", CombatType.PvE, GameVersion = "7.15")]
[SourceCode(Path = "main/BasicRotations/Tank/PLD_Default.cs")]
[Api(4)]

Expand Down Expand Up @@ -32,6 +32,9 @@ public sealed class PLD_Default : PaladinRotation
[RotationConfig(CombatType.PvE, Name = "Use Holy Spirit when out of melee range")]
private bool UseHolyWhenAway { get; set; } = false;

[RotationConfig(CombatType.PvE, Name = "Use Clemency with Requiescat")]
private bool RequiescatHealBot { get; set; } = true;

[Range(0, 1, ConfigUnitType.Percent)]
[RotationConfig(CombatType.PvE, Name = "Minimum HP threshold party member needs to be to use Clemency with Requiescat")]
public float ClemencyRequi { get; set; } = 0.2f;
Expand Down Expand Up @@ -186,8 +189,8 @@ protected override bool HealSingleGCD(out IAction? act)
{
act = null;
if (PassageProtec && Player.HasStatus(true, StatusID.PassageOfArms)) return false;
if (ClemencyPvE.Target.Target?.GetHealthRatio() < ClemencyRequi && RequiescatStacks > 0 && ClemencyPvE.CanUse(out act, skipCastingCheck: true)) return true;
if (HealBot && ClemencyPvE.Target.Target?.GetHealthRatio() < ClemencyNoRequi && ClemencyPvE.CanUse(out act)) return true;
if (RequiescatHealBot && RequiescatStacks > 0 && ClemencyPvE.CanUse(out act, skipCastingCheck: true) && ClemencyPvE.Target.Target?.GetHealthRatio() < ClemencyRequi) return true;
if (HealBot && ClemencyPvE.CanUse(out act) && ClemencyPvE.Target.Target?.GetHealthRatio() < ClemencyNoRequi) return true;
return base.HealSingleGCD(out act);
}

Expand Down
271 changes: 0 additions & 271 deletions BasicRotations/Tank/PLD_Default_Old

This file was deleted.

2 changes: 1 addition & 1 deletion RotationSolver/Commands/RSCommands_Actions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ internal static void UpdateRotationState()
(DataCenter.RightSet.SwitchCancelConditionSet?.IsTrue(DataCenter.RightNowRotation) ?? false))
{
CancelState();
if (Player.Job != _previousJob) _previousJob = Player.Job;
if (Player.Job != null && Player.Job != _previousJob) _previousJob = Player.Job;

Check warning on line 193 in RotationSolver/Commands/RSCommands_Actions.cs

View workflow job for this annotation

GitHub Actions / Build

The result of the expression is always 'true' since a value of type 'Job' is never equal to 'null' of type 'Job?'

Check warning on line 193 in RotationSolver/Commands/RSCommands_Actions.cs

View workflow job for this annotation

GitHub Actions / Build

The result of the expression is always 'true' since a value of type 'Job' is never equal to 'null' of type 'Job?'
if (ActionUpdater.AutoCancelTime != DateTime.MinValue) ActionUpdater.AutoCancelTime = DateTime.MinValue;
}
else if ((Service.Config.AutoOnPvPMatchStart && Svc.Condition[ConditionFlag.BetweenAreas] &&
Expand Down
Loading

0 comments on commit cb7dcd8

Please sign in to comment.