Skip to content

[lua] Add level penalty to picklocking Treasure Chests / Treasure Coffers#9991

Merged
Xaver-DaRed merged 1 commit into
LandSandBoat:basefrom
Hatberg:treasure-picklock-level-penalty
May 8, 2026
Merged

[lua] Add level penalty to picklocking Treasure Chests / Treasure Coffers#9991
Xaver-DaRed merged 1 commit into
LandSandBoat:basefrom
Hatberg:treasure-picklock-level-penalty

Conversation

@Hatberg
Copy link
Copy Markdown
Contributor

@Hatberg Hatberg commented May 8, 2026

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?

Adds decreased success/increased failure rate to opening Treasure Coffers and Treasure Chests when using picklocking tools.
(Leaves Trap/Mimic rate unaffected)

Based on @ThrisStraizo treasure chest and treasure coffer picking tests, a 5% penalty per THF level under the treasureLevel is applied.

Addresses bug #9986

Steps to test these changes

!changejob thf 1
!zone FeiYin
!additem living_key 99
!gotoname Treasure_Chest
Trade Living Key to Treasure Chest

  • It should no longer be possible for low level players to pick locks, unless they are within a certain level range.
  • Players that match or exceed the treasureLevel are unaffected.
  • Zone Key users are unaffected.

@Xaver-DaRed Xaver-DaRed force-pushed the treasure-picklock-level-penalty branch 2 times, most recently from ec3655d to 32f14ed Compare May 8, 2026 10:33
@Xaver-DaRed Xaver-DaRed merged commit d23cb38 into LandSandBoat:base May 8, 2026
18 of 19 checks passed
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