(ikuria) on hopper picks up item location_flagged:test not working

[misty peak]

picks up works without location_flag, with not
https://paste.denizenscript.com/View/129666
https://paste.denizenscript.com/View/129667

[lime plaza]

(ikuria) on hopper picks up item location_flagged:test not working

[lime plaza]

Hi I'm AutoThreadBot! Don't mind me, I'll just be adding the helper team to this thread so they can see it. A human will get to you soon.

[misty peak]

!logcheck https://paste.denizenscript.com/View/129666

[icy oyster]

Log Check Results

Server Version

Paper version 1.21.4-72-daddcf6 (MC: 1.21.4)

Plugin Version(s)

Citizens: 2.0.37-SNAPSHOT (build 3675) -- (Current build :white_check_mark:)
Denizen: 1.3.1-SNAPSHOT (build 7121-DEV) -- (Current build :white_check_mark:)
Depenizen: 2.1.1 (build 867) -- (Current build :white_check_mark:)
dDiscordBot: 0.7 (build 303) -- (Current build :white_check_mark:)

Checked For

@misty peak

UUID Version

4 (:white_check_mark: Online)

Java Version

21.0.3 :white_check_mark:

Other Noteworthy Plugin(s)

WorldGuard: 7.0.13-SNAPSHOT+2321-5f8e5ea, ProtocolLib: 5.3.0-SNAPSHOT-726

Possibly Relevant Plugin(s)

ViaVersion: 5.2.1-SNAPSHOT - Mixed client vs server versions can sometimes cause packet-related issues.
HeadDatabase: 4.20.4 - HeadDatabase has been known to cause issues with skins.
CMI: 9.7.8.2 - CMI tends to mess with a large variety of server features and often gets in the way of issue debugging.
Multiverse-Core: 4.3.14 - Multi-world configuration plugins may affect NPCs in unexpected ways.
FastAsyncWorldEdit: 2.12.4-SNAPSHOT-1011;7160792 - This plugin has been known to break the plugin load order on many servers, due to usage of the 'loadbefore' directive in its 'plugin.yml'.

[tawdry laurel]

!e hopper picks up item

[icy oyster]

Event: inventory picks up item

Group

Item

Event Lines

<inventory> picks up <item>

Triggers

when a hopper or hopper minecart picks up an item.

Context

<context.inventory> returns the InventoryTag that picked up the item.
<context.item> returns the ItemTag.
<context.entity> returns a EntityTag of the item entity.

Has Known Location

True - this adds switches in:<area> + location_flagged:<flag name>.

Cancellable

True - this adds <context.cancelled> and determines cancelled + cancelled:false.

[tawdry laurel]

The switch uses the location of the item

does the items location has the flag?

[misty peak]

thats strange, no way to get location_flagged for the hopper location?! Oo

or a seperate "location:" option

with this we just have to check <context.inventory.location.has_flag...

[tawdry laurel]

ye

[misty peak]

complicated 😄

[tawdry laurel]

without looking the reason behind that is probably that not every inventory has a location

[ruby spoke]

wouldn't it be more logic to have the location_flagged based on the hopper?

[misty peak]

but picksup are only for hopper and hopper minecart O.o

[tawdry laurel]

https://jd.papermc.io/paper/1.21.4/org/bukkit/event/inventory/InventoryPickupItemEvent.html

lol true

[misty peak]

I have a follow-up question about this: how does it actually affect performance if you check for "has flag" after the event line?

[tawdry laurel]

I'd assume the effect is very tiny

[agile vale]

Thread Closing Reminder

Has your issue been resolved, or your question been answered?
If so, please use the </resolved:1028673926114594866> command to close your thread.
Or </invalid:1028673926898909185> if it's not possible to resolve.

If not yet resolved, please reply below to tell us what you still need.

(Note that if there is no reply for a few days, this thread will eventually close itself.)

@misty peak