Iteminfo gives an error

Question

Iteminfo gives an error

x1p opened this issue 4 years ago · 9 comments

x1p commented 4 years ago

Plugin Version

ChestShop build 263

Plugin Config

Config

[Put the config here]

Server Version

1.16.3

Server Log

Log

No error on the console/log file The only clue is in ChestShop.log:

2021/01/04 19:11:19 Error while generating shop sign item name

What other plugins are you running?

WE 7.2.1. WG 7.0.4

What is happening?

Iteminfo gives me this error:
Me doing: /iteminfo with an enchanted book in my hand
Full Name: Enchanted Book#3X

The Shop Sign is missing in this message

What did you expect to happen?

Work as usual to give me the shop sign name.
Other books work fine. Only this book not. It's the thorn 3 enchantment, but that is prob irrelevant because it works on a test server.

Additional context

andris155 commented 4 years ago

items.zip

Phoenix616 · Answer 1 · 2021-01-04T20:01:06.000Z

Please provide the full server log as well as your config.

x1p · Answer 2 · 2021-01-04T22:10:45.000Z

After /chestshop reload, it does not output any more than we know now. So I have to await a restart by night.

Edit: I did (un)load with plugman. But no clues on the console

x1p · Answer 3 · 2021-01-04T22:29:21.000Z

Ok, I made a shop with this particular book. Maybe this helps.

[23:27:38 ERROR]: Could not pass event PreShopCreationEvent to ChestShop v3.11.1-SNAPSHOT (build 263)
java.lang.IllegalArgumentException: Cannot generate code for item ItemStack{ENCHANTED_BOOK x 1, ENCHANTED_META:{meta-type=ENCHANTED, stored-enchants={SOUL_SPEED=3}}} with maximum length of 90 (code EnchanteBook#3x results in item ItemStack{ENCHANTED_BOOK x 1, ENCHANTED_META:{meta-type=ENCHANTED, enchants={DURABILITY=1}}})
at com.Acrobot.ChestShop.Utils.ItemUtil.getName(ItemUtil.java:82) ~[?:?]
at com.Acrobot.ChestShop.Utils.ItemUtil.getSignName(ItemUtil.java:97) ~[?:?]
at com.Acrobot.ChestShop.Listeners.PreShopCreation.ItemChecker.onPreShopCreation(ItemChecker.java:63) ~[?:?]
at com.destroystokyo.paper.event.executor.StaticMethodHandleEventExecutor.execute(StaticMethodHandleEventExecutor.java:38) ~[patched_1.16.3.jar:git-Purpur-808]
at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:76) ~[patched_1.16.3.jar:git-Purpur-808]
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[patched_1.16.3.jar:git-Purpur-808]
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:607) ~[patched_1.16.3.jar:git-Purpur-808]
at com.Acrobot.ChestShop.ChestShop.callEvent(ChestShop.java:540) ~[?:?]
at com.Acrobot.ChestShop.Listeners.Block.SignCreate.onSignChange(SignCreate.java:48) ~[?:?]
at com.destroystokyo.paper.event.executor.StaticMethodHandleEventExecutor.execute(StaticMethodHandleEventExecutor.java:38) ~[patched_1.16.3.jar:git-Purpur-808]
at co.aikar.timings.TimedEventExecutor.execute(TimedEventExecutor.java:76) ~[patched_1.16.3.jar:git-Purpur-808]
at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:70) ~[patched_1.16.3.jar:git-Purpur-808]
at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:607) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.PlayerConnection.a(PlayerConnection.java:2763) ~[?:git-Purpur-808]
at net.minecraft.server.v1_16_R2.PacketPlayInUpdateSign.a(SourceFile:41) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.PacketPlayInUpdateSign.a(SourceFile:9) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.PlayerConnectionUtils.lambda$ensureMainThread$1(PlayerConnectionUtils.java:43) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.TickTask.run(SourceFile:18) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.IAsyncTaskHandler.executeTask(IAsyncTaskHandler.java:136) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.IAsyncTaskHandlerReentrant.executeTask(SourceFile:23) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.IAsyncTaskHandler.executeNext(IAsyncTaskHandler.java:109) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.MinecraftServer.ba(MinecraftServer.java:1203) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.MinecraftServer.executeNext(MinecraftServer.java:1196) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.IAsyncTaskHandler.executeAll(IAsyncTaskHandler.java:95) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.MinecraftServer.a(MinecraftServer.java:1333) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.MinecraftServer.w(MinecraftServer.java:1006) ~[patched_1.16.3.jar:git-Purpur-808]
at net.minecraft.server.v1_16_R2.MinecraftServer.lambda$a$0(MinecraftServer.java:178) ~[patched_1.16.3.jar:git-Purpur-808]
at java.lang.Thread.run(Thread.java:834) [?:?]

Phoenix616 · Answer 4 · 2021-01-04T23:03:18.000Z

Well this says that it somehow created the wrong code or got the wrong item from the code. This might have multiple reasons, either an error in the recent changes or how the server itself serialises items (seeing as you don't use Spigot/Paper that might be possible).

Can you provide your items.db file as well as test if this issue occurs on ChestShop build 260 too (and maybe even Spigot/Paper if you can but I realise that that might not be possible if that only occurs on production)

x1p · Answer 5 · 2021-01-04T21:17:17.000Z

latest.log

config.yml.txt

Phoenix616 · Answer 6 · 2021-01-04T21:28:04.000Z

Please enable LOG_TO_CONSOLE and try using the command again, apparently the external file logger does not include the full error message.

x1p · Answer 7 · 2021-01-05T03:11:14.000Z

Build 260 works fine.