Skip to content

Add Parike-Poranke, Brown Mage Practitioner#10127

Open
TiberonKalkaz wants to merge 1 commit into
LandSandBoat:basefrom
TiberonKalkaz:Brown_Mage
Open

Add Parike-Poranke, Brown Mage Practitioner#10127
TiberonKalkaz wants to merge 1 commit into
LandSandBoat:basefrom
TiberonKalkaz:Brown_Mage

Conversation

@TiberonKalkaz
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?

  1. Adds the Brown Mage, Parike-Poranke
  2. Adds a new setting to xi.settings.main for servers to determine if they wish persist his skill values between restarts, thereby writing updates to ServerVars instead of LocalVars.
    Note: I considered VolatileServerVars but given the read/write ratio - I did not run with that idea. "With Food "is a 1:1 ratio, "Without Food" is a 2:1 read ratio - so the befit would be slim to none.

Steps to test these changes

Check the following:

  1. Talk to NPC without food active - No "CS" fade, Parike-Poranke talks about people walking by
  2. Talk to NPC with food active - CS fade, Parike-Poranke suggests you trade a Gysahl Green
  3. Trading Gyshal Green with food active - CS fade, increments success counter
  4. Trading Gyshal Green without food active - CS fade, increments failure counter

Without maniuplating lua or the database via the setting PERSIST_BROWN_MAGIC_SKILL - the following are very hard to test:
5. 1,000th trade with food - Parike-Poranke gets a .1 skill up to Digestive
6. 1,000th trade without food - Parike-Poranke gets a .1 skill up to Science
7. 10,000th trade with food - Parike-Poranke gets "one" skill level of Digestive.
8. Player gets the BROWN_MAGE_GUINEA_PIG title. Player gets a RR3 effect
9. 10,000th trade without food - Parike-Poranke gets a .1 skill up to Science
8. Player gets the BROWN_MAGIC_BY_PRODUCT title. Player gets a Flee effect

Fun / Interesting Note

Parike-Poranke has his own Debug Menu - triggered by passing Event 10044 Param7 as anything but 10000, 10001, 10002, and 10003.
This Debug Menu contains the entries "Success 999, Sucess 9999, Failure 999, Failure 9999".
Then, the CS would playout based on the values set via the menu.
I have not implemented this Debug functionality.
image

Less Fun Note

As the capture playlist below shows - I have spent a long time trying to find a way to trigger the event path that exists where Parike-Poranke would stop, mid food expulsion, and say "Wait a minute... My linkshell's ringing. Let me get back to you after I take this call.".

This is entirely controlled serverside during an event update. For the life of me, I dont know how this happens and while I will continue to look for ways to trigger this event, I believe I've ruled out the following:

  • Interactions with LB 7 and LB 8 (Parike-Poranke appears in these LBs) - and LB 9
  • Spam Trading Parike-Poranke with fps 0

Captures and Information

@TiberonKalkaz TiberonKalkaz force-pushed the Brown_Mage branch 2 times, most recently from 59f0a4a to 223d708 Compare May 25, 2026 21:09
@TiberonKalkaz TiberonKalkaz marked this pull request as ready for review May 25, 2026 21:26
Comment thread scripts/zones/Lower_Jeuno/npcs/Parike-Poranke.lua Outdated
Comment thread scripts/zones/Lower_Jeuno/npcs/Parike-Poranke.lua
Comment thread settings/default/main.lua Outdated
@TiberonKalkaz TiberonKalkaz force-pushed the Brown_Mage branch 2 times, most recently from 5004015 to 672ef26 Compare May 26, 2026 23:18
Comment thread scripts/zones/Lower_Jeuno/npcs/Parike-Poranke.lua Outdated
@Xaver-DaRed Xaver-DaRed enabled auto-merge May 27, 2026 00:17
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