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.

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

Trapped chest flag issue #1912

Closed
ypxo-mc opened this issue Jan 5, 2022 · 3 comments
Closed

Trapped chest flag issue #1912

ypxo-mc opened this issue Jan 5, 2022 · 3 comments
Assignees
Labels
Status: Done This issue has been completed or answered. This pull request has been merged. Type: Bug

Comments

@ypxo-mc
Copy link

ypxo-mc commented Jan 5, 2022

Expected behavior

If island owner has enabled trapped chest to anyone, players can't steal items from double chest, only see chest content.

Observed/Actual behavior

If player open enabled trapped double chest, he can manipulate with items in chest.

Steps/models to reproduce

  1. Create new island by player x. Place double trapped chest to island.
  2. Enable trapped chest to anyone.
  3. Use account player y, open trapped chest on player x's island.
  4. Try to steal items from chest.

BentoBox version

Running PAPER 1.17.1.
BentoBox version: 1.19.0-SNAPSHOT-b${env.BUILD_NUMBER}
Database: JSON
Loaded Game Worlds:
ONEBLOCK • oneblock_world (OneBlock): Overworld, [nether_color]Nether, [end_color]End
Loaded Addons:
AOneBlock 1.9.0 (ENABLED)
Biomes 1.7.0 (ENABLED)
Challenges 0.8.3 (ENABLED)
Chat 1.1.1 (ENABLED)
Likes 2.0.0 (ENABLED)
Limits 1.14.0 (ENABLED)
Warps 1.11.2 (ENABLED)

Plugin list

AnimatedScoreboard, ArmorStandTools, BentoBox, ChestShop, Citizens, CMILib, CombatLogX, Craftconomy3, DeluxeChat, DeluxeChatHex, DeluxeMenus, Essentials, FarmControl, FastAsyncWorldEdit (WorldEdit), GSit, HeadDatabase, HolographicDisplays, IllegalStack, InventoryRollback, ItemJoin, Jobs, KixsAutoAnnouncerPremium, LeaderHeadsRevamped, LogBlock, LuckPerms, MarketPlace, MineableSpawners, Multiverse-Core, Multiverse-Inventories, OpenInv, PinataParty, PlaceholderAPI, PlayerHeads, ProtocolLib, Residence, ShopGUIPlus, SuperbVote, SuperTrailsPro*, SuperVanish, TAB, TradeSystem, Vault*, ViaBackwards, ViaPotions, ViaRewind, ViaRewind-Legacy-Support, ViaVersion, Votifier, WeatherVote*, WorldBorder, WorldGuard
and some custom plugins

Other

No response

@tastybento
Copy link
Member

@ypxo-mc Can you clarify? You say that if you put a trapped chest on and island and allow access to trapped chests for any player, then they can access the trapped chest. I don't see the bug. If you give access, then they can open the chest. Please explain more.

@tastybento tastybento added Type: Not a bug Works as intended / not a BentoBox bug. and removed Type: Bug labels Jan 8, 2022
@adameeek
Copy link

adameeek commented Jan 9, 2022

@ypxo-mc Can you clarify? You say that if you put a trapped chest on and island and allow access to trapped chests for any player, then they can access the trapped chest. I don't see the bug. If you give access, then they can open the chest. Please explain more.

It doesn't work with a double trapped chest, it works with a single trapped chest

@tastybento tastybento self-assigned this Jan 10, 2022
@tastybento tastybento added Status: Under investigation Investigating the interest and the feasability of the issue. Type: Bug and removed Type: Not a bug Works as intended / not a BentoBox bug. labels Jan 10, 2022
@tastybento
Copy link
Member

Okay, got it.

@tastybento tastybento added Status: In progress Working on the issue. and removed Status: Under investigation Investigating the interest and the feasability of the issue. labels Jan 10, 2022
@tastybento tastybento added Status: Done This issue has been completed or answered. This pull request has been merged. and removed Status: In progress Working on the issue. labels Jan 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Done This issue has been completed or answered. This pull request has been merged. Type: Bug
Projects
None yet
Development

No branches or pull requests

3 participants