search

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 56 forks source link

Specialized engineer&leatherworking trainers not correctly configured #908

Open RichardGe opened 8 years ago

RichardGe commented 8 years ago

issue :

npc 8126 (Nixx Sprocketspring) and npc 8738 (Vazario Linkgrease) are training Goblin engineering schematics. But a Gnomish engineer can learn from them.

npc 7944 (Tinkmaster Overspark) and npc 7406 (Oglethorpe Obnoticus) are training Gnomish Engineering schematics. But a Goblin engineer can learn from them

How to solve : The 'TrainerSpell' fied of npcs 8126 and 8738 should be set to 20222: http://db.vanillagaming.org/?spell=20222

The 'TrainerSpell' fied of npcs 7944 and 7406 should be set to 20219: http://db.vanillagaming.org/?spell=20219

Concerning Blacksmith, I think it's correctly set. example : npc 5164 has TrainerSpell=9788 (Armorsmith)

Concerning Leatherworking, I didn't look, let me know. (edit, they are not set correctly too, see comments below)

I don't remember if there are other specializations in Vanilla, can you tell me ?

dagochen commented 8 years ago

dragon, elementary and one more leather specc i currently dont know the name of^^

evil-at-wow commented 8 years ago

@dagochen Tribal Leatherworking.

And I think you have them all. You've mentioned Engineering, Blacksmithing and Leatherworking. Gather professions and Enchanting didn't have any. Tailoring and Alchemy got them in TBC if I remember correctly, and Jewelcrafting as a profession was even introduced in TBC. I think that rounds it up.

RichardGe commented 8 years ago

ok, we have to check that all the trainers of those specialties have the 'TrainerSpell' set correctly

RichardGe commented 8 years ago

I think leather working trainers are not set too : NPCs 7866 & 7867 for dragonscale leatherworking. 'TrainerSpell' should be = 10657 NPCs 7868 & 7869 for elemental leatherworking. 'TrainerSpell' should be = 10659 NPS 7810 & 7871 for Tribal Leatherworking. 'TrainerSpell' should be = 10661

RichardGe commented 8 years ago

And also this makes me think to an additional feature, for engineer only I think. they have membership cards. When this card expires I think you can't learn the plans from the trainer. I haven't checked if this feature is supported in this database. I don't know what parameter should be set. http://db.vanillagaming.org/?item=10790 http://db.vanillagaming.org/?item=10791

Phatcat commented 8 years ago

Actually, those membership cards were for recieving items through the mail, such as pet bombling and the gnomish engineering counterpart; lil smokey (depending on specialization).. you could get the one belonging to your specialization in the mail once you did the card renewal quests (you'd get a gift each time), but not with a 100% chance (read some of the comments on here: http://www.wowhead.com/item=11828/schematic-pet-bombling#comments:id=8877 - some people seem to have spent upwards of insane amounts of gold and not gotten it, others got it on first or second renewal) - also the membership cards were removed in wotlk and both these were turned into drops in gnomer.

so, to enlighten a bit on your thought; the membership cards have nothing to do with being able to train at the specialization trainer, they're a seperate engineering-only novelty-event revolving gifts recieved in the mail and stuff. wish I knew the name of the gifts.

RichardGe commented 8 years ago

ok, thanks for the info. So forget my last comment concerning the membership cards.