-
Notifications
You must be signed in to change notification settings - Fork 551
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
fix items not being able to be placed on ancient altar #4094
Conversation
Your Pull Request was automatically labelled as: "✨ Fix" |
Slimefun preview buildA Slimefun preview build is available for testing! https://preview-builds.walshy.dev/download/Slimefun/4094/c77b405c
|
Let's version this, I'd rather not bring back flicker for 1.20 and all future versions. |
armorStand.setRemoveWhenFarAway(false); | ||
}); | ||
// 1.19 and below don't have the consumer method so flicker exists on these versions. | ||
if (Slimefun.getMinecraftVersion().isBefore(MinecraftVersion.MINECRAFT_1_20)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wish we had a isAtMost but we don't haha
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have isatleast
I'm kinda confused I'll admit, I made this PR working in 1.19, I'm going to check Java docs some more cause I'm currently looking at some that say the method exists for spigot 1.19.1 at least |
Exists on 1.16.5 too, checked using: https://jd.andross.fr Simply search "spawn(Location" and you'll find it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving for now till we figure out a way to deal with this, seems to be broken by a signature change
Discord convo for ref: https://discord.com/channels/565557184348422174/565569196218384385/1196787007125471273 |
Description
This bug apparenttly has existed for 7 months.
On 1.19.4 you are not able to place items on an ancient altar you get the following error
https://pastebin.com/87UpB46w
Proposed changes
revert 060b59d
Related Issues (if applicable)
issue reported on discord
https://canary.discord.com/channels/565557184348422174/565569196218384385/1196756895948619797
Checklist
Nonnull
andNullable
annotations to my methods to indicate their behaviour for null values