Skip to content
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

1.7.10 Anvil works incorrectly (Required level cost is different between 1.7 and 1.8.) #496

Open
5 tasks done
Lefiy opened this issue Mar 27, 2024 · 1 comment
Open
5 tasks done

Comments

@Lefiy
Copy link

Lefiy commented Mar 27, 2024

/viaversion dump Output

https://dump.viaversion.com/0713a343db61d8ba175f4358d6e5b3ccd1d4fc1aa73837cf433f674bed592389

Console Error

No Errors

Bug Description

Sorry if this was correct behavior.
When I tried to combine enchantments in 1.7 with the same content as in 1.8 with an anvil, the required levels are different.
Then combine them and close inventory, they were canceled and the level disappeared. (The level came back when I made an action that would update the level.)
However, using the same level as 1.8 was worked.
The same thing happens not only with swords but also with armor.
Sorry for bad english.

1.8 Video (Worked)
https://github.com/ViaVersion/ViaRewind/assets/80197107/664c0c36-cc33-449a-9e54-70b1b53db589

1.7 Video
https://github.com/ViaVersion/ViaRewind/assets/80197107/2f4890ea-a6db-4b6d-b9c6-507571aa39f2

Try combining at 1.7 with the same level as 1.8. (Worked)
https://github.com/ViaVersion/ViaRewind/assets/80197107/e71abb85-d56d-4743-9174-324f94bac653

Steps to Reproduce

  1. Login with 1.8 or 1.7 client.
  2. Prepare the sword using the enchant command and the book using the enchant table.
  3. Place the anvil and work as in the video above.

Expected Behavior

If the required level is enough, it is not canceled and the levels are consumed and combined.
(I think it is unavoidable that different versions have different levels of requirements.)

Additional Server Info

I deleted all the files and downloaded PaperSpigot 1.8 again.
The only plugins installed are ViaVersion and ViaRewind, ViaRewind-Legacy-Support.
And I do not use proxies.

Checklist

@Barvalg
Copy link
Member

Barvalg commented Mar 27, 2024

Platform: git--PaperSpigot--445%20%28MC%3A%201.8.8%29
ViaVersion (4.9.4-SNAPSHOT): Even with master
ViaRewind(3.0.7-SNAPSHOT): Even with master

@Lefiy Lefiy changed the title Anvil works incorrectly (Required level cost is different between 1.7 and 1.8.) 1.7 Anvil works incorrectly (Required level cost is different between 1.7 and 1.8.) Mar 27, 2024
@Lefiy Lefiy changed the title 1.7 Anvil works incorrectly (Required level cost is different between 1.7 and 1.8.) 1.7.10 Anvil works incorrectly (Required level cost is different between 1.7 and 1.8.) Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants