canary icon indicating copy to clipboard operation
canary copied to clipboard

[Forge] Core Price Bug

Open Peixonauta2 opened this issue 5 months ago • 0 comments

Priority

High

Area

  • [X] Datapack
  • [X] Source
  • [ ] Map
  • [ ] Other

What happened?

I'm using the main branch.

It seems that the current way of dealing with exalted forge core prices for transfering tiers (non-convergence and convergence) is a bit off.

The code, as far as I investigated, isn't prepared to read the core prices the way it should:

  • If I'm trying to transfer a tier 10 without the convergence switch activated, the core price will be incorrect (1 tier lower than it should be, 9)
  • If I'm trying to transfer a tier 10 with the convergence switch activated, the core price will be correct

The problem is: the client allows the players to start the transfer (even if they don't have the number of cores needed for the transfer) because the information that is sent to the client is wrong. As the player does not have the number of cores that the server expects, an error message appears and the items are removed.

https://s11.gifyu.com/images/SAQeF.gif

As you can see, the first tentative, I've transfered the tier from the cobra hood to the golden helmet. The client asked for 60 cores, but the server was expecting to remove 85.

image

What OS are you seeing the problem on?

Windows

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

Peixonauta2 avatar Sep 24 '24 07:09 Peixonauta2