ViaVersion / ViaBackwards

Allows the connection of older clients to newer server versions for Minecraft servers.
https://hangar.papermc.io/ViaVersion/ViaBackwards
GNU General Public License v3.0
441 stars 85 forks source link

"Unknown Named Sound" check false-positives on custom sounds #380

Closed NathanWolf closed 3 years ago

NathanWolf commented 3 years ago

'/viaversion dump' Output

https://dump.viaversion.com/c303b0b3819b85502586c3a02d2752d2e14a160969c5400a71fbeebbc1f33cde

Console Error

[13:11:02 WARN]: [ViaBackwards] Unknown named sound in 1.13->1.12 protocol: minecraft:magic.zap

Bug Description

Plugins that play custom sounds from resource packs will spam the console with warnings about the sound being unrecognized.

Is there a way to disable this check via configs?

Steps to Reproduce

Use any plugin that uses custom sounds. My plugin (Magic) was reported to be console-spamming, any time a spell is cast that plays a custom sound.

https://www.spigotmc.org/resources/magic.1056/

Expected Behavior

I understand this check is nice to have for development purposes, but I imagine there are many plugins out there that use custom sounds. This should not cause such log spam.

Perhaps the check should be off by default, but if not then I think server owners should get an option to turn it off, if such an option does not exist already.

Additional Server Info

The Magic plugin from here: https://www.spigotmc.org/resources/magic.1056/

Checklist

kennytv commented 3 years ago

You can enable suppress-conversion-warnings in the ViaVersion config to hide those warnings.

NathanWolf commented 3 years ago

Whoops, thank you, sorry for wasting your time!

kennytv commented 3 years ago

No worries. We know this obviously causes "wrong" warnings for custom sounds, but is necessary for us to identify missing/wrong mappings for existing sounds.