fix: failed when trying to imbue items#3891
fix: failed when trying to imbue items#3891dudantas merged 1 commit intoopentibiabr:dudantas/protocol-15.11from
Conversation
Delete the random failure branch in Player::onApplyImbuement that used uniform_random to abort the imbuement, send a failure message and reopen the imbuement window. With this change, once prerequisites are met the code proceeds to apply imbuement stats instead of potentially failing at this random check.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs). Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
669b139
into
opentibiabr:dudantas/protocol-15.11



Description
Delete the random failure branch in Player::onApplyImbuement that used uniform_random to abort the imbuement, send a failure message and reopen the imbuement window. With this change, once prerequisites are met the code proceeds to apply imbuement stats instead of potentially failing at this random check.
Type of change
Please delete options that are not relevant.
How Has This Been Tested
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Checklist