ViaBackwards icon indicating copy to clipboard operation
ViaBackwards copied to clipboard

Item Data in Chat Component Hover Events do not display on 1.20.4 client on 1.20.6 server

Open XDMAN500 opened this issue 1 year ago • 7 comments

'/viaversion dump' Output

https://dump.viaversion.com/f607ade2ea7059165718592ab2768758d6125a478c523a89c8e5d34e753e0367

Console Error

N/A

Bug Description

The item data in chat hover events on 1.20.6 servers are not properly translated to 1.20.4 clients so item data like custom names, enchantments, attributes, and others are not visible to 1.20. clients.

Steps to Reproduce

  1. Start a server with 1.20.6
  2. Join server with a 1.20.4 client
  3. Execute command tellraw @a {"text":"Hover me!","hoverEvent":{"action":"show_item","contents":{"id":"cake","count":1, "components":{"cu stom_name":"{'text':'Special Cake Name',color:'yellow'}"}}}}
  4. The 1.20.4 client hovers over the chat message and only sees a default cake item
  5. A 1.20.6 client client sees a cake with a yellow custom name "Special Cake name"

Expected Behavior

The 1.20.4 client should see the "Special Cake Name" custom name on the cake item when hovering over the chat message.

Additional Server Info

Appearance from a 1.20.4 client image

Appearance from a 1.20.6 client image

Checklist

  • [X] Via plugins are only running on EITHER the backend servers (e.g. Paper) OR the proxy (e.g. Velocity), not on both.
  • [X] I have included a ViaVersion dump.
  • [X] If applicable, I have included a paste (not a screenshot) of the error.
  • [X] I have tried a build from https://ci.viaversion.com/ and the issue still persists.

XDMAN500 avatar Jun 16 '24 20:06 XDMAN500

Platform: 1.20.6--112--347bbe3%20%28MC%3A%201.20.6%29 ViaVersion (5.0.0): 4 commits behind master ViaBackwards(5.0.0): Even with master

Please update all Via* plugins from https://ci.viaversion.com/.In case the issue still persists send the new dump and the issue will be reopened

Barvalg avatar Jun 16 '24 20:06 Barvalg

Known, but it'll take a bit to fix

kennytv avatar Jun 16 '24 20:06 kennytv

New dump added: https://dump.viaversion.com/f607ade2ea7059165718592ab2768758d6125a478c523a89c8e5d34e753e0367

XDMAN500 avatar Jun 16 '24 20:06 XDMAN500

Platform: 1.20.6--112--347bbe3%20%28MC%3A%201.20.6%29 ViaVersion (5.0.1-SNAPSHOT): Even with master ViaBackwards(5.0.0): Even with master

Barvalg avatar Jun 16 '24 20:06 Barvalg

Is there any eta for that? 👉👈

Kryniowesegryderiusz avatar Jun 25 '24 21:06 Kryniowesegryderiusz

No, writing these conversions is quite time intensive and it will take a while, but I plan on doing it.

FlorianMichael avatar Jun 25 '24 21:06 FlorianMichael

When can we expect this to be fixed? It's been a couple of months now.

qIooIp avatar Sep 22 '24 16:09 qIooIp

Fixed in dev builds

FlorianMichael avatar Jan 15 '25 12:01 FlorianMichael