search

ZhengPeiRu21 / mod-individual-progression

AzerothCore Individual Progression Module
MIT License
117 stars 53 forks source link

Mount tab for Vanilla mounts #63

Closed justurn closed 1 year ago

justurn commented 1 year ago

I noticed that vanilla mounts are modified in the SQL mounts_and_riding.SQL It is not so clear how those changes have disabled the mounts tab that is present on WOTLK content? Maybe the spell_ID?

Is there a way to keep the level and price changes for vanilla, but also maintain access to the mounts tab? I'd prefer to not have bags slots occupied by mounts, if possible.

Thanks :)

StraysFromPath commented 1 year ago

As a workaround you can do ".learn X" where X is the ID of the mount spell.

To find the spell use ".look sp X" where X is the name of the mount.

For example, 8394 is the Striped Frostsaber for Night Elves.

Then you can delete the whistle from your inventory after buying it.

https://wotlkdb.com/?spell=8394

justurn commented 1 year ago

Thanks, would that add the mount to the tab though or just add it to your spell book? I noticed all of the blood elf mounts are going directly into the mount tab.

Maybe this is just an issue with vanilla mounts?

justurn commented 1 year ago

I found removing the changes to mounts in the mounts and riding SQL file solved this issue, perhaps these should be split into two SQL files. Making the mounts portion optional?

ZhengPeiRu21 commented 1 year ago

Thank you for the report - this is not intentional as using the mount tab is a QoL that I hoped to preserve. I will tag this as a bug and fix the SQL so the mounts are added to the tab properly.

ZhengPeiRu21 commented 1 year ago

This should now be fixed in https://github.com/ZhengPeiRu21/mod-individual-progression/commit/94d4eda797beb4311c7bc6e3afc73112506d3c99. If anyone is prefer the old behavior of requiring mount items in the bag for use, please feel free to retrieve the old version of the file from Git history.