Skip to content

[lua] Energetic Eruca Audits pt. 2#10154

Open
KiplingFFXI wants to merge 1 commit into
LandSandBoat:basefrom
KiplingFFXI:Energetic-Eruca-Audits-pt.-2
Open

[lua] Energetic Eruca Audits pt. 2#10154
KiplingFFXI wants to merge 1 commit into
LandSandBoat:basefrom
KiplingFFXI:Energetic-Eruca-Audits-pt.-2

Conversation

@KiplingFFXI
Copy link
Copy Markdown
Contributor

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

This PR is related to #10150 - I realized I forgot to include the Eruca family behavior of TP regain on Firesday. However, I did not add the Eruca family mixin specifically because this NM does not go to sleep and become non-aggressive during certain game hours. It only needs the TP regain behavior, so I've added it into the NMs lua instead.

I do not have a recorded capture of the above behavior but it is corroborated by JP wiki.

Steps to test these changes

!spawnmob ID 17027466 and witness the changes.

elseif dayElement ~= xi.element.FIRE and mobArg:getMod(xi.mod.REGAIN) ~= 0 then
mobArg:setMod(xi.mod.REGAIN, 0)
end
end)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if day changes mid-battle, does it not gain regain?
Also, this doesnt need to be a listener, just place it in onMobRoam

And while we are at it, store the mod value in a var aswell so we dont fetch it 2 times.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated

@KiplingFFXI KiplingFFXI force-pushed the Energetic-Eruca-Audits-pt.-2 branch from c6f953d to d6d2047 Compare May 27, 2026 01:30
Co-authored-by: Skold177 <113406182+Skold177@users.noreply.github.com>
@KiplingFFXI KiplingFFXI force-pushed the Energetic-Eruca-Audits-pt.-2 branch from d6d2047 to 72aeb1f Compare May 27, 2026 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants