fix Riding Speed increases Druid Flight Forms

[grimgravy]

Changes Proposed:

• added modification made at TBC

Issues Addressed:

• Closes https://github.com/azerothcore/azerothcore-wotlk/issues/16235

SOURCE:

How to Test AzerothCore PRs

When a PR is ready to be tested, it will be marked as [WAITING TO BE TESTED].

You can help by testing PRs and writing your feedback here on the PR's page on GitHub. Follow the instructions here:

http://www.azerothcore.org/wiki/How-to-test-a-PR

REMEMBER: when testing a PR that changes something generic (i.e. a part of code that handles more than one specific thing), the tester should not only check that the PR does its job (e.g. fixing spell XXX) but especially check that the PR does not cause any regression (i.e. introducing new bugs).

For example: if a PR fixes spell X by changing a part of code that handles spells X, Y, and Z, we should not only test X, but we should test Y and Z as well.

[Annamaria-CC]

~~testing in progress~~

tested Flight Form

• [ ] Skybreakers whip works on flightform
• [x] enchants gloves, mithril spurs and carrot on a stick don't work on flight form

Normal mount

• [x] Skybreakers whip works on normal mount
• [x] Enchant gloves, mithril spurs and carrot on a stick stack on top of eachother, gaining the same +10% mount speed as with a whip. Not sure if intended.

Please review and tell me if this is correct behaviour or not

[Tereneckla]

Yes, enchant, spurs and carrot should stack

[Annamaria-CC]

Yes, enchant, spurs and carrot should stack

corrected the above then <3

[Kitzunu]

Conflict

[sudlud]

Can you please resolve the conflicts again

[sudlud]

Thanks for the update!

I'm using Carbonite right now https://felbite.com/addon/4846-carbonite/ to get a flight speed info within the client and tried to test this PR.

level 70 tauren druid in "swift flight form" Riding Crop (features spell 48776) .additem 25653

unequipped speed: 280%

equipped speed: 318%

If I understand correctly, the speed should be 280% even when "Riding Crop" is equipped?

Right now to me it looks like this PR does not have the desired effect, can you please check or give a hint what's going wrong on my side?

Thanks!

[grimgravy]

Thanks for the update!

I'm using Carbonite right now https://felbite.com/addon/4846-carbonite/ to get a flight speed info within the client and tried to test this PR.

level 70 tauren druid in "swift flight form" Riding Crop (features spell 48776) .additem 25653

unequipped speed: 280%

equipped speed: 318%

If I understand correctly, the speed should be 280% even when "Riding Crop" is equipped?

Right now to me it looks like this PR does not have the desired effect, can you please check or give a hint what's going wrong on my side?

Thanks!

The Riding Crop is working with or without changes. However, the carrot on a stick and mithril spurs need modification

Interesting fact: flying mount 305% and swift flight form 295%

[sudlud]

Looks like CI is broken right now

[sudlud]

So what's the state of this PR now? Ready to be tested again?

[Kitzunu]

Conflicts

[alecsci]

What's the status for this ? if it's not ready can we label it differently ?

[Nyeriah]

TBH I don't like DBC edits that edit effect types, aura types etc. Always feel like a big hack

[grimgravy]

the effect is affecting the speed of other flight assemblies. i don't know what to do