Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Players can reloot chests even when restrict-player-reloot is true #9613

Closed
EladJenkins opened this issue Aug 16, 2023 · 1 comment · Fixed by #9615
Closed

Players can reloot chests even when restrict-player-reloot is true #9613

EladJenkins opened this issue Aug 16, 2023 · 1 comment · Fixed by #9615
Labels
priority: high This issue is either a gamebreaking bug or crash and needs to be addressed soon. status: accepted Disputed bug is accepted as valid or Feature accepted as desired to be added. type: bug Something doesn't work as it was intended to.

Comments

@EladJenkins
Copy link

Expected behavior

It is expected that players are unable to loot a chest more than once with the restrict-player-reloot configuration option set to true.

Observed/Actual behavior

Players can loot a chest more than once.

Steps/models to reproduce

Start a fresh 1.20.1 Paper server with build 119.

Set the following in paper-world-defaults.yml:
lootables:
auto-replenish: true
refresh-max: 15s
refresh-min: 15s

Log in, find a lootable chest and loot it.

Observe that after 15 seconds, you are unable to loot the chest again (it remains empty).

Upgrade to Paper build 120.

Log in and reloot the same chest repeatedly every 15 seconds.

Plugin and Datapack List

None, I used a fresh installation to test.

Paper version

This server is running Paper version git-Paper-120 (MC: 1.20.1) (Implementing API version 1.20.1-R0.1-SNAPSHOT) (Git: 20a71c5)

Other

This happens even on build 126. It was noticed after I updated my production Paper server from build 111 to 126.
@EladJenkins EladJenkins added status: needs triage type: bug Something doesn't work as it was intended to. labels Aug 16, 2023
@papermc-projects papermc-projects bot moved this to 🕑 Needs Triage in Issues: Bugs Aug 16, 2023
@Nacioszeczek Nacioszeczek added status: accepted Disputed bug is accepted as valid or Feature accepted as desired to be added. version: 1.20 and removed status: needs triage labels Aug 16, 2023
@papermc-projects papermc-projects bot moved this from 🕑 Needs Triage to ✅ Accepted in Issues: Bugs Aug 16, 2023
@Nacioszeczek Nacioszeczek added the priority: high This issue is either a gamebreaking bug or crash and needs to be addressed soon. label Aug 16, 2023
@lynxplay
Copy link
Contributor

Can you please provide the entire world default configuration.

@lynxplay lynxplay mentioned this issue Aug 16, 2023
Merged
@github-project-automation github-project-automation bot moved this from ✅ Accepted to Done in Issues: Bugs Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority: high This issue is either a gamebreaking bug or crash and needs to be addressed soon. status: accepted Disputed bug is accepted as valid or Feature accepted as desired to be added. type: bug Something doesn't work as it was intended to.
Projects
Issues: Bugs
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't let player loot if reloot time is disabled lynxplay/paper
3 participants
@lynxplay @EladJenkins @Nacioszeczek