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

Drone death does not respect Curse of Vanishing #1355

Open
bartico6 opened this issue Oct 6, 2024 · 5 comments
Open

Drone death does not respect Curse of Vanishing #1355

bartico6 opened this issue Oct 6, 2024 · 5 comments
Labels
Bug Fixed in Dev Issue will be closed after next release

Comments

@bartico6
Copy link

bartico6 commented Oct 6, 2024

Describe the bug

When a drone using curse of vanishing items is killed, it drops them on death as opposed to voiding them.
This means base defense setups in PvP, where you want to deny strong weapons your drones are using to players who manage to take them down would not work - as vanishing items would not, well, vanish,

How to reproduce the bug

Give a drone a curse of vanishing weapon, get it to equip it (such as via an entity attack command) then kill it.

Expected behavior

The drone doesn't drop the item with curse of vanishing on it upon death.

Additional details

No response

Which Minecraft version are you using?

1.20.1

Which version of PneumaticCraft: Repressurized are you using?

6.0.17

Crash log

No response

@bartico6 bartico6 added the Bug label Oct 6, 2024
@bartico6
Copy link
Author

Late addition, but for convenience sake, it should be differentiated between wrench "kills" (dismantling) and every other kill type - it might make sense (or maybe not? worth discussing balance-wise) to only let drones drop their vanish-cursed items if they're dismantled, but not when killed.

@desht
Copy link
Member

desht commented Oct 13, 2024

I guess the intention is to allow you, the drone owner, to retrieve those vanishing items if you need to wrench a drone for reprogramming purposes etc? Which kinda makes sense, as long as the items only drop if the drone is wrenched by the owning player...

desht added a commit that referenced this issue Oct 13, 2024
Exception: will still drop them if wrenched by the owning player

#1355
@desht desht added the Fixed in Dev Issue will be closed after next release label Oct 13, 2024
@bartico6
Copy link
Author

I guess the intention is to allow you, the drone owner, to retrieve those vanishing items if you need to wrench a drone for reprogramming purposes etc? Which kinda makes sense, as long as the items only drop if the drone is wrenched by the owning player...

Yep, that's exactly right - and appreciate the swift fix <3

@desht desht reopened this Oct 15, 2024
@desht
Copy link
Member

desht commented Oct 15, 2024

Keeping this open for now, till a release is done

@desht
Copy link
Member

desht commented Oct 20, 2024

Fixed in 8.2.0 for 1.21.1, will get 1.20.1 release out soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fixed in Dev Issue will be closed after next release
Projects
None yet
Development

No branches or pull requests

2 participants