chaosvolt / cdda-arcana-mod

Arcana and Magic Items mod for Cataclysm: Dark Days Ahead
Other
155 stars 62 forks source link

Hermit gives quest "Earn The Favor From Beyond" even when the player cannot complete it. #423

Closed Dracque closed 2 months ago

Dracque commented 2 months ago

Steps to reproduce

  1. Create a character, cross Dragonblood threshold.
  2. Start Hermit quest chain and complete "Retrieve The Cursed Blade" and "Restore The Ritual Blade".
  3. Ask for next quest. The Hermit doesn't take into account that the player has crossed Dragonblood threshold.

Additional context When going through the dialogue options after asking about the second task the Hermit will mention that the player bears the Sanguine Order's transformations when the option "What happened to this one?" is selected. The Hermit also mentions that the player "will not be able to make full use of the blade once it is restored". Giving the restored ritual blade to the Hermit after accepting the third quest, talking to the Hermit again and choosing "It doesn't feel like it's working. What am I doing wrong?" does not mark the quest as failed or completed.

Versions and configuration

chaosvolt commented 2 months ago

Thanks, will look into the dialogue when I can. My rough guess is, the hermit has some dialogue conditions that check for failing due to having a threshold during the preamble for the final mission, but I forgot that a failure check during the preamble of the third mission would be required.

chaosvolt commented 2 months ago

Looking into this now finally, so far the dialogue conditions seem like they should be set up to catch this correctly, but gonna test it some more. I'm assuming that some dialogue path in here is letting you bypass the intended dialogue and ask him for a mission anyway, while the mention of the "it's not working" dialogue in the mission query I assume means that it's not correctly removing the active mission when rerouting the player to the "mission failed" dialogue (or it might even itself be breaking some things due to the line's response being TALK_NONE instead of TALK_DONE).

EDIT: Oh wait I see now, it's because the dialogue trigger for TALK_HERMIT_DISCUSS_3_FAIL, the earliest dialogue response that triggers the premature end to the quest chain, requires you to have worked far enough into the quest chain that the third mission has been completed.

chaosvolt commented 2 months ago

There we go, https://github.com/chaosvolt/cdda-arcana-mod/commit/ea7d54b6ffe8e4eacc7fbb764d633fc5a1642c99 should close both loopholes, but do let me know if you find any other oddities.

Dracque commented 2 months ago

Tested the quest chain with two post threshold characters (Dragonblood and Prime). When asking for the third quest hermit takes threshold into account and doesn't give it (no mission log update).

Issue appears to be entirely fixed.