cmangos / issues

This repository is used as a centralized point for all issues regarding CMaNGOS.
179 stars 48 forks source link

Headless Horseman's Mount teaches spell (TBC) #2925

Open henhouse opened 2 years ago

henhouse commented 2 years ago

The Horseman's Reins is supposed to mount the player in TBC with spell ID 48025. Instead, this spell is learned to the player because the spell 1 is incorrectly set to 483 (Learning). Perhaps this is a bad port from WotLK?

There should only be one spell effect, the main mounting spell, not a learning one.

Fix:

UPDATE `item_template` SET `spellid_1` = '48025', `spellcharges_1` = '0', `spellcooldown_1` = '0', `spellid_2` = '0', `spelltrigger_2` = '0', `spellcooldown_2` = '0', `spellcategorycooldown_2` = '0', `description`='' WHERE `entry` = '37012';
AnonXS commented 2 years ago

Possible: https://github.com/cmangos/tbc-db/commit/b009c15016b2c581e92d55f66167ed73b19f8130#diff-b15ded4e4a3c9497db6a222585f97349cf2dc013e40044b0c774a50fc221314aR4

Linked to: https://github.com/cmangos/issues/issues/2835

yuyu-lovegelin111333 commented 2 years ago

I do tested by henhouse's Fix : update spellid1 to 48025 , but issue happened same . Issue : https://github.com/cmangos/issues/issues/2835 When Player has Level40> learn riding skill to 75 point>add item 37012> use the item horseman's mount >udner mounting riding status60% speed > levelup your player's lever +1 or any > you will see your riding skill auto increased to 210 or 225 poting > and re-ride your horseman's mount> you will see it auto changed to status100% speed . Case was you never go to trainer to learn any riding skill to 225 point .

insunaa commented 2 years ago

The riding skill issue is unrelated and fixed by this PR https://github.com/cmangos/mangos-tbc/pull/558