Wither Skeleton Skull is kept in inventory if placed in survival #2416
Labels
No Label
#P1 CRITICAL
#P2: HIGH
#P3: elevated
#P4 priority: medium
#P6: low
#Review
annoying
API
bug
code quality
combat
commands
compatibility
configurability
contribution inside
controls
core feature
creative mode
delayed for engine release
documentation
duplicate
enhancement
environment
gameplay
graphics
ground content conflict
GUI/HUD
help wanted
incomplete feature
invalid / won't fix
items
looking for contributor
mapgen
meta
mineclone2+
Minecraft >= 1.13
Minecraft >= 1.17
missing feature
mobile
mobs
mod support
model needed
multiplayer
Needs adoption
needs discussion
needs engine change
needs more information
needs research
nodes
non-Minecraft feature
non-mob entities
performance
player
possible close
redstone
release notes
schematics
Skyblock
sounds
Testing / Retest
tools
translation
unconfirmed
mcl5
mcla
Media missing
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: VoxeLibre/VoxeLibre#2416
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
title
Sure you didn't do /gamemode creative? Because that seems quite strange
Ill try it later though
Yea, confirmed.
This is gonna be fun to debug
~~I've been taking a look; since earlier.
Seems to just be missing a "return itemstack"
From the following section:
https://git.minetest.land/MineClone2/MineClone2/src/branch/master/mods/ENTITIES/mcl_wither_spawning/init.lua#L50~~
Edit: Nvm; weird, it worked in my first test world, but not in a new world...
Edit2: Wait, it does work, mostly; I was just dumb & had commented out my fix again.
Though it seems laggy; like it take a little to update when placing them down.
Edit3: Sorry, okay; now I see. It fixes most placement; like on the ground. But, it's still bugged if you place a head onto the front of another head.
Probably something to do with the mcl_heads "wall" code.
aw spoilers hehe
yeah sounds about right. might be best to have on_place immediately return old_on_place() when no soul sand below the head.
the unconditional minetest.after() is kinda sus tbh
just adding return in front of the old_on_place call fixes it though
i still kinda wanna protect that mt.after
fixed by merge of #2417