classicdb / database

Classic DB is a content database for CMaNGOS Classic: world, NPCs, objects, quests and so on.
https://github.com/cmangos/mangos-classic
Other
87 stars 59 forks source link

Master Axesmith quest #262

Open scotty0100 opened 10 years ago

scotty0100 commented 10 years ago

How to replicate the bug: Talk with Kilram in Everlook, the text balloon option is there, try to click on it What happens: Nothing What should happen: once you click on it he should teach you Master Axesmith

This is trying with Weaponsmith already learnt too.

scotty0100 commented 10 years ago

EDIT: All 3 blacksmithing specialities are bugged.

Axesmith:

Kilram in Winterspring will not teach Axesmith, when you open the chat window the chat bubble is there, you click on it, and theres another one you click on that and the chat window just closes.

Did the quest Snakestone of the Shadow Huntress in case that would affect being taught Axesmith but all that happened was I got Plans: Dawn's Edge.This could not be learned. When I clicked on 'Please teach me how to become a Axesmith this time again you click once, nothing happens, click again the window closes.

Hammersmth:

Lilith the Lithe will not teach Hammersmith, when you open the chat window the chat bubble is there, you click on it, and theres another one you click on that and the chat window just closes.

Did the quest Sweet Serenity in case that would affect being taught Hammersmith but all that happened was I got Plans: Enchanted Battlehammer and When I clicked on 'Please teach me how to become a Hammersmith this time again you click once, nothing happens, click again the window closes.

Quest Sweet Serenity is the only one out of the 3 specials that actually says it will teach you the Speciality once the quest is done. It however does not.

Swordsmith:

Seril Scourgebane will not teach Swordsmith, when you open the chat window the chat bubble is there, you click on it, and theres another one you click on that and the chat window just closes.

Did the quest Corruption in case that would affect being taught Swordsmith but all that happened was I got Plans: Blazing Rapier and When I clicked on 'Please teach me how to become a Swordsmith this time again you click once, nothing happens, click again the window closes.

cala commented 10 years ago

Probably the gossip scripts are missing

Metalica commented 10 years ago

Master blacksmith fixes Mastery of each is missing from the npc. I hope this is correct.

Master axesmith: INSERT INTO npc_trainer_template (entry, spell, reqskill, reqskillvalue, reqlevel) VALUES ('11192', '17041', '164', '250', '40')

Master Hammersmith: INSERT INTO npc_trainer_template (entry, spell, reqskill, reqskillvalue, reqlevel) VALUES ('11191', '17040', '164', '250', '40')

Master Swordsmith: INSERT INTO npc_trainer_template (entry, spell, reqskill, reqskillvalue, reqlevel) VALUES ('11193', '17039', '164', '250', '40')

NeatElves commented 10 years ago

17043 -> 17041 ?

Metalica commented 10 years ago

tnx, fixed the typo :)

Sinoxan commented 10 years ago

@Metalica That will put the skills in the training interface instead of a gossip option.

This needs entries in dbscripts_on_gossip and gossip_menu_option The NPC should cast these spells on the player when they select the gossip.

39097 - Master Swordsmith
39098 - Master Axesmith
39099 - Master Hammersmith

I'll take a look at this and try to get it working unless someone beats me to it.

Metalica commented 10 years ago

Atleast i tried, but all cool Sinoxan.

Sinoxan commented 10 years ago

Could someone clarify how this works please. From what I've gathered the specialization should be taught at the end of the quest? But are you able to get it without doing the quest as well?

It appears you can get it without doing the quest and that you can also unlearn the specialization but I can only see that after 2.0.1 so I'm not sure what it was like in classic.

Somewhere says that you can learn them without the quest if you already have Weaponsmith Specialization and someone else says that you learn the specialization from learning the plan you get from the quest.

Metalica commented 10 years ago

http://wow.allakhazam.com/forum.html?forum=69&mid=112279951511847756

I believe you need to be level 50+ (and probably at or close to 300 blacksmithing). Then go to Everlook, and find the Master Swordsmith there. He'll train you and give you a recipe for completing a quest. Note, the master axesmith and macesmith are there as well; but you can only specialize in one of these crafts.

So, vaules would be: VALUES ('11191', '17040', '164', '275', '50') / lvl 50 and 275 skill of blacksmithing.

Sinoxan commented 10 years ago

Ok, from what I can gather, you complete the quest and you automatically learn the specialization and get the plan so there shouldn't be a gossip option at all.

Metalica commented 10 years ago

http://www.wowwiki.com/Quest:The_Way_of_the_Weaponsmith_%28Alliance%29 - you need to complete this quest.

After that.

Master axsmith gives you gosip option to learn, but nothing happens. Our research and op small missinformation went to chase us around a hot meal.

Sinoxan commented 10 years ago

So, first you need to get Weaponsmith from the quest, then you can choose one of the quests to get the specific weapon specialization. The gossip options seem to be from TBC so I think they need to be removed.

The gossip commands are handled in ScriptDev2 which was backported which is why they are still in there.

Weaponsmith (Alliance) - 5284 - The Way of the Weaponsmith - Level 40 - 200 Blacksmithing
Weaponsmith (Horde)    - 5302 - The Way of the Weaponsmith - Level 40 - 200 Blacksmithing

Hammersmith - 5305 - Sweet Serenity                        - Level 50 - 300 Blacksmithing
Axesmith    - 5306 - Snakestone of the Shadow Huntress     - Level 50 - 300 Blacksmithing
Swordsmith  - 5307 - Corruption                            - Level 50 - 300 Blacksmithing

RewSpell
17039 - Master Swordsmith
17041 - Master Axesmith
17040 - Master Hammersmith

RewSpellCast
39097 - Master Swordsmith
39098 - Master Axesmith
39099 - Master Hammersmith
Sinoxan commented 10 years ago

I'm not sure about this but Sinoxan/scriptdev2-classic@5ae2e687f794157369750bf88f8f4b5757573f87 should remove the gossip options from the NPCs.

I'll make another commit adding the spells to the quests for all specializations.

Sinoxan commented 10 years ago

Ok, so prior to TBC you couldn't change your blacksmithing specialization even if you dropped the skill altogether. However, with leatherworking and engineering you could post 1.10. You would drop the skill, use 'Soothsaying for Dummies' to unlearn the specialization, take the profession again and then get the specialization, I believe with engineering you could just repeat the membership renewal quest again but for leatherworking I can't find any information on what to do.

Phatcat commented 5 years ago

@Warlockbugs so has this issue been resolved? I see your https://github.com/cmangos/mangos-classic/commit/09cdba10280df17b5a6fef3215f1a5b96146ac88 , did that fix the issue?

I know the unlearning in classic hasn't, but that has it's own raised issue, several actually)

NeatElves commented 5 years ago

No all data. https://github.com/cmangos/wotlk-db/issues/169#issuecomment-326794832