Jump to content
  • [Warrior] Execute not working properly

    • Status: Completed
      Main Category: Database
      Sub-Category: Spell
      Version: 0.20(currentmasterbranch) Milestone: 20-newbeginnings Priority: Normal
      Implemented Version: 0.20(currentmasterbranch)

    [Warrior] Execute not working properly


    Just reporting a known Mangos-Zero (and I think Mangos-One too but didn't validate) bug. When warrior is using execute, it can be that the animation is done, rage is taken (partially) but nothing appear on the client as it should. In practice, this behavior results from a miss/parry/avoid while processing the original spell sent by the client. To display properly the result on the client, it's expecting spell id 20647 to be returned to it.

    Also, fixing the above reveals another bug then, because spell 20647 is also processed, you can actually miss/parry/avoid it as well meaning there's 2 roll for the same spell (and I don't think it's the goal of it) creating a situation even worse: all the rage is gone, animation started, nothing displayed. After analyzing the spells in DBC, I've found out that Initial spell Id's get an EquippedClassItems property while the spell id 20647 doesn't have such property and it makes hit calculation different than for the initial spell id. Therefore, I propose to update spell 20647 at start-up to never miss the target.

    Impacted spell id's:

    - 5308
    - 20658
    - 20660
    - 20661
    - 20662

    - 20647

    I'm filling this bug to "normalize" the situation of my pull request: [url]https://github.com/mangoszero/server/commit/cda3ebf959502147601f2b31fc9410c00e49e62f[/url]

    Ps : If not clear enough, will make a drawing to show how it works actually and how I think it should work.

    User Feedback

    Recommended Comments

    Create an account or sign in to comment

    You need to be a member in order to leave a comment

    Create an account

    Sign up for a new account in our community. It's easy!

    Register a new account

    Sign in

    Already have an account? Sign in here.

    Sign In Now

  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. Privacy Policy Terms of Use