MUnique / OpenMU

This project aims to create an easy to use, extendable and customizable server for a MMORPG called "MU Online".
https://munique.net
MIT License
743 stars 326 forks source link

Elf: Can't re-equip Gywen Bow #517

Closed nitoygo closed 4 days ago

nitoygo commented 1 month ago

Describe the bug After removing the Gywen Bow, I can't equip it again

To Reproduce Steps to reproduce the behavior:

  1. Login Ancient, select ancientElf
  2. Unequip Gywen Bow
  3. Try to re-equip Gywen Bow
  4. // unable to equip....

Expected behavior Should be able to re-equip Gywen Bow

Screenshots image

sven-n commented 4 weeks ago

OpenMU has the wrong requirement values for some items. It has the requirements for a level 0 item in the item definition, but the actual value is calculated by an abstract value which doesn't have anything to do with the visible value. You can see the calculation at ItemExtensions.CalculateRequrement(this Item item.... This code calculates a requirement of 725 Agility and 298 Strength for this ancient silver bow, but it must be 380 Agility and 148 Strength.

I checked some config data:

So, to fix this, the data needs to be checked... e.g. by extracting the correct values from some other season 6 server files and comparing them to the ones in the database. I'll write some code to do that when I have some time.