WorldGuard

WorldGuard

8M Downloads

Hoppers in a region with chest-access deny pops outs when transfering an item

LadyCailinBot opened this issue ยท 16 comments

commented

WORLDGUARD-3253 - Reported by val59000

I'm using WorldGuard 6.0.0-SNAPSHOT.1477 and I have a major issue with hoppers in region where the flag chest-access is set to deny. As soon as one of these hoppers try to transfer an item, the hopper pops out and drops the hopper item and the item that was transferring into.

To reproduce.

  • Create a region (any size)
  • Set the flag chest-access to deny
  • Place some hoppers pointing to each other
  • Place an item in a hopper that would normally transfer into the next hopper
  • Look at the hopper which pops out

This is a major issue because all hopper-based clocks in these regions break as soon as the chunks are loaded.

Thanks

commented

Comment by sk89q

Wait, why are you setting chest-access to deny?

commented

Comment by val59000

Because I need to restrict achest to chests

commented

Comment by val59000

Access to chests *

commented

Comment by PseudoKnight

Regions deny access to chests by default. So what about your setup requires the flag to be set to deny?

commented

Comment by val59000

It doesnt mattter, it's still a bug right ?

commented

Comment by PseudoKnight

It was known this would happen when the current system was implemented. "chest-access deny" blocks owners. Hoppers act as owners. Thus you blocked hoppers. It's not so much a bug as a design decision that people may or may not agree with. I don't like how the new USE flag works, but this particular case seems like a non-issue unless you can present a reason why someone would need to set chest-access deny for everyone. And that's why you're being asked.

commented

Comment by val59000

I understand what you mean. I'll check again and see if I don't have the bug when not setting any flag. It it's OK then I'll consider this issue as a non-issue too. I'll get back here in a day or two.

commented

Comment by val59000

Ok I did some testing and I'm sorry to say PseudoKnight that you are wrong. chest-access isn't denied by default in last snapshots for Spigot 1.8. (I'm using build 1488)
Thus, I need to set it deny to prevent people from picking items in hoppers. BUt when I use "chest-access deny", it makes the hoppers pop out when transfering items to the following container (another hopper or dispenser, chest ...).

So, this is really a bug as you can't prevent a random player from picking items in hoppers without reproducing this bug.

commented

Comment by AgentLV

I can confirm the "bug", happening in regions, where I didn't set a chest-access deny flag or any flag at all, when regions overlap. At the border of the regions, you can not place a hopper looking into the other region.

commented

Comment by PseudoKnight

That's a separate issue, AgentLV, that applies to any block source event that's in region protection.

val59000, chest-access IS denied by default because permissions are denied to non-members of regions. (unless you're talking about global, which is a special case)

commented

Comment by sk89q

chest-access is indeed denied by default.

commented

Comment by sk89q

Well, this is "fixed" in that I completely disabled checks on hopper<->* item transfers, but you should not need to set chest-access=deny.

commented

Comment by val59000

Alright, thank you :)

commented

Comment by val59000

Sorry for reopening this issue. But I found a case where I still have the bug.
I have a large region where I want to allow people from using chest and buttons, so with the flag "use allow" and "chest-access allow".
And in that big region I have a hopper and a chest which I don't want anyone to open. So I create a small region around it (inside the big region) and I set the flags to "use deny" , "chest-access deny"
And when the hopper is transfering items, it pops out as I described before.

commented

Comment by PseudoKnight

What build?

commented

Comment by val59000

My mistake, I thought I was using build 1507 but I wasn't
It works perfectly with 1507, thanks !