EngineHub / CraftBook

🔧 Machines, ICs, PLCs, and more!
https://enginehub.org/craftbook/
GNU General Public License v3.0
301 stars 160 forks source link

1.21 Issues (Ammeter, Lightstone, Painting Switcher) #1330

Open andrewkm opened 1 week ago

andrewkm commented 1 week ago

CraftBook Version

CraftBook5 v5.0.0-SNAPSHOT+4845-1e3372c

Platform Version

Paper version 1.21-38-master@1f5db50

Confirmations

Bug Description

Ammeter is broken - our defined tool (coal) doesn't display redstone power strength when used. Lightstone is broken - our defined tool (glowstone_dust) doesn't display light level when used.

Both are working fine on 1.20.6

Painting switcher doesn't rotate through the 20 new 1.21 paintings but seeing we have no config settings I can find to define those I assume its up to Craftbook5 to update and add those paintings(?)

Expected Behavior

To function.

Reproduction Steps

N/A

Anything Else?

If you need any other information please let me know.

me4502 commented 1 week ago

I'll take a look at the two tools, but as for the painting switcher that just sounds like Bukkit hasn't added them to the Art enum yet. CraftBook just uses that, as Bukkit doesn't expose it via registry

me4502 commented 6 days ago

I cannot reproduce any issues with Ammeter or Lightstone, they both work fine for me on 1.21 using the listed Paper build. What else is on your server / are there any console errors? it's likely something else is going on

andrewkm commented 3 days ago

Looks like the origin of the item is what is causing it to not work. The items I was using in my original test were spawned in with /i or /give (minecraft:give). They have 7 components and don't work. https://i.imgur.com/tQuefJt.png

However if I use items naturally generated or via the Creative menu, they do work and have only 6 components: https://i.imgur.com/3W1u04r.png

me4502 commented 3 days ago

CraftBook just compares the base item type using the Bukkit API method to do that. If different components on the item are somehow breaking item type comparisons, that sounds like a bug in Spigot or Paper

Although when I did testing I did it via the /give command

andrewkm commented 3 days ago

CraftBook just compares the base item type using the Bukkit API method to do that. If different components on the item are somehow breaking item type comparisons, that sounds like a bug in Spigot or Paper

Although when I did testing I did it via the /give command

Alright, thanks, I'll investigate further and see if perhaps this is an issue with our item/give commands, specifically on our server.