i am having an issue where if i try to spill a partially empty water bucket it picks up the water in the form of a block like the one in creative

salty never misses

Would it be possable to have it still play th pickup soud or have that as an option?

Same issue as Syfe. Booted from server when I hit the hotkey.

[Error] Exception: Object reference not set to an instance of an object.
   at Vintagestory.API.Datastructures.SyncedTreeAttribute.MarkPathDirty_Patch0(SyncedTreeAttribute this, String path)
   at InstantPickUp.InstantPickupState.SetOn(String playerUid, Boolean enable)
   at InstantPickUp.InstantPickUpModSystem.<>c__DisplayClass5_0.<StartServerSide>b__0(IServerPlayer player, ToggleMsg msg)
   at Vintagestory.Server.NetworkAPI.HandleCustomPacket(Packet_Client packet, ConnectedClient client) in VintagestoryLib\Server\API\NetworkAPI.cs:line 59
   at Vintagestory.Server.ServerMain.HandleClientPacket_mainthread(ReceivedClientPacket cpk) in VintagestoryLib\Server\ServerMainNetworking.cs:line 280
   at Vintagestory.Server.ServerMain.ProcessMain() in VintagestoryLib\Server\ServerMain.cs:line 934

I also noticed the pick up is slower then previous versions. For example, i have modified xskill for more drop rates on clay per block, so it can potentially drop 6x the amount of clay per block, and i noticed that at 20 blocks, the rate that it's picking up is super slow and not instantatious. 

I also no longer get an enable/disable notification after updating from 1.0.5 to 1.1.1.

Hallowedsoul08

It's still here!!

I miss the notification that it's on or off.

hitting the hotkey in my server kicks me with this error:

14.10.2025 20:33:39 [Server Error] Exception: Object reference not set to an instance of an object.
   at Vintagestory.API.Datastructures.SyncedTreeAttribute.MarkPathDirty_Patch0(SyncedTreeAttribute this, String path)
   at InstantPickUp.InstantPickupState.SetOn(String playerUid, Boolean enable)
   at InstantPickUp.InstantPickUpModSystem.<>c__DisplayClass5_0.b__0(IServerPlayer player, ToggleMsg msg)
   at Vintagestory.Server.NetworkAPI.HandleCustomPacket(Packet_Client packet, ConnectedClient client) in VintagestoryLib\Server\API\NetworkAPI.cs:line 59
   at Vintagestory.Server.ServerMain.HandleClientPacket_mainthread(ReceivedClientPacket cpk) in VintagestoryLib\Server\ServerMainNetworking.cs:line 280
   at Vintagestory.Server.ServerMain.ProcessMain() in VintagestoryLib\Server\ServerMain.cs:line 934

getting a weird error that spams into my dedicated server that maxes its errors out and causes the server to reboot. it happens notably when i drop an item in creative. this is the error that it gives:

[Server Error] Exception: Object reference not set to an instance of an object.
   at Vintagestory.Common.InventoryPlayerCreative.GetBestSuitedSlot(ItemSlot sourceSlot, ItemStackMoveOperation op, List`1 skipSlots) in VintagestoryLib\Common\GameContent\Inventory\InventoryPlayerCreative.cs:line 263
   at Vintagestory.Common.PlayerInventoryManager.GetBestSuitedSlot(ItemSlot sourceSlot, Boolean onlyPlayerInventory, ItemStackMoveOperation op, List`1 skipSlots) in VintagestoryLib\Common\GameContent\Inventory\PlayerInventoryManager.cs:line 218
   at InstantPickUp.InstantPickUp.PlayerHasSpaceFor(IServerPlayer player, ItemStack stack)
   at InstantPickUp.InstantPickUp.OnServerTick(Single dt)
   at Vintagestory.Common.GameTickListener.OnTriggered(Int64 ellapsedMilliseconds) in VintagestoryLib\Common\Model\GameTickListener.cs:line 25
   at Vintagestory.Common.EventManager.TriggerGameTick(Int64 ellapsedMilliseconds, IWorldAccessor world) in VintagestoryLib\Common\EventManager.cs:line 102
   at Vintagestory.Server.ServerMain.Process() in VintagestoryLib\Server\ServerMain.cs:line 859

I noticed a strange bug and traced it back to this mod. I was building and a bowtorn spawned and shot at me while I had the instant pick up enabled. Instead of taking damage, 1 granite rock was added to my inventory.  When I disabled instant pick up, I took damage like normal.

@sunflowers  ahhhh this is why im here .. kept crashing.  Now i understand why

Is it possible to add a check to skip space-checking for Creative players? It’s firing every tick (OnServerTick) so it’s spamming my logs which is causing a crash if i enter creative( i suppose i could turn it off befopre but when i do it still crashed

The bit is here:

InventoryPlayerCreative.GetBestSuitedSlot(...)

PlayerInventoryManager.GetBestSuitedSlot(...)

InstantPickUp.PlayerHasSpaceFor(...)

Love this mod, but I did find a bit of an exploit - as long as it's enabled, bowtorns can't shoot you. Instead, they take a shot, the projectile disappears, and you get a granite stone in your inventory xD Maybe add an exception for projectiles?

Would it be possible to add compatibility for loot filter?

Other idea is instead of instant pick up all items on distance, just reduce the cooldown of picking up items like the "Pick up fast mod". That way items wont fly to player, player still need to reach items out of his range, but when youre harvesting stuff you pick up instant instead of the 2 secs of cooldown to pick up items from vanilla.

Seconding that it would be nice to have an option to configure instant pick up to be either a toggle or a blip.

Thanks for the mod, love your passion and dedication and really appreciate it!

May I request an option for a pickup hotkey rather than the pickup being a toggle?

Borderlands had a loot-all by holding E I believe it was, felt really good using it. It's less magic, just saves you the effort of spamming the loot action.

Just my opinion here.

• Add a hotkey to fire off a single instant pickup action.
• Add a delay on the hotkey, hold time, so it can't be spammed.

Now you can drop items and still have the instant pickup without having to run in circles :). This should also reduce the process load, especially when a player has a full inventory, as the game won't continuously pull items towards the player. It might help lessen crashes/client/server load?

The points below may also help alleviate some concerns people might have as you can currently reach beyond the vanilla range.

• Make the default radius horizontal only
• Limit vertical range in high to vanilla reach whilst jumping, and below feet to reach whilst crouching (not sure if crouching it makes a difference)

Sorry, this ended up being a LOT! Maybe a mod that fits between the "Click to Pick Up" and "Instant Pick Up" is needed ;D

This mod randomly causes an explosion of hundreds of the last item it failed to pick up, which almost always crashes my game, This doesn't happen everytime, But every like 10 to 20  minutes this will occour. The items aren't real but the mod will continue to duplicate the fake items until i toggle it off with the hotkey. Could you please just make a version that just set the built in pickup delay so 0.0 instead of the default like 0.5 seconds or some shit. This is just too buggy of a solution to a simple issue imo.

Can you implement an optional mode that checks to see if your inventory is full before instantly picking it up, so we cant cheat by just having a skirt of items around us?

Creates floating items on the ground if your inventory is full and you try picking something up

Sunflowers Works fine to me. 40+ mods installed.

crashes when i  pick up with this

Game crashes consistently when i turn this on and forget to use tongs to pick up something hot. After about a second or two of repeated pickup attempts the game gives up the ghost

Very usefull mod, but I got sometimes crash casued by items with changed stacks by another mods (eg. native copper or clay). Love the radius but sometimes bigger trees have bounced items and it needs relog to the world to pick up

Jedzia You're Welcome friend!

Thank you :)